[U-Boot] [PATCH v4 00/25] MTD defconfigs/Kconfigs/Makefiles heavy cleanup

Tom Rini trini at konsulko.com
Sun Mar 17 19:34:58 UTC 2019


On Sun, Mar 17, 2019 at 06:27:51PM +0100, Miquel Raynal wrote:
> Jagan,
> 
> Miquel Raynal <miquel.raynal at bootlin.com> wrote on Wed, 6 Mar 2019
> 10:21:44 +0100:
> 
> > Hi Jagan,
> > 
> > Miquel Raynal <miquel.raynal at bootlin.com> wrote on Wed, 20 Feb 2019
> > 08:08:58 +0100:
> > 
> > > Hi Vignesh,
> > > 
> > > Vignesh R <vigneshr at ti.com> wrote on Wed, 20 Feb 2019 10:50:18 +0530:
> > >   
> > > > Hi Jagan,
> > > > 
> > > > On 09/12/18 11:37 PM, Miquel Raynal wrote:    
> > > > > Hello,
> > > > > 
> > > > > During my last project about SPI-NAND support in U-Boot, I discovered
> > > > > when modifying Makefiles a confusing organization where:
> > > > > * Sub-directories/files are compiled from the root Makefile
> > > > > * Commands are at the root of everything
> > > > > 
> > > > > I sent a fist series a few weeks ago to move Makefile entries in their
> > > > > respective directories (which needed to be reworked). Since then, I
> > > > > have been working on clarifying all this for the MTD subsystem and
> > > > > here are the main points of such re-organization:
> > > > > * Rename CONFIG_MTD into CONFIG_DM_MTD to reserve CONFIG_MTD to what
> > > > >   is called today CONFIG_MTD_DEVICE.
> > > > > * Fix build dependencies in defconfigs, like: "UBI and NAND depend on MTD".
> > > > > * Fix the Kconfig files to reflect these dependencies (as defconfigs
> > > > >   have been updated, nothing should break).
> > > > > * Simplify the Makefiles: compiling the drivers/mtd/nand/raw/
> > > > >   sub-directory should just depend on MTD being compiled and the NAND
> > > > >   core as well, there is absolutely no logic to make it depend on
> > > > >   CMD_NAND.
> > > > > 
> > > > > New green Travis CI build for the third version of this series:
> > > > > https://travis-ci.org/miquelraynal/u-boot/builds/463486099
> > > > > There are three Sandbox tests that are failing, they have not been
> > > > > break by this series. The following Travis test has been done on the
> > > > > commit on which has been based the series and shows the same errors:
> > > > > https://travis-ci.org/miquelraynal/u-boot/builds/463593006
> > > > >       
> > > > 
> > > > I would like to revive this series. With some rebasing, this series
> > > > should still apply as is.
> > > > 
> > > > Jagan, did you get a chance to look into this series? Any comments?    
> > > 
> > > Thanks for reviving the series, actually I was sure it was merged by
> > > that time as AFAIR there was no more opposition to it.
> > > 
> > > Jagan, I see it is still marked 'new' on patchwork and assigned to
> > > you, would you mind applying it?  
> > 
> > Gentle 3-month-later ping. Jagan, please consider this series.
> > 
> 
> Almost two more weeks passed, I think I deserved some feedback now.
> 
> This series IMHO improves the MTD layer, it passed Travis CI and has
> been ready for months. It is very likely that some rebase is needed
> but I am not willing to spend more time on something you will just
> throw away for no reason. 

Jagan, can you please comment?  This sounds like something we should
pick up in the next merge window, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190317/19ada913/attachment.sig>


More information about the U-Boot mailing list