From 62b635f5129cc204fd9e64f267b9d8cfc10d0abe Mon Sep 17 00:00:00 2001 From: Diana Z Date: Thu, 30 Mar 2023 14:18:51 -0600 Subject: ISL9241: Fix frequency range checks There was a typo in the frequency ranges, which resulted in 656kHz not being set when given as an input. Correct the check point to 628kHz, the value in between both register options similar to the other clauses. BRANCH=skyrim BUG=None TEST=./twister -T ./zephyr/test Change-Id: Iffa99a6c77006bd4c11b8f6e63afafa27a788de2 Signed-off-by: Diana Z Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4387134 Reviewed-by: Keith Short Commit-Queue: Keith Short --- driver/charger/isl9241.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'driver') diff --git a/driver/charger/isl9241.c b/driver/charger/isl9241.c index e7ce455a39..5f3566095c 100644 --- a/driver/charger/isl9241.c +++ b/driver/charger/isl9241.c @@ -179,7 +179,7 @@ static enum ec_error_list isl9241_set_frequency(int chgnum, int freq_khz) freq = ISL9241_CONTROL1_SWITCHING_FREQ_808KHZ; else if (freq_khz >= 690) freq = ISL9241_CONTROL1_SWITCHING_FREQ_724KHZ; - else if (freq_khz >= 678) + else if (freq_khz >= 628) freq = ISL9241_CONTROL1_SWITCHING_FREQ_656KHZ; else freq = ISL9241_CONTROL1_SWITCHING_FREQ_600KHZ; -- cgit v1.2.1