On Mon, 14 Sep 2020 11:04:43 +0300 Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Thu, Sep 10, 2020 at 8:35 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > > > This bumper set removes all the remaining cases where of_match_ptr > > is mostly serving to prevent use of the device via ACPI PRP0001 and > > complicating the code. The main aim of this set is to reduce the chances > > of it being used in future drivers. > > It also includes some related refactorings, use of generic fw accessors, > > dropping of CONFIG_OF and lots of header changes inline with the above. > > > > For examples of how to use PRP0001 see https://github.com/westeri/meta-acpi > > which is an excellent resource from which to copy snippets of DSDT tables. > > > > It may well not be worth changing the remaining 10 IIO drivers and I > > certainly don't plan to do so anytime soon. > > > > I briefly looked at the series and didn't see anything wrong with it > (esp. taking into consideration that you are already familiar with the > topic). > Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> Thanks Andy + others that reviewed. I've fixed the typo's in descriptions Matt highlighted and applied the lot. If anyone want to add tags, it will be a few days before I push this out as a non rebasing tree. Thanks, Jonathan > > > Jonathan Cameron (38): > > iio:potentiometer:ad5272: Drop of_match_ptr and CONFIG_OF protections. > > iio:potentiometer:ds1803: Drop of_match_ptr and CONFIG_OF protections > > iio:potentiometer:max5432: Drop of_match_ptr and use generic fw > > accessors > > iio:potentiometer:max5481: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:potentiometer:max5481: Drop invalid ACPI binding. > > iio:potentiometer:mcp4018: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:potentiometer:mcp4131: Drop of_match_ptr and use generic fw > > interfaces. > > iio:potentiometer:mcp4531: Drop of_match_ptr and CONFIG_OF > > protections. > > iio:dac:ad5446: Drop of_match_ptr and CONFIG_OF protections > > iio:dac:ad5592r: Drop of_match_ptr and ACPI_PTR protections. > > iio:dac:ad5593r: Drop of_match_ptr and ACPI_PTR protections. > > iio:dac:ad7303: Drop of_match_ptr protection > > iio:dac:mcp4725: drop of_match_ptr and use generic fw properties > > iio:dac:ti-dac082s085: Drop of_match_ptr and CONFIG_OF protections > > iio:dac:ti-dac5571: Drop of_match_ptr and CONFIG_OF protections > > iio:potentiostat:lmp91000: Drop of_match_ptr and use generic fw > > accessors > > iio:pressure:icp10100: Drop of_match_ptr and CONFIG_OF protections > > iio:pressure:ms5611: Drop of_match_ptr and CONFIG_OF protections > > iio:pressure:ms5637: Drop of_match_ptr protection > > iio:pressure:zpa2326: Drop of_match_ptr protection > > iio:temperature:tsys01: Drop of_match_ptr protection > > iio:temperature:tmp007: Drop of_match_ptr protection > > iio:resolver:ad2s1200: Drop of_match_ptr protection > > iio:chemical:ams-iaq-core: Drop of_match_ptr protection > > iio:chemical:atlas-sensor: Drop of_match_ptr and use generic fw > > accessors > > iio:chemical:sgp30: Use local variable dev to simplify code > > iio:chemical:sgp30: Drop of_match_ptr and use generic fw accessors > > iio:chemical:vz89x: Introduce local struct device pointer. > > iio:chemical:vz89x: Drop of_match_ptr protection and use generic fw > > accessors > > iio:humidity:hdc100x: Drop of_match_ptr protection. > > iio:proximity:as3935: Use local struct device pointer to simplify > > code. > > iio:proximity:as3935: Drop of_match_ptr and use generic fw accessors > > iio:proximity:pulsedlight: Drop of_match_ptr protection > > iio:magn:ak8975: Drop of_match_ptr and ACPI_PTR protections. > > iio:magn:ak8974: Drop of_match_ptr protection > > iio:humidity:htu21: Drop of_match_ptr protection > > iio:humidity:si7020: Drop of_match_ptr protection > > iio:health:max30102: Drop of_match_ptr and use generic fw accessors > > > > drivers/iio/chemical/ams-iaq-core.c | 3 +- > > drivers/iio/chemical/atlas-sensor.c | 10 ++--- > > drivers/iio/chemical/sgp30.c | 28 ++++++------ > > drivers/iio/chemical/vz89x.c | 18 ++++---- > > drivers/iio/dac/ad5446.c | 5 +-- > > drivers/iio/dac/ad5592r.c | 7 ++- > > drivers/iio/dac/ad5593r.c | 7 ++- > > drivers/iio/dac/ad7303.c | 3 +- > > drivers/iio/dac/mcp4725.c | 29 +++--------- > > drivers/iio/dac/ti-dac082s085.c | 5 +-- > > drivers/iio/dac/ti-dac5571.c | 7 +-- > > drivers/iio/health/max30102.c | 11 +++-- > > drivers/iio/humidity/hdc100x.c | 3 +- > > drivers/iio/humidity/htu21.c | 3 +- > > drivers/iio/humidity/si7020.c | 3 +- > > drivers/iio/magnetometer/ak8974.c | 3 +- > > drivers/iio/magnetometer/ak8975.c | 8 ++-- > > drivers/iio/potentiometer/ad5272.c | 5 +-- > > drivers/iio/potentiometer/ds1803.c | 6 +-- > > drivers/iio/potentiometer/max5432.c | 8 ++-- > > drivers/iio/potentiometer/max5481.c | 23 ++-------- > > drivers/iio/potentiometer/mcp4018.c | 12 ++--- > > drivers/iio/potentiometer/mcp4131.c | 8 ++-- > > drivers/iio/potentiometer/mcp4531.c | 11 ++--- > > drivers/iio/potentiostat/lmp91000.c | 11 +++-- > > drivers/iio/pressure/icp10100.c | 3 +- > > drivers/iio/pressure/ms5611_i2c.c | 6 +-- > > drivers/iio/pressure/ms5611_spi.c | 6 +-- > > drivers/iio/pressure/ms5637.c | 3 +- > > drivers/iio/pressure/zpa2326_i2c.c | 6 +-- > > drivers/iio/pressure/zpa2326_spi.c | 6 +-- > > drivers/iio/proximity/as3935.c | 44 +++++++++---------- > > .../iio/proximity/pulsedlight-lidar-lite-v2.c | 3 +- > > drivers/iio/resolver/ad2s1200.c | 3 +- > > drivers/iio/temperature/tmp007.c | 4 +- > > drivers/iio/temperature/tsys01.c | 3 +- > > 36 files changed, 134 insertions(+), 190 deletions(-) > > > > -- > > 2.28.0 > > > >
