From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Fri, 20 Sep 2013 10:03:33 -0600
Subject: [U-Boot] [PATCH 1/2] ARM: tegra: support SKU b1 of Tegra30
In-Reply-To: <20130920124501.GA23804@ulmo>
References: <1378299618-29799-1-git-send-email-alban.bedel@avionic-design.de>
	<52277510.4090905@wwwdotorg.org> <20130920124501.GA23804@ulmo>
Message-ID: <523C71D5.1060509@wwwdotorg.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 09/20/2013 06:45 AM, Thierry Reding wrote:
> On Wed, Sep 04, 2013 at 11:59:44AM -0600, Stephen Warren wrote:
>> On 09/04/2013 07:00 AM, Alban Bedel wrote:
>>> Add the Tegra30 SKU b1 and treat it like other Tegra30 chips.
>> 
>> CC'ing the Tegra maintainer would be helpful (Tom Warren; I CC'd
>> him here)
>> 
>>> diff --git a/arch/arm/cpu/tegra-common/ap.c
>>> b/arch/arm/cpu/tegra-common/ap.c
>> 
>>> @@ -71,6 +71,7 @@ int tegra_get_chip_sku(void) switch (sku_id)
>>> { case SKU_ID_T33: case SKU_ID_T30: +		case SKU_ID_T30MQS:
>> 
>> Where does the name "T30MQS" come from? Tom, can you verify what
>> we call the SKUs internally?
> 
> It seems like this is indeed called T30MQS internally. Actually
> it's called TM30MQS-A3. I can provide an internal reference if
> required.

As long as it's been checked, I don't think we need to document it. In
which case, IIRC, the patch was fine.