From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AD7A3C17440 for ; Tue, 12 Nov 2019 08:11:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8272721925 for ; Tue, 12 Nov 2019 08:11:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727046AbfKLILY convert rfc822-to-8bit (ORCPT ); Tue, 12 Nov 2019 03:11:24 -0500 Received: from gloria.sntech.de ([185.11.138.130]:57632 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725775AbfKLILY (ORCPT ); Tue, 12 Nov 2019 03:11:24 -0500 Received: from ip5f5a6266.dynamic.kabel-deutschland.de ([95.90.98.102] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iURG8-0005gt-0H; Tue, 12 Nov 2019 09:10:44 +0100 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Kever Yang Cc: Soeren Moch , linux-rockchip@lists.infradead.org, Elaine Zhang , Peter Robinson , Akash Gajjar , Alexis Ballier , Andrius =?utf-8?B?xaB0aWtvbmFz?= , Andy Yan , Ezequiel Garcia , Hugh Cole-Baker , Katsuhiro Suzuki , Mark Rutland , Nick Xie , Oskari Lemmela , Pragnesh Patel , Rob Herring , Robin Murphy , Vicente Bergas , Vivek Unune , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, dianders@chromium.org Subject: Re: [PATCH 3/3] arm64: dts: rk3399: Add init voltage for vdd_log Date: Tue, 12 Nov 2019 09:10:42 +0100 Message-ID: <1780044.CQmMckQ5VN@diego> In-Reply-To: <1eaef5d5-c923-da56-b9c4-48d517b3c969@rock-chips.com> References: <20191111005158.25070-1-kever.yang@rock-chips.com> <1eaef5d5-c923-da56-b9c4-48d517b3c969@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Kever, Am Dienstag, 12. November 2019, 09:04:49 CET schrieb Kever Yang: > > On 2019/11/11 下午4:42, Soeren Moch wrote: > > On 11.11.19 01:51, Kever Yang wrote: > >> Since there is no devfreq used for vdd_log, so the vdd_log(pwm regulator) > >> will be 'enable' with the dts node at a default PWM state with high or low > >> output. Both too high or too low for vdd_log is not good for the board, > >> add init voltage for driver to make the regulator get into a know output. > >> > >> Note that this will be used by U-Boot for init voltage output, and this > >> is very important for it may get system hang somewhere during system > >> boot up with regulator enable and without this init value. > > I think it's a good idea to include this setting in the main dts for the > > boards (not in u-boot specific additions as is done now). But there is > > (for some reason?) no documented binding for regulator-init-microvolt in > > linux. > > > Ohh, I forgot the kernel driver does not support this property. > > @Heiko, can we add this 'regulator-init-microvolt' without driver > support but adding document for > > it at dt-binding? In theory yes, but you would still need to get DT maintainers and Mark Brown (regulator maintainer) involved before that. Especially as this is a individual property you will need to have a good excuse why you're not implementing it in the kernel as well. And of course you will need to keep in mind that it might be in the kernel in the future. Heiko > > > Thanks, > > - Kever > > > > > Regards, > > Soeren > >> CC: Elaine Zhang > >> CC: Peter Robinson > >> Signed-off-by: Kever Yang > >> --- > >> > >> arch/arm64/boot/dts/rockchip/rk3399-evb.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-firefly.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-hugsun-x99.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-khadas-edge.dtsi | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-leez-p710.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-orangepi.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts | 1 + > >> arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi | 1 + > >> 9 files changed, 9 insertions(+) > >> > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-evb.dts b/arch/arm64/boot/dts/rockchip/rk3399-evb.dts > >> index 77008dca45bc..fa241aeb11b0 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-evb.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-evb.dts > >> @@ -65,6 +65,7 @@ > >> regulator-name = "vdd_center"; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> regulator-always-on; > >> regulator-boot-on; > >> status = "okay"; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts > >> index 92de83dd4dbc..4e45269fcdff 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts > >> @@ -208,6 +208,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <430000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc_sys>; > >> }; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-hugsun-x99.dts b/arch/arm64/boot/dts/rockchip/rk3399-hugsun-x99.dts > >> index c133e8d64b2a..692f3154edc3 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-hugsun-x99.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-hugsun-x99.dts > >> @@ -100,6 +100,7 @@ > >> regulator-name = "vdd_log"; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> regulator-always-on; > >> regulator-boot-on; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-khadas-edge.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-khadas-edge.dtsi > >> index 4944d78a0a1c..c2ac80d99301 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-khadas-edge.dtsi > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-khadas-edge.dtsi > >> @@ -79,6 +79,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vsys_3v3>; > >> }; > >> > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-leez-p710.dts b/arch/arm64/boot/dts/rockchip/rk3399-leez-p710.dts > >> index 73be38a53796..c32abcc4ddc1 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-leez-p710.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-leez-p710.dts > >> @@ -101,6 +101,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc5v0_sys>; > >> }; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-orangepi.dts b/arch/arm64/boot/dts/rockchip/rk3399-orangepi.dts > >> index 0541dfce924d..9d674c51f025 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-orangepi.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-orangepi.dts > >> @@ -164,6 +164,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc_sys>; > >> }; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dts b/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dts > >> index 19f7732d728c..7d856ce1d156 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-roc-pc.dts > >> @@ -129,6 +129,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc3v3_sys>; > >> }; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts > >> index e544deb61d28..8fbccbc8bf47 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts > >> @@ -174,6 +174,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1700000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc5v0_sys>; > >> }; > >> }; > >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi > >> index 1bc1579674e5..f8e2cb8c0624 100644 > >> --- a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi > >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi > >> @@ -133,6 +133,7 @@ > >> regulator-boot-on; > >> regulator-min-microvolt = <800000>; > >> regulator-max-microvolt = <1400000>; > >> + regulator-init-microvolt = <950000>; > >> vin-supply = <&vcc_sys>; > >> }; > >> }; > > >