Hi Damien,

On 23/01/15 19:03, Damien GOTFROI wrote:

Hi,

I confirm, my first patch for spl_power_init.c generated DDR errors.

And my new (small) patch for spl_power_init.c fixes these errors, it is not my DDR patch.

An I can confirm that your smaller patch allows me to boot the G2C1 board. I'll just implement the smaller patch - we can tackle the rest later

Thanks for your very helpful experimentation

Regards,

Graeme

Many thanks and best regards,

Damien Gotfroi Electronics Engineer GreenWatch S.A. Avenue Lavoisier, 18A B-1300 Wavre Tel. + 32 (0) 10 471 367 dgotfroi@greenwatch.be www.greenwatch.be BCE: 0820 144 205

Retrouvez l’actu des thématiques énergétiques sur notre blog

 Please consider the environment before printing this mail note. « Ce message électronique et ses éventuels fichiers attachés est(sont) destiné(s) à l’usage exclusif de la (les) personne(s) mentionnée(s) comme destinataire. Si vous recevez ce message électronique par erreur, vous n’êtes pas autorisé à utiliser ; dévoiler ; distribuer ; imprimer ou copier tout ou partie de ce message si vous n’en êtes pas le destinataire. GreenWatch décline toute responsabilité pour tout dommage direct ou indirect lié à l’utilisation inappropriée ou frauduleuse de ce message électronique par des personnes non autorisées. En outre, les informations contenues dans ce message électronique sont à caractère exclusivement informatif et GreenWatch ne saurait en garantir l’exactitude ou l’ exhaustivité .Les préjudices occasionnés par l’utilisation d’une information erronée ou incomplète n’entraînent en aucun cas la responsabilité de GreenWatch »

-----Original Message----- From: Graeme Russ [mailto:gruss@tss-engineering.com] Sent: vendredi 23 janvier 2015 05:15 To: Fabio Estevam Cc: U-Boot-Denx; Fabio Estevam; Mårten Wikman; Damien GOTFROI; Stefano Babic; Marek Vašut Subject: Re: [U-Boot] [PATCH 2/5] Enable booting of mx28 without battery

Hi Fabio,

On 23/01/15 14:49, Fabio Estevam wrote:

...

On Fri, Jan 23, 2015 at 1:27 AM, Graeme Russ gruss@tss-engineering.com wrote:

...

I thought the DDR errors were fixed with Damien's DDR patch. Damien, can you please confirm?

His response in the forum was:

"About the patch posted on January, 6th, you can simply delete it, and empty your recycle bin . It generate DDR errors (I searched a long time in spl_mem_init to find why my DDR was not stable, but it was not a DDR configuration mistake, but a power mistake caused by my patch)."

Ah - I missed that - my bad.

I'll check if his newer patch works. If it does, I'll implement it instead. Best to make these kind of changes one small step at a time :)

Regards,

Graeme