[U-Boot] [PATCH] configs: ls1021atwr: Enable ID EEPROM for SD boot

[York Sun]

That's good. Thanks.

York


-------- Original message --------
From: Yuan Yao-B46683
Date:10/20/2015 8:59 PM (GMT-08:00)
To: Sun York-R58495 , sinan at writeme.com, Wang Huan-B18965
Cc: u-boot at lists.denx.de
Subject: RE: [U-Boot] [PATCH] configs: ls1021atwr: Enable ID EEPROM for SD boot

Hi york,

I have tested the patch on Rev1.0.

It seems there is no effect for Rev1.0.
Just the ID EEPROM can't work and will print some error info for I2C1.
There isn't any effect for others.
Is it acceptable?

Best Regards,
Yuan Yao

> -----Original Message-----
> From: York Sun [mailto:yorksun at freescale.com]
> Sent: Monday, October 19, 2015 11:02 PM
> To: sinan at writeme.com; Yuan Yao-B46683 <yao.yuan at freescale.com>; Wang
> Huan-B18965 <alison.wang at freescale.com>
> Cc: u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH] configs: ls1021atwr: Enable ID EEPROM for SD
> boot
>
>
>
> On 10/19/2015 04:14 AM, Sinan Akman wrote:
> >
> >    Hi Yuan
> >
> > On 19/10/15 05:21 AM, Yao Yuan wrote:
> >> Hi Sinan Akman,
> >>
> >> Yes, I mean the Rev 1.0 silicon.
> >> Sorry, I can't guarantee that there aren't any boards with Rev1.0 silicon are
> in user's hands.
> >> Because we have also delivery very little board with Rev1.0 silicon to
> customer  or developer for developing, assessing and verifying in the early
> stages.
> >
> >    Thanks for the follow up on this, but I don't think that there were
> > only very few rev1.0 boards delivered. These were the boards bought
> > from distributor and during the time span of many weeks.
> >
> >    I don't know the details and the impact of this patch exactly but
> > if the patch will break any rev1.0 board I don't think this would be
> > acceptable. There are definitely enough number of rev1.0 boards made
> > generally available (not specifically for early adapters etc) that we
> > should not break u-boot running on them.
> >
> >> Also we haven't support Rev1.0 in our SDK now.
> >
> >    Sorry, but this is a bit irrelevant here. I'd expect u-boot
> > mainline does not necessarily follow what your SDK stops supporting.
> > If u-boot supported rev1.0 boards and if there are still many of
> >   them out there I see no reason to break this.
> >
>
> Yuan,
>
> Please test the patch on rev 1.0 SoC and make changes if needed. This feature
> has been disabled for rev 1.0.
>
> York