From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755375AbcHWUEN (ORCPT ); Tue, 23 Aug 2016 16:04:13 -0400 Received: from down.free-electrons.com ([37.187.137.238]:42540 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753600AbcHWUEL (ORCPT ); Tue, 23 Aug 2016 16:04:11 -0400 Date: Tue, 23 Aug 2016 22:03:49 +0200 From: Maxime Ripard To: Andre Przywara Cc: Chen-Yu Tsai , linux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Herring , Mark Rutland , devicetree@vger.kernel.org Subject: Re: [PATCH v4 5/7] arm64: dts: add Allwinner A64 SoC .dtsi Message-ID: <20160823200349.GR2598@lukather> References: <20160808172149.30861-1-andre.przywara@arm.com> <20160808172149.30861-6-andre.przywara@arm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="MRBOAUz+O/XNC2GI" Content-Disposition: inline In-Reply-To: <20160808172149.30861-6-andre.przywara@arm.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --MRBOAUz+O/XNC2GI Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Mon, Aug 08, 2016 at 06:21:47PM +0100, Andre Przywara wrote: > + pmu { > + compatible =3D "arm,cortex-a53-pmu", "arm,armv8-pmuv3"; > + interrupts =3D , > + , > + , > + ; > + interrupt-affinity =3D <&cpu0>, > + <&cpu1>, > + <&cpu2>, > + <&cpu3>; > + }; The indentation looks off. > + > + psci { > + compatible =3D "arm,psci-0.2"; > + method =3D "smc"; > + }; > + > + memory { > + device_type =3D "memory"; > + reg =3D <0x40000000 0>; > + }; > + > + gic: interrupt-controller@1c81000 { > + compatible =3D "arm,gic-400"; > + interrupt-controller; > + #interrupt-cells =3D <3>; > + #address-cells =3D <0>; > + > + reg =3D <0x01c81000 0x1000>, > + <0x01c82000 0x2000>, > + <0x01c84000 0x2000>, > + <0x01c86000 0x2000>; > + interrupts =3D + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > + }; > + > + timer { > + compatible =3D "arm,armv8-timer"; > + interrupts =3D + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > + }; > + > + /include/ "sun50i-a64-clocks.dtsi" > + > + soc { > + compatible =3D "simple-bus"; > + #address-cells =3D <1>; > + #size-cells =3D <1>; > + ranges; > + > + pio: pinctrl@1c20800 { > + compatible =3D "allwinner,sun50i-a64-pinctrl"; > + reg =3D <0x01c20800 0x400>; > + interrupts =3D , > + , > + ; > + clocks =3D <&bus_gates 69>; > + gpio-controller; > + #gpio-cells =3D <3>; > + interrupt-controller; > + #interrupt-cells =3D <2>; > + > + uart0_pins_a: uart0@0 { > + allwinner,pins =3D "PB8", "PB9"; > + allwinner,function =3D "uart0"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart0_pins_b: uart0@1 { > + allwinner,pins =3D "PF2", "PF3"; > + allwinner,function =3D "uart0"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart1_2pins: uart1_2@0 { > + allwinner,pins =3D "PG6", "PG7"; > + allwinner,function =3D "uart1"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart1_4pins: uart1_4@0 { > + allwinner,pins =3D "PG6", "PG7", "PG8", "PG9"; > + allwinner,function =3D "uart1"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart2_2pins: uart2_2@0 { > + allwinner,pins =3D "PB0", "PB1"; > + allwinner,function =3D "uart2"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart2_4pins: uart2_4@0 { > + allwinner,pins =3D "PB0", "PB1", "PB2", "PB3"; > + allwinner,function =3D "uart2"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart3_pins_a: uart3@0 { > + allwinner,pins =3D "PD0", "PD1"; > + allwinner,function =3D "uart3"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart3_2pins_b: uart3_2@1 { > + allwinner,pins =3D "PH4", "PH5"; > + allwinner,function =3D "uart3"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart3_4pins_b: uart3_4@1 { > + allwinner,pins =3D "PH4", "PH5", "PH6", "PH7"; > + allwinner,function =3D "uart3"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart4_2pins: uart4_2@0 { > + allwinner,pins =3D "PD2", "PD3"; > + allwinner,function =3D "uart4"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + uart4_4pins: uart4_4@0 { > + allwinner,pins =3D "PD2", "PD3", "PD4", "PD5"; > + allwinner,function =3D "uart4"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + mmc0_pins: mmc0@0 { > + allwinner,pins =3D "PF0", "PF1", "PF2", "PF3", > + "PF4", "PF5"; > + allwinner,function =3D "mmc0"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + mmc0_default_cd_pin: mmc0_cd_pin@0 { > + allwinner,pins =3D "PF6"; > + allwinner,function =3D "gpio_in"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + mmc1_pins: mmc1@0 { > + allwinner,pins =3D "PG0", "PG1", "PG2", "PG3", > + "PG4", "PG5"; > + allwinner,function =3D "mmc1"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + mmc2_pins: mmc2@0 { > + allwinner,pins =3D "PC1", "PC5", "PC6", "PC8", > + "PC9", "PC10", "PC11", "PC12", > + "PC13", "PC14", "PC15", "PC16"; > + allwinner,function =3D "mmc2"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + i2c0_pins: i2c0_pins { > + allwinner,pins =3D "PH0", "PH1"; > + allwinner,function =3D "i2c0"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + i2c1_pins: i2c1_pins { > + allwinner,pins =3D "PH2", "PH3"; > + allwinner,function =3D "i2c1"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + > + i2c2_pins: i2c2_pins { > + allwinner,pins =3D "PE14", "PE15"; > + allwinner,function =3D "i2c2"; > + allwinner,drive =3D ; > + allwinner,pull =3D ; > + }; > + }; Our policy is that we only add the pinctrl nodes that we actually use in boards to avoid bloating the DT with unused nodes. Thanks, Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --MRBOAUz+O/XNC2GI Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXvKwlAAoJEBx+YmzsjxAgIUUP/Amew1Ea+Vd3U3iETXq9/RNy JL3XK3HzdXbYNWN9u/tF1oRiG2GeBuKhXK11S4U6x5Xfh0UbznUErWwRMUGiFXUH exnlPZu3DyXnFx6ffeDE8VeKCNH0ZxYDmbo8krZIj60SSj8vGZl/YJVQVeNdukRI FlTxA4H5nAmbtsvUKhJuuIDSg9P1XqvTYJFrYL9eJtxJoxaQefq78Cbkm2zvkN0z ZxT533knR3onHwLkLuIzINqWbq8FLDGGTpWcmYVBXUQkH4B3rOrN6pDp5Ra9xlze +0voj368FUYAdQW/jsZDMXzDpcpM+uRXSUsMOJu40hkMWg9mAzoY3NWbdzqlLl6n mDB3dPSddT8fiB7LNDKyoxbtDLnGbsa52epiHCe4ffQZ5TgOXLEU3TuveV8HAG/1 zKdbVmKryOXzMOqcgUzoO4/B6HjJGjrQIW/IDPsr06NGDEBy/X4cqceaH1nawSCy oot46ejHb21GvXfLkLyUVCXqU+Fx8q5W8ZgJNnqQMLAbymbLIgxc2sI9FF0/CZB4 c9PqhPPqmnvcIzSXAnAN8vFwLhf/lsdReUHXXNy7lg7q39uQzI/ZRtav7qMvnjLb RCzoM8eWQmtSTpBwFgdFuuq5KiCi4DqETeRHezHATf5+53/qWrn8lezld0mAzJDE nwR/p9HzE7TV6pZH3Bt3 =BcDe -----END PGP SIGNATURE----- --MRBOAUz+O/XNC2GI--