cpufreq: speedstep-lib: make several arrays static, makes code smaller
authorColin Ian King <colin.king@canonical.com>
Fri, 25 Aug 2017 17:00:16 +0000 (18:00 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 28 Aug 2017 22:21:41 +0000 (00:21 +0200)
Don't populate arrays on the stack, instead make them static.
Makes the object code smaller by over 860 bytes:

Before:
   text    data     bss     dec     hex filename
  10716    5196       0   15912    3e28 drivers/cpufreq/speedstep-lib.o

After:
   text    data     bss     dec     hex filename
   9690    5356       0   15046    3ac6 drivers/cpufreq/speedstep-lib.o

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/speedstep-lib.c

index 1b8062182c813b359200070306a228d0c0cef18b..ccab452a4ef5b739cd8294cfa3197211afa9afb8 100644 (file)
@@ -35,7 +35,7 @@ static int relaxed_check;
 static unsigned int pentium3_get_frequency(enum speedstep_processor processor)
 {
        /* See table 14 of p3_ds.pdf and table 22 of 29834003.pdf */
-       struct {
+       static const struct {
                unsigned int ratio;     /* Frequency Multiplier (x10) */
                u8 bitmap;              /* power on configuration bits
                                        [27, 25:22] (in MSR 0x2a) */
@@ -58,7 +58,7 @@ static unsigned int pentium3_get_frequency(enum speedstep_processor processor)
        };
 
        /* PIII(-M) FSB settings: see table b1-b of 24547206.pdf */
-       struct {
+       static const struct {
                unsigned int value;     /* Front Side Bus speed in MHz */
                u8 bitmap;              /* power on configuration bits [18: 19]
                                        (in MSR 0x2a) */