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 <wolfram@xxxxxxxxxxxxx> > Cc: Richard Zhao <richard.zhao@xxxxxxxxxxxxx> > --- > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
