On Monday 23 April 2012 08:43:12 Fabio Estevam wrote: > --- a/drivers/spi/mxs_spi.c > +++ b/drivers/spi/mxs_spi.c > @@ -56,12 +58,19 @@ struct spi_slave *spi_setup_slave(unsigned int bus, > > if (bus > 3) { > printf("MXS SPI: Max bus number is 3\n"); > return NULL; > } > > + if (cs > 2) { > + printf("MXS SPI: Invalid chip select number: %d\n", cs); > + return NULL; > + }
this stuff really belongs in spi_cs_is_valid(), sans the printf's. -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot