From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 7 Dec 2018 09:07:41 -0500 Subject: [U-Boot] [PATCH v3 13/28] mtd: ensure MTD is compiled when ENV_IS_IN_FLASH is selected In-Reply-To: <20181207120933.104866eb@xps13> References: <20181204235714.11805-1-miquel.raynal@bootlin.com> <20181204235714.11805-14-miquel.raynal@bootlin.com> <20181205120610.79153242268@gemini.denx.de> <20181205153218.36f6ed4e@xps13> <20181206080605.C943B242268@gemini.denx.de> <20181206100016.706330ba@bbrezillon> <20181206093214.9F4FB242268@gemini.denx.de> <20181206162300.7ccfddda@xps13> <20181206155925.GZ32109@bill-the-cat> <20181207120933.104866eb@xps13> Message-ID: <20181207140741.GC14374@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, Dec 07, 2018 at 12:09:33PM +0100, Miquel Raynal wrote: > Hi Tom, Wolfgang, [snip] > So what's our next move? I think Wolfgang had two points: > * NOR (and everything related to NOR) should not depend on MTD. > * Same for NAND. > > For NAND this is already too late, NAND code already relies on MTD so > I am just cleaning Makefiles/Kconfig/defconfigs to avoid the "xxx: > missing symbol" error. Now, what about NOR? We know that the overhead, > today, is null, so do you allow me to make NOR depend on MTD or are you > still opposed to that? I think U-Boot maintenance and future > changes would be really eased. Right, so moving forward, I think the plan is: - For SPL, we acknowledge and continue to allow for one-off drivers that don't depend on MTD for NOR/NAND for read-only case. - For full U-Boot, we continue with the path you're talking about that gives us the most long term benefits and where yes, NOR and NAND depend on MTD. We will do this in a way that doesn't grow the binary size needlessly but by the same token we aren't going to restrict NOR/NAND drivers more harshly than we do the entire rest of the project. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: