* [PATCH v2 1/2] arm64: dts: ti: k3-am65-main: Add ICSSG MDIO nodes
2021-06-01 15:00 [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs Suman Anna
@ 2021-06-01 15:00 ` Suman Anna
2021-06-01 15:00 ` [PATCH v2 2/2] arm64: dts: ti: k3-j721e-main: " Suman Anna
2021-06-07 15:04 ` [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs Nishanth Menon
2 siblings, 0 replies; 4+ messages in thread
From: Suman Anna @ 2021-06-01 15:00 UTC (permalink / raw)
To: Nishanth Menon
Cc: devicetree, Grzegorz Jaszczyk, Grygorii Strashko,
Vignesh Raghavendra, Lokesh Vutla, Jan Kiszka,
Kishon Vijay Abraham I, linux-arm-kernel
From: Roger Quadros <rogerq@ti.com>
The ICSSGs on K3 AM65x SoCs contain an MDIO controller that can
be used to control external PHYs associated with the Industrial
Ethernet peripherals within each ICSSG instance. The MDIO module
used within the ICSSG is similar to the MDIO Controller used
in TI Davinci SoCs. A bus frequency of 1 MHz is chosen for the
MDIO operations.
The nodes are added and enabled in the common k3-am65-main.dtsi
file by default, and disabled in the existing AM65 board dts
files. These nodes need pinctrl lines, and so should be enabled
only on boards where they are actually wired and pinned out for
ICSSG Ethernet. Any new board dts file should disable these if
they are not sure.
Signed-off-by: Roger Quadros <rogerq@ti.com>
[s-anna@ti.com: move the disabled status to board dts files]
Signed-off-by: Suman Anna <s-anna@ti.com>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
Acked-by: Jan Kiszka <jan.kiszka@siemens.com>
---
.../boot/dts/ti/k3-am65-iot2050-common.dtsi | 12 ++++++++
arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 30 +++++++++++++++++++
.../arm64/boot/dts/ti/k3-am654-base-board.dts | 12 ++++++++
3 files changed, 54 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index d90abda1de84..8c6b538c53f3 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -702,3 +702,15 @@ &mailbox0_cluster10 {
&mailbox0_cluster11 {
status = "disabled";
};
+
+&icssg0_mdio {
+ status = "disabled";
+};
+
+&icssg1_mdio {
+ status = "disabled";
+};
+
+&icssg2_mdio {
+ status = "disabled";
+};
diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index f97fc00c00ca..e679394d0b7e 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -1052,6 +1052,16 @@ tx_pru0_1: txpru@c000 {
reg-names = "iram", "control", "debug";
firmware-name = "am65x-txpru0_1-fw";
};
+
+ icssg0_mdio: mdio@32400 {
+ compatible = "ti,davinci_mdio";
+ reg = <0x32400 0x100>;
+ clocks = <&k3_clks 62 3>;
+ clock-names = "fck";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ bus_freq = <1000000>;
+ };
};
icssg1: icssg@b100000 {
@@ -1183,6 +1193,16 @@ tx_pru1_1: txpru@c000 {
reg-names = "iram", "control", "debug";
firmware-name = "am65x-txpru1_1-fw";
};
+
+ icssg1_mdio: mdio@32400 {
+ compatible = "ti,davinci_mdio";
+ reg = <0x32400 0x100>;
+ clocks = <&k3_clks 63 3>;
+ clock-names = "fck";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ bus_freq = <1000000>;
+ };
};
icssg2: icssg@b200000 {
@@ -1314,5 +1334,15 @@ tx_pru2_1: txpru@c000 {
reg-names = "iram", "control", "debug";
firmware-name = "am65x-txpru2_1-fw";
};
+
+ icssg2_mdio: mdio@32400 {
+ compatible = "ti,davinci_mdio";
+ reg = <0x32400 0x100>;
+ clocks = <&k3_clks 64 3>;
+ clock-names = "fck";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ bus_freq = <1000000>;
+ };
};
};
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index 60e43fd7af12..fa057a85e833 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -539,3 +539,15 @@ &mcasp2 {
&dss {
status = "disabled";
};
+
+&icssg0_mdio {
+ status = "disabled";
+};
+
+&icssg1_mdio {
+ status = "disabled";
+};
+
+&icssg2_mdio {
+ status = "disabled";
+};
--
2.30.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] 4+ messages in thread* [PATCH v2 2/2] arm64: dts: ti: k3-j721e-main: Add ICSSG MDIO nodes
2021-06-01 15:00 [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs Suman Anna
2021-06-01 15:00 ` [PATCH v2 1/2] arm64: dts: ti: k3-am65-main: Add ICSSG MDIO nodes Suman Anna
@ 2021-06-01 15:00 ` Suman Anna
2021-06-07 15:04 ` [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs Nishanth Menon
2 siblings, 0 replies; 4+ messages in thread
From: Suman Anna @ 2021-06-01 15:00 UTC (permalink / raw)
To: Nishanth Menon
Cc: devicetree, Grzegorz Jaszczyk, Grygorii Strashko,
Vignesh Raghavendra, Lokesh Vutla, Jan Kiszka,
Kishon Vijay Abraham I, linux-arm-kernel
The ICSSGs on K3 J721E SoCs contain an MDIO controller that can
be used to control external PHYs associated with the Industrial
Ethernet peripherals within each ICSSG instance. The MDIO module
used within the ICSSG is similar to the MDIO Controller used
in TI Davinci SoCs. A bus frequency of 1 MHz is chosen for the
MDIO operations.
The nodes are added and enabled in the common k3-j721e-main.dtsi
file by default, and disabled in the existing J721E board dts
file. These nodes need pinctrl lines, and so should be enabled
only on boards where they are actually wired and pinned out for
ICSSG Ethernet. Any new board dts file should disable these if
they are not sure.
Signed-off-by: Suman Anna <s-anna@ti.com>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
.../dts/ti/k3-j721e-common-proc-board.dts | 8 ++++++++
arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 20 +++++++++++++++++++
2 files changed, 28 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
index 351bb84db65b..1991f620d94b 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
@@ -718,3 +718,11 @@ &pcie3_ep {
&dss {
status = "disabled";
};
+
+&icssg0_mdio {
+ status = "disabled";
+};
+
+&icssg1_mdio {
+ status = "disabled";
+};
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
index 3bcafe4c1742..598613876b0f 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
@@ -1794,6 +1794,16 @@ tx_pru0_1: txpru@c000 {
reg-names = "iram", "control", "debug";
firmware-name = "j7-txpru0_1-fw";
};
+
+ icssg0_mdio: mdio@32400 {
+ compatible = "ti,davinci_mdio";
+ reg = <0x32400 0x100>;
+ clocks = <&k3_clks 119 1>;
+ clock-names = "fck";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ bus_freq = <1000000>;
+ };
};
icssg1: icssg@b100000 {
@@ -1925,5 +1935,15 @@ tx_pru1_1: txpru@c000 {
reg-names = "iram", "control", "debug";
firmware-name = "j7-txpru1_1-fw";
};
+
+ icssg1_mdio: mdio@32400 {
+ compatible = "ti,davinci_mdio";
+ reg = <0x32400 0x100>;
+ clocks = <&k3_clks 120 4>;
+ clock-names = "fck";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ bus_freq = <1000000>;
+ };
};
};
--
2.30.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] 4+ messages in thread* Re: [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs
2021-06-01 15:00 [PATCH v2 0/2] Add ICSSG MDIO nodes on AM65x & J721E SoCs Suman Anna
2021-06-01 15:00 ` [PATCH v2 1/2] arm64: dts: ti: k3-am65-main: Add ICSSG MDIO nodes Suman Anna
2021-06-01 15:00 ` [PATCH v2 2/2] arm64: dts: ti: k3-j721e-main: " Suman Anna
@ 2021-06-07 15:04 ` Nishanth Menon
2 siblings, 0 replies; 4+ messages in thread
From: Nishanth Menon @ 2021-06-07 15:04 UTC (permalink / raw)
To: Suman Anna
Cc: Nishanth Menon, Lokesh Vutla, Grygorii Strashko, devicetree,
linux-arm-kernel, Kishon Vijay Abraham I, Jan Kiszka,
Vignesh Raghavendra, Grzegorz Jaszczyk
On Tue, 1 Jun 2021 10:00:30 -0500, Suman Anna wrote:
> he following series is a resend of the ICSSG DT nodes [1] for the 5.14
> merge window. Patches are just rebased on top of 5.13-rc1 + your latest
> ti-k3-dts-next branch HEAD commit 69db725cdb2b ("arm64: dts: ti:
> k3-am654x/j721e/j7200-common-proc-board: Fix MCU_RGMII1_TXC direction")
> + Aswath's AM65x MMCSD1 UHS-I patch [2]. There are no code changes w.r.t
> v1, I have picked up Reviewed-by tags from Jan and Grygorii.
>
> [...]
Hi Suman Anna,
I have applied the following to branch ti-k3-dts-next on [1].
Thank you!
[1/2] arm64: dts: ti: k3-am65-main: Add ICSSG MDIO nodes
commit: d49a769dcddd4224bd631792a4d2e5bbb160aa20
[2/2] arm64: dts: ti: k3-j721e-main: Add ICSSG MDIO nodes
commit: 7ce11d4704c080ab890831b8255fbba4d2b0c5ec
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
[1] git://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
_______________________________________________
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] 4+ messages in thread