From: "Rafael J. Wysocki" <rjw@rjwysocki.net> To: Wolfram Sang <wsa@the-dreams.de> Cc: Jarkko Nikula <jarkko.nikula@linux.intel.com>, linux-i2c@vger.kernel.org, alsa-devel@alsa-project.org, linux-acpi@vger.kernel.org, lm-sensors@lm-sensors.org, Mark Brown <broonie@kernel.org>, Jean Delvare <jdelvare@suse.com>, Liam Girdwood <lgirdwood@gmail.com>, Dustin Byford <dustin@cumulusnetworks.com>, linux@roeck-us.net Subject: Re: [RFC] i2c: Revert back to old device naming for ACPI enumerated I2C slaves Date: Tue, 25 Aug 2015 02:19:36 +0200 [thread overview] Message-ID: <2688486.RKUY9k7hgx@vostro.rjw.lan> (raw) In-Reply-To: <20150824132612.GA10078@katana> [-- Attachment #1: Type: text/plain, Size: 1602 bytes --] On Monday, August 24, 2015 03:26:13 PM Wolfram Sang 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: > > > > lib/sysfs.c:665: > > if ((!subsys || !strcmp(subsys, "i2c")) && > > sscanf(dev_name, "%hd-%x", &entry.chip.bus.nr, > > &entry.chip.addr) == 2) { > > > > This patch fixes this by reverting back the old device naming in i2c-core > > but at the same avoids regression to ALSA SoC drivers that depend on stable > > device binding. > > This way we fix some userspace by going back. However, this name change > was effective for 18 months, so enough time for some other userspace to > adapt. We would break the latter by reverting. Right. Generally, reverts of such things need to be made relatively quickly. > I tend to create the symlinks to prevent any further breakage. I am open > for other suggestions, though. This ugly situation has surely happened > before somewhere somewhen in the kernel :/ It happens on a regular basis. In this particular case it looks like we need a mechanism to use the old naming scheme for hwmon and the new one for everything else or something similar. I'm afraid I am not familiar enough with the i2c core to suggest anything specific ATM, though. Thanks, Rafael [-- Attachment #2: This is a digitally signed message part. --] [-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: "Rafael J. Wysocki" <rjw@rjwysocki.net> To: Wolfram Sang <wsa@the-dreams.de> Cc: Jarkko Nikula <jarkko.nikula@linux.intel.com>, linux-i2c@vger.kernel.org, alsa-devel@alsa-project.org, linux-acpi@vger.kernel.org, lm-sensors@lm-sensors.org, Mark Brown <broonie@kernel.org>, Jean Delvare <jdelvare@suse.com>, Liam Girdwood <lgirdwood@gmail.com>, Dustin Byford <dustin@cumulusnetworks.com>, linux@roeck-us.net Subject: Re: [lm-sensors] [RFC] i2c: Revert back to old device naming for ACPI enumerated I2C slaves Date: Tue, 25 Aug 2015 00:19:36 +0000 [thread overview] Message-ID: <2688486.RKUY9k7hgx@vostro.rjw.lan> (raw) In-Reply-To: <20150824132612.GA10078@katana> [-- Attachment #1.1: Type: text/plain, Size: 1602 bytes --] On Monday, August 24, 2015 03:26:13 PM Wolfram Sang 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: > > > > lib/sysfs.c:665: > > if ((!subsys || !strcmp(subsys, "i2c")) && > > sscanf(dev_name, "%hd-%x", &entry.chip.bus.nr, > > &entry.chip.addr) == 2) { > > > > This patch fixes this by reverting back the old device naming in i2c-core > > but at the same avoids regression to ALSA SoC drivers that depend on stable > > device binding. > > This way we fix some userspace by going back. However, this name change > was effective for 18 months, so enough time for some other userspace to > adapt. We would break the latter by reverting. Right. Generally, reverts of such things need to be made relatively quickly. > I tend to create the symlinks to prevent any further breakage. I am open > for other suggestions, though. This ugly situation has surely happened > before somewhere somewhen in the kernel :/ It happens on a regular basis. In this particular case it looks like we need a mechanism to use the old naming scheme for hwmon and the new one for everything else or something similar. I'm afraid I am not familiar enough with the i2c core to suggest anything specific ATM, though. Thanks, Rafael [-- Attachment #1.2: This is a digitally signed message part. --] [-- Type: application/pgp-signature, Size: 836 bytes --] [-- Attachment #2: Type: text/plain, Size: 153 bytes --] _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2015-08-24 23:52 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-08-24 10:52 [RFC] i2c: Revert back to old device naming for ACPI enumerated I2C slaves Jarkko Nikula 2015-08-24 10:52 ` [lm-sensors] " Jarkko Nikula [not found] ` <1440413522-7855-1-git-send-email-jarkko.nikula-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> 2015-08-24 13:26 ` Wolfram Sang 2015-08-24 13:26 ` [lm-sensors] " Wolfram Sang 2015-08-25 0:19 ` Rafael J. Wysocki [this message] 2015-08-25 0:19 ` Rafael J. Wysocki 2015-08-25 14:59 ` Wolfram Sang 2015-08-25 14:59 ` [lm-sensors] " Wolfram Sang 2015-08-25 5:03 ` Dustin Byford 2015-08-25 5:03 ` [lm-sensors] " Dustin Byford [not found] ` <20150825050306.GB21569-qUQiAmfTcIp+XZJcv9eMoEEOCMrvLtNR@public.gmane.org> 2015-08-25 14:50 ` Jarkko Nikula 2015-08-25 14:50 ` [lm-sensors] " Jarkko Nikula 2015-08-25 5:25 ` Mark Brown 2015-08-25 5:25 ` [lm-sensors] " Mark Brown 2015-08-25 14:57 ` Wolfram Sang 2015-08-25 14:57 ` [lm-sensors] " Wolfram Sang [not found] ` <20150825145756.GA4066-oo5tB6JMkjKRinMKxDlMNPwbnWRJjS81@public.gmane.org> 2015-08-25 15:18 ` Guenter Roeck 2015-08-25 15:18 ` [lm-sensors] " Guenter Roeck [not found] ` <55DC8746.1060809-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> 2015-08-25 16:18 ` Wolfram Sang 2015-08-25 16:18 ` [lm-sensors] " Wolfram Sang 2015-08-25 16:22 ` Wolfram Sang 2015-08-25 16:22 ` [lm-sensors] " Wolfram Sang 2015-08-25 17:12 ` Guenter Roeck 2015-08-25 17:12 ` [lm-sensors] " Guenter Roeck 2015-08-25 16:14 ` Mark Brown 2015-08-25 16:14 ` [lm-sensors] " Mark Brown 2015-10-01 20:37 ` Wolfram Sang 2015-10-02 9:27 ` Jarkko Nikula 2015-10-09 21:47 ` Wolfram Sang 2015-10-12 8:32 ` Jarkko Nikula
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=2688486.RKUY9k7hgx@vostro.rjw.lan \ --to=rjw@rjwysocki.net \ --cc=alsa-devel@alsa-project.org \ --cc=broonie@kernel.org \ --cc=dustin@cumulusnetworks.com \ --cc=jarkko.nikula@linux.intel.com \ --cc=jdelvare@suse.com \ --cc=lgirdwood@gmail.com \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-i2c@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=lm-sensors@lm-sensors.org \ --cc=wsa@the-dreams.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.