All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Nelson <eric.nelson@boundarydevices.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mx6sabresd: Fix the fdt file for the mx6dl version
Date: Thu, 19 Sep 2013 07:01:27 -0700	[thread overview]
Message-ID: <523B03B7.5030507@boundarydevices.com> (raw)
In-Reply-To: <CAOMZO5AGpa2yM9FjtR+zt4MdxrXYZ_kmp+KscZfZ51LHnJrXsw@mail.gmail.com>

Hi Fabio,

On 09/19/2013 04:57 AM, Fabio Estevam wrote:
> Hi Eric,
>
> On Wed, Sep 18, 2013 at 10:48 AM, Eric Nelson
> <eric.nelson@boundarydevices.com> wrote:
>
>> As you know, we're loading the kernel using a boot script,
>> with various flavors for differing locations and environments,
>> so we've found it easier to pass environment variables for
>> board and CPU type to the boot script.
>>
>> https://github.com/boundarydevices/u-boot-imx6/commit/4a4fc304e4fe63cde7b3201f0daab5b1856870e4
>>
>> This allows the boot script to produce the DTB name as shown
>> here:
>>
>> https://github.com/boundarydevices/u-boot-imx6/blob/staging/board/boundary/nitrogen6x/6x_bootscript.txt#L72
>>
>> If/when you move to a single binary for DQ/DL, you won't have
>> a separate choice as shown above.
>>
>> Let me know your thoughts on this.
>
> Yes, it looks interesting for dealing with the single binary goal we
> want to achieve.
>
>> We didn't generate a proper patch for lack of time, but I
>> think that the small addition of code has merit. We also
>> may want to discuss the specifics in more detail.
>
> When you have a chance please submit a patch.
>
Will do.

> At least for the upcoming 2013.10, it would be nice to have my
> original patch of this thread applied for sabresd though.
>
Works for me.

My response wasn't a comment on this patch. You just prompted
me to start a different discussion.

Regards,


Eric

  reply	other threads:[~2013-09-19 14:01 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-18  1:55 [U-Boot] [PATCH] mx6sabresd: Fix the fdt file for the mx6dl version Fabio Estevam
2013-09-18  2:04 ` Otavio Salvador
2013-09-18 13:48 ` Eric Nelson
2013-09-19 11:57   ` Fabio Estevam
2013-09-19 14:01     ` Eric Nelson [this message]
2013-09-20 13:44     ` Stefano Babic
2013-09-20 13:43   ` Stefano Babic
2013-09-20 14:20     ` Eric Nelson
2013-09-20 16:11 ` Stefano Babic

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=523B03B7.5030507@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.