diff options
author | Mathias Krause <mathias.krause@secunet.com> | 2011-01-17 07:50:42 +0000 |
---|---|---|
committer | Mathias Krause <mathias.krause@secunet.com> | 2011-01-17 07:50:42 +0000 |
commit | a60faab83ef86bcf2884311db07cf13f445d7f47 (patch) | |
tree | 9de9787987eb6a3374e18835b196b7ac6cb9ad19 /sb600spi.c | |
parent | 2c3afa34fcfc72abe6b3e0fd034632226fa75522 (diff) | |
download | ast2050-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 'sb600spi.c')
-rw-r--r-- | sb600spi.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -208,7 +208,7 @@ int sb600_probe_spi(struct pci_dev *dev) struct pci_dev *smbus_dev; uint32_t tmp; uint8_t reg; - const char *speed_names[4] = { + static const char *const speed_names[4] = { "Reserved", "33", "22", "16.5" }; |