[U-Boot] [PATCH v1 1/3] x86: Introduce INTEL_MID quirk option

[Bin Meng]

Hi Andy,

On Wed, Feb 15, 2017 at 6:15 PM, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
> On Wed, 2017-02-15 at 18:09 +0800, Bin Meng wrote:
>> Hi Andy,
>>
>> On Wed, Feb 15, 2017 at 5:50 PM, Andy Shevchenko
>> <andriy.shevchenko at linux.intel.com> wrote:
>> > On Wed, 2017-02-15 at 11:00 +0800, Bin Meng wrote:
>> > > On Tue, Feb 14, 2017 at 10:47 PM, Andy Shevchenko
>> > > <andriy.shevchenko at linux.intel.com> wrote:
>
>> > > What's the architecture of Intel Edison? Normally such platform
>> > > Kconfig option should go to arch/x86/cpu/<platform>/Kconfig
>> >
>> > What do you mean?
>> > x86_64.
>>
>> Sorry I should say what the platform is, like Ivybridge, Baytrail,
>> Quark that U-Boot currently supports.
>
> It's misunderstanding here.
> arch/x86/cpu/<SoC/CPU/platform>/
> So, all of above either SoC or platform. A little mess.
>

Yes, it's either SoC (single chipset), or platform (CPU plus platform
controller).

> So, we will put the code to tangier folder because Intel Tangier is SoC.
> Platform is Intel Merrifield.
>
>>  If this is a new platform,
>
> Yes.
>
>>  we
>> should probably move it to arch/x86/cpu/<platform>/Kconfig.
>
> Probably not. Intel MID covers more than one SoC/platform. This option
> is x86 scope.
>

OK, so can you please add a help text for the new config to describe
what MID is and its features? I assume it has some difference that
worth introducing such a sub-arch option.

> I dunno that U-Boot will run on the rest MID devices, but for sake of
> logic and consistency it is not about Edison or Merrifield or Tangier.

Regards,
Bin