On Sun, 2020-04-19 at 16:01 +0100, jic23@xxxxxxxxxx wrote: > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Hi All, > > Given we keep having to explain to people that of_match_ptr is less > than ideal now we have the option of ACPI DSDT using PRP0001 and > the compatible, it seems sensible to reduce the number of instances > that people might copy for a new driver. > > Added theoretical benefit is that we can probe all these drivers from > appropriate DSDT (though I doubt anyone will). > > I'm sending this first set out to see if anyone has strong views against > doing this for at least the simple drivers that have no other device > tree dependence. Obviously more work would be needed to remove > use of of_match_ptr from IIO completely. > > Light sensors picked as a starting point as they tend to be simple. > > I may do follow ups in larger blocks to avoid so many small patches > (or indeed flatten these into one when applying) fwiw: i was also planning to do a sweep of these; well, tbh, the main intent was to target ADI drivers first and do a bigger conversion for them to make the slightly friendlier with ACPI; aside from this, i'm also noticing some bad patterns being copied from older drivers, when asking new people to write Linux drivers; i did not make a list, probably should have; one is 'mlock' [still] being copied; and accessing other internal fields; but the internal fields accessing requires a bit of a cleanup in the form of privatizing the fields somehow; > > Thanks > > Jonathan > > Jonathan Cameron (7): > iio: light: bh1780: use mod_devicetable.h and drop of_match_ptr macro > iio: light: cm32181: Add mod_devicetable.h and remove of_match_ptr > iio: light: cm3232: Add mod_devicetable.h include and drop > of_match_ptr > iio: light: gp2ap020a00f: Swap of.h for mod_devicetable.h + drop > of_match_ptr > iio: light: opt3001: Add mod_devicetable.h and drop use of > of_match_ptr > iio: light: st_uvis25: Add mod_devicetable.h and drop of_match_ptr > iio: light: vl6180: swap of.h for mod_devicetable.h and drop > of_match_ptr > > drivers/iio/light/bh1780.c | 6 ++---- > drivers/iio/light/cm32181.c | 3 ++- > drivers/iio/light/cm3232.c | 3 ++- > drivers/iio/light/gp2ap020a00f.c | 6 ++---- > drivers/iio/light/opt3001.c | 3 ++- > drivers/iio/light/st_uvis25_i2c.c | 3 ++- > drivers/iio/light/st_uvis25_spi.c | 3 ++- > drivers/iio/light/vl6180.c | 2 +- > 8 files changed, 15 insertions(+), 14 deletions(-) >
