]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
iio: pressure: mpl3115: use = { } instead of memset()
authorDavid Lechner <dlechner@baylibre.com>
Wed, 11 Jun 2025 22:39:16 +0000 (17:39 -0500)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Thu, 26 Jun 2025 18:32:57 +0000 (19:32 +0100)
Use { } instead of memset() to zero-initialize stack memory to simplify
the code.

Signed-off-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Nuno Sá <nuno.sa@analog.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20250611-iio-zero-init-stack-with-instead-of-memset-v1-24-ebb2d0a24302@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/pressure/mpl3115.c

index d6715997f13727e22346a76819cb7dc58a96614e..579da60ef441482a82f3bdf4f67436a13ab3e09d 100644 (file)
@@ -160,7 +160,7 @@ static irqreturn_t mpl3115_trigger_handler(int irq, void *p)
         * of the buffer may be either 16 or 32-bits.  As such we cannot
         * use a simple structure definition to express this data layout.
         */
-       u8 buffer[16] __aligned(8);
+       u8 buffer[16] __aligned(8) = { };
        int ret, pos = 0;
 
        mutex_lock(&data->lock);
@@ -170,7 +170,6 @@ static irqreturn_t mpl3115_trigger_handler(int irq, void *p)
                goto done;
        }
 
-       memset(buffer, 0, sizeof(buffer));
        if (test_bit(0, indio_dev->active_scan_mask)) {
                ret = i2c_smbus_read_i2c_block_data(data->client,
                        MPL3115_OUT_PRESS, 3, &buffer[pos]);