Alban,

Were you going to do a V2 of this patchset? 

Tom

> -----Original Message-----
> From: Stephen Warren [mailto:swar...@wwwdotorg.org]
> Sent: Friday, September 20, 2013 9:04 AM
> To: Thierry Reding
> Cc: Tom Warren; Alban Bedel; u-boot@lists.denx.de; julian.scheel@avionic-
> design.de
> Subject: Re: [U-Boot] [PATCH 1/2] ARM: tegra: support SKU b1 of Tegra30
> 
> 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.
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may 
contain
confidential information.  Any unauthorized review, use, disclosure or 
distribution
is prohibited.  If you are not the intended recipient, please contact the 
sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to