public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 10/17] blackfin: Move appropriate cpu/ dirs to arch/blackfin/cpu/
Date: Mon, 14 Dec 2009 01:19:44 -0500	[thread overview]
Message-ID: <200912140119.44922.vapier@gentoo.org> (raw)
In-Reply-To: <1260761262.18409.27.camel@ptyser-laptop>

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 <ptyser@xes-inc.com>
> > > ---
> > >  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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091214/388d43e6/attachment.pgp 

  reply	other threads:[~2009-12-14  6:19 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-13 23:04 [U-Boot] [PATCH 00/17] Reorganize directory structure Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 01/17] Move lib_<ARCH> directories to arch/<ARCH>/lib Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 02/17] Rename lib_generic/ to lib/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 03/17] Move libfdt/ into lib/ Peter Tyser
2009-12-13 23:16   ` Luigi Mantellini
2009-12-13 23:33     ` Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 04/17] Replace "#include <asm-$ARCH/$FILE>" with "#include <asm/$FILE>" Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 05/17] Move architecture-specific includes to arch/$ARCH/include/asm Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 06/17] ppc: Move appropriate cpu/ dirs to arch/ppc/cpu/ Peter Tyser
2009-12-14 20:44   ` Scott Wood
2009-12-14 22:26     ` Peter Tyser
2009-12-14 22:32       ` Scott Wood
2009-12-13 23:04 ` [U-Boot] [PATCH 07/17] sh: Move appropriate cpu/ dirs to arch/sh/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 08/17] arm: Move appropriate cpu/ dirs to arch/arm/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 09/17] m68k: Move appropriate cpu/ dirs to arch/m68k/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 10/17] blackfin: Move appropriate cpu/ dirs to arch/blackfin/cpu/ Peter Tyser
2009-12-14  2:37   ` Mike Frysinger
2009-12-14  3:27     ` Peter Tyser
2009-12-14  6:19       ` Mike Frysinger [this message]
2009-12-14  6:39         ` Peter Tyser
2009-12-14 13:58           ` Mike Frysinger
2009-12-13 23:04 ` [U-Boot] [PATCH 11/17] mips: Move appropriate cpu/ dirs to arch/mips/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 12/17] avr32: Move appropriate cpu/ dirs to arch/avr32/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 13/17] microblaze: Move appropriate cpu/ dirs to arch/microblaze/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 14/17] sparc: Move appropriate cpu/ dirs to arch/sparc/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 15/17] nios: Move appropriate cpu/ dirs to arch/nios/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 16/17] nios2: Move appropriate cpu/ dirs to arch/nios2/cpu/ Peter Tyser
2009-12-13 23:04 ` [U-Boot] [PATCH 17/17] i386: Move appropriate cpu/ dirs to arch/i386/cpu/ Peter Tyser
2010-03-20 22:51 ` [U-Boot] [PATCH 00/17] Reorganize directory structure Wolfgang Denk
2010-03-26 15:57   ` Peter Tyser

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200912140119.44922.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox