* [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node
@ 2023-09-29 12:30 Fabio Estevam
2023-10-02 17:05 ` Tim Harvey
2023-10-10 1:34 ` Shawn Guo
0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2023-09-29 12:30 UTC (permalink / raw)
To: shawnguo; +Cc: tharvey, linux-arm-kernel, Fabio Estevam
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node
2023-09-29 12:30 [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node Fabio Estevam
@ 2023-10-02 17:05 ` Tim Harvey
2023-10-10 1:34 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Tim Harvey @ 2023-10-02 17:05 UTC (permalink / raw)
To: Fabio Estevam; +Cc: shawnguo, linux-arm-kernel, Fabio Estevam
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node
2023-09-29 12:30 [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node Fabio Estevam
2023-10-02 17:05 ` Tim Harvey
@ 2023-10-10 1:34 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2023-10-10 1:34 UTC (permalink / raw)
To: Fabio Estevam; +Cc: tharvey, linux-arm-kernel, Fabio Estevam
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!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-10-10 1:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-29 12:30 [PATCH] arm64: dts: imx8mm-venice-gw790: Remove phy-mode from switch node Fabio Estevam
2023-10-02 17:05 ` Tim Harvey
2023-10-10 1:34 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).