[U-Boot-Users] [PATCH] PHYSADDR is for physical address, KSEG1ADDR is for uncached

[Shinya Kuribayashi <skuribay@ruby.dti.ne.jp>]

Hello,

I'm going to pick up this patch into -mips repos, but several problems
are there. Please see below.

Vlad Lungu wrote:
> IMO, KSEG1 should work for AU1X00 too, but I can't test it

Plase put an appropriate mail subject and patch description.
"PHYSADDR is for physical address, KSEG1ADDR is for uncached"
seems not be descriptive (at least for me).

If not native english speaker (of course I'm not), it helps a lot to see
git log of others.

> Signed-off-by: Vlad Lungu <vlad@comsys.ro>
> Signed-off-by: Shinya Kuribayashi <shinya.kuribayashi@necel.com>
> 
> ---
>  include/asm-mips/addrspace.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/include/asm-mips/addrspace.h b/include/asm-mips/addrspace.h
> index b8214b1..0e6abd7 100644
> --- a/include/asm-mips/addrspace.h
> +++ b/include/asm-mips/addrspace.h
> @@ -49,7 +49,7 @@
>     cannot access physical memory directly from core */
>  #define UNCACHED_SDRAM(a) (((unsigned long)(a)) | 0x20000000)
>  #else  /* !CONFIG_AU1X00 */

Indentation corrupted. Check your mailer settings.

> -#define UNCACHED_SDRAM(a) PHYSADDR(a)
> +#define UNCACHED_SDRAM(a) KSEG1ADDR(a)
>  #endif /* CONFIG_AU1X00 */
>  #endif /* __ASSEMBLY__ */

ditto.

>  /*

As I'm excersizing repo management, please don't expect my quick
response ;-)

thanks,

    Shinya