Re: [PATCH v6 1/6] i2c: Allow an ACPI driver to manage the device's power state during probe

From: Wolfram Sang
Date: Mon Aug 31 2020 - 05:31:20 EST



> 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