[U-Boot] [PATCH 2/2] spi: tegra20_sflash: Remove redundant code to set bus and cs of struct spi_slave

Marek Vasut marex at denx.de
Thu Jun 13 12:48:39 CEST 2013


Dear Axel Lin,

> It's done in spi_alloc_slave(), thus remove the redundant code.
> 
> Signed-off-by: Axel Lin <axel.lin at ingics.com>
> ---
>  drivers/spi/tegra20_sflash.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/spi/tegra20_sflash.c b/drivers/spi/tegra20_sflash.c
> index 9322ce7..7c3a3fc 100644
> --- a/drivers/spi/tegra20_sflash.c
> +++ b/drivers/spi/tegra20_sflash.c
> @@ -132,8 +132,6 @@ struct spi_slave *tegra20_spi_setup_slave(unsigned int
> bus, unsigned int cs, printf("SPI error: malloc of SPI structure
> failed\n");
>  		return NULL;
>  	}
> -	spi->slave.bus = bus;
> -	spi->slave.cs = cs;
>  	spi->ctrl = &spi_ctrls[bus];
>  	if (!spi->ctrl) {
>  		printf("SPI error: could not find controller for bus %d\n",

Acked-by: Marek Vasut <marex at denx.de>

Best regards,
Marek Vasut


More information about the U-Boot mailing list