All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jörg Krause" <joerg.krause@embedded.rocks>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] alsa-lib: Add option to allow calculations in float for soft float targets
Date: Sun, 08 May 2016 14:33:22 +0200	[thread overview]
Message-ID: <1462710802.22644.13.camel@embedded.rocks> (raw)
In-Reply-To: <20160507091654.0b03f2fd@free-electrons.com>

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.

[1]
http://git.alsa-project.org/?p=alsa-lib.git;a=blob;f=src/pcm/pcm_softvo
l.c;h=802aa4b7cb68e444b8b47e7140870141aad65ffc;hb=HEAD#l785

Best regards
J?rg Krause

  reply	other threads:[~2016-05-08 12:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-07  6:45 [Buildroot] [PATCH] alsa-lib: Add option to allow calculations in float for soft float targets Jörg Krause
2016-05-07  7:16 ` Thomas Petazzoni
2016-05-08 12:33   ` Jörg Krause [this message]
2016-05-08 19:53     ` Arnout Vandecappelle
2016-05-09  7:38       ` Jörg Krause
2016-07-04 19:48 ` Peter Korsgaard

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=1462710802.22644.13.camel@embedded.rocks \
    --to=joerg.krause@embedded.rocks \
    --cc=buildroot@busybox.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.