From: Matti Vaittinen <mazziesaccount@gmail.com>
To: Dario Binacchi <dario.binacchi@amarulasolutions.com>,
linux-kernel@vger.kernel.org
Cc: Simon Holesch <simon.holesch@bshg.com>,
Karthikdatt Anantharamrao
<karthikdatt.anantharamrao@in.bosch.com>,
michael@amarulasolutions.com, linux-amarula@amarulasolutions.com,
Wolfgang Birkner <wolfgang.birkner@bshg.com>,
Conor Dooley <conor+dt@kernel.org>,
Fabio Estevam <festevam@gmail.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
Rob Herring <robh@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Shawn Guo <shawnguo@kernel.org>,
devicetree@vger.kernel.org, imx@lists.linux.dev,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 9/9] arm64: dts: imx8mn-bsh-smm-s2-common: Disable PMIC SNVS reset target state
Date: Thu, 15 May 2025 12:28:43 +0300 [thread overview]
Message-ID: <fb5b60e9-eaf9-42bc-9bda-0c80bd55f81d@gmail.com> (raw)
In-Reply-To: <20250514082507.1983849-10-dario.binacchi@amarulasolutions.com>
On 14/05/2025 11:25, Dario Binacchi wrote:
> From: Wolfgang Birkner <wolfgang.birkner@bshg.com>
>
> VDD_DRAM was disabled on standby, therefore the reference hardware did not
> wake up reliable. Use PMIC reset target state READY instead of SNVS, to
> keep VDD_DRAM active during standby.
There is something I'm not quite sure I understand. Lookin at the
BD71847 data-sheet, the VDD_DRAM is OFF at READY.
(Page 27, Table 3-8. Voltage Rails ON/OFF for Respective Power State)
https://fscdn.rohm.com/en/products/databook/datasheet/ic/power/switching_regulator_system/bd71847amwv-e.pdf
Please, explain.
Yours,
-- Matti
>
> Signed-off-by: Wolfgang Birkner <wolfgang.birkner@bshg.com>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>
> ---
>
> arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi
> index ea8d741c6904..633874b3bf66 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2-common.dtsi
> @@ -94,7 +94,6 @@ bd71847: pmic@4b {
> pinctrl-0 = <&pinctrl_pmic>;
> interrupt-parent = <&gpio1>;
> interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
> - rohm,reset-snvs-powered;
>
> #clock-cells = <0>;
> clocks = <&osc_32k>;
prev parent reply other threads:[~2025-05-15 9:28 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-14 8:24 [PATCH 0/9] DTS updates for BSH SMM S2[PRO] boards Dario Binacchi
2025-05-14 8:24 ` [PATCH 1/9] arm64: dts: imx8mn-bsh-smm-s2-common: Define suspend voltage threshold Dario Binacchi
2025-05-15 8:52 ` Matti Vaittinen
2025-05-14 8:24 ` [PATCH 2/9] arm64: dts: imx8mn-bsh-smm-s2-common: Leave regulator under hw state machine Dario Binacchi
2025-05-15 8:54 ` Matti Vaittinen
2025-05-14 8:24 ` [PATCH 3/9] arm64: dts: imx8mn-bsh-smm-s2-common: Keep wifi enable during suspend Dario Binacchi
2025-05-16 4:17 ` Peng Fan
2025-05-14 8:24 ` [PATCH 4/9] arm64: dts: imx8mn-bsh-smm-s2-common: Enable GPU for smms2 and smms2pro Dario Binacchi
2025-05-16 4:23 ` Peng Fan
2025-05-14 8:24 ` [PATCH 5/9] arm64: dts: imx8mn-bsh-smm-s2-common: Pull Up Console UART pins Dario Binacchi
2025-05-16 4:24 ` Peng Fan
2025-05-14 8:25 ` [PATCH 6/9] arm64: dts: imx8mn-bsh-smm-s2-common: Set minimum value for VDD_DRAM_VPU_GPU Dario Binacchi
2025-05-15 9:19 ` Matti Vaittinen
2025-05-14 8:25 ` [PATCH 7/9] arm64: dts: imx8mn-bsh-smm-s2-common: Set minimum value for VDD_3V3 Dario Binacchi
2025-05-15 9:20 ` Matti Vaittinen
2025-05-14 8:25 ` [PATCH 8/9] arm64: dts: imx8mn-bsh-smm-s2-common: Increase drive strength of pmic irq Dario Binacchi
2025-05-16 4:24 ` Peng Fan
2025-05-14 8:25 ` [PATCH 9/9] arm64: dts: imx8mn-bsh-smm-s2-common: Disable PMIC SNVS reset target state Dario Binacchi
2025-05-15 9:28 ` Matti Vaittinen [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fb5b60e9-eaf9-42bc-9bda-0c80bd55f81d@gmail.com \
--to=mazziesaccount@gmail.com \
--cc=conor+dt@kernel.org \
--cc=dario.binacchi@amarulasolutions.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=karthikdatt.anantharamrao@in.bosch.com \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=linux-amarula@amarulasolutions.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michael@amarulasolutions.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=simon.holesch@bshg.com \
--cc=wolfgang.birkner@bshg.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.