Message ID | 20200429084740.2665893-1-arnd@arndb.de |
---|---|
State | Not Applicable |
Delegated to: | David Miller |
Headers | show |
Series | dpaa2-eth: debugfs: use div64_u64 for division | expand |
> Subject: [PATCH] dpaa2-eth: debugfs: use div64_u64 for division > > A plain 64-bit division breaks building on 32-bit architectures: > > ERROR: modpost: "__aeabi_uldivmod" > [drivers/net/ethernet/freescale/dpaa2/fsl-dpaa2-eth.ko] undefined! > > As this function is not performance critical, just use the external helper instead. > > Fixes: 460fd830dd9d ("dpaa2-eth: add channel stat to debugfs") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Thanks for the patch. A fix for this was applied some hours ago on the net-next branch. -- Ioana > --- > drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c > b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c > index 80291afff3ea..0a31e4268dfb 100644 > --- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c > +++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c > @@ -139,7 +139,7 @@ static int dpaa2_dbg_ch_show(struct seq_file *file, void > *offset) > ch->stats.dequeue_portal_busy, > ch->stats.frames, > ch->stats.cdan, > - ch->stats.frames / ch->stats.cdan, > + div64_u64(ch->stats.frames, ch->stats.cdan), > ch->buf_count); > } > > -- > 2.26.0
diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c index 80291afff3ea..0a31e4268dfb 100644 --- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c +++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c @@ -139,7 +139,7 @@ static int dpaa2_dbg_ch_show(struct seq_file *file, void *offset) ch->stats.dequeue_portal_busy, ch->stats.frames, ch->stats.cdan, - ch->stats.frames / ch->stats.cdan, + div64_u64(ch->stats.frames, ch->stats.cdan), ch->buf_count); }
A plain 64-bit division breaks building on 32-bit architectures: ERROR: modpost: "__aeabi_uldivmod" [drivers/net/ethernet/freescale/dpaa2/fsl-dpaa2-eth.ko] undefined! As this function is not performance critical, just use the external helper instead. Fixes: 460fd830dd9d ("dpaa2-eth: add channel stat to debugfs") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/net/ethernet/freescale/dpaa2/dpaa2-eth-debugfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)