Re: [PATCH v5 1/2] ASoC: qcom: document apq8016 sbc machine driver bindings

From: Srinivas Kandagatla
Date: Tue Jun 09 2015 - 13:08:39 EST




On 09/06/15 17:57, Mark Brown wrote:
On Tue, Jun 09, 2015 at 01:59:29PM +0100, Srinivas Kandagatla wrote:

>+Optional CPU/CODEC subnodes properties:
>+- external : flag to indicate if the I2S is connected to external codec
>+Example:
Missing blank line between the property and the "Example:". I'm still
I will fix it in next version.
not sure I understand why we need a boolean property indicating if an
external CODEC is in use - what is the consequence of setting this
property?
As of today, the consequence of setting this flag is to setup correct dai_link names.
Also there are some limitations on which MI2S can be configured to external or internal codecs, this flag can be used in future to validate such configurations, if required.

--srini
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/