path: root/ichspi.c
diff options
authorMathias Krause <>2011-01-17 07:50:42 +0000
committerMathias Krause <>2011-01-17 07:50:42 +0000
commita60faab83ef86bcf2884311db07cf13f445d7f47 (patch)
tree9de9787987eb6a3374e18835b196b7ac6cb9ad19 /ichspi.c
parent2c3afa34fcfc72abe6b3e0fd034632226fa75522 (diff)
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 <> Acked-by: Stefan Reinauer <>
Diffstat (limited to 'ichspi.c')
1 files changed, 2 insertions, 2 deletions
diff --git a/ichspi.c b/ichspi.c
index a087a0b..66c8d51 100644
--- a/ichspi.c
+++ b/ichspi.c
@@ -948,10 +948,10 @@ int ich_spi_send_multicommand(struct spi_command *cmds)
static void do_ich9_spi_frap(uint32_t frap, int i)
- const char *access_names[4] = {
+ static const char *const access_names[4] = {
"locked", "read-only", "write-only", "read-write"
- const char *region_names[5] = {
+ static const char *const region_names[5] = {
"Flash Descriptor", "BIOS", "Management Engine",
"Gigabit Ethernet", "Platform Data"
OpenPOWER on IntegriCloud