diff options
author | Tom Hughes <tomhughes@chromium.org> | 2022-10-11 16:30:19 -0700 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-12 16:50:12 +0000 |
commit | a1dbdc91ae1b3d60d307e5bad9514f5ed2d6b65f (patch) | |
tree | b01e4e793a3e2e3edd3d5a99f06089a5ab16dc7a | |
parent | 7929e0335ef69fedb5a2f27b5b840845792a38b4 (diff) | |
download | chrome-ec-a1dbdc91ae1b3d60d307e5bad9514f5ed2d6b65f.tar.gz |
chip/stm32: Remove self-assignment
When building with clang it warns:
chip/stm32/clock-stm32l4.c:400:9: error: explicitly assigning value of
variable of type 'uint32_t' (aka 'unsigned int') to itself
[-Werror,-Wself-assign]
val = val;
~~~ ^ ~~~
BRANCH=none
BUG=b:172020503
TEST=CC=clang make BOARD=hyperdebug
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: Ie4fae7251747d27b96ed08c3ef166fddd5411b9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3947822
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
-rw-r--r-- | chip/stm32/clock-stm32l4.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chip/stm32/clock-stm32l4.c b/chip/stm32/clock-stm32l4.c index 1c460c10fc..77e807c1ad 100644 --- a/chip/stm32/clock-stm32l4.c +++ b/chip/stm32/clock-stm32l4.c @@ -397,7 +397,7 @@ static void clock_set_osc(enum clock_osc osc, enum clock_osc pll_osc) val = STM32_FLASH_ACR; val &= ~STM32_FLASH_ACR_LATENCY_MASK; if (freq <= 16000000U) { - val = val; + /* nothing */ } else if (freq <= 32000000U) { val |= 1; } else if (freq <= 48000000U) { @@ -416,7 +416,7 @@ static void clock_set_osc(enum clock_osc osc, enum clock_osc pll_osc) val &= ~STM32_FLASH_ACR_LATENCY_MASK; if (freq <= 6000000U) { - val = val; + /* nothing */ } else if (freq <= 12000000U) { val |= 1; } else if (freq <= 18000000U) { |