Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp942231imu; Wed, 28 Nov 2018 01:58:54 -0800 (PST) X-Google-Smtp-Source: AFSGD/W1CaHw8+s9Wj5arLBF52pGRCXDf7na1H+jqHvuzYAVKcq4LD5onNUNQgqxhnWm1t9YuN9O X-Received: by 2002:a63:24c2:: with SMTP id k185mr31689426pgk.406.1543399133917; Wed, 28 Nov 2018 01:58:53 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543399133; cv=none; d=google.com; s=arc-20160816; b=J9LwSy4w1VsVLG1XVHHWeoZv+toX8lFn0wxk0hGbP6aBf9fPvoWVze3MueB9nbaVdh OCMaRVJ5uq848GYug3oi8h6gue2gDEn+A05tMFc99OlSmBykCus9Am4frK6JsdANf1az qRpcM2e4X2KMy594tcErqwxYDJl6tcVlvMaNW/hc2j4BFmp9mhe4go75WmvGqUdx81HK mlTMC9I7BK+ViraDLdlzf/8WNqMIhHHzrXlIBJ2R8sQUfsyIzC+WwfqEVr92zUSj1jFU HXtKUqtacvntzTGm7e3XDw9nrOE8JnYG7mlgnA78fTZz7LJ/XKTUtMw+J8jzacyJeOru dgdQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=tK27Och5fRdRkDJbzAptKL3PiMH+Pq9Fb8bvmMC/Vpc=; b=i2fj6WHRfJy3GSjxGlFpH2DJX2/2PIf1z3+kbc7pmK1B0z8Z795g7T7gNxTS20mM+z 7Fyk5SkyP5E/ET+KhQcvi0+JBCZAQAD6DIboVqTVGOc3r5prQgCq7q2I2Fu+9qq1DYxt Oujsadk3/dhtw2JWswK/acLY5KlVDfB5B67DdTuKjg6/Xrc5DKAhOD2fQOrudFky+8rQ Q6YG/Bcyi5m7Bb1TOkeBxwwaFuJ94an9674SseQEnRfg3jm71eHct/31yEhemsXbnM0b 3HLNE5raPndp86Z6Fgr7dvwYlwBWtzVbeU/5PfpypXMWrGendOIC8owuhXdENlmO+WcQ LVBw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y40-v6si7021710pla.391.2018.11.28.01.58.39; Wed, 28 Nov 2018 01:58:53 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727783AbeK1U54 (ORCPT + 99 others); Wed, 28 Nov 2018 15:57:56 -0500 Received: from mail.bootlin.com ([62.4.15.54]:45631 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727382AbeK1U54 (ORCPT ); Wed, 28 Nov 2018 15:57:56 -0500 Received: by mail.bootlin.com (Postfix, from userid 110) id B35C220701; Wed, 28 Nov 2018 10:56:49 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mail.bootlin.com X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT, URIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.2 Received: from localhost (aaubervilliers-681-1-94-205.w90-88.abo.wanadoo.fr [90.88.35.205]) by mail.bootlin.com (Postfix) with ESMTPSA id 79E0C2039F; Wed, 28 Nov 2018 10:56:39 +0100 (CET) Date: Wed, 28 Nov 2018 10:56:39 +0100 From: Maxime Ripard To: Priit Laes Cc: Lee Jones , Rob Herring , Mark Rutland , Chen-Yu Tsai , Sebastian Reichel , Hans de Goede , Liam Girdwood , Mark Brown , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org, Olliver Schinagl Subject: Re: [PATCH 09/14] regulator: dts: add full voltage range to LDO4 on the Lime2 Message-ID: <20181128095639.t4ogsk6ovyu2yaxu@flea> References: <0b1aa5b28cb5efe17c04150a181ef1fa4027bc55.1543245984.git-series.plaes@plaes.org> <20181127093852.2iavgo4xipjnu25l@flea> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2lymjsckr36agz46" Content-Disposition: inline In-Reply-To: <20181127093852.2iavgo4xipjnu25l@flea> User-Agent: NeoMutt/20180716 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --2lymjsckr36agz46 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Nov 27, 2018 at 10:38:52AM +0100, Maxime Ripard wrote: > On Mon, Nov 26, 2018 at 05:27:50PM +0200, Priit Laes wrote: > > From: Olliver Schinagl > >=20 > > With commit b43776d65a33b46092 ("ARM: dts: sunxi: Use axp209.dtsi for > > Olinuxino Lime2") we force them an arbitrary 2.8 volts. Granted, for > > LDO3 this may be less arbitrary, but for LDO4 this is just wrong. > >=20 > > In the defense of LDO3, LDO3 is the regulator that feeds port bank E, > > which has no other purpose then a CSI/TS interface, however the case > > may still be, that the connected IO may be just as well be 3.3 volts. > > The big misnomer is however, that the schematic names GPIO-2 pin4 > > LDO3_2.8V, rather then VDD-CSI0 or similar. > >=20 > > This is much worse for LDO4 however, which is not referenced on any > > pin, is now set to 2.8 volts, but port bank G can also support various > > other peripherals such as UARTS etc. > >=20 > > By having 2.8 volts however for LDO4, we thus now have peripherals that > > no longer function properly all of the time. > >=20 > > Ideally, we want to set a supply voltage for each port bank, but the > > monolithic nature of the sunxi pinctroller currently prevents this and > > as such, the board should at least configure the LDO4 with the proper > > ranges. > >=20 > > Until we can set the consumer at the port bank level, a child > > device-tree has to do something like: > >=20 > > ®_ldo4 { > > regulator-min-microvolt =3D <3300000>; > > regulator-max-microvolt =3D <3300000>; > > }; > >=20 > > While doing this the same way results in the same solution currently, > > we force the hack into the final devicetree rather then having it wrong > > at the board level. > >=20 > > Signed-off-by: Olliver Schinagl > > Signed-off-by: Priit Laes > > --- > > arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts | 7 ++++--- > > 1 file changed, 4 insertions(+), 3 deletions(-) > >=20 > > diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts b/arch/arm= /boot/dts/sun7i-a20-olinuxino-lime2.dts > > index ffafe97..1b9867f 100644 > > --- a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts > > +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts > > @@ -250,9 +250,10 @@ > > }; > > =20 > > ®_ldo4 { > > - regulator-min-microvolt =3D <2800000>; > > - regulator-max-microvolt =3D <2800000>; > > - regulator-name =3D "vddio-csi1"; > > + regulator-always-on; > > + regulator-min-microvolt =3D <1250000>; > > + regulator-max-microvolt =3D <3300000>; > > + regulator-name =3D "vdd-io-pg"; >=20 > As we discussed on the U-Boot ML already, this shouldn't be made > always-on but tied to the consumer device (the pinctrl one) instead. Can you test that patch (not tested): http://code.bulix.org/h02vha-514284?raw Thanks! Maxime --=20 Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --2lymjsckr36agz46 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCW/5mVwAKCRDj7w1vZxhR xVFQAQC+r68rQ7ExhxasRM817Bj/h9jucmxZwlh2jfs6MPvaCAD/RTi/ro68kT5N o+HYB8oNcmvZ0LlofxDy6ZXY8yxEMw4= =MtaY -----END PGP SIGNATURE----- --2lymjsckr36agz46--