[U-Boot] [PATCH v3 13/28] mtd: ensure MTD is compiled when ENV_IS_IN_FLASH is selected

Tom Rini
2018-12-07T14:07:41Z

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: <http://lists.denx.de/pipermail/u-boot/attachments/20181207/b5aebf49/attachment.sig>