linux-kernel - Re: [PATCH 00/10] iio: Drop unnecessary -ENOMEM messages

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250825123919.3c228ef7@jic23-huawei>
Date: Mon, 25 Aug 2025 12:39:19 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Dixit Parmar <dixitparmar19@...il.com>
Cc: David Lechner <dlechner@...libre.com>, Nuno Sá
 <nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>, Lars-Peter Clausen
 <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, Ray
 Jui <rjui@...adcom.com>, Scott Branden <sbranden@...adcom.com>, Broadcom
 internal kernel review list <bcm-kernel-feedback-list@...adcom.com>,
 Support Opensource <support.opensource@...semi.com>, Krzysztof Kozlowski
 <krzk@...nel.org>, Alim Akhtar <alim.akhtar@...sung.com>, Haibo Chen
 <haibo.chen@....com>, Shawn Guo <shawnguo@...nel.org>, Sascha Hauer
 <s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>, Cai Huoqing <cai.huoqing@...ux.dev>,
 Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue
 <alexandre.torgue@...s.st.com>, Andreas Klinger <ak@...klinger.de>, Crt
 Mori <cmo@...exis.com>, linux-iio@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-samsung-soc@...r.kernel.org, imx@...ts.linux.dev,
 linux-stm32@...md-mailman.stormreply.com
Subject: Re: [PATCH 00/10] iio: Drop unnecessary -ENOMEM messages

On Mon, 25 Aug 2025 09:31:50 +0100
Jonathan Cameron <jic23@...nel.org> wrote:

> On Fri, 22 Aug 2025 09:19:48 +0530
> Dixit Parmar <dixitparmar19@...il.com> wrote:
> 
> > The drivers do not require their own error messages for error
> > -ENOMEM, memory allocation failures. So remove the dev_err
> > messages from the probe().
> > With these patches, all the iio drivers now has uniform handling
> > of the -ENOMEM while device_allocation and trigger_allocation
> > calls.
> > 
> > Signed-off-by: Dixit Parmar <dixitparmar19@...il.com>  
> Series looks fine to me, after the minor tweaks to commit messages
> that Andy requested.  However as it touches a lot of drivers
> I'd like to leave it on list a little longer.

I changed my mind after taking the dev_err_probe() series earlier.
I'm rather too busy at the moment, so clearing this out now will reduce
what I need to keep track of.

Fixed up the () that Andy asked for in commit messages and applied
to the togreg branch of iio.git, pushed out initially as testing.

There is still the rest of the week (probably) in which I can add
tags etc if anyone wants to give them before I push that out as
a non rebasing tree.

Thanks,

Jonathan

> 
> Thanks,
> 
> Jonathan
> 
> > ---
> > Dixit Parmar (10):
> >       iio: accel: Drop unnecessary -ENOMEM messages
> >       iio: adc: Drop unnecessary -ENOMEM messages
> >       iio: dac: Drop unnecessary -ENOMEM messages
> >       iio: health: Drop unnecessary -ENOMEM messages
> >       iio: humidity: Drop unnecessary -ENOMEM messages
> >       iio: light: Drop unnecessary -ENOMEM messages
> >       iio: potentiostat: Drop unnecessary -ENOMEM messages
> >       iio: pressure: Drop unnecessary -ENOMEM messages
> >       iio: proximity: Drop unnecessary -ENOMEM messages
> >       iio: temperature: Drop unnecessary -ENOMEM messages
> > 
> >  drivers/iio/accel/bma220_spi.c      | 4 +---
> >  drivers/iio/accel/dmard06.c         | 4 +---
> >  drivers/iio/accel/dmard09.c         | 4 +---
> >  drivers/iio/accel/dmard10.c         | 4 +---
> >  drivers/iio/accel/mc3230.c          | 4 +---
> >  drivers/iio/accel/mma7660.c         | 4 +---
> >  drivers/iio/accel/stk8312.c         | 4 +---
> >  drivers/iio/accel/stk8ba50.c        | 4 +---
> >  drivers/iio/adc/ad7949.c            | 4 +---
> >  drivers/iio/adc/bcm_iproc_adc.c     | 4 +---
> >  drivers/iio/adc/cpcap-adc.c         | 6 ++----
> >  drivers/iio/adc/da9150-gpadc.c      | 5 ++---
> >  drivers/iio/adc/dln2-adc.c          | 9 +++------
> >  drivers/iio/adc/exynos_adc.c        | 4 +---
> >  drivers/iio/adc/imx7d_adc.c         | 4 +---
> >  drivers/iio/adc/imx8qxp-adc.c       | 4 +---
> >  drivers/iio/adc/mxs-lradc-adc.c     | 4 +---
> >  drivers/iio/adc/palmas_gpadc.c      | 4 +---
> >  drivers/iio/adc/rn5t618-adc.c       | 4 +---
> >  drivers/iio/adc/stm32-dfsdm-adc.c   | 4 +---
> >  drivers/iio/adc/stmpe-adc.c         | 4 +---
> >  drivers/iio/adc/ti-adc084s021.c     | 4 +---
> >  drivers/iio/adc/ti-ads131e08.c      | 8 ++------
> >  drivers/iio/adc/ti_am335x_adc.c     | 5 ++---
> >  drivers/iio/adc/twl4030-madc.c      | 4 +---
> >  drivers/iio/adc/viperboard_adc.c    | 4 +---
> >  drivers/iio/dac/ad5380.c            | 4 +---
> >  drivers/iio/dac/ad5764.c            | 4 +---
> >  drivers/iio/dac/ds4424.c            | 4 +---
> >  drivers/iio/dac/ti-dac7311.c        | 4 +---
> >  drivers/iio/dac/vf610_dac.c         | 4 +---
> >  drivers/iio/health/afe4403.c        | 4 +---
> >  drivers/iio/health/afe4404.c        | 4 +---
> >  drivers/iio/humidity/am2315.c       | 4 +---
> >  drivers/iio/humidity/dht11.c        | 4 +---
> >  drivers/iio/light/stk3310.c         | 4 +---
> >  drivers/iio/potentiostat/lmp91000.c | 4 +---
> >  drivers/iio/pressure/dlhl60d.c      | 4 +---
> >  drivers/iio/proximity/ping.c        | 4 +---
> >  drivers/iio/proximity/srf04.c       | 4 +---
> >  drivers/iio/temperature/mlx90632.c  | 4 +---
> >  41 files changed, 47 insertions(+), 130 deletions(-)
> > ---
> > base-commit: d7b8f8e20813f0179d8ef519541a3527e7661d3a
> > change-id: 20250822-enomam_logs-f66642957fb3
> > 
> > Best regards,  
> 
>