Commit f56bcc04 authored by Frank Li's avatar Frank Li Committed by Jakub Kicinski
Browse files

dt-bindings: net: dsa: microchip: Make pinctrl 'reset' optional 

Commit e469b87e ("dt-bindings: net: dsa: microchip: Add strap
description to set SPI mode") required both 'default' and 'reset' pinctrl
states for all compatible devices. However, this requirement should be only
applicable to KSZ8463.

Make the 'reset' pinctrl state optional for all other Microchip DSA
devices while keeping it mandatory for KSZ8463.

Fix below CHECK_DTBS warnings:
  arch/arm64/boot/dts/freescale/imx8mp-skov-basic.dtb: switch@5f (microchip,ksz9893): pinctrl-names: ['default'] is too short
	from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml#



Signed-off-by: default avatarFrank Li <Frank.Li@nxp.com>
Reviewed-by: default avatarMarco Felsch <m.felsch@pengutronix.de>
Reviewed-by: default avatarRob Herring (Arm) <robh@kernel.org>
Link: https://patch.msgid.link/20260106143620.126212-1-Frank.Li@nxp.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 59ba823e

Documentation/devicetree/bindings/net/dsa/microchip,ksz.yaml

+3 −0
Original line number Diff line number Diff line
@@ -40,6 +40,7 @@ properties: 
      - const: reset

        description:

          Used during reset for strap configuration.

    minItems: 1



  reset-gpios:

    description:
@@ -153,6 +154,8 @@ allOf: 
            const: microchip,ksz8463

    then:

      properties:

        pinctrl-names:

          minItems: 2

        straps-rxd-gpios:

          description:

            RXD0 and RXD1 pins, used to select SPI as bus interface.