On Tuesday, September 10, 2013 6:20 PM, Wolfram Sang wrote:
> On Thu, Sep 05, 2013 at 12:23:44PM -0700, Olof Johansson wrote:
> > commit 6d4028c644e (i2c: use dev_get_platdata()) did a bad conversion
> > of this one case:
> >
> > drivers/i2c/busses/i2c-davinci.c: In function 'davinci_i2c_probe':
> > drivers/i2c/busses/i2c-davinci.c:665:2: warning: passing argument 1 of
> > 'dev_get_platdata' from incompatible pointer type [enabled by default]
> >
> > Cc: Jingoo Han <jg1.han@xxxxxxxxxxx>
> > Signed-off-by: Olof Johansson <olof@xxxxxxxxx>
>
> Gosh, need to find out why my build-tests failed on that. Thanks!
>
> Acked-by: Wolfram Sang <wsa@xxxxxxxxxxxxx>
>
> Jingoo Han: Did you buildtest?
I always did builtest for the patch.
However, I cannot know why it makes build error.
Anyway, sorry for making this build error.
Best regards,
Jingoo Han
>
> > ---
> > drivers/i2c/busses/i2c-davinci.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/i2c/busses/i2c-davinci.c b/drivers/i2c/busses/i2c-davinci.c
> > index 5747341..132369f 100644
> > --- a/drivers/i2c/busses/i2c-davinci.c
> > +++ b/drivers/i2c/busses/i2c-davinci.c
> > @@ -662,7 +662,7 @@ static int davinci_i2c_probe(struct platform_device *pdev)
> > #endif
> > dev->dev = &pdev->dev;
> > dev->irq = irq->start;
> > - dev->pdata = dev_get_platdata(&dev->dev);
> > + dev->pdata = dev_get_platdata(&pdev->dev);
> > platform_set_drvdata(pdev, dev);
> >
> > if (!dev->pdata && pdev->dev.of_node) {
> > --
> > 1.7.10.4
> >
--
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
