linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: OMAP2+: Warn about deprecated legacy booting mode
Date: Fri, 31 Oct 2014 16:37:23 -0700	[thread overview]
Message-ID: <20141031233722.GE31454@atomide.com> (raw)
In-Reply-To: <20141031223332.GD12379@n2100.arm.linux.org.uk>

* Russell King - ARM Linux <linux@arm.linux.org.uk> [141031 15:35]:
> On Fri, Oct 31, 2014 at 02:13:26PM -0700, Tony Lindgren wrote:
> > * Russell King - ARM Linux <linux@arm.linux.org.uk> [141031 12:32]:
> > 
> > > WARNING: CPU: 0 PID: 1 at arch/arm/mach-omap2/omap_hwmod.c:2538 _init+0xc8/0x170()
> > > omap_hwmod: ssi: doesn't have mpu register target base
> > > ...
> > 
> > Not seeing this one, is this maybe from the legacy mode boot?
> > 
> > > omap_hwmod: usb_host_hs: could not associate to clkdm l3_init_clkdm
> > > omap_hwmod: usb_tll_hs: could not associate to clkdm l3_init_clkdm
> > 
> > Not seeing these either. Is this maybe from your 4430sdp instead?
> 
> This is /all/ from the 3430 LDP.
> 
> Compare these two URLs:
> 
> http://www.arm.linux.org.uk/developer/build/result.php?type=boot&idx=2891
> http://www.arm.linux.org.uk/developer/build/result.php?type=boot&idx=2892

Not able to load the second one because of the broken battery
driver :)

But based on the link above I can see that you're mixing in both
legacy mode warnings and DT mode warnings at least for the SSI,
so that explains why I'm not seeing the SSI error.

Still no idea where the l3_init_clkdm errors come from, are they
from your huge log?

Like I posted in this thread, I'm not seeing same errors as you are
with your config modified for DT:

http://muru.com/for-rmk/dmesg-ldp-dt-v3.18-rc2
 
> I'm not trying with omap2plus_defconfig as this is supposed to be a
> separate build.  Given that - as I say - the /exact/ same kernel binary
> works in legacy mode, it suggests that there's some pretty major
> differences between what devices are supported in a legacy boot mode
> and what devices are supported in DT boot mode.

Sure no problem. Your .config works just fine for me in DT mode
with the the options enabled like I posted in this thread.
 
> I really /can't/ risk letting the platform boot again in DT mode having
> just played around with some options - if it spews like it did again
> last night, it'll spew for the whole day tomorrow, and I really don't
> want the database filling up with another gigabyte of log.

Looks like the twl4030_charger.c driver just does printk in the irq
handler for errors :) I'm almost certain you'd get the same errors
in legacy mode too if board-ldp.c passed the platform_data to the
battery driver.. No ideas why the battery detection during probe
won't work for your hardware unless your PMIC is partially fried
or some earlier revision that's broken.

Anyways, let me know what happens with the options I suggested and
CONFIG_CHARGER_TWL4030 disabled so we can see if the other problems
are gone.

Regards,

Tony

  reply	other threads:[~2014-10-31 23:37 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-27 20:00 [PATCH] ARM: OMAP2+: Warn about deprecated legacy booting mode Tony Lindgren
2014-10-27 23:01 ` Aaro Koskinen
2014-10-28 22:12   ` Tony Lindgren
2014-10-29 13:43     ` Nishanth Menon
2014-10-29 18:59       ` Sebastian Reichel
2014-10-29 19:39         ` Tony Lindgren
2014-10-29 21:45           ` Nishanth Menon
2014-10-29 22:15             ` Tony Lindgren
2014-10-29 22:31             ` Aaro Koskinen
2014-10-30 13:55               ` Nishanth Menon
2014-10-29 23:07           ` Sebastian Reichel
2014-10-29 23:11           ` Aaro Koskinen
2014-10-29 23:54             ` Javier Martinez Canillas
2014-11-26 17:28   ` Pavel Machek
2014-11-26 18:19     ` Tony Lindgren
2014-11-26 18:57       ` Pali Rohár
2014-11-26 19:10         ` Tony Lindgren
2014-11-26 19:22           ` Pali Rohár
2014-11-26 20:08             ` Tony Lindgren
2014-11-26 23:01               ` Pali Rohár
2014-11-26 23:14                 ` Tony Lindgren
2014-11-26 23:38                   ` Pali Rohár
2014-11-27  1:12                     ` Tony Lindgren
2014-11-27 11:32                       ` Pali Rohár
2014-11-28 20:27                         ` Tony Lindgren
2014-11-28 21:41                           ` Pali Rohár
2014-11-28 22:24                             ` Tony Lindgren
2014-11-28 22:42                               ` Pali Rohár
2014-12-04 18:34                               ` Pali Rohár
2014-12-04 18:40                                 ` Tony Lindgren
2014-12-04 19:01                                   ` Pali Rohár
2014-11-28 22:26                             ` Aaro Koskinen
2014-11-28 22:43                               ` Pali Rohár
2014-11-28 22:41                             ` Aaro Koskinen
2014-11-28 22:49                               ` Pali Rohár
2014-11-28 22:54                                 ` Aaro Koskinen
2014-12-02 21:28                           ` Pali Rohár
2014-12-03 16:52                           ` Pavel Machek
2014-11-27 11:18               ` Pavel Machek
2014-10-31 19:30 ` Russell King - ARM Linux
2014-10-31 21:13   ` Tony Lindgren
2014-10-31 22:12     ` Tony Lindgren
2014-10-31 22:33     ` Russell King - ARM Linux
2014-10-31 23:37       ` Tony Lindgren [this message]
2014-11-01  0:44         ` Russell King - ARM Linux
2014-11-01 21:57           ` Tony Lindgren
2014-11-02 18:15             ` 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=20141031233722.GE31454@atomide.com \
    --to=tony@atomide.com \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).