Re: [PATCH v6 1/1] dt-bindings: pinctrl: Update pinctrl-single to use yaml

[Tony Lindgren <tony@xxxxxxxxxxx>]

* Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> [230607 18:17]:
> On 05/06/2023 11:52, Tony Lindgren wrote:
> > +title: Pinctrl driver for hardware with a single register for one or more pins
> 
> I asked to drop the driver references but it is still here. Bindings are
> not describing drivers.
> 
> "Generic Pin Controller with a Single Register for One or More Pins"

Oh right it's supposed to describe hardware, will update thanks.

> > +patternProperties:
> > +  '-pins$|-pin':
> 
> you did not implement my comments fully, probably we misunderstood each
> other. Why do you allow anything after '-pin'? Let's make it pure suffix
> for both cases: '-pins?$'

I'll check what kind of node renaming that would cause. At least TI
arm64 SoCs use naming like -pins-default and -pins-wakeup. Is your
preference to rename all those nodes to -default-pins and -wakeup-pins?

Regards,

Tony