From: Qais Yousef <qais.yousef@imgtec.com>
To: Andrew Bresticker <abrestic@chromium.org>
Cc: Ralf Baechle <ralf@linux-mips.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Thomas Gleixner <tglx@linutronix.de>,
"Jason Cooper" <jason@lakedaemon.net>,
Paul Burton <paul.burton@imgtec.com>,
"John Crispin" <blogic@openwrt.org>, <linux-mips@linux-mips.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 00/19] MIPS GIC cleanup, part 2
Date: Wed, 22 Oct 2014 12:03:16 +0100 [thread overview]
Message-ID: <54478EF4.7050909@imgtec.com> (raw)
In-Reply-To: <1413831846-32100-1-git-send-email-abrestic@chromium.org>
On 10/20/2014 08:03 PM, Andrew Bresticker wrote:
> Second round of cleanups for the MIPS GIC drivers:
> - Patches 1 through 5 get rid of the ugly REG() macros and instead use
> proper iomem accessors.
> - Patches 6 and 7 move the GIC header to linux/irqchip/ and clean it up.
> - Patches 8 through 10 are misc. GIC irqchip cleanups.
> - Patches 11 and 12 combine the GIC clocksource and clockevent drivers and
> move them to drivers/clocksource/.
> - Patches 13 through 19 are various cleanups for the GIC clocksource driver.
>
> Boot tested on Malta and (with additional out-of-tree patches) a platform
> based on the IMG Pistachio SoC. Build tested for SEAD-3.
I used your github tree to boot test this on sead3 and it worked fine.
Except for the one comment I had, feel free to add my
tested-and-reviewed by.
Nice work :)
Qais
> Based on 3.18-rc1 + part 1 of my GIC cleanup series [0]. A tree with both
> series is available at:
> https://github.com/abrestic/linux/commits/mips-gic-cleanup-pt2-v1
>
> [0] https://lkml.org/lkml/2014/9/18/487
>
> Andrew Bresticker (19):
> MIPS: Malta: Use gic_read_count() to read GIC timer
> irqchip: mips-gic: Export function to read counter width
> MIPS: sead3: Stop using GIC REG macros
> MIPS: Malta: Stop using GIC REG macros
> irqchip: mips-gic: Use proper iomem accessors
> MIPS: Move gic.h to include/linux/irqchip/mips-gic.h
> irqchip: mips-gic: Clean up header file
> irqchip: mips-gic: Clean up #includes
> irqchip: mips-gic: Remove gic_{pending,itrmask}_regs
> irqchip: mips-gic: Use GIC_SH_WEDGE_{SET,CLR} macros
> MIPS: Move GIC clocksource driver to drivers/clocksource/
> clocksource: mips-gic: Combine with GIC clockevent driver
> clocksource: mips-gic: Staticize local symbols
> clocksource: mips-gic: Move gic_frequency to clocksource driver
> clocksource: mips-gic: Remove gic_event_handler
> clocksource: mips-gic: Use percpu_dev_id
> clocksource: mips-gic: Use CPU notifiers to setup the timer
> clocksource: mips-gic: Use clockevents_config_and_register
> clocksource: mips-gic: Bump up rating of GIC timer
>
> arch/mips/Kconfig | 21 +-
> arch/mips/include/asm/mips-boards/maltaint.h | 2 +-
> arch/mips/include/asm/mips-boards/sead3int.h | 2 +-
> arch/mips/include/asm/time.h | 5 +-
> arch/mips/kernel/Makefile | 2 -
> arch/mips/kernel/cevt-gic.c | 103 ---------
> arch/mips/kernel/cevt-r4k.c | 2 +-
> arch/mips/kernel/csrc-gic.c | 40 ----
> arch/mips/kernel/smp-cmp.c | 2 +-
> arch/mips/kernel/smp-cps.c | 2 +-
> arch/mips/kernel/smp-gic.c | 2 +-
> arch/mips/kernel/smp-mt.c | 2 +-
> arch/mips/mti-malta/malta-int.c | 15 +-
> arch/mips/mti-malta/malta-time.c | 20 +-
> arch/mips/mti-sead3/sead3-ehci.c | 2 +-
> arch/mips/mti-sead3/sead3-int.c | 9 +-
> arch/mips/mti-sead3/sead3-net.c | 2 +-
> arch/mips/mti-sead3/sead3-platform.c | 2 +-
> arch/mips/mti-sead3/sead3-time.c | 2 +-
> drivers/clocksource/Kconfig | 4 +
> drivers/clocksource/Makefile | 1 +
> drivers/clocksource/mips-gic-timer.c | 139 ++++++++++++
> drivers/irqchip/irq-mips-gic.c | 243 +++++++++++++--------
> .../asm/gic.h => include/linux/irqchip/mips-gic.h | 203 +++--------------
> 24 files changed, 363 insertions(+), 464 deletions(-)
> delete mode 100644 arch/mips/kernel/cevt-gic.c
> delete mode 100644 arch/mips/kernel/csrc-gic.c
> create mode 100644 drivers/clocksource/mips-gic-timer.c
> rename arch/mips/include/asm/gic.h => include/linux/irqchip/mips-gic.h (61%)
>
WARNING: multiple messages have this Message-ID (diff)
From: Qais Yousef <qais.yousef@imgtec.com>
To: Andrew Bresticker <abrestic@chromium.org>
Cc: Ralf Baechle <ralf@linux-mips.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Thomas Gleixner <tglx@linutronix.de>,
Jason Cooper <jason@lakedaemon.net>,
Paul Burton <paul.burton@imgtec.com>,
John Crispin <blogic@openwrt.org>,
linux-mips@linux-mips.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/19] MIPS GIC cleanup, part 2
Date: Wed, 22 Oct 2014 12:03:16 +0100 [thread overview]
Message-ID: <54478EF4.7050909@imgtec.com> (raw)
Message-ID: <20141022110316.x8o3IswtfjQVKlPv_yWm29vRzir0MMhGYFfUus9hlnA@z> (raw)
In-Reply-To: <1413831846-32100-1-git-send-email-abrestic@chromium.org>
On 10/20/2014 08:03 PM, Andrew Bresticker wrote:
> Second round of cleanups for the MIPS GIC drivers:
> - Patches 1 through 5 get rid of the ugly REG() macros and instead use
> proper iomem accessors.
> - Patches 6 and 7 move the GIC header to linux/irqchip/ and clean it up.
> - Patches 8 through 10 are misc. GIC irqchip cleanups.
> - Patches 11 and 12 combine the GIC clocksource and clockevent drivers and
> move them to drivers/clocksource/.
> - Patches 13 through 19 are various cleanups for the GIC clocksource driver.
>
> Boot tested on Malta and (with additional out-of-tree patches) a platform
> based on the IMG Pistachio SoC. Build tested for SEAD-3.
I used your github tree to boot test this on sead3 and it worked fine.
Except for the one comment I had, feel free to add my
tested-and-reviewed by.
Nice work :)
Qais
> Based on 3.18-rc1 + part 1 of my GIC cleanup series [0]. A tree with both
> series is available at:
> https://github.com/abrestic/linux/commits/mips-gic-cleanup-pt2-v1
>
> [0] https://lkml.org/lkml/2014/9/18/487
>
> Andrew Bresticker (19):
> MIPS: Malta: Use gic_read_count() to read GIC timer
> irqchip: mips-gic: Export function to read counter width
> MIPS: sead3: Stop using GIC REG macros
> MIPS: Malta: Stop using GIC REG macros
> irqchip: mips-gic: Use proper iomem accessors
> MIPS: Move gic.h to include/linux/irqchip/mips-gic.h
> irqchip: mips-gic: Clean up header file
> irqchip: mips-gic: Clean up #includes
> irqchip: mips-gic: Remove gic_{pending,itrmask}_regs
> irqchip: mips-gic: Use GIC_SH_WEDGE_{SET,CLR} macros
> MIPS: Move GIC clocksource driver to drivers/clocksource/
> clocksource: mips-gic: Combine with GIC clockevent driver
> clocksource: mips-gic: Staticize local symbols
> clocksource: mips-gic: Move gic_frequency to clocksource driver
> clocksource: mips-gic: Remove gic_event_handler
> clocksource: mips-gic: Use percpu_dev_id
> clocksource: mips-gic: Use CPU notifiers to setup the timer
> clocksource: mips-gic: Use clockevents_config_and_register
> clocksource: mips-gic: Bump up rating of GIC timer
>
> arch/mips/Kconfig | 21 +-
> arch/mips/include/asm/mips-boards/maltaint.h | 2 +-
> arch/mips/include/asm/mips-boards/sead3int.h | 2 +-
> arch/mips/include/asm/time.h | 5 +-
> arch/mips/kernel/Makefile | 2 -
> arch/mips/kernel/cevt-gic.c | 103 ---------
> arch/mips/kernel/cevt-r4k.c | 2 +-
> arch/mips/kernel/csrc-gic.c | 40 ----
> arch/mips/kernel/smp-cmp.c | 2 +-
> arch/mips/kernel/smp-cps.c | 2 +-
> arch/mips/kernel/smp-gic.c | 2 +-
> arch/mips/kernel/smp-mt.c | 2 +-
> arch/mips/mti-malta/malta-int.c | 15 +-
> arch/mips/mti-malta/malta-time.c | 20 +-
> arch/mips/mti-sead3/sead3-ehci.c | 2 +-
> arch/mips/mti-sead3/sead3-int.c | 9 +-
> arch/mips/mti-sead3/sead3-net.c | 2 +-
> arch/mips/mti-sead3/sead3-platform.c | 2 +-
> arch/mips/mti-sead3/sead3-time.c | 2 +-
> drivers/clocksource/Kconfig | 4 +
> drivers/clocksource/Makefile | 1 +
> drivers/clocksource/mips-gic-timer.c | 139 ++++++++++++
> drivers/irqchip/irq-mips-gic.c | 243 +++++++++++++--------
> .../asm/gic.h => include/linux/irqchip/mips-gic.h | 203 +++--------------
> 24 files changed, 363 insertions(+), 464 deletions(-)
> delete mode 100644 arch/mips/kernel/cevt-gic.c
> delete mode 100644 arch/mips/kernel/csrc-gic.c
> create mode 100644 drivers/clocksource/mips-gic-timer.c
> rename arch/mips/include/asm/gic.h => include/linux/irqchip/mips-gic.h (61%)
>
next prev parent reply other threads:[~2014-10-22 11:03 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-20 19:03 [PATCH 00/19] MIPS GIC cleanup, part 2 Andrew Bresticker
2014-10-20 19:03 ` [PATCH 01/19] MIPS: Malta: Use gic_read_count() to read GIC timer Andrew Bresticker
2014-10-20 19:03 ` [PATCH 02/19] irqchip: mips-gic: Export function to read counter width Andrew Bresticker
2014-10-20 19:03 ` [PATCH 03/19] MIPS: sead3: Stop using GIC REG macros Andrew Bresticker
2014-10-22 9:33 ` Qais Yousef
2014-10-22 9:33 ` Qais Yousef
2014-10-22 18:01 ` Andrew Bresticker
2014-10-20 19:03 ` [PATCH 04/19] MIPS: Malta: " Andrew Bresticker
2014-10-20 19:03 ` [PATCH 05/19] irqchip: mips-gic: Use proper iomem accessors Andrew Bresticker
2014-10-20 19:03 ` [PATCH 06/19] MIPS: Move gic.h to include/linux/irqchip/mips-gic.h Andrew Bresticker
2014-10-20 19:03 ` [PATCH 07/19] irqchip: mips-gic: Clean up header file Andrew Bresticker
2014-10-20 19:03 ` [PATCH 08/19] irqchip: mips-gic: Clean up #includes Andrew Bresticker
2014-10-20 19:03 ` [PATCH 09/19] irqchip: mips-gic: Remove gic_{pending,itrmask}_regs Andrew Bresticker
2014-10-20 19:03 ` [PATCH 10/19] irqchip: mips-gic: Use GIC_SH_WEDGE_{SET,CLR} macros Andrew Bresticker
2014-10-20 19:03 ` [PATCH 11/19] MIPS: Move GIC clocksource driver to drivers/clocksource/ Andrew Bresticker
2014-10-20 19:03 ` [PATCH 12/19] clocksource: mips-gic: Combine with GIC clockevent driver Andrew Bresticker
2014-10-20 19:04 ` [PATCH 13/19] clocksource: mips-gic: Staticize local symbols Andrew Bresticker
2014-10-20 19:04 ` [PATCH 14/19] clocksource: mips-gic: Move gic_frequency to clocksource driver Andrew Bresticker
2014-10-20 19:04 ` [PATCH 15/19] clocksource: mips-gic: Remove gic_event_handler Andrew Bresticker
2014-10-20 19:04 ` [PATCH 16/19] clocksource: mips-gic: Use percpu_dev_id Andrew Bresticker
2014-10-20 19:04 ` [PATCH 17/19] clocksource: mips-gic: Use CPU notifiers to setup the timer Andrew Bresticker
2014-10-20 19:04 ` [PATCH 18/19] clocksource: mips-gic: Use clockevents_config_and_register Andrew Bresticker
2014-10-20 19:04 ` [PATCH 19/19] clocksource: mips-gic: Bump up rating of GIC timer Andrew Bresticker
2014-10-22 11:03 ` Qais Yousef [this message]
2014-10-22 11:03 ` [PATCH 00/19] MIPS GIC cleanup, part 2 Qais Yousef
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=54478EF4.7050909@imgtec.com \
--to=qais.yousef@imgtec.com \
--cc=abrestic@chromium.org \
--cc=blogic@openwrt.org \
--cc=daniel.lezcano@linaro.org \
--cc=jason@lakedaemon.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=paul.burton@imgtec.com \
--cc=ralf@linux-mips.org \
--cc=tglx@linutronix.de \
/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.