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] [thread-next>] [day] [month] [year] [list]
Message-ID:
 <PH0PR03MB71412F4BD377ED5E1D6729D5F9AE2@PH0PR03MB7141.namprd03.prod.outlook.com>
Date: Thu, 3 Apr 2025 07:11:39 +0000
From: "Paller, Kim Seer" <KimSeer.Paller@...log.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
CC: Jonathan Cameron <jic23@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>,
        "Hennerich, Michael" <Michael.Hennerich@...log.com>,
        Rob Herring
	<robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley
	<conor+dt@...nel.org>,
        "linux-iio@...r.kernel.org"
	<linux-iio@...r.kernel.org>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>,
        "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>
Subject: RE: [PATCH v3 2/3] dt-bindings: iio: dac: Add adi,ad3530r.yaml



> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@...nel.org>
> Sent: Thursday, April 3, 2025 3:02 PM
> To: Paller, Kim Seer <KimSeer.Paller@...log.com>
> Cc: Jonathan Cameron <jic23@...nel.org>; Lars-Peter Clausen
> <lars@...afoo.de>; Hennerich, Michael <Michael.Hennerich@...log.com>;
> Rob Herring <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>;
> Conor Dooley <conor+dt@...nel.org>; linux-iio@...r.kernel.org; linux-
> kernel@...r.kernel.org; devicetree@...r.kernel.org
> Subject: Re: [PATCH v3 2/3] dt-bindings: iio: dac: Add adi,ad3530r.yaml
> 
> [External]
> 
> On 03/04/2025 08:59, Paller, Kim Seer wrote:
> >
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <krzk@...nel.org>
> >> Sent: Thursday, April 3, 2025 2:44 PM
> >> To: Paller, Kim Seer <KimSeer.Paller@...log.com>
> >> Cc: Jonathan Cameron <jic23@...nel.org>; Lars-Peter Clausen
> >> <lars@...afoo.de>; Hennerich, Michael <Michael.Hennerich@...log.com>;
> >> Rob Herring <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>;
> >> Conor Dooley <conor+dt@...nel.org>; linux-iio@...r.kernel.org; linux-
> >> kernel@...r.kernel.org; devicetree@...r.kernel.org
> >> Subject: Re: [PATCH v3 2/3] dt-bindings: iio: dac: Add adi,ad3530r.yaml
> >>
> >> [External]
> >>
> >> On Thu, Apr 03, 2025 at 01:33:56PM +0800, Kim Seer Paller wrote:
> >>> Document the AD3530/AD3530R (8-channel) and AD3531/AD3531R (4-
> >> channel)
> >>> low-power, 16-bit, buffered voltage output DACs with software-
> >>> programmable gain controls. They provide full-scale output spans of
> >>> 2.5V or 5V for reference voltages of 2.5V. These devices operate on a
> >>> single 2.7V to 5.5V supply and are guaranteed to be monotonic by design.
> >>> The "R" variants include a 2.5V, 5ppm/°C internal reference, which is
> >>> disabled by default.
> >>>
> >>> Signed-off-by: Kim Seer Paller <kimseer.paller@...log.com>
> >>> ---
> >>>  .../devicetree/bindings/iio/dac/adi,ad3530r.yaml   | 99
> >> ++++++++++++++++++++++
> >>>  MAINTAINERS                                        |  7 ++
> >>>  2 files changed, 106 insertions(+)
> >>
> >> You got report about error in v2, which you did not respond to.
> >>
> >> You send v3... with same error. I don't understand this.
> >
> > I'm not sure, but based on the information provided, the errors might be
> > unrelated to the driver, or I might have missed something.
> 
> It's your job to respond to errors one way or another (unless there are
> obviously false positives). Ignoring them means this is marked as
> "changes requested".

My bad, I'll take note of that. I think the bindings error reported in v3 is
still unrelated to the current changes.

> 
> Best regards,
> Krzysztof

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ