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

Tom Rini
2020-06-01T11:43:18Z

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200601/7a67919a/attachment.sig>