]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
firmware: arm_scmi: Constify struct scmi_transport_ops
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Tue, 29 Jul 2025 19:15:04 +0000 (21:15 +0200)
committerSudeep Holla <sudeep.holla@arm.com>
Thu, 21 Aug 2025 13:15:19 +0000 (14:15 +0100)
Mark struct scmi_transport_ops as const since this driver never modifies
it. Constifying moves it to read-only memory, improving hardening -
especially important for structures with function pointers.

x86_64 (allmodconfig) size diff:
Before:
   text    data     bss     dec     hex filename
   6907     680      48    7635    1dd3 drivers/firmware/arm_scmi/transports/optee.o

After:
   text    data     bss     dec     hex filename
   6987     576      48    7611    1dbb drivers/firmware/arm_scmi/transports/optee.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Message-Id: <881be6ad61037ed95d5e750a2565fd9840120a08.1753816459.git.christophe.jaillet@wanadoo.fr>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
drivers/firmware/arm_scmi/transports/optee.c

index 3949a877e17dd6bc9ab61b95af68404510032a2e..dc0f46340153a190be4977f0cb71aab2696beb0c 100644 (file)
@@ -498,7 +498,7 @@ static void scmi_optee_mark_txdone(struct scmi_chan_info *cinfo, int ret,
        mutex_unlock(&channel->mu);
 }
 
-static struct scmi_transport_ops scmi_optee_ops = {
+static const struct scmi_transport_ops scmi_optee_ops = {
        .chan_available = scmi_optee_chan_available,
        .chan_setup = scmi_optee_chan_setup,
        .chan_free = scmi_optee_chan_free,