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

[Alex Dubov <oakad@yahoo.com>]

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