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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 09/06/15 18:13, Mark Brown wrote:
On Tue, Jun 09, 2015 at 06:08:24PM +0100, Srinivas Kandagatla wrote:

+- external	: flag to indicate if the I2S is connected to external codec

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.

That validation sounds like something that the SoC drivers should be
doing if required rather than the machine driver - otherwise every
machine driver using this SoC would have to implement the same
validation.

Thats a valid point, for now I can move to using dai link name property instead of external flag, this should work for now. I will add the validation logic in SOC driver if required in future.

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux