All of lore.kernel.org
 help / color / mirror / Atom feed
From: Walter Lozano <walter.lozano@collabora.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] ARM: dts: imx6qdl: Add U-Boot specific device tree files for Cubox-i
Date: Wed, 30 Oct 2019 16:58:17 -0300	[thread overview]
Message-ID: <0445b9c1-c510-9c43-8835-d2466c7847a3@collabora.com> (raw)
In-Reply-To: <CAOMZO5D_Esep7_sxiu41_=1LBg5xXnUTL9k__p=gv1pVkigwCA@mail.gmail.com>

Hi Fabio,

On 30/10/19 15:43, Fabio Estevam wrote:
> Hi Walter,
>
> On Wed, Oct 30, 2019 at 11:22 AM Walter Lozano
> <walter.lozano@collabora.com> wrote:
>
>> Thanks for your comments. My intention was to initially send the device tree patches and in a different patchset update the configuration, as it requires to run different tests (i.e. boot from SATA). Also I felt that this patch has meaning by itself, as it could be useful for people willing to test DM on this board.
> It is preferred that you resend the dts pieces only when the cubox
> board is really converted to DM, otherwise you are only adding "dead
> code".
Thanks for the suggestion. I will wait for some SATA tests before re 
sending these patches.
> Also the imx6dl-cubox-i-u-boot.dtb does not need to be explicitly
> added in the Makefile.
>
> It gets built automatically by the core U-Boot Makefile logic.

Thanks for the information. I was confused by entries like 
"r8a7790-lager-u-boot.dtb" in the Makefile but you are right. I will 
take this into account for v2.
> Thanks

Regards

  reply	other threads:[~2019-10-30 19:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-28 14:00 [U-Boot] [PATCH 1/2] ARM: dts: imx6qdl: Add device tree support for SolidRun Cubox-i Walter Lozano
2019-10-28 14:00 ` [U-Boot] [PATCH 2/2] ARM: dts: imx6qdl: Add U-Boot specific device tree files for Cubox-i Walter Lozano
2019-10-28 14:11   ` Fabio Estevam
2019-10-30 14:22     ` [U-Boot] ?==?utf-8?q? ?==?utf-8?q? [PATCH 2/2]?==?utf-8?q? ARM:?==?utf-8?q? " Walter Lozano
2019-10-30 18:43       ` [U-Boot] [PATCH 2/2] ARM: " Fabio Estevam
2019-10-30 19:58         ` Walter Lozano [this message]
2019-11-07 13:48           ` Fabio Estevam
2019-11-07 14:38             ` Walter Lozano

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=0445b9c1-c510-9c43-8835-d2466c7847a3@collabora.com \
    --to=walter.lozano@collabora.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.