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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 31870CD98E0 for ; Wed, 11 Oct 2023 02:13:36 +0000 (UTC) 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=ukYcSHJv0Vrvk/6vqM6urDiourkFdHmwHlOa64xw/BU=; b=hQJ0urMoZI+Gqb QFK2SBlubVJkxH0upJsrBbGGtTr/RDXnWhG/aUX+tZxwRk7zeM98S9n5MqCvROeEN/jm5N27huHPh 4g1YaQlvgiwEI1e/PFe7rrqlWd5mvEnyG8zo1u1he84kxbM+gI+CkvKBTcnCU9YerXsJ7laReYXdR +M/lc9ku7v8/yhxdcr6l8esmA9stJZrWQKi8/P3E6LcASEZTcWcjsNIeau/R0h4f3rDofWx7MqrzY m8WNum2vvKa/7tW5OJKeextM7iwDER+xAG9eenqs1H2KhwAnGU73bf7n4UUUR8wSi4xYTCYLbdl80 AThIspMA5tnw5uKvDqig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qqOim-00EZ8e-0L; Wed, 11 Oct 2023 02:13:12 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qqOij-00EZ6t-0p for linux-arm-kernel@lists.infradead.org; Wed, 11 Oct 2023 02:13:11 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id DE7C8B81E0E; Wed, 11 Oct 2023 02:13:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E5EF2C433C8; Wed, 11 Oct 2023 02:12:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1696990384; bh=lG6rawy4X9gm+VqftM08XTwbZ4iz+veHtEzTIwQLgfQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=BDGJJYi+xBrmCw1RnpxLc6BmrW34E9zVS1YqtTQ4SP1WvVhM2jXoIHILGxOSJUn+c 9PfOpgW4h6zNfhIZsI6YN+UaExFMUcyHnbn1O5wVg13fJTkdUemugsBMIcXMnMdh22 M+gWGtpSlfjK2CTnGfBs9wImUqpvQyt9SmNhMMav9Vxeo6IFcCJJYZlE/vV6Rbhbcj jaVUnPZc7Q1GPngxq9wPWf/c2Uq7Ol1zzvapfhPZQUeEUVOy3wF+ULZUa8OPikFZ6o TQ4UVH6lfPPN4Uz4iFYN50k0Zq0FVl/5q5PCXC9hmkhNyUjOUmcaCfTAzeAoit/tMu VF4y4EmFLSPbg== Date: Wed, 11 Oct 2023 10:12:51 +0800 From: Shawn Guo To: Marek Vasut Cc: linux-arm-kernel@lists.infradead.org, Conor Dooley , Fabio Estevam , Frieder Schrempf , Geert Uytterhoeven , Krzysztof Kozlowski , Magnus Damm , NXP Linux Team , Peng Fan , Pengutronix Kernel Team , Rob Herring , Sascha Hauer , devicetree@vger.kernel.org Subject: Re: [PATCH] arm64: dts: imx8mp: Describe VDD_ARM run and standby voltage for Data Modul i.MX8M Plus eDM SBC Message-ID: <20231011021251.GQ819755@dragon> References: <20230831181850.154813-1-marex@denx.de> <20230924142150.GM7231@dragon> <20231009123606.GA733979@dragon> <18174105-de83-436c-9e77-2c61e1ef2c71@denx.de> <20231010005813.GG733979@dragon> <884b473b-40fb-4f71-9328-9da49b5d01f9@denx.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <884b473b-40fb-4f71-9328-9da49b5d01f9@denx.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231010_191309_600035_E487D212 X-CRM114-Status: GOOD ( 38.49 ) 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 Tue, Oct 10, 2023 at 03:42:14PM +0200, Marek Vasut wrote: > On 10/10/23 02:58, Shawn Guo wrote: > > On Mon, Oct 09, 2023 at 04:03:01PM +0200, Marek Vasut wrote: > > > On 10/9/23 14:36, Shawn Guo wrote: > > > > On Sun, Oct 08, 2023 at 08:37:34PM +0200, Marek Vasut wrote: > > > > > On 9/24/23 16:21, Shawn Guo wrote: > > > > > > On Thu, Aug 31, 2023 at 08:18:50PM +0200, Marek Vasut wrote: > > > > > > > Describe VDD_ARM (BUCK2) run and standby voltage in DT. > > > > > > > > > > > > > > Signed-off-by: Marek Vasut > > > > > > > --- > > > > > > > Cc: Conor Dooley > > > > > > > Cc: Fabio Estevam > > > > > > > Cc: Frieder Schrempf > > > > > > > Cc: Geert Uytterhoeven > > > > > > > Cc: Krzysztof Kozlowski > > > > > > > Cc: Magnus Damm > > > > > > > Cc: Marek Vasut > > > > > > > Cc: NXP Linux Team > > > > > > > Cc: Peng Fan > > > > > > > Cc: Pengutronix Kernel Team > > > > > > > Cc: Rob Herring > > > > > > > Cc: Sascha Hauer > > > > > > > Cc: Shawn Guo > > > > > > > Cc: devicetree@vger.kernel.org > > > > > > > Cc: linux-arm-kernel@lists.infradead.org > > > > > > > --- > > > > > > > arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts | 2 ++ > > > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts b/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts > > > > > > > index 13674dc64be9d..d98a040860a48 100644 > > > > > > > --- a/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts > > > > > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts > > > > > > > @@ -362,6 +362,8 @@ buck1: BUCK1 { /* VDD_SOC (dual-phase with BUCK3) */ > > > > > > > }; > > > > > > > buck2: BUCK2 { /* VDD_ARM */ > > > > > > > + nxp,dvs-run-voltage = <950000>; > > > > > > > + nxp,dvs-standby-voltage = <850000>; > > > > > > > > > > > > Buck2 is not turned off in DSM on i.MX8MP? > > > > > > > > > > It is turned off in SUSPEND/SNVS/OFF , not in IDLE/RUN . > > > > > > > > Right. But nxp,dvs-standby-voltage specifies the voltage when PMIC > > > > is in STANDBY mode. My understanding is that the SoC will be in SUSPEND > > > > state while PMIC is in STANDBY mode. > > > > > > I agree > > > > > > > Is it possible that the SoC in > > > > IDLE/RUN while PMIC is in STANDBY mode at all? > > > > > > No, I don't think so, but there's still the PMIC part: > > > > > > https://www.nxp.com/docs/en/data-sheet/PCA9450.pdf > > > > > > 7.3.7 STANDBY mode > > > " > > > PCA9450 transitions to STANDBY mode from RUN mode when > > > both PMIC_ON_REQ and PMIC_STBY_REQ are driven high. BUCK1 > > > and BUCK3 output voltage is set to BUCK1OUT_DVS1 and > > > BUCK3OUT_DVS1 and BUCK2 are turned off when DVS_CTRL bit > > > in each BUCKx_CTRL register is configured to 1. > > > " > > > > > > Specifically > > > " > > > BUCK2 are turned off when DVS_CTRL bit in each > > > BUCKx_CTRL register is configured to 1. > > > " > > > > > > 8.2.19 0x13 BUCK2CTRL > > > " > > > 4 > > > DVS_CTRL > > > DVS Control configuration > > > 0b = BUCK voltage is determined by BUCK2VOUT_DVS0 register regardless of > > > PMIC_STBY_REQ > > > 1b = DVS control through PMIC_STBY_REQ > > > " > > > > > > Notice that the reset-default is '0b' , so unless the PMIC is reconfigured, > > > the BUCK2 will stay powered on even in STANDBY/SUSPEND. > > > > Hmm, isn't B2_ENMODE controlling on/off of BUCK2? > > > > BUCK2 enable mode > > 00b = OFF > > 01b = ON by PMIC_ON_REQ = H > > 10b = ON by PMIC_ON_REQ = H && PMIC_STBY_REQ = L (default) > > 11b = Always ON > > > > So unless you reconfigure the field, BUCK2 will be off when PMIC_STBY_REQ > > goes high, right? > > I think so. > > But since the buck2 in DVS behavior is user-configurable, and this can be > configured by the bootloader, I would say that it is a good idea to keep the > buck2 DVS value defined in the DT, rather than keep it undefined. > > Also note that this board is a development kit, so we just don't know what > users might do with it. Honestly, I just do not see any reason why one would change the most power efficient default configuration to something consuming much more power. Also, even if DVS control is enabled, 0.85 V is already the target voltage for STANDBY mode by default. Bottom line is that we should have some comment on this change, because it makes eDM SBC so unique among all those PCA9450 based boards and makes people like me wonder why eDM SBC board needs this setting. Shawn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel