From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mail.openembedded.org (Postfix) with ESMTP id E268C61FE1 for ; Tue, 9 Jul 2013 23:32:22 +0000 (UTC) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP; 09 Jul 2013 16:32:23 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.87,1031,1363158000"; d="scan'208";a="362835036" Received: from unknown (HELO [10.255.13.73]) ([10.255.13.73]) by fmsmga001.fm.intel.com with ESMTP; 09 Jul 2013 16:33:25 -0700 Message-ID: <51DC9D87.1000604@linux.intel.com> Date: Tue, 09 Jul 2013 16:32:23 -0700 From: Saul Wold User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 MIME-Version: 1.0 To: Darren Hart References: <51DC920E.1040305@linux.intel.com> <1373410775.3429.19.camel@envy.home> In-Reply-To: <1373410775.3429.19.camel@envy.home> Cc: Koen Kooi , openembedded-core@lists.openembedded.org Subject: Re: [PATCH] alsa-state: Provide an empty asound.conf by default X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Jul 2013 23:32:24 -0000 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit 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 > 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. 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 >>> Cc: Koen Kooi >>> --- >>> 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 >>> >