From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] imx: ddr: Move mx6q_4x_mt41j128.cfg to mx6sabresd board dir
Date: Mon, 01 Sep 2014 21:20:01 +0200 [thread overview]
Message-ID: <20140901192001.87255382237@gemini.denx.de> (raw)
In-Reply-To: <CAP9ODKrsC4O-P7+CmKKbeDa8wwWxVpNpXOyOctP6WcVoJ5gYeA@mail.gmail.com>
Dear Otavio,
In message <CAP9ODKrsC4O-P7+CmKKbeDa8wwWxVpNpXOyOctP6WcVoJ5gYeA@mail.gmail.com> you wrote:
>
> I disagree (but this is my personal view on this). The reason why I
> disagree is because the DDR calibration is very design dependant so
> if/when Freescale optimize their DDR data setting they may break any
> other board using it however they shouldn't be blamed by it as this is
> their DDR settings. Any board including this file (which can be and is
> done) takes the responsibility and risks.
Obviously, any changes to common code used by several boards needs to
be tested on the affected boards.
Also, it might be instructive for you to read the commit message for
af7ec0b "mx6q: Factor out common DDR3 init code". Apparently Fabio
considers this "common DDR3 initialization code", so what exactly are
your arguments to the contrary?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
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
To be is to program.
next prev parent reply other threads:[~2014-09-01 19:20 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-01 14:20 [U-Boot] [PATCH v2] imx: ddr: Move mx6q_4x_mt41j128.cfg to mx6sabresd board dir Nitin Garg
2014-09-01 14:24 ` Fabio Estevam
2014-09-01 14:33 ` Otavio Salvador
2014-09-01 17:59 ` Wolfgang Denk
2014-09-01 18:10 ` Otavio Salvador
2014-09-01 19:02 ` Wolfgang Denk
2014-09-01 19:05 ` Michael Trimarchi
2014-09-01 19:11 ` Otavio Salvador
2014-09-01 19:20 ` Wolfgang Denk [this message]
2014-09-01 19:24 ` Wolfgang Denk
2014-09-01 22:27 ` Fabio Estevam
2014-09-01 23:34 ` Eric Nelson
2014-09-01 23:51 ` Fabio Estevam
2014-09-12 7:53 ` Stefano Babic
2014-09-13 21:07 ` Fabio Estevam
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=20140901192001.87255382237@gemini.denx.de \
--to=wd@denx.de \
--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