From: Eric Nelson <eric.nelson@boundarydevices.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH] Add support for Boundary Devices Nitrogen6x boards
Date: Fri, 08 Mar 2013 16:25:50 -0700 [thread overview]
Message-ID: <513A737E.6070403@boundarydevices.com> (raw)
In-Reply-To: <CAOMZO5AuhodW4OmZF0T=WnBAjFzvQhwUV-7Du7V0uW-Rfo3OFw@mail.gmail.com>
Thanks for the review Fabio,
On 03/08/2013 03:49 PM, Fabio Estevam wrote:
> Hi Eric,
>
> On Wed, Jan 30, 2013 at 4:10 PM, Eric Nelson
> <eric.nelson@boundarydevices.com> wrote:
>> This patch adds support for the Nitrogen6X board(s) from
>> Boundary Devices. The boards are based on the i.MX6x
>> processor family with two major variants:
>> i.MX6Quad/6Dual
>> i.MX6Dual-Lite/6Solo
> ....
>> board/boundary/nitrogen6x/800mhz_2x128mx16.cfg | 37 +
>> board/boundary/nitrogen6x/800mhz_2x256mx16.cfg | 37 +
>> board/boundary/nitrogen6x/800mhz_4x128mx16.cfg | 37 +
>> board/boundary/nitrogen6x/800mhz_4x256mx16.cfg | 37 +
>
> Good work, I really like this series.
>
Glad to hear it.
> I managed to test it and added support for a mx6dl board.
>
> I would like to suggest if you could move the different memory inits
> (like 800mhz_4x256mx16.cfg) into a common directory so that other
> boards can pull their DDR init from there when possible.
>
Okay, but this might be a bit of a stretch. We went through a
lot of our SABRE Lite, Nitrogen6x, and Nitrogen6x SOM boards
to select the right median values for the termination values
and such.
These may not apply for other boards, so another level of
indirection might be appropriate.
IOW, while most of the registers in the files like
800mhz_4x256mx16.cfg are for the memory **arrangement**,
some of them are or could be specific to the layout and
PCB fabrication details.
> Do you think we still can make this go into 2013.04?
>
Yep.
Now that this guy at Freescale fixed up the HDMI stuff,
I can even re-send the HDMI detect patch ;)
I'll send V2 tomorrow.
Regards,
Eric
prev parent reply other threads:[~2013-03-08 23:25 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-30 18:10 [U-Boot] [RFC] Add Nitrogen6x support Eric Nelson
2013-01-30 18:10 ` [U-Boot] [RFC PATCH] Add support for Boundary Devices Nitrogen6x boards Eric Nelson
2013-02-01 20:02 ` Benoît Thébaudeau
2013-02-12 9:12 ` Stefano Babic
2013-02-14 3:54 ` Eric Nelson
2013-03-08 22:49 ` Fabio Estevam
2013-03-08 23:25 ` Eric Nelson [this message]
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=513A737E.6070403@boundarydevices.com \
--to=eric.nelson@boundarydevices.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