2017年2月17日 04:58于 Florian Fainelli <[email protected]>写道:
>
> On 02/16/2017 04:48 AM, Corentin Labbe wrote: 
> > This patch adds documentation for Device-Tree bindings for the 
> > Allwinner dwmac-sun8i driver. 
> > 
> > Signed-off-by: Corentin Labbe <[email protected]> 
> > --- 
> >  .../devicetree/bindings/net/dwmac-sun8i.txt        | 86 
> >++++++++++++++++++++++ 
> >  1 file changed, 86 insertions(+) 
> >  create mode 100644 Documentation/devicetree/bindings/net/dwmac-sun8i.txt 
> > 
> > diff --git a/Documentation/devicetree/bindings/net/dwmac-sun8i.txt 
> > b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt 
> > new file mode 100644 
> > index 0000000..ac806c6 
> > --- /dev/null 
> > +++ b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt 
> > @@ -0,0 +1,86 @@ 
> > +* Allwinner sun8i GMAC ethernet controller 
> > + 
> > +This device is a platform glue layer for stmmac. 
> > +Please see stmmac.txt for the other unchanged properties. 
> > + 
> > +Required properties: 
> > +- compatible: should be one of the following string: 
> > + "allwinner,sun8i-a83t-emac" 
> > + "allwinner,sun8i-h3-emac" 
> > + "allwinner,sun50i-a64-emac" 
> > +- reg: address and length of the register for the device. 
> > +- interrupts: interrupt for the device 
> > +- interrupt-names: should be "macirq" 
> > +- clocks: A phandle to the reference clock for this device 
> > +- clock-names: should be "stmmaceth" 
> > +- resets: A phandle to the reset control for this device 
> > +- reset-names: should be "stmmaceth" 
> > +- phy-mode: See ethernet.txt 
> > +- phy-handle: See ethernet.txt 
> > +- #address-cells: shall be 1 
> > +- #size-cells: shall be 0 
> > +- syscon: A phandle to the syscon of the SoC with one of the following 
> > + compatible string: 
> > +  - allwinner,sun8i-h3-system-controller 
> > +  - allwinner,sun8i-a64-system-controller 
> > +  - allwinner,sun8i-a83t-system-controller 
> > + 
> > +Optional properties: 
> > +- allwinner,tx-delay: TX clock delay chain value. Range value is 0-0x07. 
> > Default is 0) 
> > +- allwinner,rx-delay: RX clock delay chain value. Range value is 0-0x1F. 
> > Default is 0) 
>
> Delays should be specified in nanosecond units, and should not be a 
> value that maps directly to a HW register value, you need to convert the 
> property value into the appropriate register value. 
>
> > +Both delay properties does not have units, there are arbitrary value. 
> > +The TX/RX clock delay chain settings are board specific and could be found 
> > +in vendor FEX files. 
> > + 
> > +Optional properties for "allwinner,sun8i-h3-emac": 
> > +- allwinner,leds-active-low: EPHY LEDs are active low 
>
> Are you sure this is appropriate at the MAC node level and this is not 
> something that follows the PHY instead? 

I think it's in fact for the internal MII PHY.

Allwinner H3/H5/V3s features a internal PHY in the SoC (V3s even lacks an 
external MII interface), which is also handled in this driver.

>
> > + 
> > +Required child node of emac: 
> > +- mdio bus node: should be named mdio 
> > + 
> > +Required properties of the mdio node: 
> > +- #address-cells: shall be 1 
> > +- #size-cells: shall be 0 
> > + 
> > +The device node referenced by "phy" or "phy-handle" should be a child node 
> > +of the mdio node. See phy.txt for the generic PHY bindings. 
> > + 
> > +Required properties of the phy node with "allwinner,sun8i-h3-emac": 
> > +- clocks: an extra phandle to the reference clock for the EPHY 
>
> s/an extra/a phandle/ 
>
> > +- resets: an extra phandle to the reset control for the EPHY 
> > + 
> > +Required properties for the system controller: 
> > +- reg: address and length of the register for the device. 
> > +- compatible: should be "syscon" and one of the following string: 
> > + "allwinner,sun8i-h3-system-controller" 
> > + "allwinner,sun8i-a64-system-controller" 
> > + "allwinner,sun8i-a83t-system-controller" 
> > + 
> > +Example: 
> > + 
> > +emac: ethernet@1c0b000 { 
> > + compatible = "allwinner,sun8i-h3-emac"; 
> > + syscon = <&syscon>; 
> > + reg = <0x01c0b000 0x104>; 
> > + interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_HIGH>; 
> > + interrupt-names = "macirq"; 
> > + resets = <&ccu RST_BUS_EMAC>; 
> > + reset-names = "stmmaceth"; 
> > + clocks = <&ccu CLK_BUS_EMAC>; 
> > + clock-names = "stmmaceth"; 
> > + #address-cells = <1>; 
> > + #size-cells = <0>; 
> > + 
> > + phy = <&int_mii_phy>; 
> > + phy-mode = "mii"; 
> > + allwinner,leds-active-low; 
> > + mdio: mdio { 
> > + #address-cells = <1>; 
> > + #size-cells = <0>; 
> > + int_mii_phy: ethernet-phy@1 { 
> > + reg = <1>; 
> > + clocks = <&ccu CLK_BUS_EPHY>; 
> > + resets = <&ccu RST_BUS_EPHY>; 
> > + }; 
> > + }; 
> > +}; 
> > 
>
>
> -- 
> Florian 
>
> -- 
> You received this message because you are subscribed to the Google Groups 
> "linux-sunxi" group. 
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to [email protected]. 
> For more options, visit https://groups.google.com/d/optout. 

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to