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
