[PATCH 08/24] arm: Remove configs/P1020MBG-PC_36BIT_SDCARD_defconfig board

Tom Rini trini at konsulko.com
Fri May 29 15:27:32 CEST 2020


On Fri, May 29, 2020 at 06:06:16PM +0530, Jagan Teki wrote:
> On Fri, May 29, 2020 at 2:14 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Thu, May 28, 2020 at 02:32:39PM -0600, Simon Glass wrote:
> > > Hi,
> > >
> > > On Thu, 28 May 2020 at 14:15, Tom Rini <trini at konsulko.com> wrote:
> > > >
> > > > On Thu, May 28, 2020 at 07:07:14AM +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 08/24] arm: Remove configs/P1020MBG-
> > > > > >PC_36BIT_SDCARD_defconfig board
> > > > > >
> > > > > >This board has not been converted to CONFIG_DM_SPI by the deadline.
> > > > > >
> > > > > >Remove it.
> > > > > >
> > > > > >Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
> > > > > >---
> > > > > > arch/powerpc/cpu/mpc85xx/Kconfig             |   1 -
> > > > > > board/freescale/p1_p2_rdb_pc/Kconfig         |  19 -
> > > > > > board/freescale/p1_p2_rdb_pc/MAINTAINERS     |  51 --
> > > > > > board/freescale/p1_p2_rdb_pc/Makefile        |  24 -
> > > > > > board/freescale/p1_p2_rdb_pc/README          |  66 --
> > > > > > board/freescale/p1_p2_rdb_pc/ddr.c           | 292 ------
> > > > > > board/freescale/p1_p2_rdb_pc/law.c           |  22 -
> > > > > > board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c  | 536 -----------
> > > > > > board/freescale/p1_p2_rdb_pc/spl.c           | 123 ---
> > > > > > board/freescale/p1_p2_rdb_pc/spl_minimal.c   |  63 --
> > > > > > board/freescale/p1_p2_rdb_pc/tlb.c           | 110 ---
> > > > > > configs/P1020MBG-PC_36BIT_SDCARD_defconfig   |  67 --
> > > > > > configs/P1020MBG-PC_36BIT_defconfig          |  55 --
> > > > > > configs/P1020MBG-PC_SDCARD_defconfig         |  66 --
> > > > > > configs/P1020MBG-PC_defconfig                |  54 --
> > > > > > configs/P1020RDB-PC_36BIT_NAND_defconfig     |  85 --
> > > > > > configs/P1020RDB-PC_36BIT_SDCARD_defconfig   |  80 --
> > > > > > configs/P1020RDB-PC_36BIT_SPIFLASH_defconfig |  82 --
> > > > > > configs/P1020RDB-PC_36BIT_defconfig          |  69 --
> > > > > > configs/P1020RDB-PC_NAND_defconfig           |  84 --
> > > > > > configs/P1020RDB-PC_SDCARD_defconfig         |  79 --
> > > > > > configs/P1020RDB-PC_SPIFLASH_defconfig       |  81 --
> > > > > > configs/P1020RDB-PC_defconfig                |  68 --
> > > > > > configs/P1020RDB-PD_NAND_defconfig           |  88 --
> > > > > > configs/P1020RDB-PD_SDCARD_defconfig         |  83 --
> > > > > > configs/P1020RDB-PD_SPIFLASH_defconfig       |  85 --
> > > > > > configs/P1020RDB-PD_defconfig                |  72 --
> > > > > > configs/P1020UTM-PC_36BIT_SDCARD_defconfig   |  67 --
> > > > > > configs/P1020UTM-PC_36BIT_defconfig          |  55 --
> > > > > > configs/P1020UTM-PC_SDCARD_defconfig         |  66 --
> > > > > > configs/P1020UTM-PC_defconfig                |  54 --
> > > > > > configs/P1021RDB-PC_36BIT_NAND_defconfig     |  84 --
> > > > > > configs/P1021RDB-PC_36BIT_SDCARD_defconfig   |  79 --
> > > > > > configs/P1021RDB-PC_36BIT_SPIFLASH_defconfig |  81 --
> > > > > > configs/P1021RDB-PC_36BIT_defconfig          |  67 --
> > > > > > configs/P1021RDB-PC_NAND_defconfig           |  83 --
> > > > > > configs/P1021RDB-PC_SDCARD_defconfig         |  78 --
> > > > > > configs/P1021RDB-PC_SPIFLASH_defconfig       |  80 --
> > > > > > configs/P1021RDB-PC_defconfig                |  66 --
> > > > > > configs/P1024RDB_36BIT_defconfig             |  61 --
> > > > > > configs/P1024RDB_NAND_defconfig              |  77 --
> > > > > > configs/P1024RDB_SDCARD_defconfig            |  71 --
> > > > > > configs/P1024RDB_SPIFLASH_defconfig          |  73 --
> > > > > > configs/P1024RDB_defconfig                   |  60 --
> > > > > > configs/P1025RDB_36BIT_defconfig             |  63 --
> > > > > > configs/P1025RDB_NAND_defconfig              |  80 --
> > > > > > configs/P1025RDB_SDCARD_defconfig            |  73 --
> > > > > > configs/P1025RDB_SPIFLASH_defconfig          |  76 --
> > > > > > configs/P1025RDB_defconfig                   |  62 --
> > > > > > configs/P2020RDB-PC_36BIT_NAND_defconfig     |  90 --
> > > > > > configs/P2020RDB-PC_36BIT_SDCARD_defconfig   |  85 --
> > > > > > configs/P2020RDB-PC_36BIT_SPIFLASH_defconfig |  87 --
> > > > > > configs/P2020RDB-PC_36BIT_defconfig          |  74 --
> > > > > > configs/P2020RDB-PC_NAND_defconfig           |  89 --
> > > > > > configs/P2020RDB-PC_SDCARD_defconfig         |  84 --
> > > > > > configs/P2020RDB-PC_SPIFLASH_defconfig       |  86 --
> > > > > > configs/P2020RDB-PC_defconfig                |  73 --
> > > > > > include/configs/p1_p2_rdb_pc.h               | 881 -------------------
> > > > > > 58 files changed, 5610 deletions(-)
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/Kconfig
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/MAINTAINERS
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/Makefile
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/README
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/ddr.c
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/law.c
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/spl.c
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/spl_minimal.c
> > > > > > delete mode 100644 board/freescale/p1_p2_rdb_pc/tlb.c
> > > > > > delete mode 100644 configs/P1020MBG-PC_36BIT_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020MBG-PC_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1020MBG-PC_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020MBG-PC_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_36BIT_NAND_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_36BIT_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_36BIT_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_NAND_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PC_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PD_NAND_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PD_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PD_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1020RDB-PD_defconfig
> > > > > > delete mode 100644 configs/P1020UTM-PC_36BIT_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020UTM-PC_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1020UTM-PC_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1020UTM-PC_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_36BIT_NAND_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_36BIT_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_36BIT_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_NAND_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1021RDB-PC_defconfig
> > > > > > delete mode 100644 configs/P1024RDB_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1024RDB_NAND_defconfig
> > > > > > delete mode 100644 configs/P1024RDB_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1024RDB_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1024RDB_defconfig
> > > > > > delete mode 100644 configs/P1025RDB_36BIT_defconfig
> > > > > > delete mode 100644 configs/P1025RDB_NAND_defconfig
> > > > > > delete mode 100644 configs/P1025RDB_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P1025RDB_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P1025RDB_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_36BIT_NAND_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_36BIT_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_36BIT_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_36BIT_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_NAND_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_SDCARD_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_SPIFLASH_defconfig
> > > > > > delete mode 100644 configs/P2020RDB-PC_defconfig
> > > > > > delete mode 100644 include/configs/p1_p2_rdb_pc.h
> > > > > >
> > > > > <snip>
> > > > > NXP plans to keep maintaining these boards. 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
> > > >
> > > > Of this series, P1020MBG-PC_SDCARD, P1020UTM-PC_SDCARD,
> > > > P1021RDB-PC_36BIT_SDCARD, P1021RDB-PC_SDCARD, P1024RDB_SDCARD,
> > > > P1025RDB_SDCARD
> > > > and also sometimes:
> > > > +(P1021RDB-PC_36BIT_SDCARD) This board does not use CONFIG_DM_USB. Please update
> > > > +(P1021RDB-PC_36BIT_SDCARD) the board to use CONFIG_DM_USB before the v2019.07 release.
> > > > +(P1021RDB-PC_36BIT_SDCARD) This board does use CONFIG_LIBATA but has CONFIG_AHCI not
> > > > +(P1021RDB-PC_36BIT_SDCARD) enabled. Please update the storage controller driver to use
> > > > +(P1021RDB-PC_36BIT_SDCARD) CONFIG_AHCI before the v2019.07 release.
> > > > or:
> > > > +(P1021RDB-PC_SDCARD) This board does not use CONFIG_DM_PCI Please update
> > > > +(P1021RDB-PC_SDCARD) the board to use CONFIG_DM_PCI before the v2019.07 release.
> > > >
> > > > And of course those non-MMC issues show up on other configs of the
> > > > family.  How far along are the plans to address those?  Thanks!
> > >
> > > I think it might be best to re-introduce the board later when it is
> > > ready, since we are so far past the migration deadline. That will
> > > unblock the SPI patches, which after all we want to get in to this
> > > release or at least the next merge window.
> > >
> > > Or perhaps just disable SPI flash, etc. for the board?
> >
> > If we disable all of the far-past-migration features and fix the
> > resulting build issues (usually errata that don't have a proper guard)
> > there's little to nothing functional left in many cases, so removal and
> > reintroduction would be better.
> >
> > To be clear, I'm not suggesting any of these get dropped for v2020.07
> > but I do want to see stuff removed or fixed for v2020.10 and I will be
> 
> Yes, I have mentioned a similar point to Priyanka on the other thread,
> but I will pull whatever Priyanka acked on this series for this
> release.

To be clear, where we are in the release cycle, I want these changes
(and other migrations) for -next and not v2020.07, 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/20200529/06ffd0e7/attachment.sig>


More information about the U-Boot mailing list