public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: linux-omap@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>,
	Olof Johansson <olof@lixom.net>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>
Subject: Re: OMAP and arm-soc's for-next branch
Date: Sat, 24 Mar 2012 10:28:35 -0700	[thread overview]
Message-ID: <20120324172834.GZ2560@atomide.com> (raw)
In-Reply-To: <20120324125029.GA29335@n2100.arm.linux.org.uk>

* Russell King - ARM Linux <linux@arm.linux.org.uk> [120324 05:53]:
> On Sat, Mar 24, 2012 at 12:22:56PM +0000, Russell King - ARM Linux wrote:
> > I've just re-merged the build tree for the nightly builds so it's now
> > based on v3.3, and pulled in the latest for-next from arm-soc.
> > 
> > I notice that the problem I reported earlier (see the "arm-soc + rmk's
> > tree boot failure on OMAP4430SDP" thread), which results in OMAP
> > being totally unable to boot, is still present.
> > 
> > As far as I'm concerned, as long as this remains unfixed in arm-soc,
> > arm-soc is not to push any branch containing the broken commit
> > (3ec2decb) upstream.
> 
> Just noticed that the fix for this has already been pushed into mainline.
> So, what this means is that anyone trying to bisect across this merge
> window with an OMAP platform will hit a range of commits which just won't
> boot.  That's really great stuff when the merge window contains the most
> probable set of commits for causing issues which would need to be bisected.

Yes the boot issue got fixed by commit dc7e57fa (OMAPDSS: register dss drivers
in module init) as discussed in the "arm-soc + rmk's tree boot failure on
OMAP4430SDP" thread.

It's too bad that the problem causing commit and the fix for a driver come
from separate development trees as that causes bisect issues like you mention
above. Better coordination and testing is needed between drivers and core
omap changes to avoid issues like this in the future.

Regards,

Tony

  reply	other threads:[~2012-03-24 17:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-24 12:22 OMAP and arm-soc's for-next branch Russell King - ARM Linux
2012-03-24 12:50 ` Russell King - ARM Linux
2012-03-24 17:28   ` Tony Lindgren [this message]
2012-03-24 18:14   ` Arnd Bergmann
2012-03-25  7:56     ` Russell King - ARM Linux
2012-03-25 10:19       ` Arnd Bergmann

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=20120324172834.GZ2560@atomide.com \
    --to=tony@atomide.com \
    --cc=arnd@arndb.de \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=olof@lixom.net \
    --cc=tomi.valkeinen@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox