Thread (16 messages) 16 messages, 4 authors, 2022-08-16

Re: [PATCH 1/2] dt-bindings: power: supply: Add Richtek RT9471 battery charger

From: Krzysztof Kozlowski <hidden>
Date: 2022-08-16 09:11:14
Also in: linux-pm, lkml

On 13/08/2022 17:52, ChiYuan Huang wrote:
quoted
quoted
Some gauge HW needs this information to enhance the battery capacity accuracy.
Other supply stack pieces do it via supplies (supplied to/from in
include/linux/power_supply.h) and reporting power_supply_changed().

With such explanation, your device is an interrupt source, but it is not
an interrupt controller. If your device is interrupt controller, it
means someone routes the interrupt line to your device. Physical line.
Yap, sure. And so on, just use the SW power supply chain to do this
kind of event notification.
To remove it, it doesn't affect the internal interrupt request inside
the driver.
Just cannot be used for the outer driver to request the events directly.

If so, I think 'interrupt-controller' and even '#interrupt-cells' need
to be removed.
OK?
Yes, both should be removed. Your device is not an interrupt controller...

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help