Re: [PATCH v2 0/5] ASoC: Intel: ACPI support and machines

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

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.

-- 
jarkko