From: Peng Fan <peng.fan@oss.nxp.com>
To: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>
Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>,
linux-kernel@vger.kernel.org, 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 2/3] ARM: dts: imx6ulz-bsh-smm-m2: Enable uart2
Date: Thu, 22 May 2025 17:57:24 +0800 [thread overview]
Message-ID: <20250522095724.GA14761@nxa18884-linux> (raw)
In-Reply-To: <CAOf5uwmM=+QoaJH8vsUKL-fDab_Fmhxd1d-622pRi-hK=EXk4g@mail.gmail.com>
On Fri, May 16, 2025 at 12:11:54PM +0200, Michael Nazzareno Trimarchi wrote:
>Hi
>
>
>On Fri, May 16, 2025 at 4:56???AM Peng Fan <peng.fan@oss.nxp.com> wrote:
>>
>> On Wed, May 14, 2025 at 09:05:34AM +0200, Dario Binacchi wrote:
>> >From: Wolfgang Birkner <wolfgang.birkner@bshg.com>
>> >
>> >uart2 is used as wake up source.
>> >
>> >Signed-off-by: Wolfgang Birkner <wolfgang.birkner@bshg.com>
>> >Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>> >---
>> >
>> > arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts | 4 ++++
>> > 1 file changed, 4 insertions(+)
>> >
>> >diff --git a/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts b/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
>> >index 5c32d1e3675c..fff21f28c552 100644
>> >--- a/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
>> >+++ b/arch/arm/boot/dts/nxp/imx/imx6ulz-bsh-smm-m2.dts
>> >@@ -37,6 +37,10 @@ &snvs_poweroff {
>> > status = "okay";
>> > };
>> >
>> >+&uart2 {
>> >+ status = "okay";
>> >+};
>>
>> No need pinctrl settings?
>>
>
>We found out that in imx6ulz, if we don't enable wakeup on uart2 (or
>other uarts that do not conflict on pinout) we are not able to wake up
>from uart4, which is the console.
>According to the TRM and the code everything look fine but the real
>result it's we stuck in suspend
>
>Here is how we are able to resume.
>
>uart2
>echo enabled > wakeup
>uart4
>echo enabled > wakeup
No idea.
But at least put some info in commit log to describe why set uart2 as okay.
Thanks,
Peng
>
>Michael
>
>> Regards,
>> Peng
>>
>> >+
>> > &uart3 {
>> > pinctrl-names = "default";
>> > pinctrl-0 = <&pinctrl_uart3>;
>> >--
>> >2.43.0
>> >
>> >base-commit: e9565e23cd89d4d5cd4388f8742130be1d6f182d
>> >branch: bsh-202505-imx6ulz_smm_m2
>>
>>
>
>
>--
>Michael Nazzareno Trimarchi
>Co-Founder & Chief Executive Officer
>M. +39 347 913 2170
>michael@amarulasolutions.com
>__________________________________
>
>Amarula Solutions BV
>Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
>T. +31 (0)85 111 9172
>info@amarulasolutions.com
>www.amarulasolutions.com
next prev parent reply other threads:[~2025-05-22 8:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-14 7:05 [PATCH 1/3] ARM: dts: imx6ulz-bsh-smm-m2: Enable hardware rng Dario Binacchi
2025-05-14 7:05 ` [PATCH 2/3] ARM: dts: imx6ulz-bsh-smm-m2: Enable uart2 Dario Binacchi
2025-05-16 4:05 ` Peng Fan
2025-05-16 10:11 ` Michael Nazzareno Trimarchi
2025-05-22 9:57 ` Peng Fan [this message]
2025-05-14 7:05 ` [PATCH 3/3] ARM: dts: imx6ulz-bsh-smm-m2: Update wifi/bluetooth pinctrl Dario Binacchi
2025-05-16 4:08 ` Peng Fan
2025-05-16 4:04 ` [PATCH 1/3] ARM: dts: imx6ulz-bsh-smm-m2: Enable hardware rng Peng Fan
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=20250522095724.GA14761@nxa18884-linux \
--to=peng.fan@oss.nxp.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=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=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.