From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753091AbdLHTUF (ORCPT ); Fri, 8 Dec 2017 14:20:05 -0500 Received: from eddie.linux-mips.org ([148.251.95.138]:43820 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752431AbdLHTUE (ORCPT ); Fri, 8 Dec 2017 14:20:04 -0500 Date: Fri, 8 Dec 2017 20:19:59 +0100 From: Ladislav Michl To: Daniel Lezcano Cc: Keerthy , tony@atomide.com, aaro.koskinen@iki.fi, thierry.reding@gmail.com, t-kristo@ti.com, linux@armlinux.org.uk, grygorii.strashko@ti.com, robh+dt@kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pwm@vger.kernel.org, sebastian.reichel@collabora.co.uk, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 4/8] arm: OMAP: Move dmtimer driver out of plat-omap to drivers under clocksource Message-ID: <20171208191959.GA5801@lenoch> References: <1512105397-2544-1-git-send-email-j-keerthy@ti.com> <1512105397-2544-5-git-send-email-j-keerthy@ti.com> <20171208172031.GA4146@lenoch> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 08, 2017 at 06:50:32PM +0100, Daniel Lezcano wrote: > On 08/12/2017 18:20, Ladislav Michl wrote: > > Daniel, > > > > On Thu, Dec 07, 2017 at 11:54:27AM +0100, Daniel Lezcano wrote: > >> On 01/12/2017 06:16, Keerthy wrote: > >>> Move the dmtimer driver out of plat-omap to clocksource. > >>> So that non-omap devices also could use this. > >>> > >>> No Code changes done to the driver file. > >>> > >>> Signed-off-by: Keerthy > >>> Reviewed-by: Sebastian Reichel > >>> --- > >>> > >>> Changes in v3: > >>> > >>> * Added Sebastian's Reviewed-by. > >>> > >>> Changes in v2: > >>> > >>> * No code changes in this v2 version. Only enhanced patch > >>> statistics for renames. > >>> > >>> arch/arm/plat-omap/Kconfig | 6 ------ > >>> arch/arm/plat-omap/Makefile | 1 - > >>> drivers/clocksource/Kconfig | 6 ++++++ > >>> drivers/clocksource/Makefile | 1 + > >>> {arch/arm/plat-omap => drivers/clocksource}/dmtimer.c | 0 > >>> 5 files changed, 7 insertions(+), 7 deletions(-) > >>> rename {arch/arm/plat-omap => drivers/clocksource}/dmtimer.c (100%) > >> > >> Take the opportunity to rename it timer-dm.c > > > > seems the drivers/clocksource subdirectory contains what its name state. > > Yes, clocksource, clockevent. > > > However OMAP dual mode timers have also event capture capability, which > > I'd like to use to measure pulse/space width and pass measured values > > to IR protocol decoders. > > Do you have any idea how to split code, so that event capture capability > > could be used by other drivers? > Not 100% sure but could be drivers/pwm, using the capture API. I hoped for another answer as pwm_capture is blocking operation while something event (interrupt) driven would be more usefull. But that's for different debate, so I'll prepare quick and dirty patch to start with. Thank you, ladis From mboxrd@z Thu Jan 1 00:00:00 1970 From: ladis@linux-mips.org (Ladislav Michl) Date: Fri, 8 Dec 2017 20:19:59 +0100 Subject: [PATCH v4 4/8] arm: OMAP: Move dmtimer driver out of plat-omap to drivers under clocksource In-Reply-To: References: <1512105397-2544-1-git-send-email-j-keerthy@ti.com> <1512105397-2544-5-git-send-email-j-keerthy@ti.com> <20171208172031.GA4146@lenoch> Message-ID: <20171208191959.GA5801@lenoch> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Dec 08, 2017 at 06:50:32PM +0100, Daniel Lezcano wrote: > On 08/12/2017 18:20, Ladislav Michl wrote: > > Daniel, > > > > On Thu, Dec 07, 2017 at 11:54:27AM +0100, Daniel Lezcano wrote: > >> On 01/12/2017 06:16, Keerthy wrote: > >>> Move the dmtimer driver out of plat-omap to clocksource. > >>> So that non-omap devices also could use this. > >>> > >>> No Code changes done to the driver file. > >>> > >>> Signed-off-by: Keerthy > >>> Reviewed-by: Sebastian Reichel > >>> --- > >>> > >>> Changes in v3: > >>> > >>> * Added Sebastian's Reviewed-by. > >>> > >>> Changes in v2: > >>> > >>> * No code changes in this v2 version. Only enhanced patch > >>> statistics for renames. > >>> > >>> arch/arm/plat-omap/Kconfig | 6 ------ > >>> arch/arm/plat-omap/Makefile | 1 - > >>> drivers/clocksource/Kconfig | 6 ++++++ > >>> drivers/clocksource/Makefile | 1 + > >>> {arch/arm/plat-omap => drivers/clocksource}/dmtimer.c | 0 > >>> 5 files changed, 7 insertions(+), 7 deletions(-) > >>> rename {arch/arm/plat-omap => drivers/clocksource}/dmtimer.c (100%) > >> > >> Take the opportunity to rename it timer-dm.c > > > > seems the drivers/clocksource subdirectory contains what its name state. > > Yes, clocksource, clockevent. > > > However OMAP dual mode timers have also event capture capability, which > > I'd like to use to measure pulse/space width and pass measured values > > to IR protocol decoders. > > Do you have any idea how to split code, so that event capture capability > > could be used by other drivers? > Not 100% sure but could be drivers/pwm, using the capture API. I hoped for another answer as pwm_capture is blocking operation while something event (interrupt) driven would be more usefull. But that's for different debate, so I'll prepare quick and dirty patch to start with. Thank you, ladis