From: haojian.zhuang@linaro.org (Haojian Zhuang)
To: linux-arm-kernel@lists.infradead.org
Subject: STOP THE MADNESS
Date: Wed, 27 Nov 2013 09:22:46 +0800 [thread overview]
Message-ID: <52954966.8030706@linaro.org> (raw)
In-Reply-To: <CACRpkdY9MTUDtDxRKRHCXch98cNiMS-dumQvaTH7gp1OCVUx6w@mail.gmail.com>
On 11/27/2013 06:09 AM, Linus Walleij wrote:
> On Tue, Nov 26, 2013 at 6:29 PM, Rob Herring <robherring2@gmail.com> wrote:
>
>> IMO, the whole commit causing this problem should be reverted for
>> additional reasons. Irq generic chp and irqdomains are orthogonal
>> features/infrastructure and should not be mutually exclusive. We need
>> to fix irq generic chip.
> This cannot be easily reverted due to a lot of code and fixes building
> on top of it, and even if it could I guess it will also break device tree
> use on these: git grep pl061 arch/arm/boot/dts
>
> As DT mandates that the irqdomain be used, or it cannot translate
> the IRQs as before this patch it uses an hard-coded base from
> platform data. This is one of those odd drivers that don't need
> one line of DT-specific code in it to work with device tree, yet
> it works.
>
> I guess what I have to do is take out the very board Russell has
> and that I happen to have also and just get to the bottom of this
> bug and fix it and test it.
>
> A pointer to an "ideal" GPIO chip + irq_chip driver would be highly
> appreciated ...
>
> Yours,
> Linus Walleij
Does qemu support the board that Russell is using? If so, we can fix &
test it.
Regards
Haojian
next prev parent reply other threads:[~2013-11-27 1:22 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-26 16:30 STOP THE MADNESS Russell King - ARM Linux
2013-11-26 17:29 ` Rob Herring
2013-11-26 22:09 ` Linus Walleij
2013-11-27 1:22 ` Haojian Zhuang [this message]
2013-11-27 7:44 ` Linus Walleij
2013-11-27 3:54 ` Rob Herring
2013-11-27 7:41 ` Linus Walleij
2013-11-26 21:43 ` Linus Walleij
2013-11-29 2:01 ` Russell King - ARM Linux
2013-11-29 10:11 ` Linus Walleij
2013-11-29 10:46 ` Russell King - ARM Linux
2013-11-29 12:11 ` Linus Walleij
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=52954966.8030706@linaro.org \
--to=haojian.zhuang@linaro.org \
--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.