diff options
author | Thorsten Blum <thorsten.blum@toblux.com> | 2024-07-11 15:20:01 +0200 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2024-08-05 22:30:55 +0530 |
commit | da080d987e20a8b20b368c75826226834c9660a3 (patch) | |
tree | eabdd2c05437888e59c1007026a0461b5e20ebe1 /drivers/dma | |
parent | 8bce5522a13107605709c3b253c09cb1f90850e8 (diff) | |
download | linux-da080d987e20a8b20b368c75826226834c9660a3.tar.gz linux-da080d987e20a8b20b368c75826226834c9660a3.tar.bz2 linux-da080d987e20a8b20b368c75826226834c9660a3.zip |
dmaengine: dmatest: Explicitly cast divisor to u32
As the comment explains, the while loop ensures that runtime fits into
32 bits. Since do_div() casts the divisor to u32 anyway, explicitly cast
runtime to u32 to remove the following Coccinelle/coccicheck warning
reported by do_div.cocci:
WARNING: do_div() does a 64-by-32 division, please consider using div64_s64 instead
Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
Link: https://lore.kernel.org/r/20240711132001.92157-1-thorsten.blum@toblux.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma')
-rw-r--r-- | drivers/dma/dmatest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/dmatest.c b/drivers/dma/dmatest.c index 1f201a542b37..91b2fbc0b864 100644 --- a/drivers/dma/dmatest.c +++ b/drivers/dma/dmatest.c @@ -500,7 +500,7 @@ static unsigned long long dmatest_persec(s64 runtime, unsigned int val) per_sec *= val; per_sec = INT_TO_FIXPT(per_sec); - do_div(per_sec, runtime); + do_div(per_sec, (u32)runtime); return per_sec; } |