[U-Boot] [PATCH] mpq101: initial support for Mercury Computer Systems MPQ101 board

Alex Dubov oakad at yahoo.com
Tue Dec 28 05:42:33 CET 2010


> > +phys_size_t
> > +initdram (int board_type)
> > +{
> ...
> > +    dram_size = setup_ddr_tlbs(1ull
> << (CONFIG_SYS_SDRAM_SIZE_LOG - 20));
> > +
> > +    puts("    DDR: ");
> > +
> > +    return dram_size << 20;
> 
> You should use get_ram_size().

I still need to call setup_ddr_tlbs because get_ram_size won't do it for
me. So what get_ram_size actually does?


> > +    lbc->lcrr |= 0x00030000;
> > +    asm("sync;isync;msync");
> > +
> > +    lbc->ltesr = 0xffffffff; 
>       /* Clear LBC error interrupts */
> > +    lbc->lteir = 0xffffffff; 
>       /* Enable LBC error interrupts */
> 
> NAK.  Please use proper I/O accessors.

Do you mean out_be32 and friends?
In which case, why are not these used when writing config values to DDR
registers (for instance) even in some very recently added boards, such as
p2020ds?

> 
> ...
> > +#define
> CONFIG_SYS_LOAD_ADDR   0x2000000   
>    /* default load address */
> ...
> > +#define CONFIG_LOADADDR  1000000 /*default
> location for tftp and bootm*/
> 
> You probably want to make this consistent.
> 
> Best regards,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH,     MD:
> Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194
> Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email:
> wd at denx.de
> You can only live once, but if you do it right, once is
> enough.
> 


      


More information about the U-Boot mailing list