On Sunday 13 December 2009 22:27:42 Peter Tyser wrote: > On Sun, 2009-12-13 at 21:37 -0500, Mike Frysinger wrote: > > On Sunday 13 December 2009 18:04:33 Peter Tyser wrote: > > > Signed-off-by: Peter Tyser <[email protected]> > > > --- > > > Makefile | 2 +- > > > {cpu => arch/blackfin/cpu}/blackfin/.gitignore | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/Makefile | 0 > > > .../blackfin/cpu}/blackfin/bootrom-asm-offsets.awk | 0 > > > .../cpu}/blackfin/bootrom-asm-offsets.c.in | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/cache.S | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/cpu.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/cpu.h | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/initcode.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/interrupt.S | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/interrupts.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/jtag-console.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/os_log.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/reset.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/serial.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/serial.h | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/start.S | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/traps.c | 0 > > > {cpu => arch/blackfin/cpu}/blackfin/watchdog.c | 0 > > > > NAK; this makes no sense: > > arch/blackfin/cpu/blackfin/* > > > > the Blackfin cpu is unified, so there should only be: > > arch/blackfin/cpu/* > > I did it that way to be consistent across architectures, mostly because > it was simpler to handle 1 cpu location in the build system. I'll > respin with the ability to not have a cpu-specific sub-directory.
i know why you did it (and it makes sense for the cpus that have multiple dirs
in cpu/), but it doesnt make sense for Blackfin at this time (and probably for
quite some time).
i dont know if the nios guys want to do cpu/nios/{nios,nios2}/ ...
> Before I respin, do you have any issue with the fact that the CFLAGS_lib
> will apply to both /lib and /arch/blackfin/lib?
sorry, i dont understand what you mean. i must have missed the patch you're
implicitly referring to ?
-mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
