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.