[PATCH 07/24] arm: Remove configs/P1010RDB-PA_36BIT_NAND_SECBOOT_defconfig board

Priyanka Jain
2020-06-02T07:18:03Z

>-----Original Message-----
>From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Tom Rini
>Sent: Monday, June 1, 2020 5:13 PM
>To: Priyanka Jain <priyanka.jain@nxp.com>
>Cc: Jagan Teki <jagan@amarulasolutions.com>; Simon Glass
><sjg@chromium.org>; u-boot at lists.denx.de; linux-
>amarula at amarulasolutions.com
>Subject: Re: [PATCH 07/24] arm: Remove configs/P1010RDB-
>PA_36BIT_NAND_SECBOOT_defconfig board
>
>On Mon, Jun 01, 2020 at 05:23:09AM +0000, Priyanka Jain wrote:
>> >-----Original Message-----
>> >From: Tom Rini <trini@konsulko.com>
>> >Sent: Friday, May 29, 2020 1:32 AM
>> >To: Priyanka Jain <priyanka.jain@nxp.com>
>> >Cc: Jagan Teki <jagan@amarulasolutions.com>; Simon Glass
>> ><sjg@chromium.org>; u-boot at lists.denx.de; linux-
>> >amarula at amarulasolutions.com
>> >Subject: Re: [PATCH 07/24] arm: Remove configs/P1010RDB-
>> >PA_36BIT_NAND_SECBOOT_defconfig board
>> >
>> >On Thu, May 28, 2020 at 07:05:38AM +0000, Priyanka Jain wrote:
>> >> >-----Original Message-----
>> >> >From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Jagan
>> >> >Teki
>> >> >Sent: Wednesday, May 27, 2020 10:17 PM
>> >> >To: Simon Glass <sjg@chromium.org>; Tom Rini <trini@konsulko.com>
>> >> >Cc: u-boot at lists.denx.de; linux-amarula at amarulasolutions.com;
>> >> >Jagan Teki <jagan@amarulasolutions.com>
>> >> >Subject: [PATCH 07/24] arm: Remove configs/P1010RDB-
>> >> >PA_36BIT_NAND_SECBOOT_defconfig board
>> >> >
>> >> >This board has not been converted to CONFIG_DM_SPI by the deadline.
>> >> >
>> >> >Remove it.
>> >> >
>> >> >Patch-cc: Qiang Zhao <qiang.zhao@nxp.com>
>> >> >Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
>> >> >---
>> >> > arch/powerpc/cpu/mpc85xx/Kconfig | 1 -
>> >> > board/freescale/p1010rdb/Kconfig | 14 -
>> >> > board/freescale/p1010rdb/MAINTAINERS | 33 -
>> >> > board/freescale/p1010rdb/Makefile | 24 -
>> >> > board/freescale/p1010rdb/README.P1010RDB-PA | 208 -----
>> >> > board/freescale/p1010rdb/README.P1010RDB-PB | 188 -----
>> >> > board/freescale/p1010rdb/ddr.c | 235 ------
>> >> > board/freescale/p1010rdb/law.c | 16 -
>> >> > board/freescale/p1010rdb/p1010rdb.c | 731 -----------------
>> >> > board/freescale/p1010rdb/spl.c | 114 ---
>> >> > board/freescale/p1010rdb/spl_minimal.c | 65 --
>> >> > board/freescale/p1010rdb/tlb.c | 90 --
>> >> > .../P1010RDB-PA_36BIT_NAND_SECBOOT_defconfig | 63 --
>> >> > configs/P1010RDB-PA_36BIT_NAND_defconfig | 85 --
>> >> > .../P1010RDB-PA_36BIT_NOR_SECBOOT_defconfig | 62 --
>> >> > configs/P1010RDB-PA_36BIT_NOR_defconfig | 67 --
>> >> > configs/P1010RDB-PA_36BIT_SDCARD_defconfig | 79 --
>> >> > ...010RDB-PA_36BIT_SPIFLASH_SECBOOT_defconfig | 64 --
>> >> >configs/P1010RDB-PA_36BIT_SPIFLASH_defconfig | 81 --
>> >> > configs/P1010RDB-PA_NAND_SECBOOT_defconfig | 62 --
>> >> > configs/P1010RDB-PA_NAND_defconfig | 84 --
>> >> > configs/P1010RDB-PA_NOR_SECBOOT_defconfig | 60 --
>> >> > configs/P1010RDB-PA_NOR_defconfig | 66 --
>> >> > configs/P1010RDB-PA_SDCARD_defconfig | 78 --
>> >> > .../P1010RDB-PA_SPIFLASH_SECBOOT_defconfig | 63 --
>> >> > configs/P1010RDB-PA_SPIFLASH_defconfig | 80 --
>> >> > .../P1010RDB-PB_36BIT_NAND_SECBOOT_defconfig | 63 --
>> >> > configs/P1010RDB-PB_36BIT_NAND_defconfig | 85 --
>> >> > .../P1010RDB-PB_36BIT_NOR_SECBOOT_defconfig | 62 --
>> >> > configs/P1010RDB-PB_36BIT_NOR_defconfig | 67 --
>> >> > configs/P1010RDB-PB_36BIT_SDCARD_defconfig | 79 --
>> >> > ...010RDB-PB_36BIT_SPIFLASH_SECBOOT_defconfig | 64 --
>> >> >configs/P1010RDB-PB_36BIT_SPIFLASH_defconfig | 81 --
>> >> > configs/P1010RDB-PB_NAND_SECBOOT_defconfig | 62 --
>> >> > configs/P1010RDB-PB_NAND_defconfig | 84 --
>> >> > configs/P1010RDB-PB_NOR_SECBOOT_defconfig | 61 --
>> >> > configs/P1010RDB-PB_NOR_defconfig | 66 --
>> >> > configs/P1010RDB-PB_SDCARD_defconfig | 78 --
>> >> > .../P1010RDB-PB_SPIFLASH_SECBOOT_defconfig | 63 --
>> >> > configs/P1010RDB-PB_SPIFLASH_defconfig | 80 --
>> >> > include/configs/P1010RDB.h | 766 ------------------
>> >> > 41 files changed, 4474 deletions(-) delete mode 100644
>> >> >board/freescale/p1010rdb/Kconfig delete mode
>> >> >100644 board/freescale/p1010rdb/MAINTAINERS
>> >> > delete mode 100644 board/freescale/p1010rdb/Makefile delete mode
>> >> >100644 board/freescale/p1010rdb/README.P1010RDB-PA
>> >> > delete mode 100644 board/freescale/p1010rdb/README.P1010RDB-PB
>> >> > delete mode 100644 board/freescale/p1010rdb/ddr.c delete mode
>> >> >100644 board/freescale/p1010rdb/law.c delete mode 100644
>> >> >board/freescale/p1010rdb/p1010rdb.c
>> >> > delete mode 100644 board/freescale/p1010rdb/spl.c delete mode
>> >> >100644 board/freescale/p1010rdb/spl_minimal.c
>> >> > delete mode 100644 board/freescale/p1010rdb/tlb.c delete mode
>> >> >100644 configs/P1010RDB-PA_36BIT_NAND_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_36BIT_NAND_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>> >PA_36BIT_NOR_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_36BIT_NOR_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_36BIT_SDCARD_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>> >> >PA_36BIT_SPIFLASH_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_36BIT_SPIFLASH_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_NAND_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_NAND_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_NOR_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_NOR_defconfig delete mode
>> >> >100644 configs/P1010RDB-PA_SDCARD_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>PA_SPIFLASH_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PA_SPIFLASH_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>> >PB_36BIT_NAND_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_36BIT_NAND_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>> >PB_36BIT_NOR_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_36BIT_NOR_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_36BIT_SDCARD_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>> >> >PB_36BIT_SPIFLASH_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_36BIT_SPIFLASH_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_NAND_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_NAND_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_NOR_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_NOR_defconfig delete mode
>> >> >100644 configs/P1010RDB-PB_SDCARD_defconfig
>> >> > delete mode 100644 configs/P1010RDB-
>PB_SPIFLASH_SECBOOT_defconfig
>> >> > delete mode 100644 configs/P1010RDB-PB_SPIFLASH_defconfig
>> >> > delete mode 100644 include/configs/P1010RDB.h
>> >> >
>> >> <snip>
>> >> NXP plans to keep maintaining P1010RDB board. Please don't merge
>> >> this
>> >patch .
>> >>
>> >> The espi series of patches for DM migration were in review since
>> >> long, but
>> >could not be merged because of dependency.
>> >> NXP engineers are working on rebasing the DM espi migration series.
>> >>
>> >> http://patchwork.ozlabs.org/project/uboot/list/?series=127282&state
>> >> =*
>> >> http://patchwork.ozlabs.org/project/uboot/list/?series=138873
>> >>
>> >> Nacked-by: Priyanka Jain <priyanka.jain@nxp.com>
>> >
>> >So, _this_ platform also runs in to:
>> >+(P1010RDB-PB_SDCARD) ===================== WARNING
>> >+======================
>> >+(P1010RDB-PB_SDCARD) This board does not use CONFIG_DM_MMC. Please
>> >+update
>> >+(P1010RDB-PB_SDCARD) the board to use CONFIG_DM_MMC before the
>> >v2019.04 release.
>> >+(P1010RDB-PB_SDCARD) Failure to update by the deadline may result in
>> >board removal.
>> >+(P1010RDB-PB_SDCARD) See doc/driver-model/migration.rst for more info.
>> >+(P1010RDB-PB_SDCARD)
>> >+====================================================
>> >
>> >So how far along is NXP on doing the needed conversions there? Thanks!
>> >
>> >--
>> >Tom
>> Some of the boards from above list can be removed but not all.
>> I will send patch to remove those boards.
>
>Trimming the number of boards is good. But I started that list based on what you
>said in this thread are boards NXP cares about. So unless you're removing all of
>the ones I noted, my initial question stands.
>Thanks!
>
>--
>Tom
Yes Tom,

We will either fix DM related issues or will remove the boards which NXP no longer wants to maintain.
I will take care of all Warnings pointed by you in different patches.

Regards
Priyanka