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

[Vinod Koul <vinod.koul@intel.com>]

On Tue, Nov 04, 2014 at 10:09:40AM +0200, Jarkko Nikula wrote:
> 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.
Liam's suggestion was to remove the current driver when all the bits are
upstream

> 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?
The two firmware are *not* compatible and that is the big reason why we
can't reuse the parts available

> Optimally all machines out there would use the same drivers and get
> support for compressed audio but meanwhile we should not break
> existing machines.
Yes, I have tested this one on T100 and doesn't seem to break anything.
Please do test and revert if you find issues on any other available board (I
don't have any other board)

-- 
~Vinod