memory: tegra: make the array list static const, makes object smaller
authorColin Ian King <colin.king@canonical.com>
Thu, 19 Aug 2021 13:31:55 +0000 (14:31 +0100)
committerKrzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Wed, 15 Sep 2021 07:31:19 +0000 (09:31 +0200)
Don't populate the array list on the stack but instead it
static const. Makes the object code smaller by 110 bytes:

Before:
   text    data     bss     dec     hex filename
  37713   21992      64   59769    e979 .../tegra/tegra210-emc-cc-r21021.o

After:
   text    data     bss     dec     hex filename
  37539   22056      64   59659    e90b .../tegra/tegra210-emc-cc-r21021.o

(gcc version 10.3.0)

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Link: https://lore.kernel.org/r/20210819133155.10441-1-colin.king@canonical.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
drivers/memory/tegra/tegra210-emc-cc-r21021.c

index 0ebfa8e..cc76adb 100644 (file)
@@ -478,7 +478,7 @@ static u32 periodic_compensation_handler(struct tegra210_emc *emc, u32 type,
 static u32 tegra210_emc_r21021_periodic_compensation(struct tegra210_emc *emc)
 {
        u32 emc_cfg, emc_cfg_o, emc_cfg_update, del, value;
-       u32 list[] = {
+       static const u32 list[] = {
                EMC_PMACRO_OB_DDLL_LONG_DQ_RANK0_0,
                EMC_PMACRO_OB_DDLL_LONG_DQ_RANK0_1,
                EMC_PMACRO_OB_DDLL_LONG_DQ_RANK0_2,