[PATCH 1/4] Add MIPS arch support to barebox

[Jean-Christophe PLAGNIOL-VILLARD]

On 22:47 Sat 02 Jul     , Shinya Kuribayashi wrote:
> On 6/29/11 12:24 PM, Antony Pavlov wrote:
> >Barebox can use only uncached memory.
> 
> In addition to super-summarized one-liner commit log, I'd like to see
> the reason why it is.
> 
> >Signed-off-by: Antony Pavlov<antonynpavlov at gmail.com>
> >---
> >  arch/mips/Kconfig                   |   54 ++
> >  arch/mips/Makefile                  |   63 +++
> >  arch/mips/boot/Makefile             |    5 +
> >  arch/mips/boot/entry.c              |   79 +++
> >  arch/mips/boot/head.S               |  233 ++++++++
> >  arch/mips/boot/reset.c              |   32 ++
> 
> Is there any reason why head.S, not start.S?  Other archs in the tree
> name it start*.[cS].
> 
> >  arch/mips/include/asm/barebox.h     |   24 +
> >  arch/mips/include/asm/bitops.h      |   32 ++
> >  arch/mips/include/asm/byteorder.h   |   30 +
> >  arch/mips/include/asm/common.h      |   29 +
> >  arch/mips/include/asm/elf.h         |   30 +
> >  arch/mips/include/asm/io.h          |   73 +++
> >  arch/mips/include/asm/mipsregs.h    |  988 +++++++++++++++++++++++++++++++++++
> >  arch/mips/include/asm/module.h      |   36 ++
> >  arch/mips/include/asm/posix_types.h |   48 ++
> >  arch/mips/include/asm/regdef.h      |  100 ++++
> >  arch/mips/include/asm/sections.h    |    1 +
> >  arch/mips/include/asm/sgidefs.h     |   44 ++
> >  arch/mips/include/asm/string.h      |   31 ++
> >  arch/mips/include/asm/swab.h        |   59 ++
> >  arch/mips/include/asm/types.h       |   44 ++
> >  arch/mips/include/mach/debug_ll.h   |   35 ++
> 
> And I would like to know where these kernel headers from.  If it's the
> Linux kernel, which version they're based on, and what modifications are
> made for barebox.  Such information is useful when we add/remove bits in
> the future.  This is why I did for my patches in this regard in a careful
> manner.
> 
> If it's from U-Boot tree,it's bad idea and let's redo from the scratch.
> Starting small is good, starting based on U-Boot is not good in any area.

100% agreed

Best Regards,
J.