[meta-freescale] [meta-fsl-arm][PATCH] alsa-state: Set default state values for mx6/mx6ul/mx7

[Yuqing Zhu]

Hi Otavio,

      Thanks for your comments.

       Here we just want to use one unified version of asound.state and asound.conf to cover mx6/mx6ul/mx7 and it is added
       into our own meta-fsl-arm layer.
    
       Actually I am not very clear about why it will cause being included in another build (e.g. Intel, Marvell, Texas ones). 
       Will they use our bsp ?



B.R.
Carol

-----Original Message-----
From: Otavio Salvador [mailto:otavio.salvador at ossystems.com.br] 
Sent: Saturday, February 27, 2016 11:27 PM
To: Yuqing Zhu <carol.zhu at nxp.com>
Cc: meta-freescale at yoctoproject.org
Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH] alsa-state: Set default state values for mx6/mx6ul/mx7

On Thu, Feb 25, 2016 at 5:57 AM, Yuqing Zhu <carol.zhu at nxp.com> wrote:
> *asound.conf: Update asound.conf to set default ctrl for kernel 4.1  
> The default ctrl is missing from /etc/asound.conf.
>  If don't using “-Dhw:0”, the sound will rout from alsa to pulseaudio,  
> then to the sound device.
>
> *asound.state: Using one unified file to cover mx6/mx6ul/mx7
>  wm8958/wm8960 audio configuration for mx6ul/mx7
>  wm8962 audio configuration for mx6
>
> Signed-off-by: Yuqing Zhu <carol.zhu at nxp.com>
> ---
>  recipes-bsp/alsa-state/alsa-state.bbappend         |    3 +-
>  .../alsa-state/alsa-state/{mx6 => }/asound.conf    |    5 +
>  .../alsa-state/alsa-state/{mx7 => }/asound.state   |  800 +++++++++---
>  recipes-bsp/alsa-state/alsa-state/mx6/asound.state | 1374 
> --------------------  
> recipes-bsp/alsa-state/alsa-state/mx7/asound.conf  |  257 ----
>  5 files changed, 638 insertions(+), 1801 deletions(-)  rename 
> recipes-bsp/alsa-state/alsa-state/{mx6 => }/asound.conf (98%)  mode 
> change 100644 => 100755  rename recipes-bsp/alsa-state/alsa-state/{mx7 
> => }/asound.state (90%)  mode change 100644 => 100755  delete mode 
> 100644 recipes-bsp/alsa-state/alsa-state/mx6/asound.state
>  delete mode 100644 recipes-bsp/alsa-state/alsa-state/mx7/asound.conf
>
> diff --git a/recipes-bsp/alsa-state/alsa-state.bbappend 
> b/recipes-bsp/alsa-state/alsa-state.bbappend
> index dbfed38..915c897 100644
> --- a/recipes-bsp/alsa-state/alsa-state.bbappend
> +++ b/recipes-bsp/alsa-state/alsa-state.bbappend
> @@ -1,5 +1,4 @@
>  # Append path for freescale layer to include alsa-state asound.conf  
> FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>
> -PACKAGE_ARCH_mx7 = "${MACHINE_ARCH}"
> -PACKAGE_ARCH_mx6 = "${MACHINE_ARCH}"
> +PACKAGE_ARCH = "${MACHINE_ARCH}"

You can't drop the package override here, neither consolidate them in a single directory as we don't want those to be included in another build (e.g. Intel, Marvell, Texas ones).

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750