On 08/25/2015 09:18 AM, Wolfram Sang wrote:
On Tue, Aug 25, 2015 at 08:18:30AM -0700, Guenter Roeck wrote:
On 08/25/2015 07:57 AM, Wolfram Sang wrote:
On Tue, Aug 25, 2015 at 06:25:13AM +0100, Mark Brown wrote:
On Mon, Aug 24, 2015 at 01:52:02PM +0300, Jarkko Nikula wrote:
Commit 70762abb9f89 ("i2c: Use stable dev_name for ACPI enumerated I2C slaves") broke the lm-sensors which relies on I2C hwmon slave devices under /sys/bus/i2c/devices/ to be named as "x-00yz". However if those hwmon devices are ACPI 5 enumerated their name became "i2c-INTABCD:ij" and sysfs code in lm-sensors does not find them anymore:
Acked-by: Mark Brown broonie@kernel.org
Don't you think there will be regressions given that the new naming scheme was around for 18 months?
acpi is pretty long term. New bindings don't show up quickly.
The key is, there might be userspace applications that got used to our new "export" of ACPI bindings. The bindings were already existing, our "export" did change.
So I am not surprised that this only shows up now.
I am, to be honest. It shows running lm-sensors with ACPI on a kernel newer than 18 months. Not a rare scenario, so I thought.
With i2c (sensor) devices enumerated in acpi ? Yes, I think that is quite rare. This is not about the kernel, it is about such device bindings showing up in acpi data.
Guenter