From: Tony Lindgren <tony@atomide.com>
To: Romain Naour <romain.naour@smile.fr>
Cc: linux-omap@vger.kernel.org
Subject: Re: AM5749: tty serial 8250 omap driver crash
Date: Mon, 14 Feb 2022 09:43:44 +0200 [thread overview]
Message-ID: <YgoIMPZd7bi6XDGW@atomide.com> (raw)
In-Reply-To: <9f0cb8f9-ac94-eb50-266a-95df521a7360@smile.fr>
* Romain Naour <romain.naour@smile.fr> [220211 10:10]:
> Le 10/02/2022 à 13:05, Tony Lindgren a écrit :
> > Could be that too, but maybe it's as simple as the patch below. Care to
> > give it a try?
>
> The serial console stop working during the kernel boot with this patch applied.
Yeah and we already have the serdev taking case of PM runtime here.
> >> Are you able to reproduce it ?
> >> Maybe on a IDK574 or a Beaglebone-AI board ?
> >
> > Not sure why I'm not seeing this one with my test systems.. Have not tried
> > with the RT patches for a while though.
>
> The RT patches really makes a difference here.
Looks like the following script to just toggle the module state locks
up things for me on beagle-x15 very fast. So yeah now I'm able to
reproduce the issue. Seems like the module is not ready right after
enabling it live we've seen for dra7 iva for example.
Regards,
Tony
8< ------
#!/bin/sh
# Test rebind for serial console uart3 ttyS2, run from an ssh session
module="48020050.target-module"
driver="/sys/bus/platform/drivers/ti-sysc"
# Detach all kernel serial consoles
consoles=$(find /sys/bus/platform/devices/4*.serial/ -name console)
for console in ${consoles}; do
echo -n N > ${console}
done
# Configure PM runtime autosuspend for all uarts
uarts=$(find /sys/bus/platform/devices/4*.serial/power/ -type d)
for uart in $uarts; do
echo -n 3000 > $uart/autosuspend_delay_ms
echo -n enabled > $uart/wakeup
echo -n auto > $uart/control
done
# Configure wake-up from suspend for all uarts
uarts=$(find /sys/class/tty/tty[SO]*/power/ -type d 2>/dev/null)
for uart in $uarts; do
echo -n enabled > $uart/wakeup
done
# Keep rebinding uart3 in a loop
while true; do
echo ${module} > ${driver}/bind
echo ${module} > ${driver}/unbind
done
next prev parent reply other threads:[~2022-02-14 7:43 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-04 13:39 AM5749: tty serial 8250 omap driver crash Romain Naour
2022-02-07 8:04 ` Tony Lindgren
2022-02-09 9:13 ` Romain Naour
2022-02-10 12:05 ` Tony Lindgren
2022-02-11 10:11 ` Romain Naour
2022-02-14 7:43 ` Tony Lindgren [this message]
2022-02-14 13:08 ` Tony Lindgren
2022-02-16 9:04 ` Romain Naour
2022-02-16 11:46 ` Tony Lindgren
2022-02-16 15:51 ` Romain Naour
2022-02-17 8:08 ` Tony Lindgren
2022-02-17 9:09 ` Romain Naour
2022-02-17 12:58 ` Tony Lindgren
2022-04-02 10:15 ` Romain Naour
2022-05-03 10:05 ` Tony Lindgren
2022-05-04 12:42 ` Romain Naour
2022-05-05 4:33 ` Tony Lindgren
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=YgoIMPZd7bi6XDGW@atomide.com \
--to=tony@atomide.com \
--cc=linux-omap@vger.kernel.org \
--cc=romain.naour@smile.fr \
/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