On 25/11/2024 09:31, Sung-Chi Li wrote: > A charger chip is connect to the ChromeOS Embedded Controller (EC). > When external power input into the system, current would go through the > charger chip, generating heat in the system. The EC supports limiting > the input current, thus reducing the generated heat. As a result, EC is > a simulated passive cooling device. EC is not part of the SoC, therefore this is not a cooling device. It is not thermal sensor either, but this you pushed before I could object. > > We cannot reuse the existing charge managing mechanism in the power > framework due to: > > - The power framework requires the charger to expose its thermal status, > which is not a supported functionality on EC. Fix power framework then. > - We need to use different thermal sensors to run thermal control, > rather than using thermal sensor on the charger. Nothing stops you from using thermal sensors or thermal control. Still, this is not part of SoC, this is not thermal zone and this is not a SoC cooling device. NAK Best regards, Krzysztof
