phy: cadence: Sierra: Constify a u32[]
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Thu, 11 Jul 2024 19:34:54 +0000 (21:34 +0200)
committerVinod Koul <vkoul@kernel.org>
Wed, 31 Jul 2024 11:35:44 +0000 (17:05 +0530)
'cdns_sierra_pll_mux_table' is not modified in this driver.
And it is only used as a "const u32 *".

Constifying this structure moves some data to a read-only section, so
increase overall security.

On a x86_64, with allmodconfig:
Before:
======
   text    data     bss     dec     hex filename
  60937    3894      16   64847    fd4f drivers/phy/cadence/phy-cadence-sierra.o

After:
=====
   text    data     bss     dec     hex filename
  60897    3878      16   64791    fd17 drivers/phy/cadence/phy-cadence-sierra.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/43d881d52e3c1632ad197d4c2c18b6c481a13b24.1720723132.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/cadence/phy-cadence-sierra.c

index 947492c8f38129625b8ca1c0bb36376a7ab22bab..3010c9f24136268224af7cf452de5deea9dea915 100644 (file)
@@ -310,7 +310,7 @@ static const struct clk_parent_data pll_mux_parent_data[][SIERRA_NUM_CMN_PLLC_PA
        },
 };
 
-static u32 cdns_sierra_pll_mux_table[][SIERRA_NUM_CMN_PLLC_PARENTS] = {
+static const u32 cdns_sierra_pll_mux_table[][SIERRA_NUM_CMN_PLLC_PARENTS] = {
        [CMN_PLLLC] = { 0, 1 },
        [CMN_PLLLC1] = { 1, 0 },
 };