[U-Boot] Regression in U-boot v2019-07-rcX

[Simon Glass]

Hi Bin, Andy,

On Mon, 17 Jun 2019 at 08:49, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> On Wed, Jun 12, 2019 at 10:07:11PM +0800, Bin Meng wrote:
> > On Wed, Jun 12, 2019 at 9:49 PM Andy Shevchenko
> > <andriy.shevchenko at linux.intel.com> wrote:
> > > On Wed, Jun 12, 2019 at 04:23:37PM +0300, Andy Shevchenko wrote:
> > > > On Wed, Jun 12, 2019 at 04:18:25PM +0300, Andy Shevchenko wrote:
>
> > > commit 665cb18ea64aabbeb03d27a4c92ddec1baccb87a
> > > Author: Simon Glass <sjg at chromium.org>
> > > Date:   Thu Apr 25 21:59:06 2019 -0600
> > >
> > >     x86: Don't set up MTRRs in SPL
> > >
> > >
> > > Please revert ASAP before release, thanks!
> >
> > So it looks that MTRRs are not programmed for Intel Edison to enable cache?
> >
> > Simon, would you please take a look? I suspect simply revert this will
> > break the Chromebook SPL build?
>
> Since there is no activity on this and release is coming, I would propose to
> revert for now.

I am OK with a revert for now if we don't have another solution.

The problem here is that we want to select when the mtrrs are programmed.

With my patch, this code is enabled on edison. I think the solution
might be to set the mtrrs earlier on link. But I'll need to take a
look and it won't be for about two weeks.

Regards,
Simon