net: dsa: bcm_sf2: make const array static, makes object smaller
authorColin Ian King <colin.king@canonical.com>
Tue, 20 Oct 2020 16:50:29 +0000 (17:50 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 21 Oct 2020 03:57:57 +0000 (20:57 -0700)
Don't populate the const array rate_table on the stack but instead it
static. Makes the object code smaller by 46 bytes.

Before:
   text    data     bss     dec     hex filename
  29812    3824     192   33828    8424 drivers/net/dsa/bcm_sf2.o

After:
   text    data     bss     dec     hex filename
  29670    3920     192   33782    83f6 drivers/net/dsa/bcm_sf2.o

(gcc version 10.2.0)

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Link: https://lore.kernel.org/r/20201020165029.56383-1-colin.king@canonical.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/dsa/bcm_sf2.c

index 0b5b2b3..1e9a0ad 100644 (file)
@@ -54,7 +54,7 @@ static void bcm_sf2_recalc_clock(struct dsa_switch *ds)
        unsigned long new_rate;
        unsigned int ports_active;
        /* Frequenty in Mhz */
-       const unsigned long rate_table[] = {
+       static const unsigned long rate_table[] = {
                59220000,
                60820000,
                62500000,