devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates
@ 2023-06-16 13:18 Michael Walle
  2023-06-16 13:18 ` [PATCH 1/3] ARM: dts: lan966x: kontron-d10: fix board reset Michael Walle
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Michael Walle @ 2023-06-16 13:18 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Claudiu Beznea,
	Nicolas Ferre, Michael Walle
  Cc: devicetree, linux-kernel, Michael Walle

Fixes board reset and SPI CS# of the Kontron D10 board. Since the
MaxLinear PHY now correctly handles interrupts, we can add the correct
hardware descrption.

Signed-off-by: Michael Walle <mwalle@kernel.org>
---
Michael Walle (3):
      ARM: dts: lan966x: kontron-d10: fix board reset
      ARM: dts: lan966x: kontron-d10: fix SPI CS
      ARM: dts: lan966x: kontron-d10: add PHY interrupts

 .../dts/lan966x-kontron-kswitch-d10-mmt-8g.dts     |  2 ++
 .../boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi  | 29 +++++++++++++++++++++-
 2 files changed, 30 insertions(+), 1 deletion(-)
---
base-commit: f7efed9f38f886edb450041b82a6f15d663c98f8
change-id: 20230616-feature-d10-dt-cleanups-e79bd2e12518


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 1/3] ARM: dts: lan966x: kontron-d10: fix board reset
  2023-06-16 13:18 [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Michael Walle
@ 2023-06-16 13:18 ` Michael Walle
  2023-06-16 13:18 ` [PATCH 2/3] ARM: dts: lan966x: kontron-d10: fix SPI CS Michael Walle
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Michael Walle @ 2023-06-16 13:18 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Claudiu Beznea,
	Nicolas Ferre, Michael Walle
  Cc: devicetree, linux-kernel, Michael Walle

The pinctrl node was missing which change the pin mux to GPIO mode. Add
it.

Fixes: 79d83b3a458e ("ARM: dts: lan966x: add basic Kontron KSwitch D10 support")
Signed-off-by: Michael Walle <mwalle@kernel.org>
---
 arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
index 0097e72e3fb2..f86e2e751403 100644
--- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
+++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
@@ -17,6 +17,8 @@ chosen {
 	};
 
 	gpio-restart {
+		pinctrl-0 = <&reset_pins>;
+		pinctrl-names = "default";
 		compatible = "gpio-restart";
 		gpios = <&gpio 56 GPIO_ACTIVE_LOW>;
 		priority = <200>;
@@ -59,6 +61,12 @@ miim_c_pins: miim-c-pins {
 		function = "miim_c";
 	};
 
+	reset_pins: reset-pins {
+		/* SYS_RST# */
+		pins = "GPIO_56";
+		function = "gpio";
+	};
+
 	sgpio_a_pins: sgpio-a-pins {
 		/* SCK, D0, D1 */
 		pins = "GPIO_32", "GPIO_33", "GPIO_34";

-- 
2.30.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/3] ARM: dts: lan966x: kontron-d10: fix SPI CS
  2023-06-16 13:18 [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Michael Walle
  2023-06-16 13:18 ` [PATCH 1/3] ARM: dts: lan966x: kontron-d10: fix board reset Michael Walle
@ 2023-06-16 13:18 ` Michael Walle
  2023-06-16 13:18 ` [PATCH 3/3] ARM: dts: lan966x: kontron-d10: add PHY interrupts Michael Walle
  2023-06-21  9:31 ` [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Claudiu.Beznea
  3 siblings, 0 replies; 5+ messages in thread
From: Michael Walle @ 2023-06-16 13:18 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Claudiu Beznea,
	Nicolas Ferre, Michael Walle
  Cc: devicetree, linux-kernel, Michael Walle

The pinctrl node was missing which change the pin mux to GPIO mode.
Add it so we don't have to rely on the bootloader to set the correct
mode.

Fixes: 79d83b3a458e ("ARM: dts: lan966x: add basic Kontron KSwitch D10 support")
Signed-off-by: Michael Walle <mwalle@kernel.org>
---
 arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
index f86e2e751403..2e16217ac88e 100644
--- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
+++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
@@ -41,7 +41,7 @@ &flx3 {
 	status = "okay";
 
 	spi3: spi@400 {
-		pinctrl-0 = <&fc3_b_pins>;
+		pinctrl-0 = <&fc3_b_pins>, <&spi3_cs_pins>;
 		pinctrl-names = "default";
 		status = "okay";
 		cs-gpios = <&gpio 46 GPIO_ACTIVE_LOW>;
@@ -79,6 +79,12 @@ sgpio_b_pins: sgpio-b-pins {
 		function = "sgpio_b";
 	};
 
+	spi3_cs_pins: spi3-cs-pins {
+		/* CS# */
+		pins = "GPIO_46";
+		function = "gpio";
+	};
+
 	usart0_pins: usart0-pins {
 		/* RXD, TXD */
 		pins = "GPIO_25", "GPIO_26";

-- 
2.30.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 3/3] ARM: dts: lan966x: kontron-d10: add PHY interrupts
  2023-06-16 13:18 [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Michael Walle
  2023-06-16 13:18 ` [PATCH 1/3] ARM: dts: lan966x: kontron-d10: fix board reset Michael Walle
  2023-06-16 13:18 ` [PATCH 2/3] ARM: dts: lan966x: kontron-d10: fix SPI CS Michael Walle
@ 2023-06-16 13:18 ` Michael Walle
  2023-06-21  9:31 ` [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Claudiu.Beznea
  3 siblings, 0 replies; 5+ messages in thread
From: Michael Walle @ 2023-06-16 13:18 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Claudiu Beznea,
	Nicolas Ferre, Michael Walle
  Cc: devicetree, linux-kernel, Michael Walle

With interrupt handling fixed in the MaxLinear PHY driver, see commit
97a89ed101bb ("net: phy: mxl-gpy: disable interrupts on GPY215 by
default"), we can finally add the correct interrupt description to the
device tree.

Signed-off-by: Michael Walle <mwalle@kernel.org>
---
 arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts |  2 ++
 arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi   | 13 +++++++++++++
 2 files changed, 15 insertions(+)

diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
index 5feef9a59a79..ad5d8b56e6fa 100644
--- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
+++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
@@ -15,10 +15,12 @@ / {
 &mdio0 {
 	phy2: ethernet-phy@3 {
 		reg = <3>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 	};
 
 	phy3: ethernet-phy@4 {
 		reg = <4>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 	};
 };
 
diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
index 2e16217ac88e..83a1c23887e7 100644
--- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
+++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
@@ -49,6 +49,9 @@ spi3: spi@400 {
 };
 
 &gpio {
+	pinctrl-0 = <&phy_int_pins>;
+	pinctrl-names = "default";
+
 	fc3_b_pins: fc3-b-pins {
 		/* SCK, MISO, MOSI */
 		pins = "GPIO_51", "GPIO_52", "GPIO_53";
@@ -61,6 +64,12 @@ miim_c_pins: miim-c-pins {
 		function = "miim_c";
 	};
 
+	phy_int_pins: phy-int-pins {
+		/* PHY_INT# */
+		pins = "GPIO_24";
+		function = "gpio";
+	};
+
 	reset_pins: reset-pins {
 		/* SYS_RST# */
 		pins = "GPIO_56";
@@ -107,21 +116,25 @@ &mdio0 {
 
 	phy4: ethernet-phy@5 {
 		reg = <5>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 		coma-mode-gpios = <&gpio 37 GPIO_OPEN_DRAIN>;
 	};
 
 	phy5: ethernet-phy@6 {
 		reg = <6>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 		coma-mode-gpios = <&gpio 37 GPIO_OPEN_DRAIN>;
 	};
 
 	phy6: ethernet-phy@7 {
 		reg = <7>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 		coma-mode-gpios = <&gpio 37 GPIO_OPEN_DRAIN>;
 	};
 
 	phy7: ethernet-phy@8 {
 		reg = <8>;
+		interrupts-extended = <&gpio 24 IRQ_TYPE_LEVEL_LOW>;
 		coma-mode-gpios = <&gpio 37 GPIO_OPEN_DRAIN>;
 	};
 };

-- 
2.30.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates
  2023-06-16 13:18 [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Michael Walle
                   ` (2 preceding siblings ...)
  2023-06-16 13:18 ` [PATCH 3/3] ARM: dts: lan966x: kontron-d10: add PHY interrupts Michael Walle
@ 2023-06-21  9:31 ` Claudiu.Beznea
  3 siblings, 0 replies; 5+ messages in thread
From: Claudiu.Beznea @ 2023-06-21  9:31 UTC (permalink / raw)
  To: mwalle, robh+dt, krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre,
	michael
  Cc: devicetree, linux-kernel

On 16.06.2023 16:18, Michael Walle wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Fixes board reset and SPI CS# of the Kontron D10 board. Since the
> MaxLinear PHY now correctly handles interrupts, we can add the correct
> hardware descrption.
> 
> Signed-off-by: Michael Walle <mwalle@kernel.org>
> ---
> Michael Walle (3):
>       ARM: dts: lan966x: kontron-d10: fix board reset
>       ARM: dts: lan966x: kontron-d10: fix SPI CS
>       ARM: dts: lan966x: kontron-d10: add PHY interrupts

Applied to at91-dt, thanks!

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-06-21  9:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-16 13:18 [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Michael Walle
2023-06-16 13:18 ` [PATCH 1/3] ARM: dts: lan966x: kontron-d10: fix board reset Michael Walle
2023-06-16 13:18 ` [PATCH 2/3] ARM: dts: lan966x: kontron-d10: fix SPI CS Michael Walle
2023-06-16 13:18 ` [PATCH 3/3] ARM: dts: lan966x: kontron-d10: add PHY interrupts Michael Walle
2023-06-21  9:31 ` [PATCH 0/3] ARM: dts: lan966x: kontron-d10: device tree updates Claudiu.Beznea

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).