Skip to contentSkip to conversation

bindings: net: stmmac: add missing note about LPI interrupt

PATCH In review Niklas Cassel 4 participants 4 messages Apr 18, 2017 – Apr 21, 2017

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 bindings: net: stmmac: add missing note about LPI interrupt

From: Niklas Cassel <[email protected]>

The hardware has a LPI interrupt.
There is already code in the stmmac driver to parse and handle the interrupt. However, this information was missing from the DT binding.

At the same time, improve the description of the existing interrupts.

---
 Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
Documentation/devicetree/bindings/net/stmmac.txt +8 -5
index f652b0c384ce..c3a7be6615c5 100644
@@ -7,9 +7,12 @@ Required properties:
 - interrupt-parent: Should be the phandle for the interrupt controller
   that services interrupts for this device
 - interrupts: Should contain the STMMAC interrupts
-- interrupt-names: Should contain the interrupt names "macirq"
-  "eth_wake_irq" if this interrupt is supported in the "interrupts"
-  property
+- interrupt-names: Should contain a list of interrupt names corresponding to
+	the interrupts in the interrupts property, if available.
+	Valid interrupt names are:
+  - "macirq" (combined signal for various interrupt events)
+  - "eth_wake_irq" (the interrupt to manage the remote wake-up packet detection)
+  - "eth_lpi" (the interrupt that occurs when Tx or Rx enters/exits LPI state)
 - phy-mode: See ethernet.txt file in the same directory.
 - snps,reset-gpio 	gpio number for phy reset.
 - snps,reset-active-low boolean flag to indicate if phy reset is active low.
@@ -152,8 +155,8 @@ Examples:
 		compatible = "st,spear600-gmac";
 		reg = <0xe0800000 0x8000>;
 		interrupt-parent = <&vic1>;
-		interrupts = <24 23>;
-		interrupt-names = "macirq", "eth_wake_irq";
+		interrupts = <24 23 22>;
+		interrupt-names = "macirq", "eth_wake_irq", "eth_lpi";
 		mac-address = [000000000000]; /* Filled in by U-Boot */
 		max-frame-size = <3800>;
 		phy-mode = "gmii";
Signature · 6 lines
2.11.0

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Acked-by on v3 1/1
#1 lore raw

Hi Niklas,

Às 1:39 PM de 4/18/2017, Niklas Cassel escreveu:

Quoted 35 lines

From: Niklas Cassel <[email protected]>

The hardware has a LPI interrupt.
There is already code in the stmmac driver to parse and handle the interrupt. However, this information was missing from the DT binding.

At the same time, improve the description of the existing interrupts.

---
 Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
Documentation/devicetree/bindings/net/stmmac.txt +6 -3
index f652b0c384ce..c3a7be6615c5 100644
@@ -7,9 +7,12 @@ Required properties:
 - interrupt-parent: Should be the phandle for the interrupt controller
   that services interrupts for this device
 - interrupts: Should contain the STMMAC interrupts
-- interrupt-names: Should contain the interrupt names "macirq"
-  "eth_wake_irq" if this interrupt is supported in the "interrupts"
-  property
+- interrupt-names: Should contain a list of interrupt names corresponding to
+	the interrupts in the interrupts property, if available.
+	Valid interrupt names are:
+  - "macirq" (combined signal for various interrupt events)
+  - "eth_wake_irq" (the interrupt to manage the remote wake-up packet detection)
+  - "eth_lpi" (the interrupt that occurs when Tx or Rx enters/exits LPI state)
 - phy-mode: See ethernet.txt file in the same directory.
 - snps,reset-gpio 	gpio number for phy reset.
 - snps,reset-active-low boolean flag to indicate if phy reset is active low.

Documentation/devicetree/bindings/net/stmmac.txt Examples: · line 162

Quoted 10 lines
Documentation/devicetree/bindings/net/stmmac.txt +2 -2
@@ -152,8 +155,8 @@ Examples:
 		compatible = "st,spear600-gmac";
 		reg = <0xe0800000 0x8000>;
 		interrupt-parent = <&vic1>;
-		interrupts = <24 23>;
-		interrupt-names = "macirq", "eth_wake_irq";
+		interrupts = <24 23 22>;
+		interrupt-names = "macirq", "eth_wake_irq", "eth_lpi";
 		mac-address = [000000000000]; /* Filled in by U-Boot */
 		max-frame-size = <3800>;
 		phy-mode = "gmii";

Regards!

maintainer Acked-by on v3 1/1
#2 lore raw
In reply to Niklas Cassel
Quoted 6 lines · Niklas Cassel

From: Niklas Cassel <[email protected]>

The hardware has a LPI interrupt.
There is already code in the stmmac driver to parse and handle the interrupt. However, this information was missing from the DT binding.

Quoted 6 lines

At the same time, improve the description of the existing interrupts.

---
 Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
Acked-by on v3 1/1
#3 lore raw
In reply to Niklas Cassel
Quoted 34 lines · Niklas Cassel

From: Niklas Cassel <[email protected]>

The hardware has a LPI interrupt.
There is already code in the stmmac driver to parse and handle the interrupt. However, this information was missing from the DT binding.

At the same time, improve the description of the existing interrupts.

---
  Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++-----
  1 file changed, 8 insertions(+), 5 deletions(-)
Documentation/devicetree/bindings/net/stmmac.txt +6 -3
index f652b0c384ce..c3a7be6615c5 100644
@@ -7,9 +7,12 @@ Required properties:
  - interrupt-parent: Should be the phandle for the interrupt controller
    that services interrupts for this device
  - interrupts: Should contain the STMMAC interrupts
-- interrupt-names: Should contain the interrupt names "macirq"
-  "eth_wake_irq" if this interrupt is supported in the "interrupts"
-  property
+- interrupt-names: Should contain a list of interrupt names corresponding to
+	the interrupts in the interrupts property, if available.
+	Valid interrupt names are:
+  - "macirq" (combined signal for various interrupt events)
+  - "eth_wake_irq" (the interrupt to manage the remote wake-up packet detection)
+  - "eth_lpi" (the interrupt that occurs when Tx or Rx enters/exits LPI state)
  - phy-mode: See ethernet.txt file in the same directory.
  - snps,reset-gpio 	gpio number for phy reset.
  - snps,reset-active-low boolean flag to indicate if phy reset is active low.

Documentation/devicetree/bindings/net/stmmac.txt Examples: · line 162

Quoted 10 lines
Documentation/devicetree/bindings/net/stmmac.txt +2 -2
@@ -152,8 +155,8 @@ Examples:
  		compatible = "st,spear600-gmac";
  		reg = <0xe0800000 0x8000>;
  		interrupt-parent = <&vic1>;
-		interrupts = <24 23>;
-		interrupt-names = "macirq", "eth_wake_irq";
+		interrupts = <24 23 22>;
+		interrupt-names = "macirq", "eth_wake_irq", "eth_lpi";
  		mac-address = [000000000000]; /* Filled in by U-Boot */
  		max-frame-size = <3800>;
  		phy-mode = "gmii";

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html