[U-Boot] [PATCH 2/3] flash: complete CONFIG_SYS_NO_FLASH move with renaming

Tom Rini trini at konsulko.com
Sun Feb 12 19:28:49 UTC 2017


On Sun, Feb 12, 2017 at 10:44:20AM -0500, Tom Rini wrote:
> On Sat, Feb 11, 2017 at 10:43:54PM +0900, Masahiro Yamada wrote:
> 
> > We repeated partial moves for CONFIG_SYS_NO_FLASH, but this is
> > not completed. Finish this work by the tool.
> > 
> > During this move, let's rename it to CONFIG_MTD_NOR_FLASH.
> > Actually, we have more instances of "#ifndef CONFIG_SYS_NO_FLASH"
> > than those of "#ifdef CONFIG_SYS_NO_FLASH".  Flipping the logic will
> > make the code more readable.  Besides, negative meaning symbols do
> > not fit in obj-$(CONFIG_...) style Makefiles.
> 
> This isn't 100% right for some of the various NXP T10xx board
> combinations, and I'll go and fix those up while applying.

Ah, OK, so on further investigation the problem is the configs were
playing the game of turning something off for SPL, so this growth is OK
for now.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170212/8859c5d9/attachment.sig>


More information about the U-Boot mailing list