Re: [PATCH 1/2] iio: documentation: Document proximity sensor label use
From: Bastien Nocera <hadess@hadess.net>
Date: 2021-02-08 13:41:59
Also in:
linux-input
From: Bastien Nocera <hadess@hadess.net>
Date: 2021-02-08 13:41:59
Also in:
linux-input
On Sun, 2021-02-07 at 13:37 +0100, Hans de Goede wrote:
Add an entry to Documentation/ABI/testing/sysfs-bus-iio for the new device and channel label sysfs-attribute support. And document the standardized labels which may be used with proximity sensors to hint userspace about the intended use of the sensor. Using labels to differentiate between the multiple proximity sensors which a modern laptop/tablet may have was discussed in this thread: https://lore.kernel.org/linux-iio/9f9b0ff6-3bf1-63c4-eb36-901cecd7c4d9@redhat.com/ (local) As mentioned the "proximity-wifi*" labels are already being used in this manner on some chromebooks, see e.g.: arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi arch/arm64/boot/dts/qcom/sc7180-trogdor-lte-sku.dtsi And the "proximity-palmrest" and "proximity-lap" labels are intended to be used with the lap and palmrest sensors found in recent Lenovo ThinkPad models.
Both patches in the series look fine to me. Is IIO the interface you plan on using to implement the lap detection for the thinkpad_acpi driver? If so, don't forget to set the "nearlevel" property as well. I'll probably have to write a new iio-sensor-proxy driver for those as well, without the hardware. That'll be interesting ;) Cheers