* [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes
@ 2016-11-28 8:44 Stefan Lengfeld
2016-11-28 8:44 ` [PATCH 2/3] ARM: dts: phycore-imx6: fix missing ?xc-skew-ps settings for ethphy Stefan Lengfeld
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Stefan Lengfeld @ 2016-11-28 8:44 UTC (permalink / raw)
To: barebox
Refactor the common settings for device tree node 'fec' into the generic
phycore i.MX6 device tree file. This avoid redundant settings and makes
common fixes easier. Our kernel device tree files have the same layout.
Signed-off-by: Stefan Lengfeld <s.lengfeld@phytec.de>
---
arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts | 17 ++++-------------
arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts | 17 ++++-------------
arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts | 17 ++++-------------
arch/arm/dts/imx6q-phytec-phycore-som-nand.dts | 17 ++++-------------
arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 13 +++++++++++++
5 files changed, 29 insertions(+), 52 deletions(-)
diff --git a/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts b/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
index fc153a6..bffee5f 100644
--- a/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
+++ b/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
@@ -29,21 +29,12 @@
status = "okay";
};
+ðphy {
+ max-speed = <100>;
+};
+
&fec {
status = "okay";
- phy-handle = <ðphy>;
- phy-mode = "rgmii";
- phy-reset-gpios = <&gpio1 14 1>;
-
- mdio {
- #address-cells = <1>;
- #size-cells = <0>;
-
- ethphy: ethernet-phy@3 {
- reg = <3>;
- max-speed = <100>;
- };
- };
};
&flash {
diff --git a/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts b/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
index 3f2f1c7..1b66fda 100644
--- a/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
+++ b/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
@@ -24,21 +24,12 @@
status = "okay";
};
+ðphy {
+ max-speed = <100>;
+};
+
&fec {
status = "okay";
- phy-handle = <ðphy>;
- phy-mode = "rgmii";
- phy-reset-gpios = <&gpio1 14 1>;
-
- mdio {
- #address-cells = <1>;
- #size-cells = <0>;
-
- ethphy: ethernet-phy@3 {
- reg = <3>;
- max-speed = <100>;
- };
- };
};
&gpmi {
diff --git a/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts b/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
index 74bc09b..ecc5aa3 100644
--- a/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
+++ b/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
@@ -28,21 +28,12 @@
status = "okay";
};
+ðphy {
+ max-speed = <1000>;
+};
+
&fec {
status = "okay";
- phy-handle = <ðphy>;
- phy-mode = "rgmii";
- phy-reset-gpios = <&gpio1 14 1>;
-
- mdio {
- #address-cells = <1>;
- #size-cells = <0>;
-
- ethphy: ethernet-phy@3 {
- reg = <3>;
- max-speed = <1000>;
- };
- };
};
&flash {
diff --git a/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts b/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
index aa2c94a..9ad7eda 100644
--- a/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
+++ b/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
@@ -29,21 +29,12 @@
status = "okay";
};
+ðphy {
+ max-speed = <1000>;
+};
+
&fec {
status = "okay";
- phy-handle = <ðphy>;
- phy-mode = "rgmii";
- phy-reset-gpios = <&gpio1 14 1>;
-
- mdio {
- #address-cells = <1>;
- #size-cells = <0>;
-
- ethphy: ethernet-phy@3 {
- reg = <3>;
- max-speed = <1000>;
- };
- };
};
&flash {
diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
index d446a5e..9649817 100644
--- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
@@ -10,6 +10,7 @@
*/
#include "imx6qdl.dtsi"
+#include <dt-bindings/gpio/gpio.h>
/ {
chosen {
@@ -76,7 +77,19 @@
&fec {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_enet>;
+ phy-handle = <ðphy>;
+ phy-mode = "rgmii";
+ phy-reset-gpios = <&gpio1 14 GPIO_ACTIVE_LOW>;
status = "disabled";
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ethphy: ethernet-phy@3 {
+ reg = <3>;
+ };
+ };
};
&gpmi {
--
1.9.1
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 2/3] ARM: dts: phycore-imx6: fix missing ?xc-skew-ps settings for ethphy
2016-11-28 8:44 [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Stefan Lengfeld
@ 2016-11-28 8:44 ` Stefan Lengfeld
2016-11-28 8:45 ` [PATCH 3/3] ARM: dts: phycore-imx6: remove unnecessary include Stefan Lengfeld
2016-12-07 20:14 ` [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Lengfeld @ 2016-11-28 8:44 UTC (permalink / raw)
To: barebox
Add the missing txc-skew-ps and rxc-skew-ps settings for the Micrel PHY.
I have seen problems with 1GBit ethernet on a phyCORE-i.MX6 Solo. Oddly
enough the missing skew values are no problems for all our other
modules.
Signed-off-by: Stefan Lengfeld <s.lengfeld@phytec.de>
---
arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
index 9649817..76777e8 100644
--- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
@@ -88,6 +88,8 @@
ethphy: ethernet-phy@3 {
reg = <3>;
+ txc-skew-ps = <1680>;
+ rxc-skew-ps = <1860>;
};
};
};
--
1.9.1
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 3/3] ARM: dts: phycore-imx6: remove unnecessary include
2016-11-28 8:44 [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Stefan Lengfeld
2016-11-28 8:44 ` [PATCH 2/3] ARM: dts: phycore-imx6: fix missing ?xc-skew-ps settings for ethphy Stefan Lengfeld
@ 2016-11-28 8:45 ` Stefan Lengfeld
2016-12-07 20:14 ` [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Lengfeld @ 2016-11-28 8:45 UTC (permalink / raw)
To: barebox
The file 'imx6qdl.dtsi' is already included by DualLite/Solo and
Dual/Quad specific files 'imx6dl.dtsi' and 'imx6q.dtsi'. So the include
is not needed here.
Signed-off-by: Stefan Lengfeld <s.lengfeld@phytec.de>
---
arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
index 76777e8..1e04ce1 100644
--- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
@@ -9,7 +9,6 @@
* http://www.gnu.org/copyleft/gpl.html
*/
-#include "imx6qdl.dtsi"
#include <dt-bindings/gpio/gpio.h>
/ {
--
1.9.1
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes
2016-11-28 8:44 [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Stefan Lengfeld
2016-11-28 8:44 ` [PATCH 2/3] ARM: dts: phycore-imx6: fix missing ?xc-skew-ps settings for ethphy Stefan Lengfeld
2016-11-28 8:45 ` [PATCH 3/3] ARM: dts: phycore-imx6: remove unnecessary include Stefan Lengfeld
@ 2016-12-07 20:14 ` Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2016-12-07 20:14 UTC (permalink / raw)
To: Stefan Lengfeld; +Cc: barebox
On Mon, Nov 28, 2016 at 09:44:58AM +0100, Stefan Lengfeld wrote:
> Refactor the common settings for device tree node 'fec' into the generic
> phycore i.MX6 device tree file. This avoid redundant settings and makes
> common fixes easier. Our kernel device tree files have the same layout.
>
> Signed-off-by: Stefan Lengfeld <s.lengfeld@phytec.de>
> ---
> arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts | 17 ++++-------------
> arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts | 17 ++++-------------
> arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts | 17 ++++-------------
> arch/arm/dts/imx6q-phytec-phycore-som-nand.dts | 17 ++++-------------
> arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 13 +++++++++++++
> 5 files changed, 29 insertions(+), 52 deletions(-)
Applied, thanks
Sascha
>
> diff --git a/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts b/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
> index fc153a6..bffee5f 100644
> --- a/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
> +++ b/arch/arm/dts/imx6dl-phytec-phycore-som-emmc.dts
> @@ -29,21 +29,12 @@
> status = "okay";
> };
>
> +ðphy {
> + max-speed = <100>;
> +};
> +
> &fec {
> status = "okay";
> - phy-handle = <ðphy>;
> - phy-mode = "rgmii";
> - phy-reset-gpios = <&gpio1 14 1>;
> -
> - mdio {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - ethphy: ethernet-phy@3 {
> - reg = <3>;
> - max-speed = <100>;
> - };
> - };
> };
>
> &flash {
> diff --git a/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts b/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
> index 3f2f1c7..1b66fda 100644
> --- a/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
> +++ b/arch/arm/dts/imx6dl-phytec-phycore-som-nand.dts
> @@ -24,21 +24,12 @@
> status = "okay";
> };
>
> +ðphy {
> + max-speed = <100>;
> +};
> +
> &fec {
> status = "okay";
> - phy-handle = <ðphy>;
> - phy-mode = "rgmii";
> - phy-reset-gpios = <&gpio1 14 1>;
> -
> - mdio {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - ethphy: ethernet-phy@3 {
> - reg = <3>;
> - max-speed = <100>;
> - };
> - };
> };
>
> &gpmi {
> diff --git a/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts b/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
> index 74bc09b..ecc5aa3 100644
> --- a/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
> +++ b/arch/arm/dts/imx6q-phytec-phycore-som-emmc.dts
> @@ -28,21 +28,12 @@
> status = "okay";
> };
>
> +ðphy {
> + max-speed = <1000>;
> +};
> +
> &fec {
> status = "okay";
> - phy-handle = <ðphy>;
> - phy-mode = "rgmii";
> - phy-reset-gpios = <&gpio1 14 1>;
> -
> - mdio {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - ethphy: ethernet-phy@3 {
> - reg = <3>;
> - max-speed = <1000>;
> - };
> - };
> };
>
> &flash {
> diff --git a/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts b/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
> index aa2c94a..9ad7eda 100644
> --- a/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
> +++ b/arch/arm/dts/imx6q-phytec-phycore-som-nand.dts
> @@ -29,21 +29,12 @@
> status = "okay";
> };
>
> +ðphy {
> + max-speed = <1000>;
> +};
> +
> &fec {
> status = "okay";
> - phy-handle = <ðphy>;
> - phy-mode = "rgmii";
> - phy-reset-gpios = <&gpio1 14 1>;
> -
> - mdio {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - ethphy: ethernet-phy@3 {
> - reg = <3>;
> - max-speed = <1000>;
> - };
> - };
> };
>
> &flash {
> diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> index d446a5e..9649817 100644
> --- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> +++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> @@ -10,6 +10,7 @@
> */
>
> #include "imx6qdl.dtsi"
> +#include <dt-bindings/gpio/gpio.h>
>
> / {
> chosen {
> @@ -76,7 +77,19 @@
> &fec {
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_enet>;
> + phy-handle = <ðphy>;
> + phy-mode = "rgmii";
> + phy-reset-gpios = <&gpio1 14 GPIO_ACTIVE_LOW>;
> status = "disabled";
> +
> + mdio {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ethphy: ethernet-phy@3 {
> + reg = <3>;
> + };
> + };
> };
>
> &gpmi {
> --
> 1.9.1
>
>
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-12-07 20:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-28 8:44 [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Stefan Lengfeld
2016-11-28 8:44 ` [PATCH 2/3] ARM: dts: phycore-imx6: fix missing ?xc-skew-ps settings for ethphy Stefan Lengfeld
2016-11-28 8:45 ` [PATCH 3/3] ARM: dts: phycore-imx6: remove unnecessary include Stefan Lengfeld
2016-12-07 20:14 ` [PATCH 1/3] ARM: dts: phycore-imx6: refactor fec nodes Sascha Hauer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox