Skip to content

[PW_SID:1100485] [RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup#2012

Open
linux-riscv-bot wants to merge 1 commit into
workflow__riscv__fixesfrom
pw1100485
Open

[PW_SID:1100485] [RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup#2012
linux-riscv-bot wants to merge 1 commit into
workflow__riscv__fixesfrom
pw1100485

Conversation

@linux-riscv-bot
Copy link
Copy Markdown

PR for series 1100485 applied to workflow__riscv__fixes

Name: [RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup
URL: https://patchwork.kernel.org/project/linux-riscv/list/?series=1100485
Version: 1

The cycle register is always u32, so cycles_to_ns() can take a u32
instead of a u64. With that narrowing, cycles * NSEC_PER_SEC is at most
u32::MAX * 1e9 (~4.3e18), which fits in u64 without overflow. The
saturating arithmetic is therefore no longer needed, and the ceiling
division can use Rust's u64::div_ceil() directly instead of the
open-coded numerator/denominator form.

This also drops the TODO referring to a future
mul_u64_u64_div_u64_roundup kernel helper, which is no longer required.

Signed-off-by: Maurice Hieronymus <mhi@mailbox.org>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 115.35 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 1018.50 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1369.80 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 19.38 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 20.98 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 0.57 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 77.00 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.23 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
kdoc
Desc: Detects for kdoc errors
Duration: 0.70 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
module-param
Desc: Detect module_param changes
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.22 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

Patch 1: "[RESEND] pwm: th1520: Remove requirement for mul_u64_u64_div_u64_roundup"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.26 seconds
Result: PASS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants