From mboxrd@z Thu Jan 1 00:00:00 1970 From: daniel.lezcano@linaro.org (Daniel Lezcano) Date: Tue, 11 Feb 2014 13:50:20 +0100 Subject: [PATCH v6 0/3] Introduce clocksource driver for Keystone platform In-Reply-To: <52F8F55A.2090903@ti.com> References: <1392027058-11680-1-git-send-email-ivan.khoronzhuk@ti.com> <52F8F55A.2090903@ti.com> Message-ID: <52FA1C8C.40105@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/10/2014 04:50 PM, Santosh Shilimkar wrote: > Daniel, > > On Monday 10 February 2014 05:10 AM, Ivan Khoronzhuk wrote: >> Add a broadcast timer64 based clockevent driver for keystone arch. >> This driver uses timer in 64-bit general purpose mode as clock event >> device. >> >> Documentation: >> http://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf >> >> Based on >> git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git >> keystone/master >> >> v5..v6: >> added function to encapsulate __iowmb(). >> >> v4..v5: >> used __iowmb() insted of wmb() >> >> v3..v4: >> rebased on latest of linux-keystone.git keystone/master >> >> v2..v3: >> - clocksource: timer-keystone: introduce clocksource driver for >> changed "u64" type to "unsigned long" for hz_period as more appropriate >> hz_period rounded up by DIV_ROUND_UP(rate, HZ) >> corrected comments >> >> v1..v2: >> - clocksource: timer-keystone: introduce clocksource driver for >> renamed timer on "timer-keystone" >> in keystone_timer_interrupt() evet pointer is passed via "dev_id" >> used __relaxed variants of writel/readl and added explicit barriers >> added "keystone_timer_disable()" for using in keystone_set_mode() >> keystone_timer_config() is not used for disabling the timer any more >> in case of an unsupported mode the keystone_timer_config() returns -1. >> used request_irq() instead of setup_irq() >> assigned irq for event_device in event_dev->irq >> calculated timer.hz_period for CLOCK_EVT_MODE_PERIODIC at init >> deleted spare call of keystone_timer_config() in keystone_timer_init() >> >> Ivan Khoronzhuk (3): >> clocksource: timer-keystone: introduce clocksource driver for Keystone >> clocksource: keystone: add bindings for keystone timer >> arm: dts: keystone: add keystone timer entry >> > Can you queue the first two patches from the series ? > I will queue the dts patch via my tree. Sure. -- Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH v6 0/3] Introduce clocksource driver for Keystone platform Date: Tue, 11 Feb 2014 13:50:20 +0100 Message-ID: <52FA1C8C.40105@linaro.org> References: <1392027058-11680-1-git-send-email-ivan.khoronzhuk@ti.com> <52F8F55A.2090903@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <52F8F55A.2090903-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Santosh Shilimkar Cc: Ivan Khoronzhuk , rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, grygorii.strashko-l0cyMroinI0@public.gmane.org, joshc-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org List-Id: devicetree@vger.kernel.org On 02/10/2014 04:50 PM, Santosh Shilimkar wrote: > Daniel, > > On Monday 10 February 2014 05:10 AM, Ivan Khoronzhuk wrote: >> Add a broadcast timer64 based clockevent driver for keystone arch. >> This driver uses timer in 64-bit general purpose mode as clock event >> device. >> >> Documentation: >> http://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf >> >> Based on >> git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keyston= e.git >> keystone/master >> >> v5..v6: >> added function to encapsulate __iowmb(). >> >> v4..v5: >> used __iowmb() insted of wmb() >> >> v3..v4: >> rebased on latest of linux-keystone.git keystone/master >> >> v2..v3: >> - clocksource: timer-keystone: introduce clocksource driver for >> changed "u64" type to "unsigned long" for hz_period as more appropr= iate >> hz_period rounded up by DIV_ROUND_UP(rate, HZ) >> corrected comments >> >> v1..v2: >> - clocksource: timer-keystone: introduce clocksource driver for >> renamed timer on "timer-keystone" >> in keystone_timer_interrupt() evet pointer is passed via "dev_id" >> used __relaxed variants of writel/readl and added explicit barriers >> added "keystone_timer_disable()" for using in keystone_set_mode() >> keystone_timer_config() is not used for disabling the timer any mor= e >> in case of an unsupported mode the keystone_timer_config() returns = -1. >> used request_irq() instead of setup_irq() >> assigned irq for event_device in event_dev->irq >> calculated timer.hz_period for CLOCK_EVT_MODE_PERIODIC at init >> deleted spare call of keystone_timer_config() in keystone_timer_ini= t() >> >> Ivan Khoronzhuk (3): >> clocksource: timer-keystone: introduce clocksource driver for Key= stone >> clocksource: keystone: add bindings for keystone timer >> arm: dts: keystone: add keystone timer entry >> > Can you queue the first two patches from the series ? > I will queue the dts patch via my tree. Sure. --=20 Linaro.org =E2=94=82 Open source software fo= r ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751415AbaBKMu0 (ORCPT ); Tue, 11 Feb 2014 07:50:26 -0500 Received: from mail-we0-f173.google.com ([74.125.82.173]:57943 "EHLO mail-we0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750862AbaBKMuX (ORCPT ); Tue, 11 Feb 2014 07:50:23 -0500 Message-ID: <52FA1C8C.40105@linaro.org> Date: Tue, 11 Feb 2014 13:50:20 +0100 From: Daniel Lezcano User-Agent: Mozilla/5.0 (X11; Linux i686; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Santosh Shilimkar CC: Ivan Khoronzhuk , rob@landley.net, linux@arm.linux.org.uk, galak@codeaurora.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, tglx@linutronix.de, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, grygorii.strashko@ti.com, joshc@codeaurora.org Subject: Re: [PATCH v6 0/3] Introduce clocksource driver for Keystone platform References: <1392027058-11680-1-git-send-email-ivan.khoronzhuk@ti.com> <52F8F55A.2090903@ti.com> In-Reply-To: <52F8F55A.2090903@ti.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/10/2014 04:50 PM, Santosh Shilimkar wrote: > Daniel, > > On Monday 10 February 2014 05:10 AM, Ivan Khoronzhuk wrote: >> Add a broadcast timer64 based clockevent driver for keystone arch. >> This driver uses timer in 64-bit general purpose mode as clock event >> device. >> >> Documentation: >> http://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf >> >> Based on >> git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git >> keystone/master >> >> v5..v6: >> added function to encapsulate __iowmb(). >> >> v4..v5: >> used __iowmb() insted of wmb() >> >> v3..v4: >> rebased on latest of linux-keystone.git keystone/master >> >> v2..v3: >> - clocksource: timer-keystone: introduce clocksource driver for >> changed "u64" type to "unsigned long" for hz_period as more appropriate >> hz_period rounded up by DIV_ROUND_UP(rate, HZ) >> corrected comments >> >> v1..v2: >> - clocksource: timer-keystone: introduce clocksource driver for >> renamed timer on "timer-keystone" >> in keystone_timer_interrupt() evet pointer is passed via "dev_id" >> used __relaxed variants of writel/readl and added explicit barriers >> added "keystone_timer_disable()" for using in keystone_set_mode() >> keystone_timer_config() is not used for disabling the timer any more >> in case of an unsupported mode the keystone_timer_config() returns -1. >> used request_irq() instead of setup_irq() >> assigned irq for event_device in event_dev->irq >> calculated timer.hz_period for CLOCK_EVT_MODE_PERIODIC at init >> deleted spare call of keystone_timer_config() in keystone_timer_init() >> >> Ivan Khoronzhuk (3): >> clocksource: timer-keystone: introduce clocksource driver for Keystone >> clocksource: keystone: add bindings for keystone timer >> arm: dts: keystone: add keystone timer entry >> > Can you queue the first two patches from the series ? > I will queue the dts patch via my tree. Sure. -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog