Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-09 Thread Uwe Kleine-König
Hello, On Mon, Mar 09, 2015 at 10:12:32PM +0100, Paul Bolle wrote: > On Wed, 2015-03-04 at 13:08 +0100, Maxime Coquelin wrote: > > This is because I added also support for COMPILE_TEST coverage as per > > Uwe advice, > > and thought it was necessary to have an entry for this. > > Maybe I'm just wr

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-09 Thread Paul Bolle
On Wed, 2015-03-04 at 13:08 +0100, Maxime Coquelin wrote: > This is because I added also support for COMPILE_TEST coverage as per > Uwe advice, > and thought it was necessary to have an entry for this. > Maybe I'm just wrong? I missed that you added COMPILE_TEST. A quick scan of your idea doesn't

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-09 Thread Maxime Coquelin
2015-03-09 16:50 GMT+01:00 Linus Walleij : > On Fri, Feb 20, 2015 at 7:01 PM, Maxime Coquelin > wrote: > >> This patch adds clocksource support for ARMv7-M's System timer, >> also known as SysTick. >> >> Signed-off-by: Maxime Coquelin > (...) >> + /* If no clock found, try to get clock-freq

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-09 Thread Linus Walleij
On Fri, Feb 20, 2015 at 7:01 PM, Maxime Coquelin wrote: > This patch adds clocksource support for ARMv7-M's System timer, > also known as SysTick. > > Signed-off-by: Maxime Coquelin (...) > + /* If no clock found, try to get clock-frequency property */ > + if (!rate) { > +

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-04 Thread Maxime Coquelin
2015-03-03 20:43 GMT+01:00 Paul Bolle : > Maxime Coquelin schreef op ma 02-03-2015 om 17:53 [+0100]: >> Do you agree if I define it like this: >> >> config ARMV7M_SYSTICK >> bool "Clocksource driver for ARMv7-M System timer" >> depends on OF && (CPU_V7M || COMPILE_TEST) >> select CLKSRC

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-03 Thread Paul Bolle
Maxime Coquelin schreef op ma 02-03-2015 om 17:53 [+0100]: > Do you agree if I define it like this: > > config ARMV7M_SYSTICK > bool "Clocksource driver for ARMv7-M System timer" > depends on OF && (CPU_V7M || COMPILE_TEST) > select CLKSRC_OF > select CLKSRC_MMIO > help >

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-03-02 Thread Maxime Coquelin
Hi Paul, Uwe, 2015-02-20 22:48 GMT+01:00 Paul Bolle : > On Fri, 2015-02-20 at 20:54 +0100, Uwe Kleine-König wrote: >> On Fri, Feb 20, 2015 at 07:01:03PM +0100, Maxime Coquelin wrote: >> > This patch adds clocksource support for ARMv7-M's System timer, >> > also known as SysTick. >> > >> > Signed-o

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-02-20 Thread Paul Bolle
On Fri, 2015-02-20 at 20:54 +0100, Uwe Kleine-König wrote: > On Fri, Feb 20, 2015 at 07:01:03PM +0100, Maxime Coquelin wrote: > > This patch adds clocksource support for ARMv7-M's System timer, > > also known as SysTick. > > > > Signed-off-by: Maxime Coquelin > > --- > > drivers/clocksource/Kcon

Re: [PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-02-20 Thread Uwe Kleine-König
Hello, On Fri, Feb 20, 2015 at 07:01:03PM +0100, Maxime Coquelin wrote: > This patch adds clocksource support for ARMv7-M's System timer, > also known as SysTick. > > Signed-off-by: Maxime Coquelin > --- > drivers/clocksource/Kconfig | 7 > drivers/clocksource/Makefile |

[PATCH v2 04/18] clocksource: Add ARM System timer driver

2015-02-20 Thread Maxime Coquelin
This patch adds clocksource support for ARMv7-M's System timer, also known as SysTick. Signed-off-by: Maxime Coquelin --- drivers/clocksource/Kconfig | 7 drivers/clocksource/Makefile | 1 + drivers/clocksource/armv7m_systick.c | 78 3