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

Attachment: 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

Reply via email to