Re: [PATCH v4 1/5] dt-bindings: power: Extend battery chemistry with capacitor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 22-04-2024 13:56, Krzysztof Kozlowski wrote:
On 22/04/2024 09:50, Mike Looijmans wrote:
On 15-04-2024 17:24, Krzysztof Kozlowski wrote:
On 15/04/2024 10:13, Mike Looijmans wrote:
Another technology to store energy is a (super)capacitor.

Signed-off-by: Mike Looijmans <mike.looijmans@xxxxxxxx>
---

(no changes since v1)

   Documentation/devicetree/bindings/power/supply/battery.yaml | 1 +
   1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/power/supply/battery.yaml b/Documentation/devicetree/bindings/power/supply/battery.yaml
index 491488e7b970..a22c97dfad88 100644
--- a/Documentation/devicetree/bindings/power/supply/battery.yaml
+++ b/Documentation/devicetree/bindings/power/supply/battery.yaml
@@ -44,6 +44,7 @@ properties:
         - const: lithium-ion-polymer
         - const: lithium-ion-iron-phosphate
         - const: lithium-ion-manganese-oxide
+      - const: capacitor
Please keep some sort of order... everyone insists on adding to the end
of the lists...
Could make it alphabetical, but that requires re-ordering the existing
ones so "lithium" sorts before "nickel".
So just put it at beginning, less sorting later.

Ok. I'll wait a bit for comments on the other parts, before sending a v6 of the series.



I think the intention here was that the order is the same as the
matching defines in power-supply.h
There is no such binding as power-supply.h. And other headers do not
matter, they are not bindings.

Clear, thanks.


--
Mike Looijmans
System Expert

TOPIC Embedded Products B.V.
Materiaalweg 4, 5681 RJ Best
The Netherlands

T: +31 (0) 499 33 69 69
E: mike.looijmans@xxxxxxxx
W: www.topic.nl








[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux