> This patchset is really about changing the default of ACPI powering up I²C > devices. On OF the drivers are indeed responsible for that. So, maybe it makes sense then to move from 'device_property_present()' to 'acpi_dev_get_property()' or something alike? To clearly tell this binding is expected to be used with ACPI only. Then, we can skip this discussion now and postpone it to when someone wants to use it with DT. Which is hopefully never. Or does this approach have drawbacks? > My original series had a field in struct device_driver for this purpose but > Greg K-H suggested moving it to I²C instead: > > <URL:https://lore.kernel.org/linux-acpi/20190826084343.GA1095@xxxxxxxxx/> Ok, we can still factor it out in the unlikely case it needs to be done again. I still have the question via which tree this should go upstream? It is probably more I2C than ACPI?
Attachment:
signature.asc
Description: PGP signature
