[U-Boot] [PATCH v2] ls1088ardb_pb: Add support for ls1088ardb_pb board.

[Pramod Kumar]

>-----Original Message-----
>From: York Sun
>Sent: Friday, September 21, 2018 8:20 PM
>To: Pramod Kumar <pramod.kumar_1 at nxp.com>
>Cc: u-boot at lists.denx.de
>Subject: Re: [PATCH v2] ls1088ardb_pb: Add support for ls1088ardb_pb
>board.
>
>On 08/21/2018 02:18 AM, Pramod Kumar wrote:
>> ls1088ardb-pb is another varinat of the ls1088ardb.
>> This board support two m.2 x2 gen 3 pcie card slot.
>> There is no support of sata, pcie standard slot, miniPCIE slot and TDM
>> in this board.
>>
>> Signed-off-by: Pramod Kumar <pramod.kumar_1 at nxp.com>
>> ---
>> Changes in v2:
>>  - Fix the compilation issue when building u-boot for SD boot.
>>
>>  arch/arm/Kconfig                              |  14 +++
>>  arch/arm/cpu/armv8/Kconfig                    |   1 +
>>  board/freescale/ls1088a/Kconfig               |  43 +++++++
>>  board/freescale/ls1088a/MAINTAINERS           |  18 +++
>>  board/freescale/ls1088a/Makefile              |   1 +
>>  board/freescale/ls1088a/ddr.h                 |   6 +-
>>  board/freescale/ls1088a/ls1088a.c             |  25 +++-
>>  ... ls1088ardb_pb_qspi_SECURE_BOOT_defconfig} |   2 +-
>>  ...defconfig => ls1088ardb_pb_qspi_defconfig} |   2 +-
>>  ...ardb_pb_sdcard_qspi_SECURE_BOOT_defconfig} |   2 +-
>>  ...ig => ls1088ardb_pb_sdcard_qspi_defconfig} |   2 +-
>>  include/configs/ls1088ardb_pb.h               | 113 ++++++++++++++++++
>>  12 files changed, 219 insertions(+), 10 deletions(-)  copy
>> configs/{ls1088ardb_qspi_SECURE_BOOT_defconfig =>
>> ls1088ardb_pb_qspi_SECURE_BOOT_defconfig} (97%)  copy
>> configs/{ls1088ardb_qspi_defconfig => ls1088ardb_pb_qspi_defconfig}
>> (97%)  copy configs/{ls1088ardb_sdcard_qspi_SECURE_BOOT_defconfig =>
>> ls1088ardb_pb_sdcard_qspi_SECURE_BOOT_defconfig} (98%)  copy
>> configs/{ls1088ardb_sdcard_qspi_defconfig =>
>> ls1088ardb_pb_sdcard_qspi_defconfig} (98%)  create mode 100644
>> include/configs/ls1088ardb_pb.h
>>
>
>Is it possible to reuse ls1088ardb and detect the board at runtime?
>>
>>For distro boot we need diffrent board variable for ls1088ardb and ls1088ardb_pb, which can only be 
>>achieved by using config tagets CONFIG_TARGET_LS1088ARDB and CONFIG_TARGET_LS1088ARDB_PB respectively 
f>> or both the boards.
><snip>
>
>> diff --git a/board/freescale/ls1088a/ddr.h
>> b/board/freescale/ls1088a/ddr.h index b35c4ae2da..7a76f6bd50 100644
>> --- a/board/freescale/ls1088a/ddr.h
>> +++ b/board/freescale/ls1088a/ddr.h
>> @@ -1,6 +1,8 @@
>>  /* SPDX-License-Identifier: GPL-2.0+ */
>>  /*
>> - * Copyright 2017 NXP
>> + * Copyright 2017-2018 NXP
>> + *
>> + * SPDX-License-Identifier:	GPL-2.0+
>
>Do not add another SPDX line.
>
>> Will fix and send patch again.
>York