sravan/system76-embedded-controller
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fan: Do not make functions reentrant

Browse Source 
Signed-off-by: Tim Crawford <tcrawford@system76.com>
This commit is contained in:
Tim Crawford
2024-07-03 18:13:38 -06:00
committed by Tim Crawford
parent e75a2f1e10
commit ffd0b7cbde
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/board/system76/common/fan.c
View File

@@ -110,7 +110,7 @@ void fan_reset(void) {
// Get duty cycle based on temperature, adapted from // Get duty cycle based on temperature, adapted from
// https://github.com/pop-os/system76-power/blob/master/src/fan.rs // https://github.com/pop-os/system76-power/blob/master/src/fan.rs
static uint8_t fan_duty(const struct Fan *const fan, int16_t temp) __reentrant { static uint8_t fan_duty(const struct Fan *const fan, int16_t temp) {
for (uint8_t i = 0; i < fan->points_size; i++) { for (uint8_t i = 0; i < fan->points_size; i++) {
const struct FanPoint *cur = &fan->points[i]; const struct FanPoint *cur = &fan->points[i];
@@ -145,7 +145,7 @@ static uint8_t fan_duty(const struct Fan *const fan, int16_t temp) __reentrant {
return MAX_FAN_SPEED; return MAX_FAN_SPEED;
} }
static uint8_t fan_smooth(uint8_t last_duty, uint8_t duty) __reentrant { static uint8_t fan_smooth(uint8_t last_duty, uint8_t duty) {
uint8_t next_duty = duty; uint8_t next_duty = duty;
// ramping down // ramping down
@@ -177,7 +177,7 @@ static uint8_t fan_smooth(uint8_t last_duty, uint8_t duty) __reentrant {
return next_duty; return next_duty;
} }
static uint8_t fan_heatup(const struct Fan *const fan, uint8_t duty) __reentrant { static uint8_t fan_heatup(const struct Fan *const fan, uint8_t duty) {
uint8_t lowest = duty; uint8_t lowest = duty;
uint8_t i; uint8_t i;
@@ -193,7 +193,7 @@ static uint8_t fan_heatup(const struct Fan *const fan, uint8_t duty) __reentrant
return lowest; return lowest;
} }
static uint8_t fan_cooldown(const struct Fan *const fan, uint8_t duty) __reentrant { static uint8_t fan_cooldown(const struct Fan *const fan, uint8_t duty) {
uint8_t highest = duty; uint8_t highest = duty;
uint8_t i; uint8_t i;