| Message ID | 20210728135534.703028-2-pgwipeout@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | fixes and enablement for rk356x | expand |
Hi Peter, Am Mittwoch, 28. Juli 2021, 15:55:26 CEST schrieb Peter Geis: > The rk356x adds a debounce clock to the gpio devices. > Increase the maximum clocks to account for it. > > Signed-off-by: Peter Geis <pgwipeout@gmail.com> that binding change is also part of Jianqun's gpio driver series, also adds clock descriptions for both and even got an Ack from Rob already, so we should be going with that other variant ;-) Heiko > --- > Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > index d993e002cebe..489a5263a7a1 100644 > --- a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > +++ b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > @@ -22,7 +22,7 @@ properties: > maxItems: 1 > > clocks: > - maxItems: 1 > + maxItems: 2 > > gpio-controller: true > >
On Wed, Jul 28, 2021 at 10:10 AM Heiko Stübner <heiko@sntech.de> wrote: > > Hi Peter, > > Am Mittwoch, 28. Juli 2021, 15:55:26 CEST schrieb Peter Geis: > > The rk356x adds a debounce clock to the gpio devices. > > Increase the maximum clocks to account for it. > > > > Signed-off-by: Peter Geis <pgwipeout@gmail.com> > > that binding change is also part of Jianqun's gpio driver series, > also adds clock descriptions for both and even got an Ack from Rob > already, so we should be going with that other variant ;-) Roger, I'll drop this for the next version, thanks! > > Heiko > > > --- > > Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > > index d993e002cebe..489a5263a7a1 100644 > > --- a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > > +++ b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml > > @@ -22,7 +22,7 @@ properties: > > maxItems: 1 > > > > clocks: > > - maxItems: 1 > > + maxItems: 2 > > > > gpio-controller: true > > > > > > > >
On Wed, 28 Jul 2021 09:55:26 -0400, Peter Geis wrote: > The rk356x adds a debounce clock to the gpio devices. > Increase the maximum clocks to account for it. > > Signed-off-by: Peter Geis <pgwipeout@gmail.com> > --- > Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.example.dt.yaml: gpio@2000a000: clocks: [[4294967295, 9]] is too short From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.example.dt.yaml: gpio@2003c000: clocks: [[4294967295, 10]] is too short From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml \ndoc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/patch/1510854 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit.
diff --git a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml index d993e002cebe..489a5263a7a1 100644 --- a/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml +++ b/Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml @@ -22,7 +22,7 @@ properties: maxItems: 1 clocks: - maxItems: 1 + maxItems: 2 gpio-controller: true
The rk356x adds a debounce clock to the gpio devices. Increase the maximum clocks to account for it. Signed-off-by: Peter Geis <pgwipeout@gmail.com> --- Documentation/devicetree/bindings/gpio/rockchip,gpio-bank.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)