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=-16.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 1EBB8C4338F for ; Mon, 26 Jul 2021 01:03:38 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C80F160E09 for ; Mon, 26 Jul 2021 01:03:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C80F160E09 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=K/R3+jQtP2ggZqe0lGsfAHM/ThyAA8CEniamZPSmJKg=; b=UXRlp7RXciw+oJ PqUoaFkOKGLo6cAfS7E0dEera7rNgVbQ9w7xLSTWBJ+8i9ecQOq+gb8/VlQXZ9IYsTkffiIsngcb3 I01lPjud0/adTwGhCN6t9g6puCBB8OJz+lhnJ7e+wPS3Wc+hy7qPUyCcN6d7UjKx8++nlRB4NCYdi IXN6k+f687rpmdyMhVrwWzVI6uub9367nXahvILywIiZWac6muPkwpTnoqdpamgpPGt/Sfl/lFseT 2nkquaTpOIktVdnP00UngsPR9Ybfx15QNIYMz2xonKJ+h4gC3eps7sBfsQDiRv7IcxPKaRw29XsmN mgV6BaEYswUx2G+SYUUg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7p0A-009LQZ-KB; Mon, 26 Jul 2021 01:01:50 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7p07-009LQE-0n for linux-arm-kernel@lists.infradead.org; Mon, 26 Jul 2021 01:01:48 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 9B9D360E09; Mon, 26 Jul 2021 01:01:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1627261306; bh=PTOuMxWlEFBoK2P+EnnwEleedr8jnM9pcOAYnWVp1Es=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=fwloR04hIG2PrRzWxypmI+xrBdy7xjRmksaanuaoHtxAXl0jhKimi+gdOWGZkqxfe nwSeAqW1ojfRIERLLzr+z7FFBpfyuW3uvxreSk7nyqg3qRfKS4Z2eFJygdtMfnJlmL floEtjOnOOPPtyQw8rDaUuRonwqPbCDZY9LUJz3wJZ5YAILCNTa0Jp/ggpF1AgzAeK vFuVxAShfJ39riyrotcawXeWqAtQFG3CiWt/olYXdruFDa5UVYtbBdcJD35m38lkqG mOhriv9QcGk7gDNjdoXbwipYPFwTaln/wkQWH3JQmc3Q3dmsJlg60Uawcr/fHKQv8G 5R4QXyGdNHy8w== Date: Mon, 26 Jul 2021 09:01:40 +0800 From: Shawn Guo To: Tim Harvey Cc: Rob Herring , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Device Tree Mailing List , Linux ARM Mailing List Subject: Re: [PATCH 2/4] arm64: dts: imx8mm-venice-gw700x: fix mp5416 pmic config Message-ID: <20210726010139.GB5901@dragon> References: <20210601174917.1979-1-tharvey@gateworks.com> <20210601174917.1979-2-tharvey@gateworks.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210725_180147_149847_F3EE3CD1 X-CRM114-Status: GOOD ( 18.21 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sat, Jul 24, 2021 at 11:19:21AM -0700, Tim Harvey wrote: > On Tue, Jun 1, 2021 at 10:49 AM Tim Harvey wrote: > > > > Fix various MP5416 PMIC configurations: > > - Update regulator names per dt-bindings > > - ensure values fit among valid register values > > - add required regulator-max-microamp property > > - add regulator-always-on prop > > > > Signed-off-by: Tim Harvey > > --- > > .../dts/freescale/imx8mm-venice-gw700x.dtsi | 56 ++++++++++++------- > > 1 file changed, 37 insertions(+), 19 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi > > index 512b76cd7c3b..f4eb827baed7 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw700x.dtsi > > @@ -283,65 +283,83 @@ > > reg = <0x69>; > > > > regulators { > > + /* vdd_0p95: DRAM/GPU/VPU */ > > buck1 { > > - regulator-name = "vdd_0p95"; > > - regulator-min-microvolt = <805000>; > > + regulator-name = "buck1"; > > + regulator-min-microvolt = <800000>; > > regulator-max-microvolt = <1000000>; > > - regulator-max-microamp = <2500000>; > > + regulator-min-microamp = <3800000>; > > + regulator-max-microamp = <6800000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* vdd_soc */ > > buck2 { > > - regulator-name = "vdd_soc"; > > - regulator-min-microvolt = <805000>; > > + regulator-name = "buck2"; > > + regulator-min-microvolt = <800000>; > > regulator-max-microvolt = <900000>; > > - regulator-max-microamp = <1000000>; > > + regulator-min-microamp = <2200000>; > > + regulator-max-microamp = <5200000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* vdd_arm */ > > buck3_reg: buck3 { > > - regulator-name = "vdd_arm"; > > - regulator-min-microvolt = <805000>; > > + regulator-name = "buck3"; > > + regulator-min-microvolt = <800000>; > > regulator-max-microvolt = <1000000>; > > - regulator-max-microamp = <2200000>; > > - regulator-boot-on; > > + regulator-min-microamp = <3800000>; > > + regulator-max-microamp = <6800000>; > > + regulator-always-on; > > }; > > > > + /* vdd_1p8 */ > > buck4 { > > - regulator-name = "vdd_1p8"; > > + regulator-name = "buck4"; > > regulator-min-microvolt = <1800000>; > > regulator-max-microvolt = <1800000>; > > - regulator-max-microamp = <500000>; > > + regulator-min-microamp = <2200000>; > > + regulator-max-microamp = <5200000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* nvcc_snvs_1p8 */ > > ldo1 { > > - regulator-name = "nvcc_snvs_1p8"; > > + regulator-name = "ldo1"; > > regulator-min-microvolt = <1800000>; > > regulator-max-microvolt = <1800000>; > > - regulator-max-microamp = <300000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* vdd_snvs_0p8 */ > > ldo2 { > > - regulator-name = "vdd_snvs_0p8"; > > + regulator-name = "ldo2"; > > regulator-min-microvolt = <800000>; > > regulator-max-microvolt = <800000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* vdd_0p9 */ > > ldo3 { > > - regulator-name = "vdd_0p95"; > > - regulator-min-microvolt = <800000>; > > - regulator-max-microvolt = <800000>; > > + regulator-name = "ldo3"; > > + regulator-min-microvolt = <900000>; > > + regulator-max-microvolt = <900000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > > > + /* vdd_1p8 */ > > ldo4 { > > - regulator-name = "vdd_1p8"; > > + regulator-name = "ldo4"; > > regulator-min-microvolt = <1800000>; > > regulator-max-microvolt = <1800000>; > > regulator-boot-on; > > + regulator-always-on; > > }; > > }; > > }; > > -- > > 2.17.1 > > > > Shawn, > > Is there anything you want changed here? Tim, Could you resend the series? For some reason, I cannot find the original posting in my mailbox. Shawn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel