Re: [PATCH 3/5] lib: div64: inline __iter_div_u64_rem() into its only user

Simon Glass <[email protected]>
Newsgroups org.u-boot-project.lists.u-boot
Message-ID <CAFLszTjCk=uYhSbvtwBog_7BCfuUjCLxiUDJc+bhVZjNb7iJag@mail.gmail.com>
On 2026-07-30T18:40:37, Alexey Charkov <[email protected]> wrote:
> lib: div64: inline __iter_div_u64_rem() into its only user
>
> __iter_div_u64_rem() is only split out in Linux for its vDSO header, which
> has no use in U-Boot. Inline the body of the helper directly into its only
> user, which is the iter_div_u64_rem() wrapper in this same file, and drop
> the static inline, so that a subsequent resync of include/linux/math64.h
> against Linux does not have to carry an exception for it.
>
> include/vdso/math64.h is deliberately not created: its only other resident,
> mul_u64_u32_add_u64_shr(), has no U-Boot user.
>
> No functional change: the generated code is identical, verified by
> comparing objdump -d of lib/div64.o before and after for
> evb-rk3288-rk808_defconfig. The compiler was already inlining the sole
> call.
>
> iter_div_u64_rem() itself has no in-tree callers either, but it is
> upstream API and is left in place.
>
> Signed-off-by: Alexey Charkov <[email protected]>
>
> include/linux/math64.h | 19 -------------------
>  lib/div64.c            | 17 ++++++++++++++++-
>  2 files changed, 16 insertions(+), 20 deletions(-)

Reviewed-by: Simon Glass <[email protected]>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.