All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.