[U-Boot] worth cleaning redundancies out of some board Makefiles?

Robert P. J. Day rpjday at crashcourse.ca
Wed Apr 13 15:17:51 CEST 2016


  again, in my travels, i noticed board Makefiles like, say,
board/freescale/bsc9131rdb/Makefile:


ifdef MINIMAL

obj-y   += spl_minimal.o tlb.o law.o

else

obj-y        += bsc9131rdb.o
obj-y        += ddr.o
obj-y        += law.o
obj-y        += tlb.o
#obj-y          += bsc9131rdb_mux.o

endif


  obviously(?), it's redundant to select building tlb.o and law.o in
both parts of the conditional, when it could look like:


ifdef MINIMAL

obj-y   += spl_minimal.o

else

obj-y        += bsc9131rdb.o
obj-y        += ddr.o
#obj-y          += bsc9131rdb_mux.o

endif

obj-y        += law.o
obj-y        += tlb.o


  is it worth submitting a patch or two that cleans up that sort of
thing, perhaps on a vendor-by-vendor basis? i can do that bit by bit
if it's worth it.

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================



More information about the U-Boot mailing list