Add a label property to allow a custom name to be used for identifying
a device on the board. This is useful when multiple devices are present on
the same board. Similar change was done by commit ffae65fb1ae4
("dt-bindings: spi: spi-cadence: Add label property").
Signed-off-by: Michal Simek <[email protected]>
---
zcu102 is using this feature
---
Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
index 378d1f6aeeb3..8e5c1935b5f4 100644
--- a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
+++ b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
@@ -32,6 +32,9 @@ properties:
reg:
maxItems: 1
+ label:
+ description: A descriptive name for this device.
+
shunt-resistor:
description:
Shunt resistor value in micro-Ohm.
@@ -77,6 +80,7 @@ examples:
power-sensor@44 {
compatible = "ti,ina220";
reg = <0x44>;
+ label = "vdd_3v0";
shunt-resistor = <1000>;
vs-supply = <&vdd_3v0>;
};
--
2.36.1
There are two drivers in the Linux kernel. One is hwmon based and second
IIO. IIO version requires to define #io-channel-cells to operate.
Signed-off-by: Michal Simek <[email protected]>
---
zynqmp zcu102 uses this feature too.
---
Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
index 8e5c1935b5f4..f324b627bf9c 100644
--- a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
+++ b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
@@ -32,6 +32,9 @@ properties:
reg:
maxItems: 1
+ "#io-channel-cells":
+ const: 1
+
label:
description: A descriptive name for this device.
@@ -80,6 +83,7 @@ examples:
power-sensor@44 {
compatible = "ti,ina220";
reg = <0x44>;
+ #io-channel-cells = <1>;
label = "vdd_3v0";
shunt-resistor = <1000>;
vs-supply = <&vdd_3v0>;
--
2.36.1
On 20/12/2023 14:12, Michal Simek wrote:
> Add a label property to allow a custom name to be used for identifying
> a device on the board. This is useful when multiple devices are present on
> the same board. Similar change was done by commit ffae65fb1ae4
> ("dt-bindings: spi: spi-cadence: Add label property").
>
> Signed-off-by: Michal Simek <[email protected]>
> ---
>
> zcu102 is using this feature
Is the driver or driver core parsing it?
Best regards,
Krzysztof
On 20/12/2023 14:12, Michal Simek wrote:
> There are two drivers in the Linux kernel. One is hwmon based and second
> IIO. IIO version requires to define #io-channel-cells to operate.
>
> Signed-off-by: Michal Simek <[email protected]>
> ---
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On 12/21/23 10:06, Krzysztof Kozlowski wrote:
> On 20/12/2023 14:12, Michal Simek wrote:
>> Add a label property to allow a custom name to be used for identifying
>> a device on the board. This is useful when multiple devices are present on
>> the same board. Similar change was done by commit ffae65fb1ae4
>> ("dt-bindings: spi: spi-cadence: Add label property").
>>
>> Signed-off-by: Michal Simek <[email protected]>
>> ---
>>
>> zcu102 is using this feature
>
> Is the driver or driver core parsing it?
I see that functions to be in hwmon and iio cores.
drivers/hwmon/hwmon.c:820: err = device_property_read_string(dev, "label",
&label);
drivers/iio/industrialio-core.c:1914: fwnode_property_read_string(fwnode,
"label", &indio_dev->label);
Thanks,
Michal
On 21/12/2023 12:24, Michal Simek wrote:
>
>
> On 12/21/23 10:06, Krzysztof Kozlowski wrote:
>> On 20/12/2023 14:12, Michal Simek wrote:
>>> Add a label property to allow a custom name to be used for identifying
>>> a device on the board. This is useful when multiple devices are present on
>>> the same board. Similar change was done by commit ffae65fb1ae4
>>> ("dt-bindings: spi: spi-cadence: Add label property").
>>>
>>> Signed-off-by: Michal Simek <[email protected]>
>>> ---
>>>
>>> zcu102 is using this feature
>>
>> Is the driver or driver core parsing it?
>
> I see that functions to be in hwmon and iio cores.
>
> drivers/hwmon/hwmon.c:820: err = device_property_read_string(dev, "label",
> &label);
>
> drivers/iio/industrialio-core.c:1914: fwnode_property_read_string(fwnode,
> "label", &indio_dev->label);
I wonder why it is not in bindings/iio/common.yaml...
Best regards,
Krzysztof
On 20/12/2023 14:12, Michal Simek wrote:
> Add a label property to allow a custom name to be used for identifying
> a device on the board. This is useful when multiple devices are present on
> the same board. Similar change was done by commit ffae65fb1ae4
> ("dt-bindings: spi: spi-cadence: Add label property").
>
> Signed-off-by: Michal Simek <[email protected]>
> ---
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Best regards,
Krzysztof
On Wed, Dec 20, 2023 at 02:12:13PM +0100, Michal Simek wrote:
> Add a label property to allow a custom name to be used for identifying
> a device on the board. This is useful when multiple devices are present on
> the same board. Similar change was done by commit ffae65fb1ae4
> ("dt-bindings: spi: spi-cadence: Add label property").
>
> Signed-off-by: Michal Simek <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
Applied.
Guenter
On Wed, Dec 20, 2023 at 02:12:14PM +0100, Michal Simek wrote:
> There are two drivers in the Linux kernel. One is hwmon based and second
> IIO. IIO version requires to define #io-channel-cells to operate.
>
> Signed-off-by: Michal Simek <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
Applied.
Guenter