linux-kernel - Re: [PATCH 1/8] ASoC: fsl: Add VF610 soc audio card Kconfig

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140123183829.GB11727@sirena.org.uk>
Date:	Thu, 23 Jan 2014 18:38:29 +0000
From:	Mark Brown <broonie@...nel.org>
To:	Xiubo Li <Li.Xiubo@...escale.com>
Cc:	lgirdwood@...il.com, shawn.guo@...aro.org,
	kuninori.morimoto.gx@...esas.com, moinejf@...e.fr,
	alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/8] ASoC: fsl: Add VF610 soc audio card Kconfig

On Thu, Jan 23, 2014 at 01:02:43PM +0800, Xiubo Li wrote:

> +menuconfig SND_VF610_SOC
> +       tristate "SoC Audio for Freescale VF610 CPUs"
> +       select DMA_ENGINE
> +       help
> +         Say Y or M if you want to add support for codecs attached to
> +         the VF610 CPUs.
> +
> +         This will enable Freeacale SAI and SGTL5000 codec, and an extra
> +         TWR-AUDIO-SGTL sub-board is needed for SGTL5000.

This bit is OK but...

> +config SND_SOC_VF610_SGTL5000
> +       tristate "SoC Audio support for VF610 boards with SGTL5000"
> +       depends on OF && I2C
> +       select SND_SOC_FSL_SAI
> +       select SND_SOC_SGTL5000
> +       select SND_SIMPLE_CARD

...for simple card since the whole idea is to support any CODEC with the
same driver we should probably just make sure that all the individual
drivers can be enabled in Kconfig, that way we don't have to have
specific Kconfig entries for boards and loose some of the benefit of the
generic card.  I sent a patch earlier exposing the OF supporting CODEC
drivers, one for at least some of the Freescale CPU drivers was sent to
the list recently too but there were some review comments.

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)