On 11/04/2014 07:31 AM, Vinod Koul wrote:
This is second rev of patch series which fixes the modules split as discussed in ML. The core part is now module along with PCI and ACPI one. Added fix reported by Dan as well. Also added the BYTCR machine driver
Subhransu S. Prusty (1): ASoC: Intel: add BYTCR machine driver with RT5640
Vinod Koul (4): ASoC: Intel: mrfld - remove unnecessary check for pointer ASoC: Intel: mrfld - create separate module for pci part ASoC: Intel: mrfld - add shim save restore ASoC: Intel: mrfld- add ACPI module
I think there is need to have some consolidation with current byt-rt5640 and byt-max98090. Since existing drivers and this set are using the same ACPI IDs "80860F28" for the ADSP and "10EC5640" codec ACPIID for the machine driver probing these drivers won't work together.
One difference at the moment is that byt-rt5640 machines out there have differences in audio routings but those are not too difficult to handle in the same machine driver.
Bigger question how to handle platform part? I.e. can these use the same drivers and ADSP firmware? If they can, is the firmware available for distributions to take?
Optimally all machines out there would use the same drivers and get support for compressed audio but meanwhile we should not break existing machines.