Hi Arnd, > > -#include <linux/i2c/apds990x.h> > > +#include <linux/platform_data/apds990x.h> > > The new location is clearly better than the old, but I notice that in both > patches, there is not a single definition for the platform_data structure > in the kernel and both drivers refuse to load when they do not get > passed valid platform_data. Yes, this is true for quite some drivers I am moving around. I think there are two reasons: a) the board code never made it upstream b) DT conversion happened and platform_data is now cruft. As mentioned in the cover-letter, I didn't dive deeper for all the drivers. > Should we also remove the drivers or move them into staging? I'd prefer to keep them. If they are needed again, DT conversion is likely and easier than restarting from scratch. Guenter Roeck also prefers to not unnecessarily annoy people who might have out-of-tree board code. So, as long as they are not painful, let's keep them? Regards, Wolfram
Attachment:
signature.asc
Description: PGP signature
