diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2024-09-07 09:00:04 +0200 |
|---|---|---|
| committer | Iwona Winiarska <iwona.winiarska@intel.com> | 2024-10-25 10:58:07 +0200 |
| commit | 733dc978fab659dee9938739e2b9e88ce72f0408 (patch) | |
| tree | ca35c8392acb50c83cd51312f54348dddb789dca | |
| parent | 42f7652d3eb527d03665b09edac47f85fb600924 (diff) | |
| download | lwn-733dc978fab659dee9938739e2b9e88ce72f0408.tar.gz lwn-733dc978fab659dee9938739e2b9e88ce72f0408.zip | |
peci: npcm: Constify struct peci_controller_ops
'struct peci_controller_ops' is not modified in this driver.
Constifying this structure moves some data to a read-only section, so
increase overall security, especially when the structure holds some
function pointers.
On a x86_64, with allmodconfig:
Before:
======
text data bss dec hex filename
8003 784 48 8835 2283 drivers/peci/controller/peci-npcm.o
After:
=====
text data bss dec hex filename
8003 776 48 8827 227b drivers/peci/controller/peci-npcm.o
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Iwona Winiarska <iwona.winiarska@intel.com>
Link: https://lore.kernel.org/r/3c7d455745c2265c19ed02f026dfc9610271d3c2.1725692376.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Iwona Winiarska <iwona.winiarska@intel.com>
| -rw-r--r-- | drivers/peci/controller/peci-npcm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/peci/controller/peci-npcm.c b/drivers/peci/controller/peci-npcm.c index ec613d35c796..fa91be58f6f3 100644 --- a/drivers/peci/controller/peci-npcm.c +++ b/drivers/peci/controller/peci-npcm.c @@ -224,7 +224,7 @@ static const struct regmap_config npcm_peci_regmap_config = { .fast_io = true, }; -static struct peci_controller_ops npcm_ops = { +static const struct peci_controller_ops npcm_ops = { .xfer = npcm_peci_xfer, }; |