Re: [U-Boot] [PATCH v2] Move arch/ppc to arch/powerpc

2010-04-16 Thread Stefan Roese
Hi Anatolij,

On Friday 16 April 2010 16:35:44 Anatolij Gustschin wrote:
> > [PATCH v2] Move arch/ppc to arch/powerpc
> > 
> > As discussed on the list, move "arch/ppc" to "arch/powerpc" to
> > better match the Linux directory structure.
> > 
> > Please note that this patch also changes the "ppc" target in
> > MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
> > is kept as an alias for now, to not break compatibility with
> > scripts using this name.
> 
> Could you please also merge the patch below into your big patch?
> MAKEALL ppc reports link errors for MVSMR board without it.

Yes, I already noticed this failure myself. Will update the patch and resend a 
new version later today.

Cheers,
Stefan

--
DENX Software Engineering GmbH,  MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Move arch/ppc to arch/powerpc

2010-04-16 Thread Anatolij Gustschin
Hi Stefan,

On Fri, 16 Apr 2010 14:37:52 +0200
Stefan Roese  wrote:
...
> [PATCH v2] Move arch/ppc to arch/powerpc
> 
> As discussed on the list, move "arch/ppc" to "arch/powerpc" to
> better match the Linux directory structure.
> 
> Please note that this patch also changes the "ppc" target in
> MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
> is kept as an alias for now, to not break compatibility with
> scripts using this name.

Could you please also merge the patch below into your big patch?
MAKEALL ppc reports link errors for MVSMR board without it.

Thanks,
Anatolij

diff --git a/board/matrix_vision/mvsmr/u-boot.lds 
b/board/matrix_vision/mvsmr/u-boot.lds
index cfbb6ef..36f867d 100644
--- a/board/matrix_vision/mvsmr/u-boot.lds
+++ b/board/matrix_vision/mvsmr/u-boot.lds
@@ -55,11 +55,11 @@ SECTIONS
   {
 /* WARNING - the following is hand-optimized to fit within  */
 /* the first two sectors (=8KB) of our S29GL flash chip */
-cpu/mpc5xxx/start.o  (.text)
-cpu/mpc5xxx/traps.o  (.text)
-lib_generic/crc32.o (.text)
-lib_ppc/cache.o (.text)
-lib_ppc/time.o  (.text)
+arch/powerpc/cpu/mpc5xxx/start.o   (.text)
+arch/powerpc/cpu/mpc5xxx/traps.o   (.text)
+lib/crc32.o(.text)
+arch/powerpc/lib/cache.o   (.text)
+arch/powerpc/lib/time.o(.text)
 
 /* This is only needed to force failure if size of above code will ever */
 /* increase and grow into reserved space. */
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Move arch/ppc to arch/powerpc

2010-04-16 Thread Kumar Gala

On Apr 16, 2010, at 8:59 AM, Peter Tyser wrote:

> 
>> [PATCH v2] Move arch/ppc to arch/powerpc
>> 
>> As discussed on the list, move "arch/ppc" to "arch/powerpc" to
>> better match the Linux directory structure.
>> 
>> Please note that this patch also changes the "ppc" target in
>> MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
>> is kept as an alias for now, to not break compatibility with
>> scripts using this name.
> 
> Thanks for finishing this up!  After a quick once over of the patch a
> minor nit:
> 
> Near line 2756:
> -This should work for arch/ppc and arch/powerpc. Only Linux
> +This should work for arch/powerpc and arch/powerpc. Only Linux
> 
> The same thing happens at line 4710 in arch/powerpc/lib/board.c
> 
> Otherwise things look good to me.
> 
> Peter

I'd appreciate this going in ASAP to reduce the churn :)

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Move arch/ppc to arch/powerpc

2010-04-16 Thread Peter Tyser

> [PATCH v2] Move arch/ppc to arch/powerpc
> 
> As discussed on the list, move "arch/ppc" to "arch/powerpc" to
> better match the Linux directory structure.
> 
> Please note that this patch also changes the "ppc" target in
> MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
> is kept as an alias for now, to not break compatibility with
> scripts using this name.

Thanks for finishing this up!  After a quick once over of the patch a
minor nit:

Near line 2756:
-This should work for arch/ppc and arch/powerpc. Only Linux
+This should work for arch/powerpc and arch/powerpc. Only Linux

The same thing happens at line 4710 in arch/powerpc/lib/board.c

Otherwise things look good to me.

Peter

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] Move arch/ppc to arch/powerpc

2010-04-16 Thread Stefan Roese
Because of the size of this patch (approx. 350k), here the link
to it:

http://www.denx.de/wiki/pub/U-Boot/TooBigPatches/0001-Move-arch-ppc-to-arch-powerpc.patch

And here the patch description and statistics:


[PATCH v2] Move arch/ppc to arch/powerpc

As discussed on the list, move "arch/ppc" to "arch/powerpc" to
better match the Linux directory structure.

Please note that this patch also changes the "ppc" target in
MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
is kept as an alias for now, to not break compatibility with
scripts using this name.

Signed-off-by: Stefan Roese 
Acked-by: Wolfgang Denk 
Acked-by: Detlev Zundel 
Cc: Peter Tyser 
---
v2: Keep "ppc" as an alias to "powerpc" in MAKEALL as suggested
by Wolfgang

 MAKEALL|   14 +-
 Makefile   |  606 ++--
 README |   16 +-
 api/{api_platform-ppc.c => api_platform-powerpc.c} |0
 arch/{ppc => powerpc}/config.mk|0
 arch/{ppc => powerpc}/cpu/74xx_7xx/Makefile|0
 arch/{ppc => powerpc}/cpu/74xx_7xx/cache.S |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/config.mk   |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/cpu.c   |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/cpu_init.c  |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/interrupts.c|0
 arch/{ppc => powerpc}/cpu/74xx_7xx/io.S|0
 arch/{ppc => powerpc}/cpu/74xx_7xx/kgdb.S  |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/speed.c |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/start.S |0
 arch/{ppc => powerpc}/cpu/74xx_7xx/traps.c |2 +-
 arch/{ppc => powerpc}/cpu/mpc512x/Makefile |0
 arch/{ppc => powerpc}/cpu/mpc512x/asm-offsets.h|2 +-
 arch/{ppc => powerpc}/cpu/mpc512x/config.mk|2 +-
 arch/{ppc => powerpc}/cpu/mpc512x/cpu.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/cpu_init.c   |0
 arch/{ppc => powerpc}/cpu/mpc512x/diu.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/fixed_sdram.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/i2c.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/ide.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/iim.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/interrupts.c |0
 arch/{ppc => powerpc}/cpu/mpc512x/iopin.c  |0
 arch/{ppc => powerpc}/cpu/mpc512x/pci.c|0
 arch/{ppc => powerpc}/cpu/mpc512x/serial.c |0
 arch/{ppc => powerpc}/cpu/mpc512x/speed.c  |0
 arch/{ppc => powerpc}/cpu/mpc512x/start.S  |0
 arch/{ppc => powerpc}/cpu/mpc512x/traps.c  |0
 arch/{ppc => powerpc}/cpu/mpc512x/u-boot.lds   |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xx/Makefile  |4 +-
 arch/{ppc => powerpc}/cpu/mpc5xx/config.mk |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xx/cpu.c |0
 arch/{ppc => powerpc}/cpu/mpc5xx/cpu_init.c|0
 arch/{ppc => powerpc}/cpu/mpc5xx/interrupts.c  |0
 arch/{ppc => powerpc}/cpu/mpc5xx/serial.c  |0
 arch/{ppc => powerpc}/cpu/mpc5xx/speed.c   |0
 arch/{ppc => powerpc}/cpu/mpc5xx/spi.c |0
 arch/{ppc => powerpc}/cpu/mpc5xx/start.S   |0
 arch/{ppc => powerpc}/cpu/mpc5xx/traps.c   |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xx/u-boot.lds|2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/Makefile |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/config.mk|2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/cpu.c|0
 arch/{ppc => powerpc}/cpu/mpc5xxx/cpu_init.c   |0
 .../cpu/mpc5xxx/firmware_sc_task_bestcomm.impl.S   |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/i2c.c|0
 arch/{ppc => powerpc}/cpu/mpc5xxx/ide.c|0
 arch/{ppc => powerpc}/cpu/mpc5xxx/interrupts.c |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/io.S |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/loadtask.c   |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/pci_mpc5200.c|0
 arch/{ppc => powerpc}/cpu/mpc5xxx/serial.c |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/speed.c  |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/start.S  |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/traps.c  |2 +-
 .../cpu/mpc5xxx/u-boot-customlayout.lds|8 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/u-boot.lds   |2 +-
 arch/{ppc => powerpc}/cpu/mpc5xxx/usb.c|0
 arch/{ppc => powerpc}/cpu/mpc5xxx/usb_ohci.c   |0
 arch/{ppc => powerpc}/cpu/mpc5xxx/usb_ohci.h   |0
 arch/{ppc => powerpc}/cpu/mpc8220/Makefile |0
 arch/{ppc => powerpc}/cpu/mpc8220/config.mk|2 +-
 arch/{ppc => powerpc}/cpu/mpc8220/cpu.c|0
 arch/{ppc => powerpc}/cpu/mpc8220/cpu_init.c   |0
 arch/{ppc => powerpc}/cpu/mpc8220/dma.h|0
 arch/{ppc => powerpc}/cpu/mpc8220/dra