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 069EC6204F for ; Wed, 10 Jul 2013 16:52:09 +0000 (UTC) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 10 Jul 2013 09:52:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.87,1037,1363158000"; d="scan'208";a="343455435" Received: from envy.jf.intel.com (HELO [10.7.199.52]) ([10.7.199.52]) by orsmga001.jf.intel.com with ESMTP; 10 Jul 2013 09:52:04 -0700 Message-ID: <1373475121.3028.23.camel@envy.home> From: Darren Hart To: Saul Wold Date: Wed, 10 Jul 2013 09:52:01 -0700 In-Reply-To: <51DC9D87.1000604@linux.intel.com> References: <51DC920E.1040305@linux.intel.com> <1373410775.3429.19.camel@envy.home> <51DC9D87.1000604@linux.intel.com> Organization: Intel X-Mailer: Evolution 3.8.3 (3.8.3-2.fc19) Mime-Version: 1.0 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: Wed, 10 Jul 2013 16:52:09 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit 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 > > 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 > >>> 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 > >>> > > -- Darren Hart Intel Open Source Technology Center Yocto Project - Technical Lead - Linux Kernel