On Thu, 19 Apr 2018 16:06:18 +0200
Wolfram Sang <wsa+rene...@sang-engineering.com> wrote:

> We should get drvdata from struct device directly. Going via
> platform_device is an unneeded step back and forth.
> 
> Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
Applied, thanks,

Jonathan

> ---
> 
> Build tested only. buildbot is happy. Please apply individually.
> 
>  drivers/staging/iio/adc/ad7606_par.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/iio/adc/ad7606_par.c 
> b/drivers/staging/iio/adc/ad7606_par.c
> index 3eb6f8f312dd..a34c2a1d5373 100644
> --- a/drivers/staging/iio/adc/ad7606_par.c
> +++ b/drivers/staging/iio/adc/ad7606_par.c
> @@ -18,8 +18,7 @@
>  static int ad7606_par16_read_block(struct device *dev,
>                                  int count, void *buf)
>  {
> -     struct platform_device *pdev = to_platform_device(dev);
> -     struct iio_dev *indio_dev = platform_get_drvdata(pdev);
> +     struct iio_dev *indio_dev = dev_get_drvdata(dev);
>       struct ad7606_state *st = iio_priv(indio_dev);
>  
>       insw((unsigned long)st->base_address, buf, count);
> @@ -34,8 +33,7 @@ static const struct ad7606_bus_ops ad7606_par16_bops = {
>  static int ad7606_par8_read_block(struct device *dev,
>                                 int count, void *buf)
>  {
> -     struct platform_device *pdev = to_platform_device(dev);
> -     struct iio_dev *indio_dev = platform_get_drvdata(pdev);
> +     struct iio_dev *indio_dev = dev_get_drvdata(dev);
>       struct ad7606_state *st = iio_priv(indio_dev);
>  
>       insb((unsigned long)st->base_address, buf, count * 2);

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to