[U-Boot] [PATCH] SPEAr: Add ARM relocation support

Marek Vasut marek.vasut at gmail.com
Sat Feb 18 08:45:29 CET 2012


> Hello Marek,
> 
> On Fri, Feb 17, 2012 at 2:28 PM, Marek Vasut <marek.vasut at gmail.com> wrote:
> > > On 2/17/2012 1:13 AM, Marek Vasut wrote:
> > > >> Signed-off-by: Amit Virdi<amit.virdi at st.com>
> > > > 
> > > > Uh, are you sure the Subject correlates with the patch ?
> > > > 
> > > > M
> > > 
> > > Yeah. The fix is a part of the Relocation feature. For relocation, it
> > > is essential that dram_init stores RAM size and do nothing more i.e.
> > > no access to bss etc.
> > 
> > Then name it "fix dram size reporting?". Also, you don't need dram_init
> > at all
> > if you only call get_dram_size() if I recall correctly, check
> > arch/arm/lib/board.c, I might be mistaken. And maybe explain why you
> > removed all
> > the code in the changelog, it's not clear at all.
> 
> Thanks for your suggestions. I got your point. I'll send V2 with correct
> subject and clear message.
> 
> Regards
> Amit Virdi

Good, thanks! :)

M

> 
> 
> Thanks!
> 
> > > ~Amit Virdi
> > > 
> > > >> ---
> > > >> 
> > > >>   board/spear/common/spr_misc.c |   20 +-------------------
> > > >>   1 files changed, 1 insertions(+), 19 deletions(-)
> > > >> 
> > > >> diff --git a/board/spear/common/spr_misc.c
> > > >> b/board/spear/common/spr_misc.c index 0812c20..3ab278f 100644
> > > >> --- a/board/spear/common/spr_misc.c
> > > >> +++ b/board/spear/common/spr_misc.c
> > > >> @@ -40,27 +40,9 @@ static struct chip_data chip_data;
> > > >> 
> > > >>   int dram_init(void)
> > > >>   {
> > > >> 
> > > >> -  struct xloader_table *xloader_tb =
> > > >> -      (struct xloader_table *)XLOADER_TABLE_ADDRESS;
> > > >> -  struct xloader_table_1_1 *table_1_1;
> > > >> -  struct xloader_table_1_2 *table_1_2;
> > > >> -  struct chip_data *chip =&chip_data;
> > > >> -
> > > >> +  /* Store complete RAM size and return */
> > > >> 
> > > >>    gd->ram_size = get_ram_size(PHYS_SDRAM_1, PHYS_SDRAM_1_MAXSIZE);
> > > >> 
> > > >> -  if (XLOADER_TABLE_VERSION_1_1 == xloader_tb->table_version) {
> > > >> -          table_1_1 =&xloader_tb->table.table_1_1;
> > > >> -          chip->dramfreq = table_1_1->ddrfreq;
> > > >> -          chip->dramtype = table_1_1->ddrtype;
> > > >> -
> > > >> -  } else if (XLOADER_TABLE_VERSION_1_2 ==
> > > >> xloader_tb->table_version)
> > 
> > {
> > 
> > > >> -          table_1_2 =&xloader_tb->table.table_1_2;
> > > >> -          chip->dramfreq = table_1_2->ddrfreq;
> > > >> -          chip->dramtype = table_1_2->ddrtype;
> > > >> -  } else {
> > > >> -          chip->dramfreq = -1;
> > > >> -  }
> > > >> -
> > > >> 
> > > >>    return 0;
> > > >>   
> > > >>   }
> > > > 
> > > > .
> > 
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list