public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 4/6] ARM: Tegra210: Add support to common Tegra source/config files
Date: Wed, 22 Jul 2015 11:52:40 -0600	[thread overview]
Message-ID: <55AFD868.7020207@wwwdotorg.org> (raw)
In-Reply-To: <1437421844-5157-5-git-send-email-twarren@nvidia.com>

On 07/20/2015 01:50 PM, Tom Warren wrote:
> Derived from Tegra124, modified as appropriate during T210
> board bringup. Cleaned up debug statements to conserve
> string space, too.

> diff --git a/include/fdtdec.h b/include/fdtdec.h

> +	COMPAT_NVIDIA_TEGRA210_SDMMC,	/* Tegra210 SDMMC controller */

> +	COMPAT_NVIDIA_TEGRA210_XUSB_PADCTL,

The DT files added in the next patch contain only the Tegra124 
compatible values, implying that the Tegra210 HW modules are 
backwards-compatible with the Tegra124 HW modules. If this is actually 
true, we don't need to update the U-Boot drivers to match against the 
Tegra210 compatible values; they can continue to match against the 
Tegra124 compatible values.

That said, our downstream DT files contain only Tegra210 compatible 
values for these modules, so I suspect the bug is that the DT patch 
needs fixing to contain the correct compatible values. This might apply 
to more HW modules too, so you might actually need to add more entries 
into this table rather than fewer...

  reply	other threads:[~2015-07-22 17:52 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-20 19:50 [U-Boot] [PATCH V2 0/6] Tegra210/P2571 initial support Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 1/6] Tegra210: Fix 64-bit build warning about save_boot_params_ret() Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow 64-bit builds (T210) Tom Warren
2015-07-22 17:40   ` Stephen Warren
2015-07-22 20:05     ` Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 3/6] ARM: Tegra210: Add SoC code/include files for T210 Tom Warren
2015-07-22 17:44   ` Stephen Warren
2015-07-23 15:34     ` Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 4/6] ARM: Tegra210: Add support to common Tegra source/config files Tom Warren
2015-07-22 17:52   ` Stephen Warren [this message]
2015-07-20 19:50 ` [U-Boot] [PATCH V2 5/6] P2571: dts: Add DT files for Tegra210/P2571 board Tom Warren
2015-07-22 17:57   ` Stephen Warren
2015-07-23 17:41     ` Tom Warren
2015-07-22 18:16   ` Stephen Warren
2015-07-23 17:44     ` Tom Warren
2015-07-23 17:59       ` Stephen Warren
2015-07-23 19:06         ` Tom Warren
2015-07-23 19:30           ` Stephen Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 6/6] T210: Add support for 64-bit T210-based P2571 board Tom Warren
2015-07-22 18:04   ` Stephen Warren
2015-07-23 23:17     ` Tom Warren
2015-07-22 18:10 ` [U-Boot] [PATCH V2 0/6] Tegra210/P2571 initial support Stephen Warren
2015-07-22 20:35 ` Stephen Warren
2015-07-22 21:07   ` Tom Warren

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=55AFD868.7020207@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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