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

Priyanka Jain (OSS) priyanka.jain at oss.nxp.com
Tue Jun 2 09:18:03 CEST 2020


>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Tom Rini
>Sent: Monday, June 1, 2020 5:13 PM
>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 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
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


More information about the U-Boot mailing list