Thread (33 messages) 33 messages, 3 authors, 2017-02-21

[PATCH 08/21] ARM: dts: sun8i-h3: add dwmac-sun8i rgmii pins

From: Maxime Ripard <hidden>
Date: 2017-02-17 08:40:55
Also in: linux-devicetree, lkml, netdev

On Thu, Feb 16, 2017 at 01:48:46PM +0100, Corentin Labbe wrote:
quoted hunk ↗ jump to hunk
This patch add pinctrl node for dwmac-sun8i on H3.

Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
 arch/arm/boot/dts/sun8i-h3.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-h3.dtsi b/arch/arm/boot/dts/sun8i-h3.dtsi
index 61d56d2..59ed40e 100644
--- a/arch/arm/boot/dts/sun8i-h3.dtsi
+++ b/arch/arm/boot/dts/sun8i-h3.dtsi
@@ -349,6 +349,17 @@
 				function = "i2c2";
 			};
 
+			emac_rgmii_pins: emac0 at 0 {
+				allwinner,pins = "PD0", "PD1", "PD2", "PD3",
+						"PD4", "PD5", "PD7",
+						"PD8", "PD9", "PD10",
+						"PD12", "PD13", "PD15",
+						"PD16", "PD17";
+				allwinner,function = "emac";
Please use the generic pin config properties (ie. pins and functions).
+				allwinner,drive = <SUN4I_PINCTRL_40_MA>;
Why do you need to use 40mA?
+				allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
This is the default now.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170216/27dd5072/attachment-0001.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help