The 'quirks' variable cannot ever be negative, therefore use u8 instead of int. The 8 bit size is given from the fact that currently the quirks variable has very few statuses.
The rx_lvl_offset and tx_st_done store shift values, so that u8 is a proper size. fifo_lvl_mask stores a series of masks, to be in we will keep the 32 bit size. Signed-off-by: Andi Shyti <andi.sh...@samsung.com> Signed-off-by: Jaehoon Chung <jh80.ch...@samsung.com> --- drivers/spi/spi-s3c64xx.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/spi/spi-s3c64xx.c b/drivers/spi/spi-s3c64xx.c index 6d8486f..6c9503a 100644 --- a/drivers/spi/spi-s3c64xx.c +++ b/drivers/spi/spi-s3c64xx.c @@ -150,10 +150,10 @@ struct s3c64xx_spi_dma_data { * which is provided as driver data to the driver. */ struct s3c64xx_spi_port_config { - int fifo_lvl_mask[MAX_SPI_PORTS]; - int rx_lvl_offset; - int tx_st_done; - int quirks; + u32 fifo_lvl_mask[MAX_SPI_PORTS]; + u8 rx_lvl_offset; + u8 tx_st_done; + u8 quirks; bool high_speed; bool clk_from_cmu; }; -- 2.8.1