Hi,
On Wed, Apr 06, 2011 at 15:02:22, Ran Shalit wrote:
>
>
> It seems that currently both atmel & davinci boards supports only 1 spi
> instance at a time.
>
> In order to support different spi instances,
> In case of atmel declare SPIx_BASE as required & set
> CONFIG_DEFAULT_SPI_BUS to required bus
> In case of davinci set CONFIG_SYS_SPI_BASE to base address of required
> spi instance
>
> Also, spi_cs_is_valid() function needs to be updated to support
> different spi instances. spi_cs_is_valid() function is board dependednt.
>
> Hope this helps.
>
> Regards,
> Gururaja
>
>
>
>
> Hi Gururaja,
>
> It seems that the implementation limit the validity only for bus==0.
> This means that trying other bus(=spi port ?) will fail even if its not at
> the same time. In addition to that the CPU's data sheet make no limit.
>
> Best Regards,
> Ran
>
Don't top post.
Currenly davinci spi driver only allows (bus == 0) to pass which means spi0. So
if you want to support other or many spi port at once, then
1. you need to change spi_cs_is_valid() to check for various bus values &
2. validate the bus argument to spi_setup_slave(). may be using a switch as in
case of atmel.
from
ds->regs = (struct davinci_spi_regs *)CONFIG_SYS_SPI_BASE;
to
struct davinci_spi_regs *regs;
switch (bus) {
case 0:
regs = (struct davinci_spi_regs *)CONFIG_SYS_SPI0_BASE;
break;
case 1:
regs = (struct davinci_spi_regs *)CONFIG_SYS_SPI1_BASE;
break;
...
...
default:
return NULL;
}
ds->regs = regs;
Regards
Gururaja
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
