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

[Antonio Borneo <antonio.borneo@foss.st.com>]

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@foss.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@foss.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