Messages in this thread | | | From | Viresh Kumar <> | Subject | [PATCH 0/2] thermal: Validate new state in cur_state_store() | Date | Mon, 17 Oct 2022 15:33:00 +0530 |
| |
In cur_state_store(), the new state of the cooling device is received from user-space and is not validated by the thermal core but the same is left for the individual drivers to take care of. Apart from duplicating the code it leaves possibility for introducing bugs where a driver may not do it right.
Lets make the thermal core check the new state itself and store the max value in the cooling device structure.
Once this is merged, I will update the thermal drivers to stop checking for the same.
Based of v6.1-rc1.
Viresh Kumar (2): thermal: Validate new state in cur_state_store() thermal: sysfs: Reuse cdev->max_state
drivers/thermal/gov_fair_share.c | 6 +----- drivers/thermal/thermal_core.c | 15 ++++++------- drivers/thermal/thermal_sysfs.c | 37 +++++++++++++------------------- include/linux/thermal.h | 1 + 4 files changed, 24 insertions(+), 35 deletions(-)
-- 2.31.1.272.g89b43f80a514
| |