[PATCH v6 1/1] dt-bindings: pinctrl: Update pinctrl-single to use yaml
Tony Lindgren
tony at atomide.com
Wed Jun 7 23:36:39 PDT 2023
* Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org> [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
More information about the linux-arm-kernel
mailing list