All of lore.kernel.org
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [ARM] head.S change broke platform device registration?
Date: Fri, 30 Nov 2012 12:40:49 +0000	[thread overview]
Message-ID: <20121130124049.GO19440@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAHod+GcgvxPGUi4p9-qVuRXfyODrYwDvU8u5cChgiXjFYpwOEg@mail.gmail.com>

On Fri, Nov 30, 2012 at 01:28:33PM +0100, Marko Kati? wrote:
> On Fri, Nov 30, 2012 at 1:17 PM, Dave Martin <dave.martin@linaro.org> wrote:
> > This is indeed strange, since there is not much difference to the kernel
> > if CONFIG_ARM_VIRT_EXT is not set.
> >
> > Unless a bug is causing the kernel to trample on some random memory,
> > there is no reason for this to cause I2C devices to disappear.
> >
> > It could be something unrelated, like additional and removal of code
> > changing the position or alignment of other parts of the kernel image.
> >
> > Have you could try undoing parts of the patch to narrow the bug down
> > further?
> >
> > Cheers
> > ---Dave
> 
> No, not yet. As i said my knowledge of what goes on in head.S is limited
> so i'd rather not mess with it.

Well, I think everyone's confused how this change could cause an I2C
device to vanish.  I've been through this code carefully and I don't
see anything that would affect an I2C device here.  The code in question
is not even touching any hardware.

Please post your .config file so we can see how your kernel is configured.

  reply	other threads:[~2012-11-30 12:40 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-30  1:38 [ARM] head.S change broke platform device registration? Marko Katić
2012-11-30 10:28 ` Marc Zyngier
2012-11-30 12:20   ` Marko Katić
2012-11-30 13:45     ` Marc Zyngier
2012-11-30 14:34     ` Russell King - ARM Linux
2012-12-04 21:48       ` Marko Katić
2012-12-04 22:18         ` Russell King - ARM Linux
2012-12-05 22:18           ` Marko Katić
2012-12-05 23:58             ` Russell King - ARM Linux
2012-12-07 12:28               ` Russell King - ARM Linux
2012-12-07 13:49                 ` Dave Martin
2012-12-10 17:23                   ` Marko Katić
2012-12-10 17:31                     ` Dave Martin
2012-12-10 17:21               ` Marko Katić
2012-12-11  0:25                 ` Russell King - ARM Linux
2012-12-12 15:55                   ` Richard Purdie
2012-12-12 17:22                     ` Marko Katić
2012-12-12 17:59                     ` Russell King - ARM Linux
2012-12-12 22:29                       ` Richard Purdie
2012-11-30 12:17 ` Dave Martin
2012-11-30 12:28   ` Marko Katić
2012-11-30 12:40     ` Russell King - ARM Linux [this message]
2012-11-30 13:07       ` Marko Katić
2012-11-30 13:11         ` Marko Katić

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=20121130124049.GO19440@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --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 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.