On 13/08/2022 17:52, ChiYuan Huang wrote: >>> 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