From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759777AbbKTLMz (ORCPT ); Fri, 20 Nov 2015 06:12:55 -0500 Received: from down.free-electrons.com ([37.187.137.238]:36587 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758812AbbKTLMw (ORCPT ); Fri, 20 Nov 2015 06:12:52 -0500 Date: Fri, 20 Nov 2015 12:12:38 +0100 From: Maxime Ripard To: Chen-Yu Tsai Cc: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Lee Jones , Samuel Ortiz , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Hans de Goede , Mark Brown Subject: Re: [PATCH v3 5/9] ARM: dts: sun8i: sinlinx-sina33: Add AXP223 PMIC device and regulator nodes Message-ID: <20151120111238.GV32142@lukather> References: <1447691908-25456-1-git-send-email-wens@csie.org> <1447691908-25456-6-git-send-email-wens@csie.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="YwTTlJgQ7QoYB9ta" Content-Disposition: inline In-Reply-To: <1447691908-25456-6-git-send-email-wens@csie.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --YwTTlJgQ7QoYB9ta Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Tue, Nov 17, 2015 at 12:38:24AM +0800, Chen-Yu Tsai wrote: > This board has a X-Powers AXP223 PMIC connected via RSB. Its regulators > provide power to various parts of the SoC and the board. >=20 > Also update the regulator supply phandles. >=20 > Signed-off-by: Chen-Yu Tsai > --- > arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts | 79 ++++++++++++++++++++= +++++- > 1 file changed, 76 insertions(+), 3 deletions(-) >=20 > diff --git a/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts b/arch/arm/bo= ot/dts/sun8i-a33-sinlinx-sina33.dts > index 13ce68f06dd6..91a0fde47fdd 100644 > --- a/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts > +++ b/arch/arm/boot/dts/sun8i-a33-sinlinx-sina33.dts > @@ -68,7 +68,7 @@ > }; > =20 > &lradc { > - vref-supply =3D <®_vcc3v0>; > + vref-supply =3D <®_dcdc1>; > status =3D "okay"; > =20 > button@200 { > @@ -96,7 +96,7 @@ > &mmc0 { > pinctrl-names =3D "default"; > pinctrl-0 =3D <&mmc0_pins_a>, <&mmc0_cd_pin_sina33>; > - vmmc-supply =3D <®_vcc3v0>; > + vmmc-supply =3D <®_dcdc1>; > bus-width =3D <4>; > cd-gpios =3D <&pio 1 4 GPIO_ACTIVE_HIGH>; /* PB4 */ > cd-inverted; > @@ -106,7 +106,7 @@ > &mmc2 { > pinctrl-names =3D "default"; > pinctrl-0 =3D <&mmc2_8bit_pins>; > - vmmc-supply =3D <®_vcc3v0>; > + vmmc-supply =3D <®_dcdc1>; > bus-width =3D <8>; > non-removable; > status =3D "okay"; > @@ -132,6 +132,79 @@ > =20 > &r_rsb { > status =3D "okay"; > + > + axp22x: pmic@3e3 { > + compatible =3D "x-powers,axp223"; > + reg =3D <0x3e3>; > + interrupt-parent =3D <&nmi_intc>; > + interrupts =3D <0 IRQ_TYPE_LEVEL_LOW>; > + eldoin-supply =3D <®_dcdc1>; > + }; > +}; > + > +#include "axp22x.dtsi" > + > +®_aldo1 { > + regulator-always-on; > + regulator-min-microvolt =3D <3000000>; > + regulator-max-microvolt =3D <3000000>; > + regulator-name =3D "vcc-io"; > +}; > + > +®_aldo2 { > + regulator-always-on; > + regulator-min-microvolt =3D <2350000>; > + regulator-max-microvolt =3D <2650000>; > + regulator-name =3D "vdd-dll"; > +}; > + > +®_aldo3 { > + regulator-always-on; > + regulator-min-microvolt =3D <2700000>; > + regulator-max-microvolt =3D <3300000>; > + regulator-name =3D "vcc-pll-avcc"; > +}; > + > +®_dc5ldo { > + regulator-always-on; > + regulator-min-microvolt =3D <900000>; > + regulator-max-microvolt =3D <1400000>; > + regulator-name =3D "vdd-cpus"; > +}; > + > +®_dcdc1 { > + regulator-always-on; > + regulator-min-microvolt =3D <3000000>; > + regulator-max-microvolt =3D <3000000>; > + regulator-name =3D "vcc-3v0"; > +}; > + > +®_dcdc2 { > + regulator-always-on; > + regulator-min-microvolt =3D <900000>; > + regulator-max-microvolt =3D <1400000>; > + regulator-name =3D "vdd-sys"; > +}; > + > +®_dcdc3 { > + regulator-always-on; > + regulator-min-microvolt =3D <900000>; > + regulator-max-microvolt =3D <1400000>; > + regulator-name =3D "vdd-cpu"; > +}; > + > +®_dcdc5 { > + regulator-always-on; > + regulator-min-microvolt =3D <1500000>; > + regulator-max-microvolt =3D <1500000>; > + regulator-name =3D "vcc-dram"; > +}; > + > +®_rtc_ldo { > + regulator-always-on; > + regulator-min-microvolt =3D <3000000>; > + regulator-max-microvolt =3D <3000000>; > + regulator-name =3D "vcc-rtc"; > }; Isn't this supposed to be in the AXP DTSI? Thanks! Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --YwTTlJgQ7QoYB9ta Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWTwAmAAoJEBx+YmzsjxAgg3EP/io5absB6NKa46IQYds2C9Pu owXgTr7b0Lg87Z41CIC5PvOxzVL9aagnIoqi/2mjwDzjxYTMS2WbXXyk28LRwBx5 AMmLCsYejAEInpXxyuNUaWwfvwubjoJIaIdowtgkuKl61J36dPF0j8piJxYgZw1A N6NvDQC4ocgUAIA76iE14On5K429xBeVqzbjRt68IRu1d2zwX5Azp1eHoRNHJWLV 2E5gGP39+gE0/+Ll933/NgNt1Vt0La/bT/2FF383VBTxKcO8GB2KXvPw7jLSAWbc cE+Hc2S3nliYoNFmjm1tgHGHYTRvUFfjAIUjJGzx82Ix7wKfcU8DzUqj/8q777Wj uRQgCk5TSNDj/0cbpTVdawk67XXJaGKLnqP4DqTDzK2yN91ZLKaqh03Pj7oJLWJ1 ZtGwIKR1Kbh+msUAMZtHNkcRPsfalchTfOULIU/Py/Rx1TCUmJvBB+IMAfMQ681F 9/hHSkzPBaYvTHgqCrS+LeOkW5+x3yE0amKXRUXApCkh78rrg24D+vE96eIlYWij N3gQUvjzgOt2rxvWJPjHMxJU+/DBwfrtMhcQllJZbZjspYV4jttqjJy9ICatMQjY YfZC7d1gHsCPbcBn6Xf67U54wOrU2HlU0625OchcswahN5kFzAAYQ99ZQ2i//9ju fQvUNQLL78e0hjdD06Q9 =PXDM -----END PGP SIGNATURE----- --YwTTlJgQ7QoYB9ta--