Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-18 Thread Lorenzo Pieralisi
Hi Samuel, On Wed, Jul 17, 2013 at 10:07:00PM +0100, Samuel Ortiz wrote: > Hi Lorenzo, > > On Tue, Jul 16, 2013 at 05:05:42PM +0100, Lorenzo Pieralisi wrote: > > Hello, > > > > version v5 of VExpress SPC driver, please read on the changelog for major > > changes and explanations. > > > > The pr

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Samuel Ortiz
On Wed, Jul 17, 2013 at 06:22:46PM -0400, Nicolas Pitre wrote: > On Wed, 17 Jul 2013, Samuel Ortiz wrote: > > > On Wed, Jul 17, 2013 at 02:29:02PM -0400, Nicolas Pitre wrote: > > > At this point I don't really care about the name. I just want the damn > > > thing merged upstream. But after seve

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Nicolas Pitre
On Wed, 17 Jul 2013, Samuel Ortiz wrote: > On Wed, Jul 17, 2013 at 02:29:02PM -0400, Nicolas Pitre wrote: > > At this point I don't really care about the name. I just want the damn > > thing merged upstream. But after several iterations to either fit one > > or another maintainers taste, each

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Samuel Ortiz
Hi Nicolas, On Wed, Jul 17, 2013 at 02:29:02PM -0400, Nicolas Pitre wrote: > On Wed, 17 Jul 2013, Russell King - ARM Linux wrote: > > > On Wed, Jul 17, 2013 at 11:57:55AM -0400, Nicolas Pitre wrote: > > > The sanest location at this point might simply be > > > drivers/platform/vexpress_spc.c or

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Samuel Ortiz
Hi Pawel, On Wed, Jul 17, 2013 at 03:20:11PM +0100, Pawel Moll wrote: > On Wed, 2013-07-17 at 15:16 +0100, Nicolas Pitre wrote: > > On Wed, 17 Jul 2013, Pawel Moll wrote: > > > > > On Wed, 2013-07-17 at 13:33 +0100, Nicolas Pitre wrote: > > > > If this is really miscelaneous code that really does

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Samuel Ortiz
Hi Lorenzo, On Tue, Jul 16, 2013 at 05:05:42PM +0100, Lorenzo Pieralisi wrote: > Hello, > > version v5 of VExpress SPC driver, please read on the changelog for major > changes and explanations. > > The probing scheme is unchanged, since after trying the early platform > devices approach it appea

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Nicolas Pitre
On Wed, 17 Jul 2013, Russell King - ARM Linux wrote: > On Wed, Jul 17, 2013 at 11:57:55AM -0400, Nicolas Pitre wrote: > > The sanest location at this point might simply be > > drivers/platform/vexpress_spc.c or drivers/platform/vexpress/spc.c > > depending on whether or not more such driver glue

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Russell King - ARM Linux
On Wed, Jul 17, 2013 at 11:57:55AM -0400, Nicolas Pitre wrote: > The sanest location at this point might simply be > drivers/platform/vexpress_spc.c or drivers/platform/vexpress/spc.c > depending on whether or not more such driver glue is expected in the > vexpress future. No point putting "arm

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Nicolas Pitre
On Wed, 17 Jul 2013, Pawel Moll wrote: > On Wed, 2013-07-17 at 15:16 +0100, Nicolas Pitre wrote: > > On Wed, 17 Jul 2013, Pawel Moll wrote: > > > > > On Wed, 2013-07-17 at 13:33 +0100, Nicolas Pitre wrote: > > > > If this is really miscelaneous code that really doesn't fit > > > > anywhere else,

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Nicolas Pitre
On Wed, 17 Jul 2013, Pawel Moll wrote: > On Wed, 2013-07-17 at 13:33 +0100, Nicolas Pitre wrote: > > If this is really miscelaneous code that really doesn't fit > > anywhere else, it should rather go into drivers/misc/ as a last resort. > > Interestingly enough drivers/misc was my first choice f

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Pawel Moll
On Wed, 2013-07-17 at 15:16 +0100, Nicolas Pitre wrote: > On Wed, 17 Jul 2013, Pawel Moll wrote: > > > On Wed, 2013-07-17 at 13:33 +0100, Nicolas Pitre wrote: > > > If this is really miscelaneous code that really doesn't fit > > > anywhere else, it should rather go into drivers/misc/ as a last re

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Pawel Moll
On Wed, 2013-07-17 at 13:33 +0100, Nicolas Pitre wrote: > If this is really miscelaneous code that really doesn't fit > anywhere else, it should rather go into drivers/misc/ as a last resort. Interestingly enough drivers/misc was my first choice for all the vexpress stuff, but it wasn't received

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Nicolas Pitre
On Wed, 17 Jul 2013, Pawel Moll wrote: > On Tue, 2013-07-16 at 17:05 +0100, Lorenzo Pieralisi wrote: > > /drivers/mfd is probably not the right place for this code as it stands (but > > probably will be when the entire driver, with DVFS and config interface, is > > complete). > > Not that it real

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Lorenzo Pieralisi
On Wed, Jul 17, 2013 at 10:18:25AM +0100, Pawel Moll wrote: > On Tue, 2013-07-16 at 17:05 +0100, Lorenzo Pieralisi wrote: > > /drivers/mfd is probably not the right place for this code as it stands (but > > probably will be when the entire driver, with DVFS and config interface, is > > complete). >

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-17 Thread Pawel Moll
On Tue, 2013-07-16 at 17:05 +0100, Lorenzo Pieralisi wrote: > /drivers/mfd is probably not the right place for this code as it stands (but > probably will be when the entire driver, with DVFS and config interface, is > complete). Not that it really matters now, but my vexpress-sysreg rework will -

Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-16 Thread Nicolas Pitre
On Tue, 16 Jul 2013, Lorenzo Pieralisi wrote: > Hello, > > version v5 of VExpress SPC driver, please read on the changelog for major > changes and explanations. > > The probing scheme is unchanged, since after trying the early platform > devices approach it appeared that the end result was no be

[RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support

2013-07-16 Thread Lorenzo Pieralisi
Hello, version v5 of VExpress SPC driver, please read on the changelog for major changes and explanations. The probing scheme is unchanged, since after trying the early platform devices approach it appeared that the end result was no better than the current one. The only clean solution relies eit