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

[Antony Pavlov <antonynpavlov@gmail.com>]

On 2 July 2011 17:47, Shinya Kuribayashi <skuribay@pobox.com> 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.

Because the using of KSEG1 is most elementary way to make barebox work on MIPS.

If you use KSEG1 you have no code for cache setup and no cache exceptions.


> Is there any reason why head.S, not start.S?  Other archs in the tree
> name it start*.[cS].

x86 and sandbox use other names.
In mips linux this file have name head.S.

But I don't insist on this name.


>>  arch/mips/include/asm/barebox.h     |   24 +
>>  arch/mips/include/asm/bitops.h      |   32 ++
...
>>  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.

Good idea! I will use it.

> 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.

:))))
is U-Boot so terrible?

But I agree, now there is no reason starting based on U-Boot.

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox