diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2025-07-29 21:15:03 +0200 |
|---|---|---|
| committer | Sudeep Holla <sudeep.holla@arm.com> | 2025-08-21 14:14:59 +0100 |
| commit | abd658d3e7332bc88ae910664c7d5f4e116d6bde (patch) | |
| tree | 4b5df3860d2e9e14b5fee7d9c9533405b05264b0 /drivers/firmware | |
| parent | dt-bindings: firmware: arm,scmi: Allow multiple instances (diff) | |
| download | linux-abd658d3e7332bc88ae910664c7d5f4e116d6bde.tar.gz linux-abd658d3e7332bc88ae910664c7d5f4e116d6bde.zip | |
firmware: arm_scmi: Constify struct scmi_voltage_proto_ops
Constify struct scmi_voltage_proto_ops since the driver never modifies
it. Placing it in read-only memory (.rodata) improves hardening,
particularly because it contains function pointers.
x86_64 (allmodconfig) size impact:
Before:
text data bss dec hex filename
13142 1808 0 14950 3a66 drivers/firmware/arm_scmi/voltage.o
After:
text data bss dec hex filename
13238 1712 0 14950 3a66 drivers/firmware/arm_scmi/voltage.o
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Message-Id: <2091660c072dd2d5599897243a5d69e89d46ed4d.1753816459.git.christophe.jaillet@wanadoo.fr>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Diffstat (limited to 'drivers/firmware')
| -rw-r--r-- | drivers/firmware/arm_scmi/voltage.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/firmware/arm_scmi/voltage.c b/drivers/firmware/arm_scmi/voltage.c index fda6a1573609..17127880e10a 100644 --- a/drivers/firmware/arm_scmi/voltage.c +++ b/drivers/firmware/arm_scmi/voltage.c @@ -393,7 +393,7 @@ static int scmi_voltage_domains_num_get(const struct scmi_protocol_handle *ph) return vinfo->num_domains; } -static struct scmi_voltage_proto_ops voltage_proto_ops = { +static const struct scmi_voltage_proto_ops voltage_proto_ops = { .num_domains_get = scmi_voltage_domains_num_get, .info_get = scmi_voltage_info_get, .config_set = scmi_voltage_config_set, |