[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<TY3PR01MB1134699387B914674E72B33F3869BA@TY3PR01MB11346.jpnprd01.prod.outlook.com>
Date: Tue, 3 Feb 2026 17:10:29 +0000
From: Biju Das <biju.das.jz@...renesas.com>
To: "Rob Herring (Arm)" <robh@...nel.org>, biju.das.au <biju.das.au@...il.com>
CC: Paolo Abeni <pabeni@...hat.com>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>, Geert Uytterhoeven
<geert+renesas@...der.be>, Eric Dumazet <edumazet@...gle.com>, "David S.
Miller" <davem@...emloft.net>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>, "linux-renesas-soc@...r.kernel.org"
<linux-renesas-soc@...r.kernel.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, Prabhakar Mahadev Lad
<prabhakar.mahadev-lad.rj@...renesas.com>, Andrew Lunn
<andrew+netdev@...n.ch>, Stefan Eichenberger <eichest@...il.com>, Jakub
Kicinski <kuba@...nel.org>, Conor Dooley <conor+dt@...nel.org>
Subject: RE: [PATCH net-next] dt-bindings: net: micrel: Fix dtbs compatible
too long warnings
Hi Rob,
> -----Original Message-----
> From: Rob Herring (Arm) <robh@...nel.org>
> Sent: 03 February 2026 14:47
> Subject: Re: [PATCH net-next] dt-bindings: net: micrel: Fix dtbs compatible too long warnings
>
>
> On Tue, 03 Feb 2026 12:17:21 +0000, Biju wrote:
> > From: Biju Das <biju.das.jz@...renesas.com>
> >
> > The KSZ9131 PHY is suitable for IEEE 802.3 applications.
> >
> > Fix the below dtbs warning "compatible: ['ethernet-phy-id0022.1640',
> > 'ethernet-phy-ieee802.3-c22'] is too long" on the below dts:
> >
> > r8a774b1-beacon-rzg2n-kit.dtb
> > r8a774a1-beacon-rzg2m-kit.dtb
> > r8a774e1-beacon-rzg2h-kit.dtb
> > r9a08g046l48-smarc.dtb
> > r9a07g043u11-smarc.dtb
> > r9a07g044c2-smarc.dtb
> > r9a07g044l2-smarc.dtb
> > r9a07g044l2-smarc.dtb
> > r9a07g054l2-smarc.dtb
> > r9a07g054l2-smarc.dtb
> > r9a09g047e57-smarc.dtb
> > r9a09g047e57-smarc.dtb
> > r9a09g056n48-rzv2n-evk.dtb
> > r9a09g056n48-rzv2n-evk.dtb
> > r9a09g057h44-rzv2h-evk.dtb
> > r9a09g057h44-rzv2h-evk.dtb
> > r9a07g043u11-smarc-cru-csi-ov5645.dtb
> > r9a07g043u11-smarc-pmod.dtb
> > r9a07g043u11-smarc-du-adv7513.dtb
> > r9a09g047e57-smarc-cru-csi-ov5645.dtb
> > r9a07g044c2-smarc-cru-csi-ov5645.dtb
> > r9a09g047e57-smarc-cru-csi-ov5645.dtb
> > r9a07g044l2-smarc-cru-csi-ov5645.dtb
> > r9a07g044l2-smarc-cru-csi-ov5645.dtb
> > r9a07g054l2-smarc-cru-csi-ov5645.dtb
> > r9a07g054l2-smarc-cru-csi-ov5645.dtb
> > r9a09g056n48-rzv2n-evk-cn15-emmc.dtb
> > r9a09g056n48-rzv2n-evk-cn15-emmc.dtb
> > r9a09g056n48-rzv2n-evk-cn15-sd.dtb
> > r9a09g057h44-rzv2h-evk-cn15-sd.dtb
> > r9a09g056n48-rzv2n-evk-cn15-sd.dtb
> > r9a09g057h44-rzv2h-evk-cn15-emmc.dtb
> > r9a09g057h44-rzv2h-evk-cn15-sd.dtb
> > r9a09g057h44-rzv2h-evk-cn15-emmc.dtb
> >
> > Signed-off-by: Biju Das <biju.das.jz@...renesas.com>
> > ---
> > .../bindings/net/micrel,gigabit.yaml | 23 +++++++++++--------
> > 1 file changed, 14 insertions(+), 9 deletions(-)
> >
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/mscc-phy-
> vsc8531.example.dtb: ethernet-phy@0 (ethernet-phy-id0007.0772): compatible: 'oneOf' conditional
> failed, one must be fixed:
> ['ethernet-phy-id0007.0772', 'ethernet-phy-ieee802.3-c22'] is too long
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-id0007.0772' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/mscc-phy-
> vsc8531.example.dtb: ethernet-phy@0 (ethernet-phy-id0007.0772): Unevaluated properties are not allowed
> ('load-save-gpios', 'vsc8531,edge-slowdown', 'vsc8531,led-0-mode', 'vsc8531,led-1-mode',
> 'vsc8531,vddmac' were unexpected)
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
The above bot warnings are not related to micrel PHY.
Can you please check?
Cheers,
Biju
> /builds/robherring/dt-review-
> ci/linux/Documentation/devicetree/bindings/net/faraday,ftgmac100.example.dtb: ethernet-phy@1
> (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed, one must be fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/qcom,ethqos.example.dtb:
> phy@4 (ethernet-phy-ieee802.3-c22): $nodename:0: 'phy@4' does not match '^ethernet-phy(@[a-f0-9]+)?$'
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/qcom,ethqos.example.dtb:
> phy@4 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed, one must be fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/qcom,ethqos.example.dtb:
> phy@4 (ethernet-phy-ieee802.3-c22): Unevaluated properties are not allowed ('#phy-cells',
> 'device_type', 'reg' were unexpected)
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/brcm,unimac-
> mdio.example.dtb: ethernet-phy@0 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed,
> one must be fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-
> ci/linux/Documentation/devicetree/bindings/net/renesas,etheravb.example.dtb: ethernet-phy@0 (ethernet-
> phy-id0022.1622): compatible: 'oneOf' conditional failed, one must be fixed:
> ['ethernet-phy-id0022.1622', 'ethernet-phy-ieee802.3-c22'] is too long
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-id0022.1622' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-
> ci/linux/Documentation/devicetree/bindings/net/renesas,etheravb.example.dtb: ethernet-phy@0 (ethernet-
> phy-id0022.1622): Unevaluated properties are not allowed ('rxc-skew-ps' was unexpected)
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/socionext,synquacer-
> netsec.example.dtb: ethernet-phy@1 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional
> failed, one must be fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/amlogic,g12a-mdio-
> mux.example.dtb: ethernet-phy@8 (ethernet-phy-id0180.3301): compatible: 'oneOf' conditional failed,
> one must be fixed:
> ['ethernet-phy-id0180.3301', 'ethernet-phy-ieee802.3-c22'] is too long
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-id0180.3301' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/aspeed,ast2600-
> mdio.example.dtb: ethernet-phy@0 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed,
> one must be fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/fsl,fec.example.dtb:
> ethernet-phy@0 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed, one must be
> fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/brcm,bcmgenet.example.dtb:
> ethernet-phy@1 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed, one must be
> fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/brcm,bcmgenet.example.dtb:
> ethernet-phy@0 (ethernet-phy-ieee802.3-c22): compatible: 'oneOf' conditional failed, one must be
> fixed:
> ['ethernet-phy-ieee802.3-c22'] is too short
> 'ethernet-phy-id0022.1640' was expected
> 'ethernet-phy-ieee802.3-c22' is not one of ['ethernet-phy-id0022.1610', 'ethernet-phy-
> id0022.1611', 'ethernet-phy-id0022.1620', 'ethernet-phy-id0022.1631', 'ethernet-phy-id0022.1640',
> 'ethernet-phy-id0022.1650', 'ethernet-phy-id0022.1660', 'ethernet-phy-id0022.1670']
> from schema $id: http://devicetree.org/schemas/net/micrel,gigabit.yaml
>
> doc reference errors (make refcheckdocs):
>
> See https://patchwork.kernel.org/project/devicetree/patch/20260203121723.312336-1-
> biju.das.jz@...renesas.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.
Powered by blists - more mailing lists