> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> On Behalf Of [email protected]
> Sent: Friday, March 18, 2011 1:40 PM
> To: [email protected]
> Subject: [U-Boot] [PATCH 1/7] Orion5x: Correct DRAM bank detection
> 

Some more explanation about the patch may find help full.
Please have a practice to provide some detailed comments.
 
> From: Rogan Dawes <[email protected]>
> 
> ---
>  arch/arm/cpu/arm926ejs/orion5x/dram.c       |    2 +-
>  arch/arm/include/asm/arch-orion5x/orion5x.h |    1 +
>  2 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/cpu/arm926ejs/orion5x/dram.c
> b/arch/arm/cpu/arm926ejs/orion5x/dram.c
> index b749282..2ae8f48 100644
> --- a/arch/arm/cpu/arm926ejs/orion5x/dram.c
> +++ b/arch/arm/cpu/arm926ejs/orion5x/dram.c
> @@ -38,7 +38,7 @@ u32 orion5x_sdram_bar(enum memory_bank bank)
>  {
>       struct orion5x_ddr_addr_decode_registers *winregs =
>               (struct orion5x_ddr_addr_decode_registers *)
> -             ORION5X_CPU_WIN_BASE;
> +             ORION5X_DRAM_BASE;

Any explanation about previous definition was wrong?
Does struct orion5x_ddr_addr_decode_registers still valid for this change?

Regards..
Prafulla . .
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to