Re: [PATCH] alsa-state: Provide an empty asound.conf by default

[Darren Hart <dvhart@linux.intel.com>]

On Tue, 2013-07-09 at 16:32 -0700, Saul Wold wrote:
> On 07/09/2013 03:59 PM, Darren Hart wrote:
> > On Tue, 2013-07-09 at 15:43 -0700, Saul Wold wrote:
> >> On 07/09/2013 03:15 PM, Darren Hart wrote:
> >>> An empty asound.conf should work for most cases. The existing asound.conf
> >>> uses plug dmix which causes arecord to fail without a special -D device
> >>> specification on the command line (as no capture device is defined).
> >>>
> >>> Clear asound.conf for the default case. Any needed changes should be
> >>> made in the BSP layers.
> >>>
> >> Does there need to be any corresponding change for the meta-yocto-bsps?
> >
> > I don't believe so, but I have not built and tested every one. The old
> > one setup an alias "mixer0" to hardware card 0, which isn't necessary
> > for alsamixer to work and should be the default anyway.
> >
> > I don't know what the motivation was to use dmix for the default pcm.
> > This was pulled in from open-embedded, which has a minimal commit
> > message:
> >
> > commit bbc2776c0d955da7ec774ae934c541f33133f876
> > Author: Michael Lauer <mickey@vanille-media.de>
> > Date:   Thu Sep 13 01:10:23 2007 +0000
> >
> Given it's 5 years old, it might be that alsa now "does the right thing" 
> and your change is valid.

Apparently Angstrom has been doing this for a while now. Koen, do you
agree this is the right thing for oe-core?

I missed the PR bump... I can't remember, do we do that anymore?

--
Darren

> 
> Thanks
> 	Sau!
> 
> >      alsa-state: ship standard asoundrc (simple dmix configuration)
> >      that should be a reasonable default for most devices
> >
> > According to the alsa wiki, the asound.conf should not be required for
> > basic functionality:
> >
> > http://www.alsa-project.org/main/index.php/Asoundrc
> >
> > --
> > Darren
> >
> >>
> >>
> >> Sau!
> >>
> >>
> >>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> >>> Cc: Koen Kooi <koen@dominion.thruhere.net>
> >>> ---
> >>>    meta/recipes-bsp/alsa-state/alsa-state/asound.conf | 13 +------------
> >>>    1 file changed, 1 insertion(+), 12 deletions(-)
> >>>
> >>> diff --git a/meta/recipes-bsp/alsa-state/alsa-state/asound.conf b/meta/recipes-bsp/alsa-state/alsa-state/asound.conf
> >>> index 6be0242..03653cf 100644
> >>> --- a/meta/recipes-bsp/alsa-state/alsa-state/asound.conf
> >>> +++ b/meta/recipes-bsp/alsa-state/alsa-state/asound.conf
> >>> @@ -1,12 +1 @@
> >>> -# default dmix configuration
> >>> -
> >>> -pcm.!default {
> >>> -    type plug
> >>> -    slave.pcm "dmix"
> >>> -}
> >>> -
> >>> -ctl.mixer0 {
> >>> -    type hw
> >>> -    card 0
> >>> -}
> >>> -
> >>> +# Global alsa-lib configuration
> >>>
> >

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Technical Lead - Linux Kernel