From nobody Wed Dec 17 08:44:11 2025 Received: from smtp.smtpout.orange.fr (smtp-16.smtpout.orange.fr [80.12.242.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2598816F855 for ; Thu, 11 Jul 2024 19:35:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=80.12.242.16 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726509; cv=none; b=HDMAEXw/XUT50VTk2FdAkJ8jtIKLByrpjvZFzmiPBiX60Hyzh6q/Ygr05eefWRtoD9i908tVWo19v2PCvjOh9W1e1V5eCSNQzH6/AgUez4QQp72Gflxb6ZraojJvFQP7Pn1B5OVpZeEyx1ZYtCewu08gIJUXnIIbZw4byXp9i6E= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726509; c=relaxed/simple; bh=fxqHbexqa/XBA+b4kSaSE8iCXVy+Xhb2hvj+JR297Bk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=KN6bwkpAT83GLWGpCKD29PxEJKyNULxbjwvLl7LXa0Hx9xgJiCYfl3/tobZPqH/avPp4dB6Y6L94O0NK2HCUNdKODMae2miJ+CW7uknlEIbdXaju2RTMkVVoTai70uXT7gSqUDoUGdKxK52xRWu0pDFVwTqR/pKVPBTOPxoCG0M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr; spf=pass smtp.mailfrom=wanadoo.fr; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b=uKlfxmlZ; arc=none smtp.client-ip=80.12.242.16 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b="uKlfxmlZ" Received: from fedora.. ([77.131.3.214]) by smtp.orange.fr with ESMTPA id RzZBsCMFrj4pfRzZEsIiRe; Thu, 11 Jul 2024 21:35:01 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wanadoo.fr; s=t20230301; t=1720726501; bh=MaLM7B8grFGEA8Qhxc325Ffgm/J2Yg0HpAuW5vXtFvE=; h=From:To:Subject:Date:Message-ID:MIME-Version; b=uKlfxmlZm+QDK9dDFuhV//L6spwodsLDPA9Fqhg/ncr3a1Z9vkzcdMNckcJEsP7Cy HHrKThLpkL6iNE/eHu3mDTKRFwbfrtvF3/VxzHRGzXyDptCEI+uX1bEd4BHazLP+nC eeFMytPSvArCHyalNCDhdiJgVewvlteO9DNw9p/a6cj193aaaETA4ClxIsU1Tl2QMu K4MKrlnKmSLdL/+8GRxsg6iTbNm9OF0tmxMBhp9S2RpOgXT62e9FSBfimxm1cgzrFH rxx0zb5ZjRpC5bNoXBSuzOiW787NX+GytCf2gheKLtWTQVOaCKndKY+89u5KB85tm9 +lan4gygVrKsA== X-ME-Helo: fedora.. X-ME-Auth: Y2hyaXN0b3BoZS5qYWlsbGV0QHdhbmFkb28uZnI= X-ME-Date: Thu, 11 Jul 2024 21:35:01 +0200 X-ME-IP: 77.131.3.214 From: Christophe JAILLET To: vkoul@kernel.org, kishon@kernel.org Cc: linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET Subject: [PATCH 1/3] phy: cadence: Sierra: Constify struct cdns_reg_pairs Date: Thu, 11 Jul 2024 21:34:52 +0200 Message-ID: X-Mailer: git-send-email 2.45.2 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" 'struct cdns_reg_pairs' is not modified in this driver. Constifying this structure moves some data to a read-only section, so increase overall security. On a x86_64, with allmodconfig: Before: =3D=3D=3D=3D=3D=3D text data bss dec hex filename 60489 4342 16 64847 fd4f drivers/phy/cadence/phy-cadence-sie= rra.o After: =3D=3D=3D=3D=3D text data bss dec hex filename 60521 4310 16 64847 fd4f drivers/phy/cadence/phy-cadence-sie= rra.o Signed-off-by: Christophe JAILLET --- Compile tested-only. --- drivers/phy/cadence/phy-cadence-sierra.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/phy/cadence/phy-cadence-sierra.c b/drivers/phy/cadence= /phy-cadence-sierra.c index d4eb93ce8232..6bb0ca3443f1 100644 --- a/drivers/phy/cadence/phy-cadence-sierra.c +++ b/drivers/phy/cadence/phy-cadence-sierra.c @@ -1544,7 +1544,7 @@ static void cdns_sierra_phy_remove(struct platform_de= vice *pdev) } =20 /* SGMII PHY PMA lane configuration */ -static struct cdns_reg_pairs sgmii_phy_pma_ln_regs[] =3D { +static const struct cdns_reg_pairs sgmii_phy_pma_ln_regs[] =3D { {0x9010, SIERRA_PHY_PMA_XCVR_CTRL} }; =20 @@ -1609,7 +1609,7 @@ static struct cdns_sierra_vals sgmii_100_no_ssc_plllc= 1_opt3_ln_vals =3D { }; =20 /* QSGMII PHY PMA lane configuration */ -static struct cdns_reg_pairs qsgmii_phy_pma_ln_regs[] =3D { +static const struct cdns_reg_pairs qsgmii_phy_pma_ln_regs[] =3D { {0x9010, SIERRA_PHY_PMA_XCVR_CTRL} }; =20 @@ -1675,7 +1675,7 @@ static struct cdns_sierra_vals qsgmii_100_no_ssc_plll= c1_ln_vals =3D { }; =20 /* PCIE PHY PCS common configuration */ -static struct cdns_reg_pairs pcie_phy_pcs_cmn_regs[] =3D { +static const struct cdns_reg_pairs pcie_phy_pcs_cmn_regs[] =3D { {0x0430, SIERRA_PHY_PIPE_CMN_CTRL1} }; =20 --=20 2.45.2 From nobody Wed Dec 17 08:44:11 2025 Received: from msa.smtpout.orange.fr (msa-208.smtpout.orange.fr [193.252.23.208]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BE2C716F82F for ; Thu, 11 Jul 2024 19:35:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=193.252.23.208 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726511; cv=none; b=stPm4lO3Uzrx1+JW6zx/Mgk0PNGqwRDIxl3nZiYn514YJLQadXQIj8qX89MLk3uy7dNEi5jhOvggV44VoaFhHxfMR8ITMEQZHCn8gNqTndGO5eu7gH2EPNCAWKHWZmeiwcGkPHVD8cWH1KBrozHvWp+qvb80VTRbKiY0qTy4qHU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726511; c=relaxed/simple; bh=eP8D2AlULiplo11sxaqVpd06Seg2ZUNBL6SsUgrO1NI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=HLLGB4Z6Bun++9HMeaZvS6ycOKoSdYa5KqNu/2OwaEv5LN47VXNYeI/GuBwy+cwU70SrZxa1ScnPuT01W59HYESyeJ3Xzw8UERKBxw6kjBWQUPfZAm9QgfHangZHLbQ5+csZREuelkTmGYTebzyf7GQdSlZsAw8OvlumlAewTGg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr; spf=pass smtp.mailfrom=wanadoo.fr; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b=JvB3Mkxq; arc=none smtp.client-ip=193.252.23.208 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b="JvB3Mkxq" Received: from fedora.. ([77.131.3.214]) by smtp.orange.fr with ESMTPA id RzZBsCMFrj4pfRzZGsIiRu; Thu, 11 Jul 2024 21:35:02 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wanadoo.fr; s=t20230301; t=1720726502; bh=dRpz9tTr7QVvo4jsxN8ZGItmvHwQfohDQmxginrSsQ8=; h=From:To:Subject:Date:Message-ID:MIME-Version; b=JvB3Mkxqfq/Pfy0T3tXD7mgqhA5FuqwkjqyumSBu0lgO/lY8fivaCu31HACrQ20YU P3tfrUqTh6fM8pKdeV2JQ5gxTdgj9Go4/wrrrZ5TL1NYHxIGxCDY6wP7kWvS7eIifz 5oMdO0+rByWN8ZsIJR9GsXmX5zjN3YC/LAbKFFwkF5ZMWJmQdzh96XS+8qpOPRK4zo 1QEQTW2/DAPSw2hprvNje09kutnRZOEHPe3xQZQprH3uCEWGG4Hc+ql1VufPaCiruu sUlhEsieKEBD9PincvVNIGubprXECVnkVmnt/E1ZHd8WwPk16NEsnxNEX9KG/3ZOAI wi9OpXxg1SMyA== X-ME-Helo: fedora.. X-ME-Auth: Y2hyaXN0b3BoZS5qYWlsbGV0QHdhbmFkb28uZnI= X-ME-Date: Thu, 11 Jul 2024 21:35:02 +0200 X-ME-IP: 77.131.3.214 From: Christophe JAILLET To: vkoul@kernel.org, kishon@kernel.org Cc: linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET Subject: [PATCH 2/3] phy: cadence: Sierra: Constify struct cdns_sierra_vals Date: Thu, 11 Jul 2024 21:34:53 +0200 Message-ID: <0ccfd259fdf40475c89cfb22a64c6388e7c646b4.1720723132.git.christophe.jaillet@wanadoo.fr> X-Mailer: git-send-email 2.45.2 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" 'struct cdns_sierra_vals' is not modified in this driver. Constifying this structure moves some data to a read-only section, so increase overall security. On a x86_64, with allmodconfig: Before: =3D=3D=3D=3D=3D=3D text data bss dec hex filename 60521 4310 16 64847 fd4f drivers/phy/cadence/phy-cadence-sie= rra.o After: =3D=3D=3D=3D=3D text data bss dec hex filename 60937 3894 16 64847 fd4f drivers/phy/cadence/phy-cadence-sie= rra.o Signed-off-by: Christophe JAILLET --- Compile tested-only. --- drivers/phy/cadence/phy-cadence-sierra.c | 80 ++++++++++++------------ 1 file changed, 40 insertions(+), 40 deletions(-) diff --git a/drivers/phy/cadence/phy-cadence-sierra.c b/drivers/phy/cadence= /phy-cadence-sierra.c index 6bb0ca3443f1..947492c8f381 100644 --- a/drivers/phy/cadence/phy-cadence-sierra.c +++ b/drivers/phy/cadence/phy-cadence-sierra.c @@ -362,14 +362,14 @@ struct cdns_sierra_data { u32 id_value; u8 block_offset_shift; u8 reg_offset_shift; - struct cdns_sierra_vals *pcs_cmn_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] - [NUM_SSC_MODE]; - struct cdns_sierra_vals *phy_pma_ln_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] - [NUM_SSC_MODE]; - struct cdns_sierra_vals *pma_cmn_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] - [NUM_SSC_MODE]; - struct cdns_sierra_vals *pma_ln_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] - [NUM_SSC_MODE]; + const struct cdns_sierra_vals *pcs_cmn_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] + [NUM_SSC_MODE]; + const struct cdns_sierra_vals *phy_pma_ln_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] + [NUM_SSC_MODE]; + const struct cdns_sierra_vals *pma_cmn_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] + [NUM_SSC_MODE]; + const struct cdns_sierra_vals *pma_ln_vals[NUM_PHY_TYPE][NUM_PHY_TYPE] + [NUM_SSC_MODE]; }; =20 struct cdns_regmap_cdb_context { @@ -539,12 +539,12 @@ static int cdns_sierra_phy_init(struct phy *gphy) struct cdns_sierra_inst *ins =3D phy_get_drvdata(gphy); struct cdns_sierra_phy *phy =3D dev_get_drvdata(gphy->dev.parent); const struct cdns_sierra_data *init_data =3D phy->init_data; - struct cdns_sierra_vals *pma_cmn_vals, *pma_ln_vals; + const struct cdns_sierra_vals *pma_cmn_vals, *pma_ln_vals; enum cdns_sierra_phy_type phy_type =3D ins->phy_type; + const struct cdns_sierra_vals *phy_pma_ln_vals; enum cdns_sierra_ssc_mode ssc =3D ins->ssc_mode; - struct cdns_sierra_vals *phy_pma_ln_vals; + const struct cdns_sierra_vals *pcs_cmn_vals; const struct cdns_reg_pairs *reg_pairs; - struct cdns_sierra_vals *pcs_cmn_vals; struct regmap *regmap; u32 num_regs; int i, j; @@ -1244,12 +1244,12 @@ static int cdns_sierra_phy_get_resets(struct cdns_s= ierra_phy *sp, =20 static int cdns_sierra_phy_configure_multilink(struct cdns_sierra_phy *sp) { + const struct cdns_sierra_vals *pma_cmn_vals, *pma_ln_vals; const struct cdns_sierra_data *init_data =3D sp->init_data; - struct cdns_sierra_vals *pma_cmn_vals, *pma_ln_vals; + const struct cdns_sierra_vals *phy_pma_ln_vals; + const struct cdns_sierra_vals *pcs_cmn_vals; enum cdns_sierra_phy_type phy_t1, phy_t2; - struct cdns_sierra_vals *phy_pma_ln_vals; const struct cdns_reg_pairs *reg_pairs; - struct cdns_sierra_vals *pcs_cmn_vals; int i, j, node, mlane, num_lanes, ret; enum cdns_sierra_ssc_mode ssc; struct regmap *regmap; @@ -1548,7 +1548,7 @@ static const struct cdns_reg_pairs sgmii_phy_pma_ln_r= egs[] =3D { {0x9010, SIERRA_PHY_PMA_XCVR_CTRL} }; =20 -static struct cdns_sierra_vals sgmii_phy_pma_ln_vals =3D { +static const struct cdns_sierra_vals sgmii_phy_pma_ln_vals =3D { .reg_pairs =3D sgmii_phy_pma_ln_regs, .num_regs =3D ARRAY_SIZE(sgmii_phy_pma_ln_regs), }; @@ -1598,12 +1598,12 @@ static const struct cdns_reg_pairs sgmii_100_no_ssc= _plllc1_opt3_ln_regs[] =3D { {0x0002, SIERRA_RXBUFFER_RCDFECTRL_PREG} }; =20 -static struct cdns_sierra_vals sgmii_100_no_ssc_plllc1_opt3_cmn_vals =3D { +static const struct cdns_sierra_vals sgmii_100_no_ssc_plllc1_opt3_cmn_vals= =3D { .reg_pairs =3D sgmii_100_no_ssc_plllc1_opt3_cmn_regs, .num_regs =3D ARRAY_SIZE(sgmii_100_no_ssc_plllc1_opt3_cmn_regs), }; =20 -static struct cdns_sierra_vals sgmii_100_no_ssc_plllc1_opt3_ln_vals =3D { +static const struct cdns_sierra_vals sgmii_100_no_ssc_plllc1_opt3_ln_vals = =3D { .reg_pairs =3D sgmii_100_no_ssc_plllc1_opt3_ln_regs, .num_regs =3D ARRAY_SIZE(sgmii_100_no_ssc_plllc1_opt3_ln_regs), }; @@ -1613,7 +1613,7 @@ static const struct cdns_reg_pairs qsgmii_phy_pma_ln_= regs[] =3D { {0x9010, SIERRA_PHY_PMA_XCVR_CTRL} }; =20 -static struct cdns_sierra_vals qsgmii_phy_pma_ln_vals =3D { +static const struct cdns_sierra_vals qsgmii_phy_pma_ln_vals =3D { .reg_pairs =3D qsgmii_phy_pma_ln_regs, .num_regs =3D ARRAY_SIZE(qsgmii_phy_pma_ln_regs), }; @@ -1664,12 +1664,12 @@ static const struct cdns_reg_pairs qsgmii_100_no_ss= c_plllc1_ln_regs[] =3D { {0x0002, SIERRA_RXBUFFER_RCDFECTRL_PREG} }; =20 -static struct cdns_sierra_vals qsgmii_100_no_ssc_plllc1_cmn_vals =3D { +static const struct cdns_sierra_vals qsgmii_100_no_ssc_plllc1_cmn_vals =3D= { .reg_pairs =3D qsgmii_100_no_ssc_plllc1_cmn_regs, .num_regs =3D ARRAY_SIZE(qsgmii_100_no_ssc_plllc1_cmn_regs), }; =20 -static struct cdns_sierra_vals qsgmii_100_no_ssc_plllc1_ln_vals =3D { +static const struct cdns_sierra_vals qsgmii_100_no_ssc_plllc1_ln_vals =3D { .reg_pairs =3D qsgmii_100_no_ssc_plllc1_ln_regs, .num_regs =3D ARRAY_SIZE(qsgmii_100_no_ssc_plllc1_ln_regs), }; @@ -1679,7 +1679,7 @@ static const struct cdns_reg_pairs pcie_phy_pcs_cmn_r= egs[] =3D { {0x0430, SIERRA_PHY_PIPE_CMN_CTRL1} }; =20 -static struct cdns_sierra_vals pcie_phy_pcs_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_phy_pcs_cmn_vals =3D { .reg_pairs =3D pcie_phy_pcs_cmn_regs, .num_regs =3D ARRAY_SIZE(pcie_phy_pcs_cmn_regs), }; @@ -1745,12 +1745,12 @@ static const struct cdns_reg_pairs ml_pcie_100_no_s= sc_ln_regs[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_no_ssc_plllc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_no_ssc_plllc_cmn_vals =3D { .reg_pairs =3D pcie_100_no_ssc_plllc_cmn_regs, .num_regs =3D ARRAY_SIZE(pcie_100_no_ssc_plllc_cmn_regs), }; =20 -static struct cdns_sierra_vals ml_pcie_100_no_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ml_pcie_100_no_ssc_ln_vals =3D { .reg_pairs =3D ml_pcie_100_no_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ml_pcie_100_no_ssc_ln_regs), }; @@ -1810,7 +1810,7 @@ static const struct cdns_reg_pairs ti_ml_pcie_100_no_= ssc_ln_regs[] =3D { {0x0002, SIERRA_TX_RCVDET_OVRD_PREG} }; =20 -static struct cdns_sierra_vals ti_ml_pcie_100_no_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ti_ml_pcie_100_no_ssc_ln_vals =3D { .reg_pairs =3D ti_ml_pcie_100_no_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ti_ml_pcie_100_no_ssc_ln_regs), }; @@ -1886,12 +1886,12 @@ static const struct cdns_reg_pairs ml_pcie_100_int_= ssc_ln_regs[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_int_ssc_plllc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_int_ssc_plllc_cmn_vals =3D { .reg_pairs =3D pcie_100_int_ssc_plllc_cmn_regs, .num_regs =3D ARRAY_SIZE(pcie_100_int_ssc_plllc_cmn_regs), }; =20 -static struct cdns_sierra_vals ml_pcie_100_int_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ml_pcie_100_int_ssc_ln_vals =3D { .reg_pairs =3D ml_pcie_100_int_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ml_pcie_100_int_ssc_ln_regs), }; @@ -1954,7 +1954,7 @@ static const struct cdns_reg_pairs ti_ml_pcie_100_int= _ssc_ln_regs[] =3D { {0x0002, SIERRA_TX_RCVDET_OVRD_PREG} }; =20 -static struct cdns_sierra_vals ti_ml_pcie_100_int_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ti_ml_pcie_100_int_ssc_ln_vals =3D { .reg_pairs =3D ti_ml_pcie_100_int_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ti_ml_pcie_100_int_ssc_ln_regs), }; @@ -2024,12 +2024,12 @@ static const struct cdns_reg_pairs ml_pcie_100_ext_= ssc_ln_regs[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_ext_ssc_plllc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_ext_ssc_plllc_cmn_vals =3D { .reg_pairs =3D pcie_100_ext_ssc_plllc_cmn_regs, .num_regs =3D ARRAY_SIZE(pcie_100_ext_ssc_plllc_cmn_regs), }; =20 -static struct cdns_sierra_vals ml_pcie_100_ext_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ml_pcie_100_ext_ssc_ln_vals =3D { .reg_pairs =3D ml_pcie_100_ext_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ml_pcie_100_ext_ssc_ln_regs), }; @@ -2092,7 +2092,7 @@ static const struct cdns_reg_pairs ti_ml_pcie_100_ext= _ssc_ln_regs[] =3D { {0x0002, SIERRA_TX_RCVDET_OVRD_PREG} }; =20 -static struct cdns_sierra_vals ti_ml_pcie_100_ext_ssc_ln_vals =3D { +static const struct cdns_sierra_vals ti_ml_pcie_100_ext_ssc_ln_vals =3D { .reg_pairs =3D ti_ml_pcie_100_ext_ssc_ln_regs, .num_regs =3D ARRAY_SIZE(ti_ml_pcie_100_ext_ssc_ln_regs), }; @@ -2152,12 +2152,12 @@ static const struct cdns_reg_pairs cdns_pcie_ln_reg= s_no_ssc[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_no_ssc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_no_ssc_cmn_vals =3D { .reg_pairs =3D cdns_pcie_cmn_regs_no_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_cmn_regs_no_ssc), }; =20 -static struct cdns_sierra_vals pcie_100_no_ssc_ln_vals =3D { +static const struct cdns_sierra_vals pcie_100_no_ssc_ln_vals =3D { .reg_pairs =3D cdns_pcie_ln_regs_no_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_ln_regs_no_ssc), }; @@ -2227,12 +2227,12 @@ static const struct cdns_reg_pairs cdns_pcie_ln_reg= s_int_ssc[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_int_ssc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_int_ssc_cmn_vals =3D { .reg_pairs =3D cdns_pcie_cmn_regs_int_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_cmn_regs_int_ssc), }; =20 -static struct cdns_sierra_vals pcie_100_int_ssc_ln_vals =3D { +static const struct cdns_sierra_vals pcie_100_int_ssc_ln_vals =3D { .reg_pairs =3D cdns_pcie_ln_regs_int_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_ln_regs_int_ssc), }; @@ -2296,12 +2296,12 @@ static const struct cdns_reg_pairs cdns_pcie_ln_reg= s_ext_ssc[] =3D { {0x4432, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals pcie_100_ext_ssc_cmn_vals =3D { +static const struct cdns_sierra_vals pcie_100_ext_ssc_cmn_vals =3D { .reg_pairs =3D cdns_pcie_cmn_regs_ext_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_cmn_regs_ext_ssc), }; =20 -static struct cdns_sierra_vals pcie_100_ext_ssc_ln_vals =3D { +static const struct cdns_sierra_vals pcie_100_ext_ssc_ln_vals =3D { .reg_pairs =3D cdns_pcie_ln_regs_ext_ssc, .num_regs =3D ARRAY_SIZE(cdns_pcie_ln_regs_ext_ssc), }; @@ -2413,12 +2413,12 @@ static const struct cdns_reg_pairs cdns_usb_ln_regs= _ext_ssc[] =3D { {0x4243, SIERRA_RXBUFFER_DFECTRL_PREG} }; =20 -static struct cdns_sierra_vals usb_100_ext_ssc_cmn_vals =3D { +static const struct cdns_sierra_vals usb_100_ext_ssc_cmn_vals =3D { .reg_pairs =3D cdns_usb_cmn_regs_ext_ssc, .num_regs =3D ARRAY_SIZE(cdns_usb_cmn_regs_ext_ssc), }; =20 -static struct cdns_sierra_vals usb_100_ext_ssc_ln_vals =3D { +static const struct cdns_sierra_vals usb_100_ext_ssc_ln_vals =3D { .reg_pairs =3D cdns_usb_ln_regs_ext_ssc, .num_regs =3D ARRAY_SIZE(cdns_usb_ln_regs_ext_ssc), }; @@ -2443,7 +2443,7 @@ static const struct cdns_reg_pairs sgmii_pma_cmn_vals= [] =3D { {0x0013, SIERRA_CMN_PLLLC1_DCOCAL_CTRL_PREG}, }; =20 -static struct cdns_sierra_vals sgmii_cmn_vals =3D { +static const struct cdns_sierra_vals sgmii_cmn_vals =3D { .reg_pairs =3D sgmii_pma_cmn_vals, .num_regs =3D ARRAY_SIZE(sgmii_pma_cmn_vals), }; @@ -2489,7 +2489,7 @@ static const struct cdns_reg_pairs sgmii_ln_regs[] = =3D { {0x321F, SIERRA_CPICAL_RES_STARTCODE_MODE01_PREG}, }; =20 -static struct cdns_sierra_vals sgmii_pma_ln_vals =3D { +static const struct cdns_sierra_vals sgmii_pma_ln_vals =3D { .reg_pairs =3D sgmii_ln_regs, .num_regs =3D ARRAY_SIZE(sgmii_ln_regs), }; --=20 2.45.2 From nobody Wed Dec 17 08:44:11 2025 Received: from msa.smtpout.orange.fr (msa-211.smtpout.orange.fr [193.252.23.211]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0F6D616F0E9 for ; Thu, 11 Jul 2024 19:35:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=193.252.23.211 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726506; cv=none; b=a6AmLV8rUoiiq64bK0MnzhU1YRV6kIUNoi2i+LendAd1NO5rOaC5rcRMMhT/vuQTLD6DMaBgrCVJ6RWbF72TuaH95+OmyPcrBbY5GxEwDyuo2rc8k8uaUJN3A/jpEMjSYQq/eWXrU2Jg1TI/2W8J9rDhKNAtRMB0iRr02epLM4E= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720726506; c=relaxed/simple; bh=x68ESisQH80WW9Hmoe5cxe8M7THJVffjZhGe8eKvGQw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=FAHd6m71jGkd18WJyJUXEyAlejWzW0YUgQnQGpO1xbq6apKMTqomlyIL3OkOCVcpG5KgCkJbYr/eg2HrwwVkRkM2/Q92fOjGhMWu4JHJj87/ZsHcOUrrxjeShcsmWbWQze/P0ZV+W8dNDC1qxmFFY90+iUAGta4ZiMFihlEFR+Y= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr; spf=pass smtp.mailfrom=wanadoo.fr; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b=G4FCmE10; arc=none smtp.client-ip=193.252.23.211 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=wanadoo.fr Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=wanadoo.fr header.i=@wanadoo.fr header.b="G4FCmE10" Received: from fedora.. ([77.131.3.214]) by smtp.orange.fr with ESMTPA id RzZBsCMFrj4pfRzZHsIiSB; Thu, 11 Jul 2024 21:35:03 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wanadoo.fr; s=t20230301; t=1720726503; bh=+R7FMy9AXoamPmQBHl0PouZlAVVAloz2ln5nMlCPFco=; h=From:To:Subject:Date:Message-ID:MIME-Version; b=G4FCmE10keN47ZT0oy7vcEVw+P05COp971q569t5kqgym7lD3ENBPU+VwI9cntx/z 8dSrndaG7iv76mVM2IFztFqCb2qAkQxgXmkP4zqZVUAIeEufn8LHOKY7roFiDPX8cr zPjaAm3PLkMLfkHAH3XnOLIgptNnUnIv0BkilwdDbOVmaT5eQQx1MxFzDu+0CrEfKI 3ophi6NDVNA9LE02ljoqzR2wTMy+rbdO79oU5RB+06pGlNFUDN1ioCU5p/fjoXYmdh LuQgx9ZgX2eK+x9d40YpbLS9p8SyhhG/1dgwZqNY3FfttuZr8FXZTesBvHN1JQp3be I+aJFk5GhtzRw== X-ME-Helo: fedora.. X-ME-Auth: Y2hyaXN0b3BoZS5qYWlsbGV0QHdhbmFkb28uZnI= X-ME-Date: Thu, 11 Jul 2024 21:35:03 +0200 X-ME-IP: 77.131.3.214 From: Christophe JAILLET To: vkoul@kernel.org, kishon@kernel.org Cc: linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET Subject: [PATCH 3/3] phy: cadence: Sierra: Constify a u32[] Date: Thu, 11 Jul 2024 21:34:54 +0200 Message-ID: <43d881d52e3c1632ad197d4c2c18b6c481a13b24.1720723132.git.christophe.jaillet@wanadoo.fr> X-Mailer: git-send-email 2.45.2 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" '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: =3D=3D=3D=3D=3D=3D text data bss dec hex filename 60937 3894 16 64847 fd4f drivers/phy/cadence/phy-cadence-sie= rra.o After: =3D=3D=3D=3D=3D text data bss dec hex filename 60897 3878 16 64791 fd17 drivers/phy/cadence/phy-cadence-sie= rra.o Signed-off-by: Christophe JAILLET --- Compile tested-only. --- drivers/phy/cadence/phy-cadence-sierra.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/phy/cadence/phy-cadence-sierra.c b/drivers/phy/cadence= /phy-cadence-sierra.c index 947492c8f381..3010c9f24136 100644 --- a/drivers/phy/cadence/phy-cadence-sierra.c +++ b/drivers/phy/cadence/phy-cadence-sierra.c @@ -310,7 +310,7 @@ static const struct clk_parent_data pll_mux_parent_data= [][SIERRA_NUM_CMN_PLLC_PA }, }; =20 -static u32 cdns_sierra_pll_mux_table[][SIERRA_NUM_CMN_PLLC_PARENTS] =3D { +static const u32 cdns_sierra_pll_mux_table[][SIERRA_NUM_CMN_PLLC_PARENTS] = =3D { [CMN_PLLLC] =3D { 0, 1 }, [CMN_PLLLC1] =3D { 1, 0 }, }; --=20 2.45.2