[Buildroot] [PATCH] alsa-lib: Add option to allow calculations in float for soft float targets

Jörg Krause joerg.krause at embedded.rocks
Mon May 9 07:38:21 UTC 2016


On So, 2016-05-08 at 21:53 +0200, Arnout Vandecappelle wrote:
> On 05/08/16 14:33, Jörg Krause wrote:
> > On Sa, 2016-05-07 at 09:16 +0200, Thomas Petazzoni wrote:
> > > Hello,
> > > 
> > > On Sat,  7 May 2016 08:45:25 +0200, Jörg Krause wrote:
> > > 
> > > > -ifeq ($(BR2_SOFT_FLOAT),y)
> > > > +ifeq ($(BR2_PACKAGE_ALSA_LIB_NO_FLOAT),y)
> > > >  ALSA_LIB_CONF_OPTS += --with-softfloat
> > > >  endif
> > > 
> > > So we were already passing --with-softfloat when
> > > BR2_SOFT_FLOAT=y. So
> > > what your option brings is:
> > > 
> > >  - Ability to use floating point operations when
> > > BR2_SOFT_FLOAT=y.
> > > 
> > >  - Ability to not use floating point operations when
> > > BR2_SOFT_FLOAT
> > > is
> > >    disabled.
> > > 
> > > Do you have any actual use for such possibilities ?
> > 
> > Yes, for the first case.
> > 
> > I want to use a feature from the alsa-lib which is opted-out at
> > compile
> > time for softfloat targets by #ifndef HAVE_SOFT_FLOAT [1]. This
> > functionality requires floating point data types and libmath
> > operations.
> > 
> > I do not see any point in disabling this unconditionally for
> > softfloat
> > targets, as a software floating point library can emulate this.
> 
>   The question is: is there a point in disabling it at all, ever?
> Does it have a 
> big impact on size, or does it require functionality which may not be
> available 
> for a particular target arch?

No, it does not. Setting softfloat only disables functionality using
floating point types and routines.

>   If not, I'd vote for unconditionally disabling the softfloat
> option.

I agree. However, this changes the current behaviour how alsa-lib is
build for soft-float targets. That's why I added an option which
preserves the current configuration.

>   That said, I remember that there was some option (not sure if it
> was alsalib 
> or another lib) to choose between floating point (accurate but
> slower) and fixed 
> point (less accurate but faster) for some filtering operations.
> *That* would be 
> a relevant user-settable option.

Yes, but it does not apply to alsa-lib.

Best regards
Jörg Krause



More information about the buildroot mailing list