linux-kernel - Re: [PATCH 1/2] dt-bindings: iio: afe: current-sense-amplifier: Add io-channel-cells

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: <b6006a1b-0b46-4e0d-9d49-6cc0dbe84814@norik.com>
Date: Thu, 4 Sep 2025 12:15:52 +0200
From: Primoz Fiser <primoz.fiser@...ik.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: Peter Rosin <peda@...ntia.se>, Jonathan Cameron <jic23@...nel.org>,
 David Lechner <dlechner@...libre.com>, Nuno Sa <nuno.sa@...log.com>,
 Andy Shevchenko <andy@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
 Sascha Hauer <s.hauer@...gutronix.de>,
 Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>, linux-iio@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
 upstream@...ts.phytec.de
Subject: Re: [PATCH 1/2] dt-bindings: iio: afe: current-sense-amplifier: Add
 io-channel-cells

Hi Krzysztof,

On 4. 09. 25 11:17, Krzysztof Kozlowski wrote:
> On Wed, Sep 03, 2025 at 01:36:59PM +0200, Primoz Fiser wrote:
>> The current-sense-amplifier is an IIO provider thus can be referenced by
>> IIO consumers (via "io-channels" property in consumer device node). Such
>> provider is required to describe number of cells used in phandle lookup
>> with "io-channel-cells" property, otherwise the following kernel error
>> is present:
>>
>>    OF: /iio-hwmon: could not get #io-channel-cells for /current-sense
> 
> Also, how can I reproduce this error? Do you paste here real errors or
> just some invented ones from some out of tree code (hint: such would not
> be relevant).

I get this error if I apply patch #2 without patch #1. See [1] for more 
details.

BR,
Primoz


[1] 
https://lore.kernel.org/all/20250903113700.3079626-1-primoz.fiser@norik.com/

> 
>>
>> Also fixes the following dtbs_check warning:
>>
>>    current-sense: '#io-channel-cells' does not match any of the regexes:
>>    'pinctrl-[0-9]+' from schema $id:
>>    http://devicetree.org/schemas/iio/afe/current-sense-amplifier.yaml#
> 
> Best regards,
> Krzysztof
>