Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Aneesh V
Hi Scott, On Tuesday 17 May 2011 10:20 PM, Scott Wood wrote: > On Tue, 17 May 2011 12:24:34 +0530 > Aneesh V wrote: > >> On Tuesday 17 May 2011 12:02 AM, Scott Wood wrote: >>> On Sun, 15 May 2011 20:51:24 +0530 >>> Aneesh V wrote: diff --git a/arch/arm/include/asm/global_data.h b/arc

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Scott Wood
On Tue, 17 May 2011 12:24:34 +0530 Aneesh V wrote: > On Tuesday 17 May 2011 12:02 AM, Scott Wood wrote: > > On Sun, 15 May 2011 20:51:24 +0530 > > Aneesh V wrote: > >> diff --git a/arch/arm/include/asm/global_data.h > >> b/arch/arm/include/asm/global_data.h > >> index 2a84d27..2ce020e 100644 >

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Aneesh V
Hi Wolfgang, On Tuesday 17 May 2011 06:03 PM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<4dd26719.5090...@ti.com> you wrote: >> >> I was thinking that it may be faster. More number of registers at >> disposal may mean less number of pushes to the stack, right? I am not >> sure if this w

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Wolfgang Denk
Dear Aneesh V, In message <4dd26719.5090...@ti.com> you wrote: > > I was thinking that it may be faster. More number of registers at > disposal may mean less number of pushes to the stack, right? I am not > sure if this will make a significant difference. It does not make a significant differenc

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Aneesh V
Hi Wolfgang, On Tuesday 17 May 2011 04:47 PM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<4dd24e63.3020...@ti.com> you wrote: >> >>> But that's not what you are doing. You are not changing the storage >>> of the global data itself, you are changing the storage of the POINTER >>> TO the

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Wolfgang Denk
Dear Aneesh V, In message <4dd24e63.3020...@ti.com> you wrote: > > > But that's not what you are doing. You are not changing the storage > > of the global data itself, you are changing the storage of the POINTER > > TO the global data - and this makes no sense to me. The pointer can > > certain

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Aneesh V
Hi Aneesh, On Tuesday 17 May 2011 01:45 PM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<4dd21baa.6000...@ti.com> you wrote: >> >>> What is MLO? >> >> MLO is the name of SPL created for OMAP. ROM code expects a file with >> this name as the first image when it boots from FAT. > > What doe

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-17 Thread Wolfgang Denk
Dear Aneesh V, In message <4dd21baa.6000...@ti.com> you wrote: > > > What is MLO? > > MLO is the name of SPL created for OMAP. ROM code expects a file with > this name as the first image when it boots from FAT. What does MLO mean? > >> +#ifdef CONFIG_PRELOADER > >> +/* SPL works from internal

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-16 Thread Aneesh V
Hi Scott, On Tuesday 17 May 2011 12:02 AM, Scott Wood wrote: > On Sun, 15 May 2011 20:51:24 +0530 > Aneesh V wrote: > >> diff --git a/Makefile b/Makefile >> index 384a59e..d3f4bef 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -289,6 +289,22 @@ LDPPFLAGS += \ >> $(shell $(LD) --version | \

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-16 Thread Scott Wood
On Sun, 15 May 2011 20:51:24 +0530 Aneesh V wrote: > diff --git a/Makefile b/Makefile > index 384a59e..d3f4bef 100644 > --- a/Makefile > +++ b/Makefile > @@ -289,6 +289,22 @@ LDPPFLAGS += \ > $(shell $(LD) --version | \ > sed -ne 's/GNU ld version > \([0-9][0-9]*\)\.\([0-9][0-9]*\)

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-16 Thread Wolfgang Denk
Dear Aneesh V, In message <4dd11d1f.8020...@ti.com> you wrote: > > > - Get rid of xloader. I cannot see any good reasons why we need it, > >i. e. which functions if performs that cannot be as well (and > >eventually even more efficiently) be performed in the U-Boot SPL > >code. > > I

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-16 Thread Aneesh V
Hi Wolfgang, On Monday 16 May 2011 01:18 AM, Wolfgang Denk wrote: > Dear Aneesh V, > > In message<1305472900-4004-7-git-send-email-ane...@ti.com> you wrote: >> Define a new type of SPL that is not tied to any particular media. >> - Create a top level directory 'spl' that has a structure similar >

Re: [U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-15 Thread Wolfgang Denk
Dear Aneesh V, In message <1305472900-4004-7-git-send-email-ane...@ti.com> you wrote: > Define a new type of SPL that is not tied to any particular media. > - Create a top level directory 'spl' that has a structure similar > to the existing 'nand_spl' > - Make necessary changes to top-level Make

[U-Boot] [PATCH v2 06/22] Add generic spl infrastructure

2011-05-15 Thread Aneesh V
Define a new type of SPL that is not tied to any particular media. - Create a top level directory 'spl' that has a structure similar to the existing 'nand_spl' - Make necessary changes to top-level Makefile to build such an spl Rationale for this approach: - There may be SPLs(like the OMAP x-loa