[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<FR2PPF4571F02BCFAD85EB34A3830AF75708C57A@FR2PPF4571F02BC.DEUP281.PROD.OUTLOOK.COM>
Date: Tue, 15 Jul 2025 09:18:53 +0000
From: Remi Buisson <Remi.Buisson@....com>
To: Andy Shevchenko <andriy.shevchenko@...el.com>,
kernel test robot
<lkp@...el.com>
CC: Remi Buisson via B4 Relay <devnull+remi.buisson.tdk.com@...nel.org>,
Jonathan Cameron <jic23@...nel.org>,
David Lechner <dlechner@...libre.com>,
Nuno Sá <nuno.sa@...log.com>,
Andy Shevchenko
<andy@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski
<krzk@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
"oe-kbuild-all@...ts.linux.dev" <oe-kbuild-all@...ts.linux.dev>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: RE: [PATCH v2 6/8] iio: imu: inv_icm45600: add SPI driver for
inv_icm45600 driver
>
>
>From: Andy Shevchenko mailto:andriy.shevchenko@...el.com
>Sent: Friday, July 11, 2025 8:10 AM
>To: kernel test robot mailto:lkp@...el.com
>Cc: Remi Buisson via B4 Relay mailto:devnull+remi.buisson.tdk.com@...nel.org; Jonathan Cameron mailto:jic23@...nel.org; David Lechner mailto:dlechner@...libre.com; Nuno Sá mailto:nuno.sa@...log.com; Andy Shevchenko mailto:andy@...nel.org; Rob Herring mailto:robh@...nel.org; Krzysztof Kozlowski mailto:krzk@...nel.org; Conor Dooley mailto:conor+dt@...nel.org; mailto:oe-kbuild-all@...ts.linux.dev; mailto:linux-kernel@...r.kernel.org; mailto:linux-iio@...r.kernel.org; mailto:devicetree@...r.kernel.org; Remi Buisson mailto:Remi.Buisson@....com
>Subject: Re: [PATCH v2 6/8] iio: imu: inv_icm45600: add SPI driver for inv_icm45600 driver
>On Fri, Jul 11, 2025 at 12:55:25PM +0800, kernel test robot wrote:
>> Hi Remi,
>>
>
>> drivers/iio/imu/inv_icm45600/inv_icm45600_accel.c:100:20: note: initialize the variable 'sleep' to silence this warning
>> 100 | unsigned int sleep;
>> | ^
>> | = 0
>
>For the record, this is usually bad advice by the compiler. You need to either
>check for errors, or do something else.
Thanks, I'll follow your recommendation.
>
>> drivers/iio/imu/inv_icm45600/inv_icm45600_gyro.c:100:20: note: initialize the variable 'sleep' to silence this warning
>> 100 | unsigned int sleep;
>> | ^
>> | = 0
>
>Ditto.
>
>--
>With Best Regards,
>Andy Shevchenko
>
>
>
Powered by blists - more mailing lists