From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Baruch Siach <baruch@tkos.co.il>, Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 0/2] clocksource: Conexant CX92755 timers support
Date: Mon, 26 Jan 2015 22:23:56 +0100 [thread overview]
Message-ID: <54C6B06C.20907@linaro.org> (raw)
In-Reply-To: <cover.1422296650.git.baruch@tkos.co.il>
Hi Baruch,
applied the series for 3.20.
Thanks !
-- Daniel
On 01/26/2015 07:35 PM, Baruch Siach wrote:
> This short series adds support for the Conexant CX92755 SoC timers. This SoC is
> part of the Conexant Digicolor series of SoCs.
>
> v6:
> * Use a dedicated config symbol, CONFIG_DIGICOLOR_TIMER, to enable the
> driver (Daniel Lezcano)
>
> v5:
> http://thread.gmane.org/gmane.linux.kernel/1874237
>
> * Switch back to use of_iomap (not of_io_request_and_map), as the watchdog
> driver also needs access to the same registers; add a comment clarifying
> this
>
> Address more review comments by Daniel Lezcano:
>
> * Properly encapsulate the static objects for the clock_event code
>
> * Use dc_timer_{disable,enable,set_count} accessors to make the code clearer
>
> v4:
> http://thread.gmane.org/gmane.linux.kernel/1874017
>
> Address review comments by Daniel Lezcano:
>
> * Use macros for register configuration values
>
> * Encapsulate static objects in a struct
>
> * Use of_io_request_and_map to exclusively map registers
>
> * Use UINT_MAX instead of ~0
>
> * Use request_irq instead of setup_irq, obviating the need for a separate
> irqaction struct
>
> v3:
> http://article.gmane.org/gmane.linux.kernel/1870628
> http://article.gmane.org/gmane.linux.kernel/1870627
>
> * Split into a separate clocksource series
>
> v2:
> http://article.gmane.org/gmane.linux.kernel/1861850
> http://article.gmane.org/gmane.linux.kernel/1861853
>
> * Change the timer dt binding, so that the 'reg' property points to the
> first "Agent Communication" register. This should improve the chance of
> reusing this binding for other SoCs in this series.
>
> * Add the CONTROL() and COUNT() macros to the timer driver to make the code
> clearer.
>
> * Move arch/arm Kconfig changes from the clocksource driver patch to the
> base arch support patch to reduce dependency between them
>
> v1:
> http://article.gmane.org/gmane.linux.kernel/1855028
> http://article.gmane.org/gmane.linux.kernel/1855025
>
> Baruch Siach (2):
> clocksource: devicetree: document Conexant Digicolor timer binding
> clocksource: driver for Conexant Digicolor SoC timer
>
> .../devicetree/bindings/timer/digicolor-timer.txt | 18 ++
> drivers/clocksource/Kconfig | 3 +
> drivers/clocksource/Makefile | 1 +
> drivers/clocksource/timer-digicolor.c | 199 +++++++++++++++++++++
> 4 files changed, 221 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/timer/digicolor-timer.txt
> create mode 100644 drivers/clocksource/timer-digicolor.c
>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: daniel.lezcano@linaro.org (Daniel Lezcano)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 0/2] clocksource: Conexant CX92755 timers support
Date: Mon, 26 Jan 2015 22:23:56 +0100 [thread overview]
Message-ID: <54C6B06C.20907@linaro.org> (raw)
In-Reply-To: <cover.1422296650.git.baruch@tkos.co.il>
Hi Baruch,
applied the series for 3.20.
Thanks !
-- Daniel
On 01/26/2015 07:35 PM, Baruch Siach wrote:
> This short series adds support for the Conexant CX92755 SoC timers. This SoC is
> part of the Conexant Digicolor series of SoCs.
>
> v6:
> * Use a dedicated config symbol, CONFIG_DIGICOLOR_TIMER, to enable the
> driver (Daniel Lezcano)
>
> v5:
> http://thread.gmane.org/gmane.linux.kernel/1874237
>
> * Switch back to use of_iomap (not of_io_request_and_map), as the watchdog
> driver also needs access to the same registers; add a comment clarifying
> this
>
> Address more review comments by Daniel Lezcano:
>
> * Properly encapsulate the static objects for the clock_event code
>
> * Use dc_timer_{disable,enable,set_count} accessors to make the code clearer
>
> v4:
> http://thread.gmane.org/gmane.linux.kernel/1874017
>
> Address review comments by Daniel Lezcano:
>
> * Use macros for register configuration values
>
> * Encapsulate static objects in a struct
>
> * Use of_io_request_and_map to exclusively map registers
>
> * Use UINT_MAX instead of ~0
>
> * Use request_irq instead of setup_irq, obviating the need for a separate
> irqaction struct
>
> v3:
> http://article.gmane.org/gmane.linux.kernel/1870628
> http://article.gmane.org/gmane.linux.kernel/1870627
>
> * Split into a separate clocksource series
>
> v2:
> http://article.gmane.org/gmane.linux.kernel/1861850
> http://article.gmane.org/gmane.linux.kernel/1861853
>
> * Change the timer dt binding, so that the 'reg' property points to the
> first "Agent Communication" register. This should improve the chance of
> reusing this binding for other SoCs in this series.
>
> * Add the CONTROL() and COUNT() macros to the timer driver to make the code
> clearer.
>
> * Move arch/arm Kconfig changes from the clocksource driver patch to the
> base arch support patch to reduce dependency between them
>
> v1:
> http://article.gmane.org/gmane.linux.kernel/1855028
> http://article.gmane.org/gmane.linux.kernel/1855025
>
> Baruch Siach (2):
> clocksource: devicetree: document Conexant Digicolor timer binding
> clocksource: driver for Conexant Digicolor SoC timer
>
> .../devicetree/bindings/timer/digicolor-timer.txt | 18 ++
> drivers/clocksource/Kconfig | 3 +
> drivers/clocksource/Makefile | 1 +
> drivers/clocksource/timer-digicolor.c | 199 +++++++++++++++++++++
> 4 files changed, 221 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/timer/digicolor-timer.txt
> create mode 100644 drivers/clocksource/timer-digicolor.c
>
--
<http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
next prev parent reply other threads:[~2015-01-26 21:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-26 18:35 [PATCH v6 0/2] clocksource: Conexant CX92755 timers support Baruch Siach
2015-01-26 18:35 ` Baruch Siach
2015-01-26 18:35 ` [PATCH v6 1/2] clocksource: devicetree: document Conexant Digicolor timer binding Baruch Siach
2015-01-26 18:35 ` Baruch Siach
2015-01-26 18:35 ` [PATCH v6 2/2] clocksource: driver for Conexant Digicolor SoC timer Baruch Siach
2015-01-26 18:35 ` Baruch Siach
2015-01-26 21:23 ` Daniel Lezcano [this message]
2015-01-26 21:23 ` [PATCH v6 0/2] clocksource: Conexant CX92755 timers support Daniel Lezcano
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=54C6B06C.20907@linaro.org \
--to=daniel.lezcano@linaro.org \
--cc=baruch@tkos.co.il \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.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.