Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-19 Thread Arnd Bergmann
On Thursday 05 April 2012, Trilok Soni wrote: Couple of suggestions: drivers/platform/omap/avs? drivers/misc/omap/avs? I would definitely prefer something under drivers/power, drivers/regulators or a new top-level directory under drivers. IIRC, David Brownell was referring to the

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-19 Thread Jean Pihet
Hi Arnd, On Thu, Apr 19, 2012 at 3:54 PM, Arnd Bergmann a...@arndb.de wrote: On Thursday 05 April 2012, Trilok Soni wrote: Couple of suggestions: drivers/platform/omap/avs? drivers/misc/omap/avs? I would definitely prefer something under drivers/power, drivers/regulators or a new

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-19 Thread Kevin Hilman
Arnd Bergmann a...@arndb.de writes: On Thursday 05 April 2012, Trilok Soni wrote: Couple of suggestions: drivers/platform/omap/avs? drivers/misc/omap/avs? I would definitely prefer something under drivers/power, drivers/regulators or a new top-level directory under drivers.

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-05 Thread Trilok Soni
Hi Benoit, The main motivation is that it's a driver and thus does not have anything to do inside mach-omap2. Right, I understood that. mach-omap2 is not suitable for full fledged drivers. Where will you put that otherwise? Couple of suggestions: drivers/platform/omap/avs?

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-05 Thread Jean Pihet
Hi! On Thu, Apr 5, 2012 at 8:53 AM, Trilok Soni ts...@codeaurora.org wrote: Hi Benoit, The main motivation is that it's a driver and thus does not have anything to do inside mach-omap2. Right, I understood that. mach-omap2 is not suitable for full fledged drivers. The initial

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-05 Thread Trilok Soni
Hi Jean, The initial motivation is to provide a generic framework for this type of drivers, by cleaning up the current OMAP code and by providing as much generic code as possible. Cf. the patch sets I submitted before this very one: - the SR code clean-up [1], which is needed to make the code

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-04 Thread Trilok Soni
Hi Jean, On 3/19/2012 9:42 PM, jean.pi...@newoldbits.com wrote: From: Jean Pihetj-pi...@ti.com Move the driver specific macros from the smartreflex header file (arch/arm/mach-omap2/smartreflex.h) in a new header file include/linux/power/smartreflex.h. This change makes the SmartReflex

Re: [PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-04-04 Thread Cousson, Benoit
On 4/4/2012 9:10 PM, Trilok Soni wrote: Hi Jean, On 3/19/2012 9:42 PM, jean.pi...@newoldbits.com wrote: From: Jean Pihetj-pi...@ti.com Move the driver specific macros from the smartreflex header file (arch/arm/mach-omap2/smartreflex.h) in a new header file include/linux/power/smartreflex.h.

[PATCH 2/9] ARM: OMAP2+: SmartReflex: move the driver specific macros in include/linux/power

2012-03-19 Thread jean . pihet
From: Jean Pihet j-pi...@ti.com Move the driver specific macros from the smartreflex header file (arch/arm/mach-omap2/smartreflex.h) in a new header file include/linux/power/smartreflex.h. This change makes the SmartReflex implementation ready for the move to drivers/. Signed-off-by: Jean Pihet