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

2017-08-17 Thread Rob Herring
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. > > Signed-off-by: Jeffy Chen > --- > > Changes in v2: > Let rockchip,codec-names be a required property. > >

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

2017-08-17 Thread Rob Herring
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. > > Signed-off-by: Jeffy Chen > --- > > Changes in v2: > Let rockchip,codec-names be a required property. > >

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

2017-08-10 Thread jeffy
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

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

2017-08-10 Thread jeffy
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

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

2017-08-10 Thread Mark Brown
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... signature.asc Description: PGP signature

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

2017-08-10 Thread Mark Brown
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... signature.asc Description: PGP signature