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.shyti@xxxxxxxxxxx> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> --- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html