pwm: atmel-tcb: Use min() macro

Shen Lichuan · Uwe Kleine-König · commit 51f813702275 · 2024-10-25T11:33:34.000+02:00
Use the min() macro to simplify the atmel_tcb_pwm_apply() function and improve its readability. Signed-off-by: Shen Lichuan <shenlichuan@vivo.com> Link: https://lore.kernel.org/r/20240827075749.67583-1-shenlichuan@vivo.com Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
diff --git a/drivers/pwm/pwm-atmel-tcb.c b/drivers/pwm/pwm-atmel-tcb.c
@@ -342,8 +342,8 @@ static int atmel_tcb_pwm_apply(struct pwm_chip *chip, struct pwm_device *pwm,
 		return 0;
 	}
 
-	period = state->period < INT_MAX ? state->period : INT_MAX;
-	duty_cycle = state->duty_cycle < INT_MAX ? state->duty_cycle : INT_MAX;
+	period = min(state->period, INT_MAX);
+	duty_cycle = min(state->duty_cycle, INT_MAX);
 
 	ret = atmel_tcb_pwm_config(chip, pwm, duty_cycle, period);
 	if (ret)

Original file line number	Diff line number	Diff line change
`@@ -342,8 +342,8 @@ static int atmel_tcb_pwm_apply(struct pwm_chip chip, struct pwm_device pwm,`
`342`	`342`	`return 0;`
`343`	`343`	`}`
`344`	`344`
`345`		`- period = state->period < INT_MAX ? state->period : INT_MAX;`
`346`		`- duty_cycle = state->duty_cycle < INT_MAX ? state->duty_cycle : INT_MAX;`
	`345`	`+ period = min(state->period, INT_MAX);`
	`346`	`+ duty_cycle = min(state->duty_cycle, INT_MAX);`
`347`	`347`
`348`	`348`	`ret = atmel_tcb_pwm_config(chip, pwm, duty_cycle, period);`
`349`	`349`	`if (ret)`