From: "Bedia, Vaibhav" <vaibhav.bedia@ti.com> To: Tony Lindgren <tony@atomide.com> Cc: Mark Jackson <mpfj-list@newflow.co.uk>, "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: RE: Boot hang regression 3.10.0-rc4 -> 3.10.0 Date: Fri, 5 Jul 2013 13:20:10 +0000 [thread overview] Message-ID: <B5906170F1614E41A8A28DE3B8D121433EE94489@DBDE04.ent.ti.com> (raw) In-Reply-To: <20130705115959.GQ5523@atomide.com> Hi Tony, On Fri, Jul 05, 2013 at 17:29:59, Tony Lindgren wrote: > * Bedia, Vaibhav <vaibhav.bedia@ti.com> [130705 01:17]: > > > > I just checked the behavior on my AM335x-EVM. Current mainline boots fine > > provided I don't use earlyprintk. The offending patch [1] in this case is the one > > that tries to get rid of omap_serial_early_init() for DT boot. This change inadvertently > > also results in the console UART getting reset and idled during bootup and that's where > > the boot stops for you. I think if you skip earlyprintk from the bootargs you should see > > the system booting fine. > > > > I guess we need to retain the NO_IDLE and NO_RESET aspect for the console UART in > > omap_serial_early_init() to get earlyprintk working again. > > Hmm nothing should get idled while earlyprintk is running, and then when the > serial driver kicks in it should not idle anything by default. And for DT based > booting we should not have mach-omap2/serial.c initialize anything. > If I add in the HWMOD flags without any reverts I get to the point where the serial driver comes up but the boot eventually stops [1]. Without the flags the boot stops much earlier [2] just like Mark reported. > I wonder if this is because the timeouts get now initialized to 0 instead > of -1 for the serial driver? > You meant initialized to -1, right? There's an additional check for timeout being 0. Unless i am missing something DT-boot will start off with timeout set to 0 and then get forced to -1. Regards, Vaibhav === [1] [ 0.000000] Booting Linux on physical CPU 0x0 <...> [ 0.190107] SMP: Total of 1 processors activated (363.72 BogoMIPS). [ 0.196711] CPU: All CPU(s) started in SVC mode. [ 0.205580] devtmpfs: initialized <stops here if booted with earlyprintk> [2] [ 0.000000] Booting Linux on physical CPU 0x0 <...> [ 1.453364] pinctrl-single 44e10800.pinmux: 142 pins at pa f9e10800 size 568 [ 1.467121] Serial: 8250/16550 driver, 4 ports, IRQ sharing enabled <stops here if booted with earlyprintk>
WARNING: multiple messages have this Message-ID (diff)
From: vaibhav.bedia@ti.com (Bedia, Vaibhav) To: linux-arm-kernel@lists.infradead.org Subject: Boot hang regression 3.10.0-rc4 -> 3.10.0 Date: Fri, 5 Jul 2013 13:20:10 +0000 [thread overview] Message-ID: <B5906170F1614E41A8A28DE3B8D121433EE94489@DBDE04.ent.ti.com> (raw) In-Reply-To: <20130705115959.GQ5523@atomide.com> Hi Tony, On Fri, Jul 05, 2013 at 17:29:59, Tony Lindgren wrote: > * Bedia, Vaibhav <vaibhav.bedia@ti.com> [130705 01:17]: > > > > I just checked the behavior on my AM335x-EVM. Current mainline boots fine > > provided I don't use earlyprintk. The offending patch [1] in this case is the one > > that tries to get rid of omap_serial_early_init() for DT boot. This change inadvertently > > also results in the console UART getting reset and idled during bootup and that's where > > the boot stops for you. I think if you skip earlyprintk from the bootargs you should see > > the system booting fine. > > > > I guess we need to retain the NO_IDLE and NO_RESET aspect for the console UART in > > omap_serial_early_init() to get earlyprintk working again. > > Hmm nothing should get idled while earlyprintk is running, and then when the > serial driver kicks in it should not idle anything by default. And for DT based > booting we should not have mach-omap2/serial.c initialize anything. > If I add in the HWMOD flags without any reverts I get to the point where the serial driver comes up but the boot eventually stops [1]. Without the flags the boot stops much earlier [2] just like Mark reported. > I wonder if this is because the timeouts get now initialized to 0 instead > of -1 for the serial driver? > You meant initialized to -1, right? There's an additional check for timeout being 0. Unless i am missing something DT-boot will start off with timeout set to 0 and then get forced to -1. Regards, Vaibhav === [1] [ 0.000000] Booting Linux on physical CPU 0x0 <...> [ 0.190107] SMP: Total of 1 processors activated (363.72 BogoMIPS). [ 0.196711] CPU: All CPU(s) started in SVC mode. [ 0.205580] devtmpfs: initialized <stops here if booted with earlyprintk> [2] [ 0.000000] Booting Linux on physical CPU 0x0 <...> [ 1.453364] pinctrl-single 44e10800.pinmux: 142 pins at pa f9e10800 size 568 [ 1.467121] Serial: 8250/16550 driver, 4 ports, IRQ sharing enabled <stops here if booted with earlyprintk>
next prev parent reply other threads:[~2013-07-05 13:20 UTC|newest] Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-07-04 13:25 Boot hang regression 3.10.0-rc4 -> 3.10.0 Mark Jackson 2013-07-04 13:25 ` Mark Jackson 2013-07-04 15:14 ` Mark Jackson 2013-07-04 15:14 ` Mark Jackson 2013-07-04 16:00 ` Mark Jackson 2013-07-04 16:00 ` Mark Jackson 2013-07-05 8:11 ` Bedia, Vaibhav 2013-07-05 8:11 ` Bedia, Vaibhav 2013-07-05 11:59 ` Tony Lindgren 2013-07-05 11:59 ` Tony Lindgren 2013-07-05 13:20 ` Bedia, Vaibhav [this message] 2013-07-05 13:20 ` Bedia, Vaibhav 2013-07-05 13:31 ` Bedia, Vaibhav 2013-07-05 13:31 ` Bedia, Vaibhav 2013-07-08 11:25 ` Tony Lindgren 2013-07-08 11:25 ` Tony Lindgren 2013-07-08 12:16 ` Tony Lindgren 2013-07-08 12:16 ` Tony Lindgren 2013-07-08 12:41 ` Rajendra Nayak 2013-07-08 12:41 ` Rajendra Nayak 2013-07-08 13:10 ` Tony Lindgren 2013-07-08 13:10 ` Tony Lindgren 2013-07-08 13:20 ` Rajendra Nayak 2013-07-08 13:20 ` Rajendra Nayak 2013-07-08 13:25 ` Rajendra Nayak 2013-07-08 13:25 ` Rajendra Nayak 2013-07-08 13:35 ` Felipe Balbi 2013-07-08 13:35 ` Felipe Balbi 2013-07-09 5:33 ` Rajendra Nayak 2013-07-09 5:33 ` Rajendra Nayak 2013-07-09 6:42 ` Felipe Balbi 2013-07-09 6:42 ` Felipe Balbi 2013-07-09 7:19 ` Rajendra Nayak 2013-07-09 7:19 ` Rajendra Nayak 2013-07-09 7:40 ` Felipe Balbi 2013-07-09 7:40 ` Felipe Balbi 2013-07-09 18:59 ` Grygorii Strashko 2013-07-09 18:59 ` Grygorii Strashko 2013-07-09 19:41 ` Felipe Balbi 2013-07-09 19:41 ` Felipe Balbi 2013-07-10 12:16 ` Grygorii Strashko 2013-07-10 12:16 ` Grygorii Strashko 2013-07-10 12:25 ` Felipe Balbi 2013-07-10 12:25 ` Felipe Balbi 2013-07-10 8:22 ` Kevin Hilman 2013-07-10 8:22 ` Kevin Hilman 2013-07-10 12:10 ` Tony Lindgren 2013-07-10 12:10 ` Tony Lindgren 2013-07-10 12:27 ` Tony Lindgren 2013-07-10 12:27 ` Tony Lindgren 2013-07-10 14:26 ` Tony Lindgren 2013-07-10 14:26 ` Tony Lindgren 2013-07-10 16:07 ` Felipe Balbi 2013-07-10 16:07 ` Felipe Balbi 2013-07-10 16:11 ` Felipe Balbi 2013-07-10 16:11 ` Felipe Balbi 2013-07-11 6:32 ` Tony Lindgren 2013-07-11 6:32 ` Tony Lindgren 2013-07-11 9:59 ` Grygorii Strashko 2013-07-11 9:59 ` Grygorii Strashko 2013-07-12 0:40 ` Suman Anna 2013-07-12 0:40 ` Suman Anna 2013-07-15 6:44 ` Rajendra Nayak 2013-07-15 6:44 ` Rajendra Nayak 2013-07-15 10:01 ` Rajendra Nayak 2013-07-15 10:01 ` Rajendra Nayak 2013-07-15 19:23 ` Suman Anna 2013-07-15 19:23 ` Suman Anna 2013-07-16 6:30 ` Rajendra Nayak 2013-07-16 6:30 ` Rajendra Nayak 2013-07-11 9:17 ` Rajendra Nayak 2013-07-11 9:17 ` Rajendra Nayak 2013-07-11 9:26 ` Felipe Balbi 2013-07-11 9:26 ` Felipe Balbi 2013-07-11 10:16 ` [PATCH] arm: omap2plus: unidle devices which are about to probe Felipe Balbi 2013-07-11 10:16 ` Felipe Balbi 2013-07-12 11:58 ` Grygorii Strashko 2013-07-12 11:58 ` Grygorii Strashko 2013-07-12 12:10 ` Felipe Balbi 2013-07-12 12:10 ` Felipe Balbi 2013-07-12 12:27 ` Rajendra Nayak 2013-07-12 12:27 ` Rajendra Nayak 2013-07-13 22:21 ` Kevin Hilman 2013-07-13 22:21 ` Kevin Hilman 2013-07-11 9:59 ` Boot hang regression 3.10.0-rc4 -> 3.10.0 Grygorii Strashko 2013-07-11 9:59 ` Grygorii Strashko 2013-07-16 10:27 ` Grygorii Strashko 2013-07-16 10:27 ` Grygorii Strashko 2013-07-17 7:10 ` Rajendra Nayak 2013-07-17 7:10 ` Rajendra Nayak 2013-07-11 6:18 ` Rajendra Nayak 2013-07-11 6:18 ` Rajendra Nayak 2013-07-11 6:24 ` Tony Lindgren 2013-07-11 6:24 ` Tony Lindgren 2013-07-11 9:11 ` Rajendra Nayak 2013-07-11 9:11 ` Rajendra Nayak
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=B5906170F1614E41A8A28DE3B8D121433EE94489@DBDE04.ent.ti.com \ --to=vaibhav.bedia@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=mpfj-list@newflow.co.uk \ --cc=tony@atomide.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: linkBe 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.