Hi Andy,
On Wed, Feb 15, 2017 at 6:15 PM, Andy Shevchenko andriy.shevchenko@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@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@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