From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: All OMAP platforms: MMC is broken
Date: Mon, 5 Oct 2015 04:23:04 -0700 [thread overview]
Message-ID: <20151005112304.GZ23801@atomide.com> (raw)
In-Reply-To: <20151001100326.GV21513@n2100.arm.linux.org.uk>
* Russell King - ARM Linux <linux@arm.linux.org.uk> [151001 03:07]:
> On Thu, Oct 01, 2015 at 11:50:00AM +0200, Ulf Hansson wrote:
> > On 1 October 2015 at 11:33, Russell King - ARM Linux
> > <linux@arm.linux.org.uk> wrote:
> > > On Thu, Sep 24, 2015 at 04:37:56PM -0700, Tony Lindgren wrote:
> > >> * Russell King - ARM Linux <linux@arm.linux.org.uk> [150924 02:04]:
> > >> > Nightly testing has revealed that both the OMAP3430 LDP and the OMAP4430
> > >> > SDP fail to boot due to lack of working MMC. Both platforms fail to
> > >> > find their rootfs, which is on a SD card.
> > >> >
> > >> > The breakage occurred somewhere between trees of September 9th (commit
> > >> > 4e4adb2f4628) and September 12th (commit b0a1ea51bda4), so during the
> > >> > merge window.
> > >>
> > >> Yes sorry things got messed up in multiple ways :( I've summarized
> > >> the mess here earlier:
> > >>
> > >> http://article.gmane.org/gmane.linux.kernel.mmc/33911
> > >>
> > >> And today commit b9c93646fd5c ("regulator: pbias: program pbias register
> > >> offset in pbias driver") hit mainline so I'll send a pull request for
> > >> the related dts change.
> > >
> > > It's still broken and untestable. We're 8 days off it being a full
> > > month worth of failed testing for OMAP3 and OMAP4 platforms.
> > >
> > > I think OMAP and MMC people need to do a post-mortem and work out why
> > > this happened, and how to stop it happening again in the future.
> >
> > You are probably right!
> >
> > I think I should have been more persistent when asking on how to deal
> > with these patches. Typically they should all have gone together via
> > one tree, or we needed a slower way forward keeping changes more
> > standalone.
> >
> > Anyway, according to kernelci.org, which builds and boot my next
> > branch omap3/4 seems to boot now...
> > http://kernelci.org/boot/all/job/ulfh/kernel/v4.3-rc3-27-gf7734d097520/
>
> It continues to fail here.
>
> Okay, sod it, I'm at the point of just not giving a damn about whether
> OMAP3 and OMAP4 boot anymore.
>
> I'm taking all OMAP platforms out of my boot farm. I'm totally fed up
> with this kind of regression happening. It's probably some config option
> that someone's recently introduced which defaults to being off, thereby
> breaking the ability for people to move forward without constantly having
> to re-configure. This is not acceptable.
>
> STOP BREAKING THE KERNEL.
I totally agree, this is unacceptable. And I'm fed up chasing driver
breakage and trying to test everything myself.
If Kihon is not starting to respond to these issues, we better start
reverting some of the MMC patches. And I'd say in the future non-trivial
patches really have to sit in linux next for two weeks before merging.
We have still the omap3 legacy booting issue remaining, and warnings
at least on omap4 overo.
Regards,
Regards,
Tony
next prev parent reply other threads:[~2015-10-05 11:23 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-24 9:00 All OMAP platforms: MMC is broken Russell King - ARM Linux
2015-09-24 22:51 ` Grygorii Strashko
2015-09-24 22:53 ` Russell King - ARM Linux
2015-09-24 23:37 ` Tony Lindgren
2015-09-25 1:13 ` Tony Lindgren
2015-09-26 2:54 ` Nishanth Menon
2015-10-01 9:33 ` Russell King - ARM Linux
2015-10-01 9:50 ` Ulf Hansson
2015-10-01 10:03 ` Russell King - ARM Linux
2015-10-05 11:23 ` Tony Lindgren [this message]
2015-10-05 14:35 ` Tony Lindgren
2015-10-05 14:51 ` Tony Lindgren
2015-10-05 17:11 ` Tony Lindgren
2015-10-05 18:38 ` Russell King - ARM Linux
2015-10-06 9:00 ` Russell King - ARM Linux
2015-10-06 9:44 ` Tony Lindgren
2015-10-06 10:11 ` Ulf Hansson
2015-10-06 10:36 ` Kishon Vijay Abraham I
2015-10-06 15:07 ` Russell King - ARM Linux
2015-10-06 19:29 ` Kishon Vijay Abraham I
2015-10-06 19:57 ` Russell King - ARM Linux
2015-10-08 0:46 ` Kishon Vijay Abraham I
2015-10-06 15:00 ` Russell King - ARM Linux
2015-10-06 15:37 ` Tony Lindgren
2015-10-07 12:45 ` Russell King - ARM Linux
2015-10-07 13:26 ` Tony Lindgren
2015-10-07 13:41 ` Ulf Hansson
2015-10-07 15:52 ` Tony Lindgren
2015-10-07 19:40 ` Ulf Hansson
2015-10-07 23:13 ` Kishon Vijay Abraham I
2015-10-08 8:40 ` Tony Lindgren
2015-10-08 9:35 ` Russell King - ARM Linux
2015-10-08 9:56 ` Tony Lindgren
2015-10-08 10:00 ` Russell King - ARM Linux
2015-10-07 17:53 ` Russell King - ARM Linux
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=20151005112304.GZ23801@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).