On Monday 14 May 2012 01:37:10 Kyungmin Park wrote: > On 5/14/12, Mike Frysinger <vap...@gentoo.org> wrote: > > On Thursday 10 May 2012 01:23:40 Donghwa Lee wrote: > >> Makefile | 1 + > >> include/libtizen.h | 30 + > >> lib/tizen/Makefile | 46 + > >> lib/tizen/tizen.c | 40 + > >> lib/tizen/tizen_hd_logo.h | 5071 > >> > >> lib/tizen/tizen_hd_logo_data.h | 30 + > > > > do you really need your own lib/tizen/ dir for a logo ? every one who > > doesn't > > build this still has to waste time going into lib/tizen/ to create an > > empty > > > > object. at least make the addition of lib/tizen/ depend on CONFIG_TIZEN > > too. > > It uses the same method as others does. > e.g., libfdt > > In Makefile > LIBS += lib/libfdt/libfdt.o > > In "lib/libfdt/Makefile" > SOBJS = > > COBJS-libfdt += fdt.o fdt_ro.o fdt_rw.o fdt_strerror.o fdt_sw.o fdt_wip.o > > COBJS-$(CONFIG_OF_LIBFDT) += $(COBJS-libfdt) > COBJS-$(CONFIG_FIT) += $(COBJS-libfdt)
the difference is they have more than one CONFIG symbol going on, and their code base is sizable, and many arches/boards utilize it. you're adding a single file with a single function with a single CONFIG knob for a single board. -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot