[U-Boot] [PATCH 02/10] x86: ivybridge: Add FSP support

Bin Meng bmeng.cn at gmail.com
Mon Dec 21 03:27:09 CET 2015


Hi Simon,

On Sat, Dec 19, 2015 at 10:52 AM, Simon Glass <sjg at chromium.org> wrote:
> On 11 December 2015 at 03:55, Bin Meng <bmeng.cn at gmail.com> wrote:
>> IvyBridge FSP package is built with a base address at 0xfff80000,
>> and does not use UPD data region. This adds basic FSP support.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>  arch/x86/cpu/ivybridge/Kconfig                     |  8 ++++
>>  arch/x86/cpu/ivybridge/Makefile                    |  4 ++
>>  arch/x86/cpu/ivybridge/fsp_configs.c               | 45 ++++++++++++++++++++++
>>  arch/x86/cpu/ivybridge/ivybridge.c                 | 22 +++++++++++
>>  .../include/asm/arch-ivybridge/fsp/fsp_configs.h   | 40 +++++++++++++++++++
>>  arch/x86/include/asm/arch-ivybridge/fsp/fsp_vpd.h  | 12 ++++++
>>  6 files changed, 131 insertions(+)
>>  create mode 100644 arch/x86/cpu/ivybridge/fsp_configs.c
>>  create mode 100644 arch/x86/cpu/ivybridge/ivybridge.c
>>  create mode 100644 arch/x86/include/asm/arch-ivybridge/fsp/fsp_configs.h
>>  create mode 100644 arch/x86/include/asm/arch-ivybridge/fsp/fsp_vpd.h
>
> Acked-by: Simon Glass <sjg at chromium.org>
> Tested on link (ivybridge non-FSP)
> Tested-by: Simon Glass <sjg at chromium.org>
>
> One question: why do we need an Intel license on this code?
>

It was a copy and paste from other platform (queensbay and baytrail)
files. Do you think we should change it to GPLv2+?

Regards,
Bin


More information about the U-Boot mailing list