Re: [PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
On Wed, Dec 20, 2017 at 2:02 AM, Jassi Brarwrote: > Hi Mark, > > On Tue, Dec 12, 2017 at 10:59 PM, Mark Rutland wrote: >> Hi, >> >> On Tue, Dec 12, 2017 at 10:45:21PM +0530, jassisinghb...@gmail.com wrote: >>> From: Jassi Brar >>> >>> This patch adds documentation for Device-Tree bindings for the >>> Socionext NetSec Controller driver. >>> >>> Signed-off-by: Ard Biesheuvel >>> Signed-off-by: Jassi Brar >>> --- >>> .../devicetree/bindings/net/socionext-netsec.txt | 43 >>> ++ >>> 1 file changed, 43 insertions(+) >>> create mode 100644 >>> Documentation/devicetree/bindings/net/socionext-netsec.txt >>> >>> diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt >>> b/Documentation/devicetree/bindings/net/socionext-netsec.txt >>> new file mode 100644 >>> index 000..4695969 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt >>> @@ -0,0 +1,45 @@ >>> +* Socionext NetSec Ethernet Controller IP >>> + >>> +Required properties: >>> +- compatible: Should be "socionext,synquacer-netsec" >>> +- reg: Address and length of the control register area, followed by the >>> + address and length of the EEPROM holding the MAC address and >>> + microengine firmware >>> +- interrupts: Should contain ethernet controller interrupt >>> +- clocks: phandle to the PHY reference clock, and any other clocks to be >>> + switched by runtime_pm runtime_pm is a Linux thing and driver detail. >>> +- clock-names: Required only if more than a single clock is listed in >>> 'clocks'. >>> + The PHY reference clock must be named 'phy_refclk' >> >> Please define the full set of clocks (and their names) explicitly. This >> should be well-known. >> > The issue is some implementations have just the 'rate-reference' clock > going in, while others may also have 1or2 optional 'enable' clocks > (which may go to other devices as well). > The driver only needs to know which clock to read the freq from, so it > expects that clock to be named 'phy_refclk', while the 'enable' clocks > can be named anything. It still needs to be documented. If there's differing number of clocks, then I expect a compatible string for each possible clock setup. Of course, differing number of clocks for the same block is often an error when multiple clock inputs are driven by the same source clock. Rob
Re: [PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
On 13 December 2017 at 02:07, Andrew Lunnwrote: > On Tue, Dec 12, 2017 at 10:45:21PM +0530, jassisinghb...@gmail.com wrote: >> From: Jassi Brar >> >> This patch adds documentation for Device-Tree bindings for the >> Socionext NetSec Controller driver. >> >> Signed-off-by: Ard Biesheuvel >> Signed-off-by: Jassi Brar >> --- >> .../devicetree/bindings/net/socionext-netsec.txt | 43 >> ++ >> 1 file changed, 43 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/net/socionext-netsec.txt >> >> diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt >> b/Documentation/devicetree/bindings/net/socionext-netsec.txt >> new file mode 100644 >> index 000..4695969 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt >> @@ -0,0 +1,45 @@ >> +* Socionext NetSec Ethernet Controller IP >> + >> +Required properties: >> +- compatible: Should be "socionext,synquacer-netsec" >> +- reg: Address and length of the control register area, followed by the >> + address and length of the EEPROM holding the MAC address and >> + microengine firmware >> +- interrupts: Should contain ethernet controller interrupt >> +- clocks: phandle to the PHY reference clock, and any other clocks to be >> + switched by runtime_pm >> +- clock-names: Required only if more than a single clock is listed in >> 'clocks'. >> + The PHY reference clock must be named 'phy_refclk' >> +- phy-mode: See ethernet.txt file in the same directory >> +- phy-handle: phandle to select child phy >> + >> +Optional properties: (See ethernet.txt file in the same directory) >> +- dma-coherent: Boolean property, must only be present if memory >> + accesses performed by the device are cache coherent >> +- local-mac-address >> +- mac-address >> +- max-speed >> +- max-frame-size >> + >> +Required properties for the child phy: >> +- reg: phy address > > Hi Jassi > > Just reference phy.txt > >> + >> +Example: >> + eth0: netsec@522D { >> + compatible = "socionext,synquacer-netsec"; >> + reg = <0 0x522D 0x0 0x1>, <0 0x1000 0x0 0x1>; >> + interrupts = ; >> + clocks = <_netsec>; >> + phy-mode = "rgmii"; >> + max-speed = <1000>; >> + max-frame-size = <9000>; >> + phy-handle = <>; >> + >> + #address-cells = <1>; >> + #size-cells = <0>; >> + > > Please add an mdio node here, and list all the phys and possibly > Ethernet switches as children of it. > OK. Though in order to avoid breaking dtbs in the wild already, the driver falls back on using the parent node if no mdio subnode is found. Thank you.
Re: [PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
Hi Mark, On Tue, Dec 12, 2017 at 10:59 PM, Mark Rutlandwrote: > Hi, > > On Tue, Dec 12, 2017 at 10:45:21PM +0530, jassisinghb...@gmail.com wrote: >> From: Jassi Brar >> >> This patch adds documentation for Device-Tree bindings for the >> Socionext NetSec Controller driver. >> >> Signed-off-by: Ard Biesheuvel >> Signed-off-by: Jassi Brar >> --- >> .../devicetree/bindings/net/socionext-netsec.txt | 43 >> ++ >> 1 file changed, 43 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/net/socionext-netsec.txt >> >> diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt >> b/Documentation/devicetree/bindings/net/socionext-netsec.txt >> new file mode 100644 >> index 000..4695969 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt >> @@ -0,0 +1,45 @@ >> +* Socionext NetSec Ethernet Controller IP >> + >> +Required properties: >> +- compatible: Should be "socionext,synquacer-netsec" >> +- reg: Address and length of the control register area, followed by the >> + address and length of the EEPROM holding the MAC address and >> + microengine firmware >> +- interrupts: Should contain ethernet controller interrupt >> +- clocks: phandle to the PHY reference clock, and any other clocks to be >> + switched by runtime_pm >> +- clock-names: Required only if more than a single clock is listed in >> 'clocks'. >> + The PHY reference clock must be named 'phy_refclk' > > Please define the full set of clocks (and their names) explicitly. This > should be well-known. > The issue is some implementations have just the 'rate-reference' clock going in, while others may also have 1or2 optional 'enable' clocks (which may go to other devices as well). The driver only needs to know which clock to read the freq from, so it expects that clock to be named 'phy_refclk', while the 'enable' clocks can be named anything. Thanks
Re: [PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
On Tue, Dec 12, 2017 at 10:45:21PM +0530, jassisinghb...@gmail.com wrote: > From: Jassi Brar> > This patch adds documentation for Device-Tree bindings for the > Socionext NetSec Controller driver. > > Signed-off-by: Ard Biesheuvel > Signed-off-by: Jassi Brar > --- > .../devicetree/bindings/net/socionext-netsec.txt | 43 > ++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/socionext-netsec.txt > > diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt > b/Documentation/devicetree/bindings/net/socionext-netsec.txt > new file mode 100644 > index 000..4695969 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt > @@ -0,0 +1,45 @@ > +* Socionext NetSec Ethernet Controller IP > + > +Required properties: > +- compatible: Should be "socionext,synquacer-netsec" > +- reg: Address and length of the control register area, followed by the > + address and length of the EEPROM holding the MAC address and > + microengine firmware > +- interrupts: Should contain ethernet controller interrupt > +- clocks: phandle to the PHY reference clock, and any other clocks to be > + switched by runtime_pm > +- clock-names: Required only if more than a single clock is listed in > 'clocks'. > + The PHY reference clock must be named 'phy_refclk' > +- phy-mode: See ethernet.txt file in the same directory > +- phy-handle: phandle to select child phy > + > +Optional properties: (See ethernet.txt file in the same directory) > +- dma-coherent: Boolean property, must only be present if memory > + accesses performed by the device are cache coherent > +- local-mac-address > +- mac-address > +- max-speed > +- max-frame-size > + > +Required properties for the child phy: > +- reg: phy address Hi Jassi Just reference phy.txt > + > +Example: > + eth0: netsec@522D { > + compatible = "socionext,synquacer-netsec"; > + reg = <0 0x522D 0x0 0x1>, <0 0x1000 0x0 0x1>; > + interrupts = ; > + clocks = <_netsec>; > + phy-mode = "rgmii"; > + max-speed = <1000>; > + max-frame-size = <9000>; > + phy-handle = <>; > + > + #address-cells = <1>; > + #size-cells = <0>; > + Please add an mdio node here, and list all the phys and possibly Ethernet switches as children of it. Andrew
Re: [PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
Hi, On Tue, Dec 12, 2017 at 10:45:21PM +0530, jassisinghb...@gmail.com wrote: > From: Jassi Brar> > This patch adds documentation for Device-Tree bindings for the > Socionext NetSec Controller driver. > > Signed-off-by: Ard Biesheuvel > Signed-off-by: Jassi Brar > --- > .../devicetree/bindings/net/socionext-netsec.txt | 43 > ++ > 1 file changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/socionext-netsec.txt > > diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt > b/Documentation/devicetree/bindings/net/socionext-netsec.txt > new file mode 100644 > index 000..4695969 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt > @@ -0,0 +1,45 @@ > +* Socionext NetSec Ethernet Controller IP > + > +Required properties: > +- compatible: Should be "socionext,synquacer-netsec" > +- reg: Address and length of the control register area, followed by the > + address and length of the EEPROM holding the MAC address and > + microengine firmware > +- interrupts: Should contain ethernet controller interrupt > +- clocks: phandle to the PHY reference clock, and any other clocks to be > + switched by runtime_pm > +- clock-names: Required only if more than a single clock is listed in > 'clocks'. > + The PHY reference clock must be named 'phy_refclk' Please define the full set of clocks (and their names) explicitly. This should be well-known. Otherwise, this looks ok. Thanks, Mark. > +- phy-mode: See ethernet.txt file in the same directory > +- phy-handle: phandle to select child phy > + > +Optional properties: (See ethernet.txt file in the same directory) > +- dma-coherent: Boolean property, must only be present if memory > + accesses performed by the device are cache coherent > +- local-mac-address > +- mac-address > +- max-speed > +- max-frame-size > + > +Required properties for the child phy: > +- reg: phy address > + > +Example: > + eth0: netsec@522D { > + compatible = "socionext,synquacer-netsec"; > + reg = <0 0x522D 0x0 0x1>, <0 0x1000 0x0 0x1>; > + interrupts = ; > + clocks = <_netsec>; > + phy-mode = "rgmii"; > + max-speed = <1000>; > + max-frame-size = <9000>; > + phy-handle = <>; > + > + #address-cells = <1>; > + #size-cells = <0>; > + > + ethphy0: ethernet-phy@1 { > + compatible = "ethernet-phy-ieee802.3-c22"; > + reg = <1>; > + }; > + }; > -- > 2.7.4 >
[PATCHv2 1/3] dt-bindings: net: Add DT bindings for Socionext Netsec
From: Jassi BrarThis patch adds documentation for Device-Tree bindings for the Socionext NetSec Controller driver. Signed-off-by: Ard Biesheuvel Signed-off-by: Jassi Brar --- .../devicetree/bindings/net/socionext-netsec.txt | 43 ++ 1 file changed, 43 insertions(+) create mode 100644 Documentation/devicetree/bindings/net/socionext-netsec.txt diff --git a/Documentation/devicetree/bindings/net/socionext-netsec.txt b/Documentation/devicetree/bindings/net/socionext-netsec.txt new file mode 100644 index 000..4695969 --- /dev/null +++ b/Documentation/devicetree/bindings/net/socionext-netsec.txt @@ -0,0 +1,45 @@ +* Socionext NetSec Ethernet Controller IP + +Required properties: +- compatible: Should be "socionext,synquacer-netsec" +- reg: Address and length of the control register area, followed by the + address and length of the EEPROM holding the MAC address and + microengine firmware +- interrupts: Should contain ethernet controller interrupt +- clocks: phandle to the PHY reference clock, and any other clocks to be + switched by runtime_pm +- clock-names: Required only if more than a single clock is listed in 'clocks'. + The PHY reference clock must be named 'phy_refclk' +- phy-mode: See ethernet.txt file in the same directory +- phy-handle: phandle to select child phy + +Optional properties: (See ethernet.txt file in the same directory) +- dma-coherent: Boolean property, must only be present if memory +accesses performed by the device are cache coherent +- local-mac-address +- mac-address +- max-speed +- max-frame-size + +Required properties for the child phy: +- reg: phy address + +Example: + eth0: netsec@522D { + compatible = "socionext,synquacer-netsec"; + reg = <0 0x522D 0x0 0x1>, <0 0x1000 0x0 0x1>; + interrupts = ; + clocks = <_netsec>; + phy-mode = "rgmii"; + max-speed = <1000>; + max-frame-size = <9000>; + phy-handle = <>; + + #address-cells = <1>; + #size-cells = <0>; + + ethphy0: ethernet-phy@1 { + compatible = "ethernet-phy-ieee802.3-c22"; + reg = <1>; + }; + }; -- 2.7.4