Re: [PATCH 8/8] dt-bindings: hwmon: allow specifying channels for tmp421
From: Oskar Senft <hidden>
Date: 2021-09-21 21:21:43
Also in:
linux-hwmon
From: Oskar Senft <hidden>
Date: 2021-09-21 21:21:43
Also in:
linux-hwmon
Hi
quoted
quoted
+ temperature-sensors { + ltd { + status = "disabled"; + }; + + rtd1 { + status = "okay"; + type = <4> /* thermistor */;'type' is a bit generic. We don't want the same property name to possibly have multiple definitions.How about sensor-type ?
In the datasheet this is called "mode". I called it "type" since it's tempX_type in sysfs and I wanted to stay in sync with that.
quoted
If it is how you address an instance of something which seems to be the case here, then 'reg' should be used.
The reason I didn't do that is because you'd effectively have to
duplicate the ID in both the name (e.g. sensor@1) and the reg property
(e.g. reg = <1>). But maybe that's just what is is in device tree, I
can live with that.
However, we'd also have to find out whether the "local" sensor ("LTD")
would simply be #4, as it is in sysfs today. I'd also be ok with that
as it would keep sysfs and device tree "in sync" wrt. naming.
Thanks
Oskar.