Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-12 Thread Segher Boessenkool
>>> But, many SoCs do have a notion of device "number", which is relevant >>> for programming other general control registers in places. We need >>> to >>> encode it somehow, and it would be good to have a consistent way of >>> doing it. >> >> I'm not convinced there isn't a more direct way to re

Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-12 Thread David Gibson
On Fri, Aug 10, 2007 at 10:42:51PM +0200, Segher Boessenkool wrote: > >>> We should co-ordinate better on this, if it's to become a > >>> convention... > >> > >> That means we shouldn't coordinate on this, right? > > > > Heh. Either one is kind of ugly, I'll grant you. > > > > But, many SoCs do ha

Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-10 Thread Segher Boessenkool
>>> We should co-ordinate better on this, if it's to become a >>> convention... >> >> That means we shouldn't coordinate on this, right? > > Heh. Either one is kind of ugly, I'll grant you. > > But, many SoCs do have a notion of device "number", which is relevant > for programming other general co

Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-09 Thread David Gibson
On Thu, Aug 09, 2007 at 10:15:44PM +0200, Segher Boessenkool wrote: > > Hrm... Freescale is using these 'fsl,device-id' properties, I'm > > guessing they're basically the same thing as the 'cell-index' used in > > the EMAC binding. > > > > We should co-ordinate better on this, if it's to become a >

Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-09 Thread Segher Boessenkool
> Hrm... Freescale is using these 'fsl,device-id' properties, I'm > guessing they're basically the same thing as the 'cell-index' used in > the EMAC binding. > > We should co-ordinate better on this, if it's to become a > convention... That means we shouldn't coordinate on this, right? Segher _

Re: [PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-08 Thread David Gibson
On Wed, Aug 08, 2007 at 09:09:21PM +0400, Anton Vorontsov wrote: > mmc_spi already tested to work. When it will hit mainline > the only change that will be needed is replacing "spidev" > by "mmc_spi". > > Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> > --- > arch/powerpc/boot/dts/mpc832x_rdb

[PATCH v3 1/2] [POWERPC] MPC832x_RDB: update dts to use spi, register mmc_spi stub

2007-08-08 Thread Anton Vorontsov
mmc_spi already tested to work. When it will hit mainline the only change that will be needed is replacing "spidev" by "mmc_spi". Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> --- arch/powerpc/boot/dts/mpc832x_rdb.dts |4 ++- arch/powerpc/platforms/83xx/mpc832x_rdb.c | 52 +