summaryrefslogtreecommitdiffstats
path: root/stm50flw0x0x.c
diff options
context:
space:
mode:
authorMathias Krause <mathias.krause@secunet.com>2011-01-17 07:50:42 +0000
committerMathias Krause <mathias.krause@secunet.com>2011-01-17 07:50:42 +0000
commita60faab83ef86bcf2884311db07cf13f445d7f47 (patch)
tree9de9787987eb6a3374e18835b196b7ac6cb9ad19 /stm50flw0x0x.c
parent2c3afa34fcfc72abe6b3e0fd034632226fa75522 (diff)
downloadast2050-flashrom-a60faab83ef86bcf2884311db07cf13f445d7f47.zip
ast2050-flashrom-a60faab83ef86bcf2884311db07cf13f445d7f47.tar.gz
Convince compilers to put constant data into the .rodata section
This patch reduces the stack usage by declaring 'const' stack variables as 'static const' so they end up in the .rodata section instead of being copied from there to the stack for every invocation of the corresponding function. As a plus we end up in having a smaller binary as the "copy from .rodata to stack" code isn't emitted by the compiler any more (roughly -100 bytes). Corresponding to flashrom svn r1252. Signed-off-by: Mathias Krause <mathias.krause@secunet.com> Acked-by: Stefan Reinauer <stepan@coreboot.org>
Diffstat (limited to 'stm50flw0x0x.c')
-rw-r--r--stm50flw0x0x.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/stm50flw0x0x.c b/stm50flw0x0x.c
index 80f518e..60e5c45 100644
--- a/stm50flw0x0x.c
+++ b/stm50flw0x0x.c
@@ -39,7 +39,7 @@
static int unlock_block_stm50flw0x0x(struct flashchip *flash, int offset)
{
chipaddr wrprotect = flash->virtual_registers + 2;
- const uint8_t unlock_sector = 0x00;
+ static const uint8_t unlock_sector = 0x00;
int j;
/*
OpenPOWER on IntegriCloud