Re: [PATCH v5 03/21] dt-bindings: pinctrl: Add starfive,jhb100-sys0-pinctrl

[Rob Herring <robh@kernel.org>]

On Mon, Aug 10, 2026 at 09:51:07AM +0000, Changhuang Liang wrote:
> Hi, Rob
> 
> > On Sun, 09 Aug 2026 23:36:32 -0700, Changhuang Liang wrote:
> > > Add pinctrl bindings for StarFive JHB100 SoC System-0(sys0) pinctrl
> > > controller.
> > >
> > > Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>
> > > ---
> > >  .../pinctrl/starfive,jhb100-sys0-pinctrl.yaml | 164 ++++++++++++++++++
> > >  .../pinctrl/starfive,jhb100-pinctrl.h         |  17 ++
> > >  2 files changed, 181 insertions(+)
> > >  create mode 100644
> > > Documentation/devicetree/bindings/pinctrl/starfive,jhb100-sys0-pinctrl
> > > .yaml  create mode 100644
> > > include/dt-bindings/pinctrl/starfive,jhb100-pinctrl.h
> > >
> > 
> > My bot found errors running 'make dt_binding_check' on your patch:
> > 
> > yamllint warnings/errors:
> > 
> > dtschema/dtc warnings/errors:
> > Documentation/devicetree/bindings/pinctrl/starfive,jhb100-sys0-pinctrl.exa
> > mple.dtb: pinctrl@13080000 (starfive,jhb100-sys0-pinctrl): gpio-ranges:1: [4]
> > is too short
> > 	from schema $id: http://devicetree.org/schemas/gpio/gpio.yaml
> > 
> > doc reference errors (make refcheckdocs):
> > 
> > See
> > https://patchwork.kernel.org/project/devicetree/patch/20260810063650.23
> > 9303-4-changhuang.liang@starfivetech.com
> > 
> > The base for the series is generally the latest rc1. A different dependency
> > should be noted in *this* patch.
> > 
> > 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 after running the above command yourself. Note
> > that DT_SCHEMA_FILES can be set to your schema file to speed up checking
> > your schema. However, it must be unset to test all examples with your
> > schema.
> 
> Hi Rob,
> 
> Regarding this link: https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/gpio/gpio.yaml#L71
> 
> It seems that the configuration to support #gpio-cells = <3>; has not been added here yet, is that correct?

Shrug. Based on the description, gpio-ranges doesn't depend the # of 
cells.

Why do you need 3 cells? You haven't described the cell content.

Rob