Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1625302 > unrolled thread
| Started by | Niklas Cassel <niklas.cassel@axis.com> |
|---|---|
| First post | 2017-04-18 14:50 +0200 |
| Last post | 2017-04-21 07:30 +0200 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
[PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt Niklas Cassel <niklas.cassel@axis.com> - 2017-04-18 14:50 +0200
Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt Joao Pinto <Joao.Pinto@synopsys.com> - 2017-04-19 17:30 +0200
Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt Rob Herring <robh@kernel.org> - 2017-04-20 22:20 +0200
Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-04-21 07:30 +0200
| From | Niklas Cassel <niklas.cassel@axis.com> |
|---|---|
| Date | 2017-04-18 14:50 +0200 |
| Subject | [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt |
| Message-ID | <txAJX-6e5-11@gated-at.bofh.it> |
From: Niklas Cassel <niklas.cassel@axis.com> 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. Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> --- Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/net/stmmac.txt b/Documentation/devicetree/bindings/net/stmmac.txt index f652b0c384ce..c3a7be6615c5 100644 --- a/Documentation/devicetree/bindings/net/stmmac.txt +++ b/Documentation/devicetree/bindings/net/stmmac.txt @@ -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"; -- 2.11.0
[toc] | [next] | [standalone]
| From | Joao Pinto <Joao.Pinto@synopsys.com> |
|---|---|
| Date | 2017-04-19 17:30 +0200 |
| Subject | Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt |
| Message-ID | <txZIp-56L-87@gated-at.bofh.it> |
| In reply to | #1625302 |
Hi Niklas, Às 1:39 PM de 4/18/2017, Niklas Cassel escreveu: > From: Niklas Cassel <niklas.cassel@axis.com> > > 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. > > Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> > --- > Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/stmmac.txt b/Documentation/devicetree/bindings/net/stmmac.txt > index f652b0c384ce..c3a7be6615c5 100644 > --- a/Documentation/devicetree/bindings/net/stmmac.txt > +++ b/Documentation/devicetree/bindings/net/stmmac.txt > @@ -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"; > Acked-By: Joao Pinto <jpinto@synopsys.com> Regards!
[toc] | [prev] | [next] | [standalone]
| From | Rob Herring <robh@kernel.org> |
|---|---|
| Date | 2017-04-20 22:20 +0200 |
| Subject | Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt |
| Message-ID | <tyqIx-504-3@gated-at.bofh.it> |
| In reply to | #1625302 |
On Tue, Apr 18, 2017 at 02:39:53PM +0200, Niklas Cassel wrote: > From: Niklas Cassel <niklas.cassel@axis.com> > > 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. > > Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> > --- > Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) Acked-by: Rob Herring <robh@kernel.org>
[toc] | [prev] | [next] | [standalone]
| From | Giuseppe CAVALLARO <peppe.cavallaro@st.com> |
|---|---|
| Date | 2017-04-21 07:30 +0200 |
| Subject | Re: [PATCH net-next v3] bindings: net: stmmac: add missing note about LPI interrupt |
| Message-ID | <tyziN-1Tc-7@gated-at.bofh.it> |
| In reply to | #1625302 |
On 4/18/2017 2:39 PM, Niklas Cassel wrote: > From: Niklas Cassel <niklas.cassel@axis.com> > > 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. > > Signed-off-by: Niklas Cassel <niklas.cassel@axis.com> > --- > Documentation/devicetree/bindings/net/stmmac.txt | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/stmmac.txt b/Documentation/devicetree/bindings/net/stmmac.txt > index f652b0c384ce..c3a7be6615c5 100644 > --- a/Documentation/devicetree/bindings/net/stmmac.txt > +++ b/Documentation/devicetree/bindings/net/stmmac.txt > @@ -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"; Acked-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web