Hi Mike, On Thu, Nov 3, 2011 at 6:44 PM, Mike Frysinger <vap...@gentoo.org> wrote: > On Thursday 03 November 2011 18:41:37 Simon Glass wrote: >> Since we are adding a second file to board/nvidia/common, we create >> a proper Makefile there and remove the direct board.o include from >> board/nvidia/seaboard/Makefile > > so who is including board/nvidia/common/libnvidia.o now ?
No one - it doesn't exist and that directory doesn't have a Makefile. The top-level Makefile doesn't require it, but since I now have more than one object file I want to create a library. > >> --- /dev/null >> +++ b/arch/arm/include/asm/arch-tegra2/uart-spi-switch.h >> >> +#ifndef __ASSEMBLY__ > > the asm checking is incomplete. so it must not matter -> drop it. OK > >> --- /dev/null >> +++ b/board/nvidia/common/uart-spi-switch.c >> >> +#include <asm/io.h> >> +#include <libfdt.h> >> +#include <fdt_support.h> > > seems to be unused > -mike > OK - yes this isn't needed until FDT support is in there (will be a later patch set). Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot