diff options
author | Diana Z <dzigterman@chromium.org> | 2023-03-30 14:18:51 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-03-31 20:50:36 +0000 |
commit | 62b635f5129cc204fd9e64f267b9d8cfc10d0abe (patch) | |
tree | 34fa064f4330051de799cd81b082f13f4b392cc1 /driver | |
parent | d32a3897184e7507a310b0d4a351d9dbd8f3bc0b (diff) | |
download | chrome-ec-62b635f5129cc204fd9e64f267b9d8cfc10d0abe.tar.gz |
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 <dzigterman@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4387134
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
Diffstat (limited to 'driver')
-rw-r--r-- | driver/charger/isl9241.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |