Re: [PATCH] ASoC: add RT5639 CODEC driver

[Jarkko Nikula <jarkko.nikula@linux.intel.com>]

On 03/26/2014 04:57 AM, Oder Chiou wrote:
>> -----Original Message-----
>> From: Mark Brown [mailto:broonie@kernel.org]
>> Sent: Wednesday, March 26, 2014 12:58 AM
>> To: Stephen Warren
>> Cc: Oder Chiou; Bard Liao; alsa-devel@alsa-project.org; lgirdwood@gmail.com;
>> Flove
>> Subject: Re: [alsa-devel] [PATCH] ASoC: add RT5639 CODEC driver
>>
>> On Tue, Mar 25, 2014 at 09:33:42AM -0600, Stephen Warren wrote:
>>> FWIW, I was informed internally (although this certainly isn't
>>> normative) that RT5639 is RT5640 plus a DSP, and otherwise compatible.
>>> Related, I have a board that has a RT5639 and I'm running the RT5640
>>> driver on it for at least a simple use-case, without issue.
> The RT5639 is a simplified version of RT5640. There was only a pair of DACs and there was no MONO out.
> The RT5639 and RT5640 didn't have the DSP function, and the RT5642 was the RT5640 plus a DSP.
> In our side, the Tegra4 was broken, and it also was EOL in your company.
> So, do you have the code base of Tegra5 which can use for verifying the upsteam driver?
>
FWIW, we are using RT5640 upstream driver on Intel reference designs 
that have RT5642 so if those DSP features are going to be supported some 
day I propose to have them implemented in a same driver.

sound/soc/codecs/tlv320aic3x.c comes to my mind how to implement support 
for different codec variants in the same driver.

-- 
Jarkko