From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 07/10] ARM: clps711x: Add CLPS711X irqchip driver
Date: Mon, 5 Aug 2013 21:26:07 +0200 [thread overview]
Message-ID: <201308052126.08078.arnd@arndb.de> (raw)
In-Reply-To: <20130805221631.f740b0d39de0e14e641f477c@mail.ru>
On Monday 05 August 2013, Alexander Shiyan wrote:
> On Sun, 4 Aug 2013 22:57:58 +0200
> Arnd Bergmann <arnd@arndb.de> wrote:
>
> > Handling them in the irqchip driver probably seemed like a logical solution
> > when the code was initially written, but if you move that access into the
> > device drivers, you can probably remove the irqchip driver entirely and just
> > use the generic irqchip implementation.
>
> ioremap for a whole set of CPU registers is called from map_io, so ioremap
> returns already mapped address, that is, it's just converting physical to
> virtual addresses.
I see, that does make things better at least.
It would be nice to change the DT binding in some way so the register
ranges are non-overlapping, but I'm not going to insist if the other
reviewers are ok with the small inconsistency here.
Do you have any reason for not using the syscon driver for these
registers?
Arnd
next prev parent reply other threads:[~2013-08-05 19:26 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-18 18:34 [PATCH 00/10] ARM: clps711x: Initial DT support / Fixes Alexander Shiyan
2013-07-18 18:34 ` [PATCH 01/10] ARM: clps711x: Remove the special name for the syscon driver Alexander Shiyan
2013-08-14 6:29 ` Olof Johansson
2013-08-14 6:29 ` Olof Johansson
2013-07-18 18:34 ` [PATCH 02/10] ARM: clps711x: Drop fortunet board support Alexander Shiyan
2013-08-14 6:29 ` Olof Johansson
2013-07-18 18:34 ` [PATCH 03/10] ARM: clps711x: autcpu12: Remove incorrect config checking Alexander Shiyan
2013-08-14 6:30 ` Olof Johansson
2013-07-18 18:34 ` [PATCH 04/10] ARM: clps711x: edb7211: Remove extra iotable_init() call Alexander Shiyan
2013-08-14 6:30 ` Olof Johansson
2013-07-18 18:34 ` [PATCH 05/10] ARM: clps711x: Add CLPS711X clk driver Alexander Shiyan
2013-08-02 10:25 ` Mark Rutland
2013-07-18 18:34 ` [PATCH 06/10] ARM: clps711x: Add CLPS711X clocksource driver Alexander Shiyan
2013-07-19 14:38 ` Daniel Lezcano
2013-07-20 3:44 ` Alexander Shiyan
2013-07-20 21:48 ` Daniel Lezcano
2013-07-21 4:30 ` Alexander Shiyan
2013-08-02 10:46 ` Mark Rutland
2013-08-04 12:31 ` Alexander Shiyan
2013-08-05 17:02 ` Mark Rutland
2013-07-18 18:34 ` [PATCH 07/10] ARM: clps711x: Add CLPS711X irqchip driver Alexander Shiyan
2013-08-02 10:57 ` Mark Rutland
2013-08-02 15:55 ` Alexander Shiyan
2013-08-03 19:45 ` Arnd Bergmann
2013-08-04 4:50 ` Alexander Shiyan
2013-08-04 20:46 ` Arnd Bergmann
2013-08-04 20:57 ` Arnd Bergmann
2013-08-05 18:16 ` Alexander Shiyan
2013-08-05 19:26 ` Arnd Bergmann [this message]
2013-08-05 20:42 ` Alexander Shiyan
2013-08-05 21:00 ` Arnd Bergmann
2013-08-06 15:25 ` Alexander Shiyan
2013-08-05 16:36 ` H Hartley Sweeten
2013-07-18 18:34 ` [PATCH 08/10] ARM: clps711x: Add CLPS711X cpuidle driver Alexander Shiyan
2013-07-20 21:42 ` Daniel Lezcano
2013-07-21 4:11 ` Alexander Shiyan
2013-07-21 8:32 ` Daniel Lezcano
2013-07-21 10:04 ` Alexander Shiyan
2013-07-22 11:40 ` Russell King - ARM Linux
2013-08-02 11:10 ` Mark Rutland
2013-07-18 18:35 ` [PATCH 09/10] ARM: clps711x: Migrate CLPS711X subarch to the new basic drivers Alexander Shiyan
2013-07-18 18:35 ` [PATCH 10/10] ARM: clps711x: Add initial DT support Alexander Shiyan
2013-08-02 11:15 ` Mark Rutland
2013-08-03 3:54 ` [PATCH 00/10] ARM: clps711x: Initial DT support / Fixes Alexander Shiyan
2013-08-14 6:29 ` Olof Johansson
2013-08-17 4:32 ` Alexander Shiyan
2013-08-22 5:00 ` Olof Johansson
2013-08-14 6:31 ` Olof Johansson
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=201308052126.08078.arnd@arndb.de \
--to=arnd@arndb.de \
--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.