On Sun, Jul 26, 2015 at 12:43:23AM +0200, Hartmut Knaack wrote: > Use augmented assignment to subtract the offset for negative temperature > values. > Specify the amount of private data to be allocated through > devm_iio_device_alloc() with sizeof(*priv), as it is shorter and common > practice in IIO. > > Signed-off-by: Hartmut Knaack <knaack.h@xxxxxx> Acked-by: Antoine Tenart <antoine.tenart@xxxxxxxxxxxxxxxxxx> > --- > drivers/iio/adc/berlin2-adc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/iio/adc/berlin2-adc.c b/drivers/iio/adc/berlin2-adc.c > index b7df94a0b879..1d14290ba6f2 100644 > --- a/drivers/iio/adc/berlin2-adc.c > +++ b/drivers/iio/adc/berlin2-adc.c > @@ -221,7 +221,7 @@ static int berlin2_adc_read_raw(struct iio_dev *indio_dev, > return temp; > > if (temp > 2047) > - temp = -(4096 - temp); > + temp -= 4096; > > /* Convert to milli Celsius */ > *val = ((temp * 100000) / 264 - 270000); > @@ -286,8 +286,7 @@ static int berlin2_adc_probe(struct platform_device *pdev) > int irq, tsen_irq; > int ret; > > - indio_dev = devm_iio_device_alloc(&pdev->dev, > - sizeof(struct berlin2_adc_priv)); > + indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*priv)); > if (!indio_dev) > return -ENOMEM; > > -- > 2.4.3 > -- Antoine Ténart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html