[U-Boot] [PATCH] zynq: spi: Remove unnecessary error condition

Jagan Teki jagannadh.teki at gmail.com
Fri Apr 17 13:50:51 CEST 2015


On 15 April 2015 at 16:43, Michal Simek <michal.simek at xilinx.com> wrote:
> From: Siva Durga Prasad Paladugu <siva.durga.paladugu at xilinx.com>
>
> Removed the unnecessary error check from spi_xfer
> as the bitlen zero is possible now to deassert the
> chip select for which no data is required to be transfered.
>
> Signed-off-by: Siva Durga Prasad Paladugu <sivadur at xilinx.com>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
>  drivers/spi/zynq_spi.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/drivers/spi/zynq_spi.c b/drivers/spi/zynq_spi.c
> index 5da87591ceef..e9129da79d99 100644
> --- a/drivers/spi/zynq_spi.c
> +++ b/drivers/spi/zynq_spi.c
> @@ -227,9 +227,6 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
>         debug("spi_xfer: bus:%i cs:%i bitlen:%i len:%i flags:%lx\n",
>               slave->bus, slave->cs, bitlen, len, flags);
>
> -       if (bitlen == 0)
> -               return -1;
> -
>         if (bitlen % 8) {
>                 debug("spi_xfer: Non byte aligned SPI transfer\n");
>                 return -1;
> --
> 2.3.5
>

Applied to  u-boot-spi/master

thanks!
-- 
Jagan.


More information about the U-Boot mailing list