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

[Tom Rini]

On Mon, Jun 01, 2020 at 05:23:09AM +0000, Priyanka Jain wrote:
> >-----Original Message-----
> >From: Tom Rini <trini at konsulko.com>
> >Sent: Friday, May 29, 2020 1:32 AM
> >To: Priyanka Jain <priyanka.jain at nxp.com>
> >Cc: Jagan Teki <jagan at amarulasolutions.com>; Simon Glass
> ><sjg at 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 at lists.denx.de> On Behalf Of Jagan Teki
> >> >Sent: Wednesday, May 27, 2020 10:17 PM
> >> >To: Simon Glass <sjg at chromium.org>; Tom Rini <trini at konsulko.com>
> >> >Cc: u-boot at lists.denx.de; linux-amarula at amarulasolutions.com; Jagan
> >> >Teki <jagan at 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 at nxp.com>
> >> >Signed-off-by: Jagan Teki <jagan at 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 at 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>