public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Graeme Russ <gruss@tss-engineering.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/5] Enable booting of mx28 without battery
Date: Fri, 23 Jan 2015 19:11:08 +1100	[thread overview]
Message-ID: <54C2021C.9010604@tss-engineering.com> (raw)
In-Reply-To: <DB3PR05MB252F1D6A1521E87599A5991CB360@DB3PR05MB252.eurprd05.prod.outlook.com>

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 at 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 at 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
>

  parent reply	other threads:[~2015-01-23  8:11 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-21 11:55 [U-Boot] [PATCH 0/5] Add support for G2C1 mx28 board (which has no battery) Graeme Russ
2015-01-21 11:55 ` [U-Boot] [PATCH 1/5] Add debug output to mx29 SPL source files Graeme Russ
2015-01-21 20:49   ` Fabio Estevam
2015-01-21 23:24   ` Marek Vasut
2015-01-22  4:24     ` Graeme Russ
2015-01-21 11:55 ` [U-Boot] [PATCH 2/5] Enable booting of mx28 without battery Graeme Russ
2015-01-21 12:03   ` Graeme Russ
2015-01-21 21:08     ` Fabio Estevam
2015-01-21 23:27   ` Marek Vasut
2015-01-23  0:58     ` Graeme Russ
2015-01-22 12:42   ` Fabio Estevam
2015-01-22 13:51     ` Marek Vasut
2015-01-23  2:58     ` Graeme Russ
2015-01-23  3:01       ` Fabio Estevam
2015-01-23  3:27         ` Graeme Russ
2015-01-23  3:49           ` Fabio Estevam
2015-01-23  4:14             ` Graeme Russ
     [not found]               ` <DB3PR05MB252F1D6A1521E87599A5991CB360@DB3PR05MB252.eurprd05.prod.outlook.com>
2015-01-23  8:11                 ` Graeme Russ [this message]
2015-01-23 13:01                   ` Stefano Babic
2015-01-21 11:55 ` [U-Boot] [PATCH 3/5] Add 'Wait for JTAG user' if booted in JTAG mode Graeme Russ
2015-01-21 15:35   ` Stefan Roese
2015-01-21 11:55 ` [U-Boot] [PATCH 4/5] Move PLL power-up from power init to memory init Graeme Russ
2015-01-21 23:33   ` Marek Vasut
2015-01-21 11:55 ` [U-Boot] [PATCH 5/5] Add support for ReachTech G2C1 board Graeme Russ
2015-01-21 20:53   ` Fabio Estevam
2015-01-21 23:03     ` Marek Vasut
2015-01-22  4:21       ` Graeme Russ
2015-01-21 23:36   ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54C2021C.9010604@tss-engineering.com \
    --to=gruss@tss-engineering.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox