linux-kernel - Re: [PATCH v7 7/8] iio: dac: ad3552r: add high-speed platform driver

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: <20241026183502.40662850@jic23-huawei>
Date: Sat, 26 Oct 2024 18:35:02 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: David Lechner <dlechner@...libre.com>
Cc: Angelo Dureghello <adureghello@...libre.com>, Nuno Sá
 <noname.nuno@...il.com>, Nuno Sá <nuno.sa@...log.com>,
 Lars-Peter Clausen <lars@...afoo.de>, Michael Hennerich
 <Michael.Hennerich@...log.com>, Rob Herring <robh@...nel.org>, Krzysztof
 Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Olivier
 Moysan <olivier.moysan@...s.st.com>, linux-iio@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, Mark Brown
 <broonie@...nel.org>
Subject: Re: [PATCH v7 7/8] iio: dac: ad3552r: add high-speed platform
 driver

On Thu, 24 Oct 2024 10:13:25 -0500
David Lechner <dlechner@...libre.com> wrote:

> On 10/24/24 10:02 AM, Angelo Dureghello wrote:
> > Hi Nuno,
> > 
> > On 24.10.2024 15:05, Nuno Sá wrote:  
> >> On Tue, 2024-10-22 at 18:40 +0200, Angelo Dureghello wrote:  
> >>> Hi Nuno,
> >>>
> >>> On 22.10.2024 14:28, Nuno Sá wrote:  
> >>>> On Mon, 2024-10-21 at 14:40 +0200, Angelo Dureghello wrote:  
> >>>>> From: Angelo Dureghello <adureghello@...libre.com>
> >>>>>
> >>>>> Add High Speed ad3552r platform driver.  
> 
> ...
> 
> >>>>> +	switch (mask) {
> >>>>> +	case IIO_CHAN_INFO_RAW:
> >>>>> +		iio_device_claim_direct_scoped(return -EBUSY, indio_dev) {
> >>>>> +			return st->data->bus_reg_write(st->back,
> >>>>> +				    AD3552R_REG_ADDR_CH_DAC_16B(chan-  
> >>>>>> channel),  
> >>>>> +				    val, 2);
> >>>>> +		}  
> >>>>
> >>>> Maybe we'll get the new stuff in time for this :)
> >>>>  
> > This is not clear, sorry.
> >   
> 
> Probably this :-)
> 
> https://lore.kernel.org/all/20241023105757.GA9767@noisy.programming.kicks-ass.net/
> 
> But it hasn't reached the stable tree yet.

Agreed. It is going to be nice to get rid of so much ugly code.
I guess mostly next cycle however.

Jonathan