From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andre Schwarz Date: Wed, 10 Nov 2010 09:35:36 +0100 Subject: [U-Boot] [PATCH v2] Switch from archive libraries to partial linking In-Reply-To: <20101109174414.GA28687@safe.home.local> References: <1289253872-16891-1-git-send-email-sebastien.carlier@gmail.com> <4CD91133.2000704@matrix-vision.de> <20101109174414.GA28687@safe.home.local> Message-ID: <4CDA5958.9080406@matrix-vision.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Sebastien, >> I also don't understand why ftd.c is not depending on CONFIG_QE. >> That's another question ... but leads to error having your patch applied. >> > Right, fdt.c should not be included in the build unless both CONFIG_QE > and CONFIG_OF_LIBFDT are defined. Previously this issue was hidden > because all of qe.a was discarded by the linker. Would the following > be a good fix for this problem? > > -COBJS-$(CONFIG_OF_LIBFDT) += fdt.o > +COBJS-$(and $(CONFIG_QE),$(CONFIG_OF_LIBFDT)) += fdt.o > yes - this should do it. Within fdt.c there's a CONFIG_QE conditional - you can remove this also with your approach. Regards, Andr? MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler Registergericht: Amtsgericht Stuttgart, HRB 271090 Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner