[media.git/next] media: platform: synopsys: hdmirx: Fix 64-bit division for 32-bit targets

[media-ci]

On Thu Mar 6 22:45:16 2025 +0100, Nathan Chancellor wrote:
> The build fails for 32-bit targets with:
> 
>   arm-linux-gnueabi-ld: drivers/media/platform/synopsys/hdmirx/snps_hdmirx.o: 
> in function `hdmirx_get_timings':
>   snps_hdmirx.c:(.text.hdmirx_get_timings+0x46c): undefined reference to 
> `__aeabi_uldivmod'
> 
> bt->pixelclock is __u64, which causes the compiler to emit a libcall for
> 64-bit division. Use the optimized kernel helper, div_u64(), to resolve
> this.
> 
> Fixes: 7b59b132ad43 ("media: platform: synopsys: Add support for HDMI input 
> driver")
> Signed-off-by: Nathan Chancellor <nat...@kernel.org>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/synopsys/hdmirx/snps_hdmirx.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/synopsys/hdmirx/snps_hdmirx.c 
b/drivers/media/platform/synopsys/hdmirx/snps_hdmirx.c
index 4d42da7255f3..e4cdd8b5745d 100644
--- a/drivers/media/platform/synopsys/hdmirx/snps_hdmirx.c
+++ b/drivers/media/platform/synopsys/hdmirx/snps_hdmirx.c
@@ -17,6 +17,7 @@
 #include <linux/interrupt.h>
 #include <linux/irq.h>
 #include <linux/mfd/syscon.h>
+#include <linux/math64.h>
 #include <linux/module.h>
 #include <linux/of.h>
 #include <linux/of_platform.h>
@@ -291,7 +292,7 @@ static void hdmirx_get_timings(struct snps_hdmirx_dev 
*hdmirx_dev,
        hfp = htotal - hact - hs - hbp;
        vfp = vtotal - vact - vs - vbp;
 
-       fps = (bt->pixelclock + (htotal * vtotal) / 2) / (htotal * vtotal);
+       fps = div_u64(bt->pixelclock + (htotal * vtotal) / 2, htotal * vtotal);
        bt->width = hact;
        bt->height = vact;
        bt->hfrontporch = hfp;