Re: [linux-sunxi] [PATCH 3/4] sunxi: add stub EMAC device node in A83T device tree

2017-07-06 Thread Andre Przywara
Hi,

On 02/07/17 08:02, Icenowy Zheng wrote:
> The Allwinner A83T SoC has an EMAC which is already supported by
> sun8i_emac driver in U-Boot now.
> 
> Add a stub device node for it.
> 
> The device node cannot work for Linux, because it now lacks the proper
> clock definition; however, it can satisfy sun8i_emac driver in U-Boot.

if you rebase your series on top of mine [1], you should be able to
directly use mainline Linux DT nodes, namely ...

> 
> Signed-off-by: Icenowy Zheng 
> ---
>  arch/arm/dts/sun8i-a83t.dtsi | 25 +
>  1 file changed, 25 insertions(+)
> 
> diff --git a/arch/arm/dts/sun8i-a83t.dtsi b/arch/arm/dts/sun8i-a83t.dtsi
> index 0fe73e173f..9aac3a7929 100644
> --- a/arch/arm/dts/sun8i-a83t.dtsi
> +++ b/arch/arm/dts/sun8i-a83t.dtsi
> @@ -52,6 +52,10 @@
>  / {
>   interrupt-parent = <>;
>  
> + aliases {
> + ethernet0 = 
> + };
> +
>   cpus {
>   #address-cells = <1>;
>   #size-cells = <0>;
> @@ -166,6 +170,17 @@
>   #interrupt-cells = <3>;
>   #gpio-cells = <3>;
>  
> + emac_rgmii_pins: emac-rgmii {
> + allwinner,pins = "PD2", "PD3", "PD4", "PD5",
> + "PD6", "PD7", "PD11",
> + "PD12", "PD13", "PD14",
> + "PD18", "PD19", "PD21",
> + "PD22", "PD23";
> + allwinner,function = "emac";
> + allwinner,drive = ;
> + allwinner,pull = ;

... using the new generic pinctrl bindings here ...

> + };
> +
>   mmc0_pins_a: mmc0@0 {
>   allwinner,pins = "PF0", "PF1", "PF2",
>"PF3", "PF4", "PF5";
> @@ -214,6 +229,16 @@
>   status = "disabled";
>   };
>  
> + emac: ethernet@1c3 {
> + compatible = "allwinner,sun8i-a83t-emac";
> + reg = <0x01c3 0x104>, <0x01c00030 0x4>;
> + reg-names = "emac", "syscon";
> + interrupts = ;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + status = "disabled";

... and using the new binding scheme here, with a "syscon" property, for
instance.

Also the Linux binding requires an mdio child node, which should be
introduced here.

Cheers,
Andre.


> + };
> +
>   gic: interrupt-controller@01c81000 {
>   compatible = "arm,cortex-a7-gic", "arm,cortex-a15-gic";
>   reg = <0x01c81000 0x1000>,
> 

-- 
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 linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[linux-sunxi] [PATCH 3/4] sunxi: add stub EMAC device node in A83T device tree

2017-07-02 Thread Icenowy Zheng
The Allwinner A83T SoC has an EMAC which is already supported by
sun8i_emac driver in U-Boot now.

Add a stub device node for it.

The device node cannot work for Linux, because it now lacks the proper
clock definition; however, it can satisfy sun8i_emac driver in U-Boot.

Signed-off-by: Icenowy Zheng 
---
 arch/arm/dts/sun8i-a83t.dtsi | 25 +
 1 file changed, 25 insertions(+)

diff --git a/arch/arm/dts/sun8i-a83t.dtsi b/arch/arm/dts/sun8i-a83t.dtsi
index 0fe73e173f..9aac3a7929 100644
--- a/arch/arm/dts/sun8i-a83t.dtsi
+++ b/arch/arm/dts/sun8i-a83t.dtsi
@@ -52,6 +52,10 @@
 / {
interrupt-parent = <>;
 
+   aliases {
+   ethernet0 = 
+   };
+
cpus {
#address-cells = <1>;
#size-cells = <0>;
@@ -166,6 +170,17 @@
#interrupt-cells = <3>;
#gpio-cells = <3>;
 
+   emac_rgmii_pins: emac-rgmii {
+   allwinner,pins = "PD2", "PD3", "PD4", "PD5",
+   "PD6", "PD7", "PD11",
+   "PD12", "PD13", "PD14",
+   "PD18", "PD19", "PD21",
+   "PD22", "PD23";
+   allwinner,function = "emac";
+   allwinner,drive = ;
+   allwinner,pull = ;
+   };
+
mmc0_pins_a: mmc0@0 {
allwinner,pins = "PF0", "PF1", "PF2",
 "PF3", "PF4", "PF5";
@@ -214,6 +229,16 @@
status = "disabled";
};
 
+   emac: ethernet@1c3 {
+   compatible = "allwinner,sun8i-a83t-emac";
+   reg = <0x01c3 0x104>, <0x01c00030 0x4>;
+   reg-names = "emac", "syscon";
+   interrupts = ;
+   #address-cells = <1>;
+   #size-cells = <0>;
+   status = "disabled";
+   };
+
gic: interrupt-controller@01c81000 {
compatible = "arm,cortex-a7-gic", "arm,cortex-a15-gic";
reg = <0x01c81000 0x1000>,
-- 
2.13.0

-- 
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 linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.