On Sun, Jul 08, 2012 at 01:24:47PM +0200, Wolfram Sang wrote:
> sparse found this assignment of u32 to an int. Fix it:
> 
> drivers/i2c/busses/i2c-imx.c:540:56: warning: incorrect type in argument 3 
> (different signedness)
Could you change imxi2c_platform_data.bitrate to unsigned int too?

Thanks
Richard
> 
> Signed-off-by: Wolfram Sang <[email protected]>
> Cc: Richard Zhao <[email protected]>
> ---
>  drivers/i2c/busses/i2c-imx.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c
> index a93e846..3f5dfe8 100644
> --- a/drivers/i2c/busses/i2c-imx.c
> +++ b/drivers/i2c/busses/i2c-imx.c
> @@ -471,8 +471,8 @@ static int __init i2c_imx_probe(struct platform_device 
> *pdev)
>       struct imxi2c_platform_data *pdata = pdev->dev.platform_data;
>       struct pinctrl *pinctrl;
>       void __iomem *base;
> -     int irq, bitrate;
> -     int ret;
> +     int irq, ret;
> +     u32 bitrate;
>  
>       dev_dbg(&pdev->dev, "<%s>\n", __func__);
>  
> -- 
> 1.7.10
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to