[U-Boot] [PATCH 1/2] ddr: altera: Replace float multiplication with integer one

Marek Vasut marex at denx.de
Wed Aug 19 00:07:26 CEST 2015


On Tuesday, August 18, 2015 at 10:13:42 PM, Dinh Nguyen wrote:
> On 8/10/15 5:59 PM, Marek Vasut wrote:
> > This gem is really really rare, there was an actual float used in
> > the Altera DDR init code, which pulled in floating point ops from
> > the libgcc, just wow.
> > 
> > Since we don't support floating point operations the same way Linux
> > does not support them, replace this with an integer multiplication
> > and division combo. This removes some 2kiB of size from the SPL as
> > the floating point ops are no longer pulled in from libgcc.
> > 
> > This was detected by enabling CONFIG_USE_PRIVATE_LIBGCC=y , which
> > does not contain the floating point bits.

Private libgcc is indeed helpful! :-)

> > Signed-off-by: Marek Vasut <marex at denx.de>
> > ---
> > 
> >  drivers/ddr/altera/sequencer.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/ddr/altera/sequencer.c
> > b/drivers/ddr/altera/sequencer.c index 2bd0109..f3621cf 100644
> 
> Acked-by: Dinh Nguyen <dinguyen at opensource.altera.com>

Applied to u-boot-socfpga/master, thanks!

Best regards,
Marek Vasut


More information about the U-Boot mailing list