diff mbox series

arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node

Message ID 20230929123029.642424-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node | expand

Commit Message

Fabio Estevam Sept. 29, 2023, 12:30 p.m. UTC
From: Fabio Estevam <festevam@denx.de>

Per microchip,ksz.yaml, phy-mode is not a valid property in the
top-level switch node.

phy-mode = "rgmii-id" is already passed in the CPU port switch (port@5).

Remove it from the top-level switch node to fix the following
schema warning:

switch@5f: Unevaluated properties are not allowed ('phy-mode' was unexpected)
from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts | 1 -
 1 file changed, 1 deletion(-)

Comments

Tim Harvey Oct. 2, 2023, 5:05 p.m. UTC | #1
On Fri, Sep 29, 2023 at 5:30 AM Fabio Estevam <festevam@gmail.com> wrote:
>
> From: Fabio Estevam <festevam@denx.de>
>
> Per microchip,ksz.yaml, phy-mode is not a valid property in the
> top-level switch node.
>
> phy-mode = "rgmii-id" is already passed in the CPU port switch (port@5).
>
> Remove it from the top-level switch node to fix the following
> schema warning:
>
> switch@5f: Unevaluated properties are not allowed ('phy-mode' was unexpected)
> from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> index f45c22d8fb50..6c1e863f4d5a 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> @@ -642,7 +642,6 @@ switch: switch@5f {
>                 pinctrl-0 = <&pinctrl_ksz>;
>                 interrupt-parent = <&gpio4>;
>                 interrupts = <18 IRQ_TYPE_EDGE_FALLING>;
> -               phy-mode = "rgmii-id";
>
>                 ports {
>                         #address-cells = <1>;
> --
> 2.34.1
>

Acked-by: Tim Harvey <tharvey@gateworks.com>

Best Regards,

Tim
Shawn Guo Oct. 10, 2023, 1:34 a.m. UTC | #2
On Fri, Sep 29, 2023 at 09:30:29AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Per microchip,ksz.yaml, phy-mode is not a valid property in the
> top-level switch node.
> 
> phy-mode = "rgmii-id" is already passed in the CPU port switch (port@5).
> 
> Remove it from the top-level switch node to fix the following
> schema warning:
> 
> switch@5f: Unevaluated properties are not allowed ('phy-mode' was unexpected)
> from schema $id: http://devicetree.org/schemas/net/dsa/microchip,ksz.yaml
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied, thanks!
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
index f45c22d8fb50..6c1e863f4d5a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
@@ -642,7 +642,6 @@  switch: switch@5f {
 		pinctrl-0 = <&pinctrl_ksz>;
 		interrupt-parent = <&gpio4>;
 		interrupts = <18 IRQ_TYPE_EDGE_FALLING>;
-		phy-mode = "rgmii-id";
 
 		ports {
 			#address-cells = <1>;