From: Frieder Schrempf <frieder.schrempf@kontron.de>
To: Frank Li <Frank.li@nxp.com>, Frieder Schrempf <frieder@fris.de>
Cc: Conor Dooley <conor+dt@kernel.org>,
devicetree@vger.kernel.org, imx@lists.linux.dev,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Rob Herring <robh@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Shawn Guo <shawnguo@kernel.org>,
Annette Kobou <annette.kobou@kontron.de>,
Fabio Estevam <festevam@gmail.com>,
Pengutronix Kernel Team <kernel@pengutronix.de>
Subject: Re: [PATCH 3/4] arm64: dts: imx8mp-kontron: Fix boot order for PMIC and RTC
Date: Thu, 19 Feb 2026 08:54:53 +0100 [thread overview]
Message-ID: <eff7e6c7-77aa-40db-a115-3c8366f1ed0d@kontron.de> (raw)
In-Reply-To: <aZXwdC0R1gzwVqeU@lizhi-Precision-Tower-5810>
On 18.02.26 18:01, Frank Li wrote:
> On Wed, Feb 18, 2026 at 02:25:07PM +0100, Frieder Schrempf wrote:
>> From: Annette Kobou <annette.kobou@kontron.de>
>>
>> The PMIC provides a level-shifter for the I2C lines to the RTC.
>> As the level shifter needs to be enabled before the RTC can be
>> accessed, we need to make sure that the PMIC driver is probed
>> first.
>>
>> As the PMIC also provides the supply voltage for the RTC through
>> the 3.3V regulator, we can simply express this in the DT to
>> create the required dependency.
>>
>> This fixes sporadic boot hangs that occurred when the RTC was
>> accessed before the level-shifter was enabled.
>>
>> Fixes: 946ab10e3f40f ("arm64: dts: Add support for Kontron OSM-S i.MX8MP SoM and BL carrier board")
>> Signed-off-by: Annette Kobou <annette.kobou@kontron.de>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>> ---
>> arch/arm64/boot/dts/freescale/imx8mp-kontron-osm-s.dtsi | 6 ++++++
>> 1 file changed, 6 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-kontron-osm-s.dtsi
>> index b97bfeb1c30f8..bc1a261bb000e 100644
>> --- a/arch/arm64/boot/dts/freescale/imx8mp-kontron-osm-s.dtsi
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-kontron-osm-s.dtsi
>> @@ -330,6 +330,12 @@ rv3028: rtc@52 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&pinctrl_rtc>;
>> interrupts-extended = <&gpio3 24 IRQ_TYPE_LEVEL_LOW>;
>> + /*
>> + * While specifying the vdd-supply is normally not strictly necessary,
>> + * here it also makes sure that the PMIC driver enables the level-
>> + * shifter for the RTC before the RTC is probed.
>> + */
>> + vdd-supply = <®_vdd_3v3>;
>
> Dose reg_vdd_3v3 controller by gpio, or actually it is level shifter?
As I explained in the commit message, reg_vdd_3v3 is a regulator
provided by the PCA9450 PMIC that feeds the RTC.
At the same time the PCA9450 PMIC also provides a level-shifter that is
used to bring the I2C data lines to the correct level for the RTC.
next prev parent reply other threads:[~2026-02-19 7:54 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-18 13:25 [PATCH 0/4] Kontron i.MX8MP OSM Devicetree Fixups Frieder Schrempf
2026-02-18 13:25 ` [PATCH 1/4] arm64: dts: imx8mp-kontron: Fix touch reset configuration on DL devices Frieder Schrempf
2026-02-18 16:55 ` Frank Li
2026-02-18 13:25 ` [PATCH 2/4] arm64: dts: imx8mp-kontron: Fix SD card supply on SMARC eval carrier Frieder Schrempf
2026-02-18 16:59 ` Frank Li
2026-02-19 7:51 ` Frieder Schrempf
2026-02-19 17:38 ` Frank Li
2026-02-20 10:35 ` Frieder Schrempf
2026-02-18 13:25 ` [PATCH 3/4] arm64: dts: imx8mp-kontron: Fix boot order for PMIC and RTC Frieder Schrempf
2026-02-18 17:01 ` Frank Li
2026-02-19 7:54 ` Frieder Schrempf [this message]
2026-02-18 13:25 ` [PATCH 4/4] arm64: dts: imx8mp-kontron: Use GPIO/IRQ defines in DL devicetree Frieder Schrempf
2026-02-18 17:02 ` Frank Li
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=eff7e6c7-77aa-40db-a115-3c8366f1ed0d@kontron.de \
--to=frieder.schrempf@kontron.de \
--cc=Frank.li@nxp.com \
--cc=annette.kobou@kontron.de \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=frieder@fris.de \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox