[PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-08 Thread Thomas Abraham
With the spi controller hardware configuration moved into the driver data, there are no more default hardware configuration data that is passed through platform data. Accordingly, the s3c64xx_spi{0|1|2}_set_platdata functions are adapted to these changes. Cc: Jaswinder Singh Signed-off-by: Thomas

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Mark Brown
On Wed, May 09, 2012 at 03:34:50AM +0530, Thomas Abraham wrote: > + s3c64xx_spi0_set_platdata("s3c6410-spi", NULL, 0, 1); Shouldn't we just set the name in the struct platform_device rather than requiring the machine to pass it through by hand? ___

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Heiko Stübner
Am Mittwoch, 9. Mai 2012, 10:56:17 schrieb Mark Brown: > On Wed, May 09, 2012 at 03:34:50AM +0530, Thomas Abraham wrote: > > + s3c64xx_spi0_set_platdata("s3c6410-spi", NULL, 0, 1); > > Shouldn't we just set the name in the struct platform_device rather than > requiring the machine to pass it thr

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Mark Brown
On Wed, May 09, 2012 at 11:10:14AM +0200, Heiko Stübner wrote: > Similar to the adc and rtc driver, all Samsung platforms reuse a common > platform-device definition for the s3c64xx-spi and simply will set the > correct > name when the machine type is determined during boot. Right, that doesn'

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Thomas Abraham
On 9 May 2012 18:55, Mark Brown wrote: > On Wed, May 09, 2012 at 11:10:14AM +0200, Heiko Stübner wrote: > >> Similar to the adc and rtc driver, all Samsung platforms reuse a common >> platform-device definition for the s3c64xx-spi and simply will set the >> correct >> name when the machine type i

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Mark Brown
On Wed, May 09, 2012 at 10:22:26PM +0800, Thomas Abraham wrote: > On 9 May 2012 18:55, Mark Brown wrote: > > Yes, that's the normal way of handling this and is actually what the > > code was originally doing - there's a bunch of ifdefed devices in > > plat-samsung/devs.c.  You usually have to do

Re: [PATCH 06/10] ARM: Samsung: Modify s3c64xx_spi{0|1|2}_set_platdata function

2012-05-09 Thread Thomas Abraham
On 9 May 2012 22:33, Mark Brown wrote: > On Wed, May 09, 2012 at 10:22:26PM +0800, Thomas Abraham wrote: >> On 9 May 2012 18:55, Mark Brown wrote: > >> > Yes, that's the normal way of handling this and is actually what the >> > code was originally doing - there's a bunch of ifdefed devices in >>