[U-Boot] [PATCH 4/5] nios2: Enable support for fitImage

Thomas Chou
2015-12-16T00:13:15Z

Hi Marek,

On 2015?12?16? 07:18, Marek Vasut wrote:
> On Tuesday, December 15, 2015 at 02:26:44 PM, Thomas Chou wrote:
>> Hi Marek,
>>
>> On 2015?12?15? 18:26, Marek Vasut wrote:
>>> On Tuesday, December 15, 2015 at 06:31:11 AM, Thomas Chou wrote:
>>>> Hi Marek,
>>>>
>>>> On 2015?12?15? 10:09, Marek Vasut wrote:
>>>>> The uImage format is legacy for years now, enable support for the
>>>>> fitImage format, which allows combining multiple files (kernel and
>>>>> dtb) into a single file, offers better protection of the payload
>>>>> and so on.
>>>>>
>>>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>>>> Cc: Thomas Chou <thomas@wytron.com.tw>
>>>>> ---
>>>>>
>>>>> include/configs/10m50_devboard.h | 2 ++
>>>>> include/configs/3c120_devboard.h | 2 ++
>>>>> 2 files changed, 4 insertions(+)
>>>>>
>>>>> diff --git a/include/configs/10m50_devboard.h
>>>>> b/include/configs/10m50_devboard.h index 608127f..0aa79f7 100644
>>>>> --- a/include/configs/10m50_devboard.h
>>>>> +++ b/include/configs/10m50_devboard.h
>>>>> @@ -14,6 +14,8 @@
>>>>>
>>>>> */
>>>>>
>>>>> #define CONFIG_DISPLAY_CPUINFO
>>>>> #define CONFIG_DISPLAY_BOARDINFO_LATE
>>>>>
>>>>> +#define CONFIG_OF_LIBFDT
>>>>
>>>> Already defined.
>>>>
>>>>> +#define CONFIG_FIT
>>>>>
>>>>> /*
>>>>>
>>>>> * SERIAL
>>>>>
>>>>> diff --git a/include/configs/3c120_devboard.h
>>>>> b/include/configs/3c120_devboard.h index 6c9dc3f..0054d8b 100644
>>>>> --- a/include/configs/3c120_devboard.h
>>>>> +++ b/include/configs/3c120_devboard.h
>>>>> @@ -14,6 +14,8 @@
>>>>>
>>>>> */
>>>>>
>>>>> #define CONFIG_DISPLAY_CPUINFO
>>>>> #define CONFIG_DISPLAY_BOARDINFO_LATE
>>>>>
>>>>> +#define CONFIG_OF_LIBFDT
>>>>
>>>> Ditto.
>>>>
>>>>> +#define CONFIG_FIT
>>>>>
>>>>> /*
>>>>>
>>>>> * SERIAL
>>>
>>> The compiler is not complaining, is it really ?
>>
>> Hmm, it might be the compiler's problem.
>>
>> It might be better to move CONFIG_FIT to the misc (the last) config
>> section.
>
> Apparently, if the macro is defined in the same file twice and it is the same,
> CPP will not complain.
>

Please note that CONFIG_FIT is moved to Kconfig. So it should be added
to defcofnig.

Best regards,
Thomas