From: Peng Fan <van.freenix@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] video: ipu: Fix build with hard-float ARM toolchain
Date: Thu, 19 May 2016 09:09:39 +0800 [thread overview]
Message-ID: <20160519010937.GA5356@linux-7smt.suse> (raw)
In-Reply-To: <1463573983-18597-1-git-send-email-guillaume.gardet@free.fr>
Hi Guillaume,
On Wed, May 18, 2016 at 02:19:43PM +0200, Guillaume GARDET wrote:
>Build with hard-float ARM toolchain was broken by commit 3cb4f25c:
>'video: ipu: avoid overflow issue'.
>To fix it, we use do_div function.
>
>Build tested for mx6qsabrelite_defconfig.
>
>Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
>Cc: Peng Fan <van.freenix@gmail.com>
>Cc: Sandor Yu <sandor.yu@nxp.com>
>Cc: Anatolij Gustschin <agust@denx.de>
>Cc: Stefano Babic <sbabic@denx.de>
>Cc: Fabio Estevam <fabio.estevam@nxp.com>
>Cc: Tom Rini <trini@konsulko.com>
>
>---
> drivers/video/ipu_common.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c
>index 36d4b23..e319c82 100644
>--- a/drivers/video/ipu_common.c
>+++ b/drivers/video/ipu_common.c
>@@ -352,7 +352,8 @@ static int ipu_pixel_clk_set_rate(struct clk *clk, unsigned long rate)
> */
> __raw_writel((div / 16) << 16, DI_BS_CLKGEN1(clk->id));
>
>- clk->rate = (u64)(clk->parent->rate * 16) / div;
>+ do_div(parent_rate, div);
>+ clk->rate = parent_rate;
This issue was already fixed,.
http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commitdiff;h=c510f2e436008e55a50b063f2180cb1e63984224
Thanks,
Peng.
>
> return 0;
> }
>--
>1.8.4.5
>
next prev parent reply other threads:[~2016-05-19 1:09 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-18 12:19 [U-Boot] [PATCH] video: ipu: Fix build with hard-float ARM toolchain Guillaume GARDET
2016-05-19 1:09 ` Peng Fan [this message]
2016-05-19 7:40 ` Guillaume Gardet
2016-05-19 10:27 ` Stefano Babic
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160519010937.GA5356@linux-7smt.suse \
--to=van.freenix@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox