Re: [PATCH v3 4/4] ASoC: Intel: rename SND_SST_MFLD_PLATFORM to SND_SST_MID_PLATFORM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 1/4/17 4:44 AM, Andy Shevchenko wrote:
On Tue, 2017-01-03 at 08:18 -0600, Pierre-Louis Bossart wrote:
On 01/02/2017 09:20 AM, Andy Shevchenko wrote:
Rename SND_SST_MFLD_PLATFORM to SND_SST_MID_PLATFORM to make it
clear that is
not only about Medfield platform.

Does this change make sense? Medfield was a completely different
platform and lumping everything under the same Kconfig option
doesn't seem right to me... Plus we've stopped using 'mid' for
years, no one would know what it stands for. The only clear
architecture feature is the presence of the switch matrix that
uniquely identifies all platforms based on Baytrail, Cherrytrail,
Merrifield and Anniedale. Medfield/Clovertrail did not expose a
switch matrix.

Mea culpa..

So we started this with medfield and the driver construction is
pretty much
the same untill skylake. So clubbing under mfld bucket made sense to
me.

MID is not flavour of the season now.

Would be, though it covers at least few out of them, MFLD just refers to
one.


If anyone comes with a better name, am for that :)

_ATOM ? Sounds too broad...


SND_SST_HIFI2_MOBILE_PLATFORM ?

Where HIFI2 part came from? Any documentation (maybe internal) available
to look at? (Because it's not obvious to me how all listed above has
something in common)

And I would get rid of _MOBILE part.

SND_SST_ATOM_HIFI2_PLATFORM then

HIFI2 is the DSP version, it's public information for those Field/Trail parts, see https://www.alsa-project.org/main/index.php/Firmware If you combine HIFI2 with ATOM you have a unique non-ambiguous description of the core+DSP hardware for Medfield through Cherrytrail.


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux