Re: [PATCH] dt-bindings: power: supply: bq256xx: Add ts-ignore property

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

 



On 2023/3/9 18:12, Krzysztof Kozlowski wrote:
On 09/03/2023 03:39, Hermes Zhang wrote:
Add a new property: ts-ignore to allow user to enable the TS_IGNORE flag
in chip. Ignore TS pin will allow user to control the charging
parameters instead of the default JEITA profile in chip.
You miss users of it.
Could you give some more info about what I missed?
@@ -68,6 +68,12 @@ properties:
        Interrupt sends an active low, 256 μs pulse to host to report the charger
        device status and faults.
+ ts-ignore:
Missing vendor prefix... is this generic property? ts-ignore is very
cryptic. You should describe here rather desired system characteristic.
Why anyone would need to use it per-board level?

OK, I will fix it. Will "ti,ignore-ts-pin" be better? Yes, it's a generic. To disable the TS pin, actually we also could do it from HW, but from software, it could be more flexble. But if HW already disable it, then you don't need to do it again from dts.

BTW, I find I may miss some information here: the ts-ignore feature is not support by all the chips list here, I will add some comment for which chip support it.


Best Regards,

Hermes





[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