Re: [PATCH v3 1/2] dt-bindings: hwmon: Add ina3221 documentation
From: Nicolin Chen <nicoleotsuka@gmail.com>
Date: 2018-09-22 18:03:57
Also in:
linux-doc, linux-hwmon, lkml
From: Nicolin Chen <nicoleotsuka@gmail.com>
Date: 2018-09-22 18:03:57
Also in:
linux-doc, linux-hwmon, lkml
quoted
+2) child nodes + The names of child nodes should indicate input source names + + Required properties: + - input-id: Must be 1, 2 or 3 + + Optional properties: + - shunt-resistor: Shunt resistor value in micro-Ohm + - status: Should be "disabled" if no input source + + Example: + + input1 { + input-id = <0x1>; + status = "disabled"; + }; + VDD_GPU { + input-id = <0x2>; + shunt-resistor = <5000>; + };Using child nodes is a good idea. However, you are converting the node name into the hwmon 'label' attribute which I can not accept. First, it is undocumented, second, it effectively creates an undocumented property (if one wants to configure the shunt resistor value, one has to configure a child node which is converted into a label), and third, it violates the hwmon ABI ('input1' is not a "hint about what this voltage channel is being used for").
Oh. I see the point here now. Then a child name could be just input[123], and I will add a separate optional child property to indicate the label. Will fix it in next ver. Thanks Nicolin