From mboxrd@z Thu Jan 1 00:00:00 1970 From: Miquel Raynal Date: Sun, 17 Mar 2019 18:27:51 +0100 Subject: [U-Boot] [PATCH v4 00/25] MTD defconfigs/Kconfigs/Makefiles heavy cleanup In-Reply-To: <20190306102144.68bbcf18@xps13> References: <20181209180747.24575-1-miquel.raynal@bootlin.com> <20190220080858.70ca276f@xps13> <20190306102144.68bbcf18@xps13> Message-ID: <20190317182751.757a4b06@xps13> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de Jagan, Miquel Raynal wrote on Wed, 6 Mar 2019 10:21:44 +0100: > Hi Jagan, >=20 > Miquel Raynal wrote on Wed, 20 Feb 2019 > 08:08:58 +0100: >=20 > > Hi Vignesh, > >=20 > > Vignesh R wrote on Wed, 20 Feb 2019 10:50:18 +0530: > > =20 > > > Hi Jagan, > > >=20 > > > On 09/12/18 11:37 PM, Miquel Raynal wrote: =20 > > > > Hello, > > > >=20 > > > > During my last project about SPI-NAND support in U-Boot, I discover= ed > > > > when modifying Makefiles a confusing organization where: > > > > * Sub-directories/files are compiled from the root Makefile > > > > * Commands are at the root of everything > > > >=20 > > > > I sent a fist series a few weeks ago to move Makefile entries in th= eir > > > > 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 NA= ND > > > > core as well, there is absolutely no logic to make it depend on > > > > CMD_NAND. > > > >=20 > > > > 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 > > > > =20 > > >=20 > > > I would like to revive this series. With some rebasing, this series > > > should still apply as is. > > >=20 > > > Jagan, did you get a chance to look into this series? Any comments? = =20 > >=20 > > Thanks for reviving the series, actually I was sure it was merged by > > that time as AFAIR there was no more opposition to it. > >=20 > > Jagan, I see it is still marked 'new' on patchwork and assigned to > > you, would you mind applying it? =20 >=20 > Gentle 3-month-later ping. Jagan, please consider this series. >=20 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.=20 Thanks, Miqu=C3=A8l