linux-kernel - Re: [PATCH v2] staging: iio: adc: ad7816: Drop unnecessary dev_set

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: <20250819192044.5637c96b@jic23-huawei>
Date: Tue, 19 Aug 2025 19:20:44 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: David Lechner <dlechner@...libre.com>
Cc: Salah Triki <salah.triki@...il.com>, Lars-Peter Clausen
 <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, Nuno
 Sá <nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-iio@...r.kernel.org,
 linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] staging: iio: adc: ad7816: Drop unnecessary
 dev_set_drvdata() call

On Tue, 19 Aug 2025 10:05:09 -0500
David Lechner <dlechner@...libre.com> wrote:

> On 8/19/25 10:00 AM, Salah Triki wrote:
> > Remove the call to `dev_set_drvdata()` and the associated comment
> > since private data is never read.
> > 
> > Signed-off-by: Salah Triki <salah.triki@...il.com>
> > ---
> > Changes in v2 :  
> 
> Reviewed-by: David Lechner <dlechner@...libre.com>

Applied, thanks!

This motivated to do my periodic can I drop this driver yet check
against my usual rule of once it's obsolete from the supplier and
no significant work on the driver (staging only I'm less keen to remove
drives in good condition!)
Unfortunately 7817 which it supports is still in production :(
I then checked the other drivers still there and they all cover production
parts as well.

This is a pretty simple device and probably a good target for someone
who wants to try their first driver. It would be a near rewrite as this
code is miles off ABI wise.

Doesn't look like there is a currently available dev board, but it's
available in a TSSOP and will run on wide range of voltages so not
a challenge for someone to give a go on a break out board.

Just thought I'd mention that and see if anyone jumped on it ;)

Jonathan

p.s. One day I'll just decide to delete all the remaining staging drivers.
     Analog folk, they are all yours!  I can't blame any of you though
     as I think the first stuff from Michael was the following year.