On January 21, 2015 16:07, Varka Bhadram wrote: > On Wednesday 21 January 2015 09:16 PM, Adam Thomson wrote: > > This patch adds support for DA9150 Charger & Fuel-Gauge IC GPADC. > > > > Signed-off-by: Adam Thomson <Adam.Thomson.Opensource@xxxxxxxxxxx> > > Reviewed-by: Hartmut Knaack <knaack.h@xxxxxx> > > Acked-by: Jonathan Cameron <jic23@xxxxxxxxxx> > > (...) > > > + > > +static int da9150_gpadc_probe(struct platform_device *pdev) > > +{ > > + struct device *dev = &pdev->dev; > > + struct da9150 *da9150 = dev_get_drvdata(dev->parent); > > + struct da9150_gpadc *gpadc; > > + struct iio_dev *indio_dev; > > + int irq, ret; > > + > > + indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*gpadc)); > > You can directly use *dev* instead of *&pdev->dev* Yes agreed. Can follow up with change for this as it's not urgent and would rather leave current patch set as is. ��.n��������+%������w��{.n�����{��(��)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥