From mboxrd@z Thu Jan 1 00:00:00 1970 From: york sun Date: Wed, 4 Jan 2017 21:41:53 +0000 Subject: [U-Boot] [PATCH] arm: layerscape: Enable UUID & GPT partition for NXP's ARM SoC References: <1482734708-23418-1-git-send-email-prabhakar.kushwaha@nxp.com> <757137c1-6823-cd5a-28ee-3bb7c2b3c61a@compulab.co.il> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >> --- >> 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. York