Re: [PATCH] spi: dw: Remove unused members from struct chip_data

[Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>]

On Fri, 2018-01-26 at 16:24 +0200, Jarkko Nikula wrote:
> Local struct chip_data has two members that are not used:
> 
> - cs. Looks like was never used
> - enable_dma. Became unused by the commit f89a6d8f43eb ("spi: dw-mid:
> move
>   to use core SPI DMA mappings").

Good clean up!

Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

> Signed-off-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxxxxxxx>
> ---
>  drivers/spi/spi-dw.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c
> index b217c22ff72f..211cc7d75bf8 100644
> --- a/drivers/spi/spi-dw.c
> +++ b/drivers/spi/spi-dw.c
> @@ -30,13 +30,11 @@
>  
>  /* Slave spi_dev related */
>  struct chip_data {
> -	u8 cs;			/* chip select pin */
>  	u8 tmode;		/* TR/TO/RO/EEPROM */
>  	u8 type;		/* SPI/SSP/MicroWire */
>  
>  	u8 poll_mode;		/* 1 means use poll mode */
>  
> -	u8 enable_dma;
>  	u16 clk_div;		/* baud rate divider */
>  	u32 speed_hz;		/* baud rate */
>  	void (*cs_control)(u32 command);

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html