linux-kernel - Re: [PATCH v2 02/15] dt-bindings: pinctrl: stm32: use strings for enum property 'st,package'

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: <c74339727ef2d9c106129cba2de58cc20645ce9a.camel@foss.st.com>
Date: Thu, 20 Nov 2025 10:50:49 +0100
From: Antonio Borneo <antonio.borneo@...s.st.com>
To: Linus Walleij <linus.walleij@...aro.org>,
        Krzysztof Kozlowski
	<krzk+dt@...nel.org>
CC: Rob Herring <robh@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
        "Maxime
 Coquelin" <mcoquelin.stm32@...il.com>,
        Alexandre Torgue
	<alexandre.torgue@...s.st.com>,
        <linux-gpio@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-stm32@...md-mailman.stormreply.com>,
        <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
        Clément Le Goffic <legoffic.clement@...il.com>,
        "Amelie
 Delaunay" <amelie.delaunay@...s.st.com>,
        Pascal Paillet
	<p.paillet@...s.st.com>
Subject: Re: [PATCH v2 02/15] dt-bindings: pinctrl: stm32: use strings for
 enum property 'st,package'

On Wed, 2025-11-19 at 15:16 +0100, Linus Walleij wrote:
> On Tue, Nov 18, 2025 at 5:20 PM Antonio Borneo
> <antonio.borneo@...s.st.com> wrote:
> 
> > The property 'st,package' reports the SoC package used in the
> > board DT and is used to inform the driver about which pins are
> > available for use by the pinctrl driver.
> > It has historically been declared as an uint32 enum, where each
> > value is a power of 2.
> > 
> > Deprecate the use of the numeric value and replace it with more
> > readable string values.
> > 
> > Signed-off-by: Antonio Borneo <antonio.borneo@...s.st.com>
> 
> I think Krzysztof is right about this, deprecate the property and
> (re-)introduce it on the top level in:
> Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> 

Yes, I agree with Krzysztof comment. This is not a property that belongs to pinctrl.
I will deprecate it in the existing bindings.


> It may be a bit quirky to get to this property from the pin control
> Linux driver, but it is the right thing to do.
> 
> The Vexpress has some inspirational top-level custom properties:
> Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml

I was looking for some top-level custom property, but I've missed this. Thanks for the hint.

Best Regards,
Antonio