]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
iio: proximity: aw96103: Constify struct iio_info​
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Fri, 6 Dec 2024 22:07:23 +0000 (23:07 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 7 Dec 2024 17:54:21 +0000 (17:54 +0000)
'struct iio_info' 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
  17366    1454      16   18836    4994 drivers/iio/proximity/aw96103.o

After:
=====
   text    data     bss     dec     hex filename
  17526    1294      16   18836    4994 drivers/iio/proximity/aw96103.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://patch.msgid.link/da4918af46fef03903ab0e9fdcb4f23e014f3821.1733522812.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/proximity/aw96103.c

index cdd254da9e5035e3c3e062fe27982b4108cc6a45..3472a2c36e440df7afba00635d17b9a48e5e8bcf 100644 (file)
@@ -433,7 +433,7 @@ static int aw96103_write_event_config(struct iio_dev *indio_dev,
                                  state ? BIT(chan->channel) : 0);
 }
 
-static struct iio_info iio_info = {
+static const struct iio_info iio_info = {
        .read_raw = aw96103_read_raw,
        .read_event_value = aw96103_read_event_val,
        .write_event_value = aw96103_write_event_val,