devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rob Herring <rob.herring@calxeda.com>
To: Meelis Roos <mroos@linux.ee>,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Thierry Reding <thierry.reding@avionic-design.de>,
	Linux Kernel list <linux-kernel@vger.kernel.org>,
	Grant Likely <grant.likely@secretlab.ca>,
	sparclinux@vger.kernel.org, devicetree-discuss@lists.ozlabs.org
Subject: Re: bisected regression since 3.5-rc5 - hang on boot on some sparc64 machines
Date: Tue, 10 Jul 2012 10:00:41 -0500	[thread overview]
Message-ID: <4FFC4399.3050407@calxeda.com> (raw)
In-Reply-To: <alpine.SOC.1.00.1207091627480.17307@math.ut.ee>

On 07/10/2012 06:09 AM, Meelis Roos wrote:
> A regression since 3.5-rc5 stops Sun Fire V100 and Sun Netra X1 sparc64 
> machines from booting, hanging after enabling serial console. Several 
> different other sparc64 systems work fine with 3.5-rc6. prtconf and full 
> dmesg below.
> 
> Bisecting leads to
> 107a84e61cdd3406c842a0e4be7efffd3a05dba6 is the first bad commit
> commit 107a84e61cdd3406c842a0e4be7efffd3a05dba6
> Author: Thierry Reding <thierry.reding@avionic-design.de>
> Date:   Thu Jun 14 09:12:35 2012 +0200
> 
>     of: match by compatible property first
>     
>     When matching devices against an OF device ID table, the first string of
>     the compatible property that is listed in the table should match,
>     regardless of its position in the table.
>     
>     Cc: Grant Likely <grant.likely@secretlab.ca>
>     Cc: Rob Herring <rob.herring@calxeda.com>
>     Cc: devicetree-discuss@lists.ozlabs.org
>     Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
>     Signed-off-by: Rob Herring <rob.herring@calxeda.com>

The problem is match combinations of compatible plus name and/or type
fail to match correctly. I have a fix for this, but given how late it is
for 3.5 I think it is best to revert this for now. There could be other
cases that rely on the current although wrong behavior. I will post an
updated version for 3.6.

Linus, Please revert 107a84e61cdd3406c842a0e4be7efffd3a05dba6.

Rob

      reply	other threads:[~2012-07-10 15:00 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-10 11:09 bisected regression since 3.5-rc5 - hang on boot on some sparc64 machines Meelis Roos
2012-07-10 15:00 ` Rob Herring [this message]

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=4FFC4399.3050407@calxeda.com \
    --to=rob.herring@calxeda.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mroos@linux.ee \
    --cc=sparclinux@vger.kernel.org \
    --cc=thierry.reding@avionic-design.de \
    --cc=torvalds@linux-foundation.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).