Re: [PATCH v2 3/3] dt-bindings: ASoC: rockchip: Add rockchip,codec-names property

From	jeffy <jeffy.chen@rock-chips.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 3/3] dt-bindings: ASoC: rockchip: Add rockchip,codec-names property
Date	2017-08-11 03:40 +0200
Message-ID	<ud75E-5qQ-13@gated-at.bofh.it> (permalink)
References	<ucNJE-Jv-5@gated-at.bofh.it> <ucNJD-Jv-3@gated-at.bofh.it> <ucX6h-723-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Mark,

On 08/10/2017 10:56 PM, Mark Brown wrote:
> On Thu, Aug 10, 2017 at 12:54:58PM +0800, Jeffy Chen wrote:
>> Add a new rockchip,codec-names property, so that the driver can parse
>> the codecs by name.
>
> Why?  You're already referencing the CODECs by phandle and these names
> are not part of any ABI...
>

currently we are binding the phandles to a fixed codec list in the 
driver. but we want to make it dynamic, since some exist codecs could be 
optional, and some new codecs could be add for new board too.

and the support of this new property in the driver is added in 
https://patchwork.kernel.org/patch/9892737

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v2 3/3] dt-bindings: ASoC: rockchip: Add rockchip,codec-names property Jeffy Chen <jeffy.chen@rock-chips.com> - 2017-08-10 07:00 +0200
  Re: [PATCH v2 3/3] dt-bindings: ASoC: rockchip: Add  rockchip,codec-names property Mark Brown <broonie@kernel.org> - 2017-08-10 17:00 +0200
    Re: [PATCH v2 3/3] dt-bindings: ASoC: rockchip: Add rockchip,codec-names  property jeffy <jeffy.chen@rock-chips.com> - 2017-08-11 03:40 +0200

csiph-web