[U-Boot] [PATCHv3 2/3] driver/ddr/altera/: Add the sdram calibration portion

Wolfgang Denk wd at denx.de
Mon May 25 15:23:58 CEST 2015


Dear Pavel,

In message <20150525123750.GD9943 at amd> you wrote:
> 
> > + ** All global variables that are explicitly initialized (including          **
> > + ** explicitly initialized to zero), are only initialized once, during       **
> > + ** configuration time, and not again on reset.  This means that they        **
> > + ** preserve their current contents across resets, which is needed for some  **
> > + ** special cases involving communication with external modules.  In         **
> > + ** addition, this avoids paying the price to have the memory initialized,   **
> > + ** even for zeroed data, provided it is explicitly set to zero in the code, **
> > + ** and doesn't rely on implicit initialization.                             **
> > + ******************************************************************************
> > +
> 
> Is this sane thing to do? How does it work for variables in other
> sources?

My concern is if this is actually true (and I asked this before, in an
earlier round ov reviews).   I cannot make heads or tails of this
comment, as I don't understand what "configuration time" and "reset"
are supposed to mean in U-Boot context.  In my understanding, after a
reset the memory content is uninitialized, i. e. random, and thus MUST
always be properly initialized. 

Also, what are "external modules"?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
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
It may be bad manners to talk with your mouth full, but it isn't  too
good either if you speak when your head is empty.


More information about the U-Boot mailing list