Message ID | 20221115231021.2389-1-nm@ti.com |
---|---|
State | New |
Headers | show |
Series | dt-bindings: gpio: gpio-davinci: Increase maxItems in gpio-line-names | expand |
On 16/11/2022 00:10, Nishanth Menon wrote: > gpio-line-names really depends on ti,ngpios. However, the maximum value > we have seen across the board is on K2G and da850 platforms where it can > be upto 144. > > Link: https://lore.kernel.org/linux-arm-kernel/20221115200357.qa2rvw3clbz7unzq@symptom/T/#u > Fixes: c830b87a761b ("dt-bindings: gpio: gpio-davinci: Convert to json-schema") > Reported-by: Robert Nelson <robertcnelson@gmail.com> > Signed-off-by: Nishanth Menon <nm@ti.com> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Wed, Nov 16, 2022 at 12:10 AM Nishanth Menon <nm@ti.com> wrote: > > gpio-line-names really depends on ti,ngpios. However, the maximum value > we have seen across the board is on K2G and da850 platforms where it can > be upto 144. > > Link: https://lore.kernel.org/linux-arm-kernel/20221115200357.qa2rvw3clbz7unzq@symptom/T/#u > Fixes: c830b87a761b ("dt-bindings: gpio: gpio-davinci: Convert to json-schema") > Reported-by: Robert Nelson <robertcnelson@gmail.com> > Signed-off-by: Nishanth Menon <nm@ti.com> > --- > > I dont see a practical scheme to set the maxItems based on ti,ngpios, > but deleting the maxItems was softening the check as well. > > Documentation/devicetree/bindings/gpio/gpio-davinci.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml b/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml > index f32e09ef937c..10e56cf306db 100644 > --- a/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml > +++ b/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml > @@ -35,7 +35,7 @@ properties: > gpio-line-names: > description: strings describing the names of each gpio line. > minItems: 1 > - maxItems: 100 > + maxItems: 144 > > "#gpio-cells": > const: 2 > -- > 2.31.1 > Applied, thanks! Bart
diff --git a/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml b/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml index f32e09ef937c..10e56cf306db 100644 --- a/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml +++ b/Documentation/devicetree/bindings/gpio/gpio-davinci.yaml @@ -35,7 +35,7 @@ properties: gpio-line-names: description: strings describing the names of each gpio line. minItems: 1 - maxItems: 100 + maxItems: 144 "#gpio-cells": const: 2
gpio-line-names really depends on ti,ngpios. However, the maximum value we have seen across the board is on K2G and da850 platforms where it can be upto 144. Link: https://lore.kernel.org/linux-arm-kernel/20221115200357.qa2rvw3clbz7unzq@symptom/T/#u Fixes: c830b87a761b ("dt-bindings: gpio: gpio-davinci: Convert to json-schema") Reported-by: Robert Nelson <robertcnelson@gmail.com> Signed-off-by: Nishanth Menon <nm@ti.com> --- I dont see a practical scheme to set the maxItems based on ti,ngpios, but deleting the maxItems was softening the check as well. Documentation/devicetree/bindings/gpio/gpio-davinci.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)