[U-Boot] [PATCH 1/2] ARM: tegra: support SKU b1 of Tegra30

Tom Warren twarren.nvidia at gmail.com
Thu Oct 17 17:45:26 CEST 2013


Alban,


On Thu, Oct 17, 2013 at 8:26 AM, Thierry Reding <thierry.reding at gmail.com>wrote:

> On Thu, Oct 17, 2013 at 08:59:22AM -0600, Stephen Warren wrote:
> > On 10/17/2013 04:50 AM, Thierry Reding wrote:
> > > On Thu, Oct 17, 2013 at 11:29:06AM +0200, Alban Bedel wrote:
> > >> On Mon, 23 Sep 2013 17:23:12 -0700 Tom Warren
> > >> <TWarren at nvidia.com> wrote:
> > >>
> > >>> It's fine as-is for now. Send a V2 with any changes Stephen, et
> > >>> al requested (if any), and I'll get it into tegra-next when I
> > >>> return from vacation next Monday (assuming it's been Acked).
> > >>
> > >> What is the status with this patch serie? I'd really like to get
> > >> it merged during this window if possible.
> > >
> > > One of the comments was that the SKU is internally listed as
> > > TM30MQS-A3, not TM30MQS. I assume Tom was expecting you to spin a
> > > v2 of the patch to incorporate that change.
> >
> > Are you sure A3 isn't the silicon revision, and hence not part of the
> > SKU name?
>
> I'm pretty sure. There's a T30MQS-A2 in the same file I found the other
> one and that has SKU 0xb3. Well, I suppose that might not mean anything
> regarding the silicon, but at least if we stripped -A3 and -A2, then
> we'd end up with two different values for the same SKU name.
>
> Thierry
>

Whatever you decide to do here to clear up the SKU naming, I need an
Ack'ed-by patch before I can stick it in u-boot-tegra/next.

Tom


More information about the U-Boot mailing list