cpufreq: longhaul: Make array speeds static const
authorColin Ian King <colin.i.king@gmail.com>
Thu, 3 Nov 2022 15:04:31 +0000 (15:04 +0000)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 3 Nov 2022 18:13:34 +0000 (19:13 +0100)
Don't populate the read-only array speeds on the stack but instead
make it static. Also makes the object code a little smaller. Replace
hard-coded loop array bounds with ARRAY_SIZE.

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

index 3e000e1..4c57c67 100644 (file)
@@ -407,10 +407,10 @@ static int guess_fsb(int mult)
 {
        int speed = cpu_khz / 1000;
        int i;
-       int speeds[] = { 666, 1000, 1333, 2000 };
+       static const int speeds[] = { 666, 1000, 1333, 2000 };
        int f_max, f_min;
 
-       for (i = 0; i < 4; i++) {
+       for (i = 0; i < ARRAY_SIZE(speeds); i++) {
                f_max = ((speeds[i] * mult) + 50) / 100;
                f_max += (ROUNDING / 2);
                f_min = f_max - ROUNDING;