Re: [PATCH v2 00/12] ARM: samsung-time: Prepare for multiplatform support

From: Tomasz Figa <tomasz.figa@gmail.com>
To: "Heiko Stübner" <heiko@sntech.de>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org,
	Kukjin Kim <kgene.kim@samsung.com>,
	kyungmin.park@samsung.com, linux@simtec.co.uk,
	broonie@opensource.wolfsonmicro.com, kwangwoo.lee@gmail.com,
	jacmet@sunsite.dk, augulis.darius@gmail.com,
	mcuelenaere@gmail.com, linux@arm.linux.org.uk,
	Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
	buserror@gmail.com, christer@weinigel.se, jekhor@gmail.com,
	ghcstop@gmail.com, Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH v2 00/12] ARM: samsung-time: Prepare for multiplatform support
Date: Sun, 17 Feb 2013 17:13:52 +0100	[thread overview]
Message-ID: <2491436.7RNovL0hQ1@flatron> (raw)
In-Reply-To: <201302171705.55706.heiko@sntech.de>

Hi Heiko,

On Sunday 17 of February 2013 17:05:54 Heiko Stübner wrote:
> Am Sonntag, 17. Februar 2013, 00:05:07 schrieb Heiko Stübner:
> > Am Samstag, 16. Februar 2013, 17:43:52 schrieb Tomasz Figa:
> > > This series is an attempt to make the samsung-time clocksource
> > > driver
> > > ready for multiplatform kernels. It moves the driver to
> > > drivers/clocksource, cleans it up from uses of static
> > > platform-specific
> > > definitions, simplifies timer interrupt handling and adds Device
> > > Tree
> > > support.
> > > 
> > > Tested on a Tiny6410 board (Mini6410-compatible) both with and
> > > without
> > > Devicee Tree (with my DT patches for S3C64xx). Compile tested for
> > > other
> > > related SoCs.
> > > 
> > > Changes since v1:
> > > (http://thread.gmane.org/gmane.linux.kernel.samsung-soc/16005)
> > > 
> > >  - Addressed comments from Mark Rutland
> > >  - Documented struct samsung_timer_variant
> > >  - Dropped inactive mail addresses from CC
> > > 
> > > Tomasz Figa (12):
> > >   ARM: SAMSUNG: Move samsung-time to drivers/clocksource
> > >   clocksource: samsung-time: Set platform-specific parameters at
> > >   runtime
> > >   clocksource: samsung-time: Drop useless defines from public header
> > >   ARM: SAMSUNG: Move samsung-time.h header to inlude/clocksource
> > >   clocksource: samsung-time: Use local register definitions
> > >   clocksource: samsung-time: Remove use of static register mapping
> > >   clocksource: samsung-time: Use clk_get_sys for getting clocks
> > >   ARM: SAMSUNG: devs: Drop unnecessary IRQ resources of timer
> > >   devices
> > >   clocksource: samsung-time: Do not use static IRQ definition
> > >   clocksource: samsung-time: Move IRQ mask/ack handling to the
> > >   driver
> > >   ARM: SAMSUNG: Remove unused PWM timer IRQ chip code
> > >   clocksource: samsung-time: Add Device Tree support
> > 
> > on a non-dt s3c2416 based board
> > Tested-by: Heiko Stuebner <heiko@sntech.de>
> > 
> > 
> > once I've ported my s3c2416 dt patches to all the recent changes, I
> > will also test the dt side.
> 
> also on my upcoming s3c2416 dt patches the clocksource works like a
> charm, so to update:
> 
> on a s3c2416 based board both with and without devicetree
> Tested-by: Heiko Stuebner <heiko@sntech.de>

Thank you for your testing.

Best regards,
Tomasz