* [PATCH v2 2/3] ARM: dts: rockchip: fix rk3036 emac node compatible string
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
@ 2022-06-03 16:35 ` Johan Jonker
2022-06-03 16:35 ` [PATCH v2 3/3] ARM: dts: rockchip: restyle emac nodes Johan Jonker
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Johan Jonker @ 2022-06-03 16:35 UTC (permalink / raw)
To: heiko
Cc: robh+dt, krzysztof.kozlowski+dt, davem, edumazet, kuba, pabeni,
netdev, linux-arm-kernel, linux-rockchip, devicetree,
linux-kernel
The Linux kernel has no logic to decide which driver to probe first.
To prevent race conditions remove the rk3036 emac node
fall back compatible string.
Signed-off-by: Johan Jonker <jbx6244@gmail.com>
---
arch/arm/boot/dts/rk3036.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/rk3036.dtsi b/arch/arm/boot/dts/rk3036.dtsi
index 9b0f04975..e240b89b0 100644
--- a/arch/arm/boot/dts/rk3036.dtsi
+++ b/arch/arm/boot/dts/rk3036.dtsi
@@ -225,7 +225,7 @@
};
emac: ethernet@10200000 {
- compatible = "rockchip,rk3036-emac", "snps,arc-emac";
+ compatible = "rockchip,rk3036-emac";
reg = <0x10200000 0x4000>;
interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
#address-cells = <1>;
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 3/3] ARM: dts: rockchip: restyle emac nodes
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
2022-06-03 16:35 ` [PATCH v2 2/3] ARM: dts: rockchip: fix rk3036 emac node compatible string Johan Jonker
@ 2022-06-03 16:35 ` Johan Jonker
2022-06-03 18:30 ` [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Rob Herring
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Johan Jonker @ 2022-06-03 16:35 UTC (permalink / raw)
To: heiko
Cc: robh+dt, krzysztof.kozlowski+dt, davem, edumazet, kuba, pabeni,
netdev, linux-arm-kernel, linux-rockchip, devicetree,
linux-kernel
The emac_rockchip.txt file is converted to YAML.
Phy nodes are now a subnode of mdio, so restyle
the emac nodes of rk3036/rk3066/rk3188.
Signed-off-by: Johan Jonker <jbx6244@gmail.com>
---
Changed V2:
use phy
---
arch/arm/boot/dts/rk3036-evb.dts | 16 ++++++++++------
arch/arm/boot/dts/rk3036-kylin.dts | 16 ++++++++++------
arch/arm/boot/dts/rk3036.dtsi | 2 --
arch/arm/boot/dts/rk3066a-marsboard.dts | 17 ++++++++++-------
arch/arm/boot/dts/rk3066a-rayeager.dts | 15 ++++++++++-----
arch/arm/boot/dts/rk3188-radxarock.dts | 19 +++++++++++--------
arch/arm/boot/dts/rk3xxx.dtsi | 2 --
7 files changed, 51 insertions(+), 36 deletions(-)
diff --git a/arch/arm/boot/dts/rk3036-evb.dts b/arch/arm/boot/dts/rk3036-evb.dts
index 2a7e6624e..9fd4d9db9 100644
--- a/arch/arm/boot/dts/rk3036-evb.dts
+++ b/arch/arm/boot/dts/rk3036-evb.dts
@@ -15,16 +15,20 @@
};
&emac {
- pinctrl-names = "default";
- pinctrl-0 = <&emac_xfer>, <&emac_mdio>;
phy = <&phy0>;
- phy-reset-gpios = <&gpio2 RK_PC6 GPIO_ACTIVE_LOW>; /* PHY_RST */
phy-reset-duration = <10>; /* millisecond */
-
+ phy-reset-gpios = <&gpio2 RK_PC6 GPIO_ACTIVE_LOW>; /* PHY_RST */
+ pinctrl-names = "default";
+ pinctrl-0 = <&emac_xfer>, <&emac_mdio>;
status = "okay";
- phy0: ethernet-phy@0 {
- reg = <0>;
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy0: ethernet-phy@0 {
+ reg = <0>;
+ };
};
};
diff --git a/arch/arm/boot/dts/rk3036-kylin.dts b/arch/arm/boot/dts/rk3036-kylin.dts
index e817eba8c..67e1e0413 100644
--- a/arch/arm/boot/dts/rk3036-kylin.dts
+++ b/arch/arm/boot/dts/rk3036-kylin.dts
@@ -80,16 +80,20 @@
};
&emac {
- pinctrl-names = "default";
- pinctrl-0 = <&emac_xfer>, <&emac_mdio>;
phy = <&phy0>;
- phy-reset-gpios = <&gpio2 RK_PC6 GPIO_ACTIVE_LOW>; /* PHY_RST */
phy-reset-duration = <10>; /* millisecond */
-
+ phy-reset-gpios = <&gpio2 RK_PC6 GPIO_ACTIVE_LOW>; /* PHY_RST */
+ pinctrl-names = "default";
+ pinctrl-0 = <&emac_xfer>, <&emac_mdio>;
status = "okay";
- phy0: ethernet-phy@0 {
- reg = <0>;
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy0: ethernet-phy@0 {
+ reg = <0>;
+ };
};
};
diff --git a/arch/arm/boot/dts/rk3036.dtsi b/arch/arm/boot/dts/rk3036.dtsi
index e240b89b0..78686fc72 100644
--- a/arch/arm/boot/dts/rk3036.dtsi
+++ b/arch/arm/boot/dts/rk3036.dtsi
@@ -228,8 +228,6 @@
compatible = "rockchip,rk3036-emac";
reg = <0x10200000 0x4000>;
interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
- #address-cells = <1>;
- #size-cells = <0>;
rockchip,grf = <&grf>;
clocks = <&cru HCLK_MAC>, <&cru SCLK_MACREF>, <&cru SCLK_MAC>;
clock-names = "hclk", "macref", "macclk";
diff --git a/arch/arm/boot/dts/rk3066a-marsboard.dts b/arch/arm/boot/dts/rk3066a-marsboard.dts
index a66d915aa..8beecd628 100644
--- a/arch/arm/boot/dts/rk3066a-marsboard.dts
+++ b/arch/arm/boot/dts/rk3066a-marsboard.dts
@@ -150,18 +150,21 @@
#include "tps65910.dtsi"
&emac {
- status = "okay";
-
phy = <&phy0>;
phy-supply = <&vcc_rmii>;
-
pinctrl-names = "default";
pinctrl-0 = <&emac_xfer>, <&emac_mdio>, <&phy_int>;
+ status = "okay";
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
- phy0: ethernet-phy@0 {
- reg = <0>;
- interrupt-parent = <&gpio1>;
- interrupts = <RK_PD2 IRQ_TYPE_LEVEL_LOW>;
+ phy0: ethernet-phy@0 {
+ reg = <0>;
+ interrupt-parent = <&gpio1>;
+ interrupts = <RK_PD2 IRQ_TYPE_LEVEL_LOW>;
+ };
};
};
diff --git a/arch/arm/boot/dts/rk3066a-rayeager.dts b/arch/arm/boot/dts/rk3066a-rayeager.dts
index 12b2e59ae..9aeef36ca 100644
--- a/arch/arm/boot/dts/rk3066a-rayeager.dts
+++ b/arch/arm/boot/dts/rk3066a-rayeager.dts
@@ -142,15 +142,20 @@
};
&emac {
- pinctrl-names = "default";
- pinctrl-0 = <&emac_xfer>, <&emac_mdio>, <&rmii_rst>;
phy = <&phy0>;
phy-supply = <&vcc_rmii>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&emac_xfer>, <&emac_mdio>, <&rmii_rst>;
status = "okay";
- phy0: ethernet-phy@0 {
- reg = <0>;
- reset-gpios = <&gpio1 RK_PD6 GPIO_ACTIVE_LOW>;
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy0: ethernet-phy@0 {
+ reg = <0>;
+ reset-gpios = <&gpio1 RK_PD6 GPIO_ACTIVE_LOW>;
+ };
};
};
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts b/arch/arm/boot/dts/rk3188-radxarock.dts
index 36c0945f4..bb0942d3e 100644
--- a/arch/arm/boot/dts/rk3188-radxarock.dts
+++ b/arch/arm/boot/dts/rk3188-radxarock.dts
@@ -126,18 +126,21 @@
};
&emac {
- status = "okay";
-
+ phy = <&phy0>;
+ phy-supply = <&vcc_rmii>;
pinctrl-names = "default";
pinctrl-0 = <&emac_xfer>, <&emac_mdio>, <&phy_int>;
+ status = "okay";
- phy = <&phy0>;
- phy-supply = <&vcc_rmii>;
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
- phy0: ethernet-phy@0 {
- reg = <0>;
- interrupt-parent = <&gpio3>;
- interrupts = <RK_PD2 IRQ_TYPE_LEVEL_LOW>;
+ phy0: ethernet-phy@0 {
+ reg = <0>;
+ interrupt-parent = <&gpio3>;
+ interrupts = <RK_PD2 IRQ_TYPE_LEVEL_LOW>;
+ };
};
};
diff --git a/arch/arm/boot/dts/rk3xxx.dtsi b/arch/arm/boot/dts/rk3xxx.dtsi
index 616a828e0..bf285091a 100644
--- a/arch/arm/boot/dts/rk3xxx.dtsi
+++ b/arch/arm/boot/dts/rk3xxx.dtsi
@@ -186,8 +186,6 @@
compatible = "snps,arc-emac";
reg = <0x10204000 0x3c>;
interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
- #address-cells = <1>;
- #size-cells = <0>;
rockchip,grf = <&grf>;
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
2022-06-03 16:35 ` [PATCH v2 2/3] ARM: dts: rockchip: fix rk3036 emac node compatible string Johan Jonker
2022-06-03 16:35 ` [PATCH v2 3/3] ARM: dts: rockchip: restyle emac nodes Johan Jonker
@ 2022-06-03 18:30 ` Rob Herring
2022-06-09 17:44 ` Rob Herring
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Rob Herring @ 2022-06-03 18:30 UTC (permalink / raw)
To: Johan Jonker
Cc: netdev, krzysztof.kozlowski+dt, linux-rockchip, heiko, edumazet,
linux-kernel, davem, pabeni, kuba, linux-arm-kernel, devicetree,
robh+dt
On Fri, 03 Jun 2022 18:35:37 +0200, Johan Jonker wrote:
> Convert emac_rockchip.txt to YAML.
>
> Changes against original bindings:
> Add mdio sub node.
> Add extra clock for rk3036
>
> Signed-off-by: Johan Jonker <jbx6244@gmail.com>
> ---
>
> Changed V2:
> use phy
> rename to rockchip,emac.yaml
> add more requirements
> ---
> .../devicetree/bindings/net/emac_rockchip.txt | 52 --------
> .../bindings/net/rockchip,emac.yaml | 115 ++++++++++++++++++
> 2 files changed, 115 insertions(+), 52 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/net/emac_rockchip.txt
> create mode 100644 Documentation/devicetree/bindings/net/rockchip,emac.yaml
>
Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.
Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.
Full log is available here: https://patchwork.ozlabs.org/patch/
ethernet@10200000: compatible: ['rockchip,rk3036-emac', 'snps,arc-emac'] is too long
arch/arm/boot/dts/rk3036-evb.dtb
arch/arm/boot/dts/rk3036-kylin.dtb
ethernet@10200000: 'mdio' is a required property
arch/arm/boot/dts/rk3036-evb.dtb
arch/arm/boot/dts/rk3036-kylin.dtb
ethernet@10204000: 'mdio' is a required property
arch/arm/boot/dts/rk3066a-marsboard.dtb
arch/arm/boot/dts/rk3066a-rayeager.dtb
arch/arm/boot/dts/rk3188-radxarock.dtb
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
` (2 preceding siblings ...)
2022-06-03 18:30 ` [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Rob Herring
@ 2022-06-09 17:44 ` Rob Herring
2022-08-01 14:30 ` Rob Herring
2022-09-09 23:10 ` (subset) " Heiko Stuebner
5 siblings, 0 replies; 7+ messages in thread
From: Rob Herring @ 2022-06-09 17:44 UTC (permalink / raw)
To: Johan Jonker
Cc: heiko, edumazet, linux-kernel, davem, robh+dt, linux-rockchip,
kuba, pabeni, devicetree, krzysztof.kozlowski+dt, netdev,
linux-arm-kernel
On Fri, 03 Jun 2022 18:35:37 +0200, Johan Jonker wrote:
> Convert emac_rockchip.txt to YAML.
>
> Changes against original bindings:
> Add mdio sub node.
> Add extra clock for rk3036
>
> Signed-off-by: Johan Jonker <jbx6244@gmail.com>
> ---
>
> Changed V2:
> use phy
> rename to rockchip,emac.yaml
> add more requirements
> ---
> .../devicetree/bindings/net/emac_rockchip.txt | 52 --------
> .../bindings/net/rockchip,emac.yaml | 115 ++++++++++++++++++
> 2 files changed, 115 insertions(+), 52 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/net/emac_rockchip.txt
> create mode 100644 Documentation/devicetree/bindings/net/rockchip,emac.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
` (3 preceding siblings ...)
2022-06-09 17:44 ` Rob Herring
@ 2022-08-01 14:30 ` Rob Herring
2022-09-09 23:10 ` (subset) " Heiko Stuebner
5 siblings, 0 replies; 7+ messages in thread
From: Rob Herring @ 2022-08-01 14:30 UTC (permalink / raw)
To: Johan Jonker
Cc: robh+dt, kuba, linux-arm-kernel, edumazet, devicetree,
linux-kernel, netdev, linux-rockchip, davem, heiko,
krzysztof.kozlowski+dt, pabeni
On Fri, 03 Jun 2022 18:35:37 +0200, Johan Jonker wrote:
> Convert emac_rockchip.txt to YAML.
>
> Changes against original bindings:
> Add mdio sub node.
> Add extra clock for rk3036
>
> Signed-off-by: Johan Jonker <jbx6244@gmail.com>
> ---
>
> Changed V2:
> use phy
> rename to rockchip,emac.yaml
> add more requirements
> ---
> .../devicetree/bindings/net/emac_rockchip.txt | 52 --------
> .../bindings/net/rockchip,emac.yaml | 115 ++++++++++++++++++
> 2 files changed, 115 insertions(+), 52 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/net/emac_rockchip.txt
> create mode 100644 Documentation/devicetree/bindings/net/rockchip,emac.yaml
>
Looks like this fell through the cracks. Applied, thanks!
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: (subset) [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML
2022-06-03 16:35 [PATCH v2 1/3] dt-bindings: net: convert emac_rockchip.txt to YAML Johan Jonker
` (4 preceding siblings ...)
2022-08-01 14:30 ` Rob Herring
@ 2022-09-09 23:10 ` Heiko Stuebner
5 siblings, 0 replies; 7+ messages in thread
From: Heiko Stuebner @ 2022-09-09 23:10 UTC (permalink / raw)
To: Johan Jonker
Cc: Heiko Stuebner, linux-kernel, devicetree, netdev, pabeni, kuba,
davem, linux-rockchip, edumazet, linux-arm-kernel,
krzysztof.kozlowski+dt, robh+dt
On Fri, 3 Jun 2022 18:35:37 +0200, Johan Jonker wrote:
> Convert emac_rockchip.txt to YAML.
>
> Changes against original bindings:
> Add mdio sub node.
> Add extra clock for rk3036
Applied, thanks!
[2/3] ARM: dts: rockchip: fix rk3036 emac node compatible string
commit: d28b680a34948d7634b824b1fc7546e9dc8422fb
[3/3] ARM: dts: rockchip: restyle emac nodes
commit: 1dabb74971b38d966ecef566bafddc4a34f4db9d
Best regards,
--
Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 7+ messages in thread