Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
thermal: gov_power_allocator: Allow binding before cooling devices
Commit e83747c ("thermal: gov_power_allocator: Set up trip points earlier") added a check that would fail binding the governer if there is no cooling devices bound to the thermal zone. Unfortunately this causes issues in cases when the TZ is bound to the governer before the cooling devices are attached to it. (I.e. when the tz is registered using thermal_zone_device_register_with_trips().) Additionally, the documentation across gov_power_allocator suggests it's intended to allow it to be bound to thermal zones without cooling devices (and thus without passive/active trip points), however the same commit added a check for the trip point to be present, causing those TZ to fail probing. Those changes cause all thermal zones to fail on some devices (such as sc7180-acer-aspire1) and prevent the kernel from controlling the cpu/gpu frequency based on the temperature, as well as loosing other "informational" thermal zones. This commit partially reverts the referenced one by dropping the trip point check and by allowing the TZ to probe even if no actor bufer was allocated. Fixes: e83747c ("thermal: gov_power_allocator: Set up trip points earlier") Signed-off-by: Nikita Travkin <nikita@trvn.ru>
- Loading branch information