[U-Boot] [PATCH] arm: layerscape: Enable UUID & GPT partition for NXP's ARM SoC

[Igor Grinberg]

On 01/05/17 18:25, york sun wrote:
> On 01/05/2017 01:51 AM, Igor Grinberg wrote:
>> On 01/04/17 23:41, york sun wrote:
>>> On 12/25/2016 11:29 PM, Igor Grinberg wrote:
>>>> Hi Prabhakar Kushwaha,
>>>>
>>>> On 12/26/16 08:45, Prabhakar Kushwaha wrote:
>>>>> Enable UUID and GPT partition support for NXP's ARM based SoCs
>>>>> i.e. LS1012A, LS1021A, LS1043A, LS1046A and LS2080A.
>>>>>
>>>>> Also enable DOS partition for LS1012AFRDM boards.
>>>>>
>>>>> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>
>>>>> ---
>>>>>  include/configs/ls1012afrdm.h | 5 +++++
>>>>>  include/configs/ls1012aqds.h  | 4 ++++
>>>>>  include/configs/ls1012ardb.h  | 4 ++++
>>>>>  include/configs/ls1021aiot.h  | 3 +++
>>>>>  include/configs/ls1021aqds.h  | 3 +++
>>>>>  include/configs/ls1021atwr.h  | 3 +++
>>>>>  include/configs/ls1043aqds.h  | 4 ++++
>>>>>  include/configs/ls1043ardb.h  | 4 ++++
>>>>>  include/configs/ls1046aqds.h  | 4 ++++
>>>>>  include/configs/ls1046ardb.h  | 4 ++++
>>>>>  include/configs/ls2080aqds.h  | 3 +++
>>>>>  include/configs/ls2080ardb.h  | 3 +++
>>>>
>>>> I'm not sure this is the expected way today...
>>>>
>>>>>  12 files changed, 44 insertions(+)
>>>>>
>>>>> diff --git a/include/configs/ls1012afrdm.h b/include/configs/ls1012afrdm.h
>>>>> index f6f88e8..94f7460 100644
>>>>> --- a/include/configs/ls1012afrdm.h
>>>>> +++ b/include/configs/ls1012afrdm.h
>>>>> @@ -42,6 +42,11 @@
>>>>>  #define CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS      2
>>>>>  #endif
>>>>>
>>>>> +#define CONFIG_DOS_PARTITION
>>>>> +#define CONFIG_PARTITION_UUIDS
>>>>> +#define CONFIG_EFI_PARTITION
>>>>> +#define CONFIG_CMD_GPT
>>>>> +
>>>>
>>>> These are the common (not board specific) things and I think today we expect
>>>> it to come through the Kconfig instead of the old config files.
>>>
>>> Igor,
>>>
>>> They are not totally common. The board has to support the device. I
>>> would recommend to use Kconfig as well if such config is in Kconfig.
>>> However it is not the case here.
>>
>> Right. They are not in Kconfig yet...
>> We are making efforts to move stuff to Kconfig.
>> Care to land a hand and move these ones to Kconfig?
>>
> 
> Can't promise. I have been doing just that for powerpc. Still have many 
> left.

Yeah, great job on those patches! Thanks!
I think it is still worth waiting instead of applying and moving afterwards...

-- 
Regards,
Igor.