From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bin Meng <bmeng.cn@gmail.com>
Date: Fri, 27 Mar 2020 00:22:01 +0800
Subject: [PATCH] x86: spi: Only use the fast SPI peripheral when support
In-Reply-To: <CAPnjgZ0T1yRsuCv9yCysG9ykPA2fF6fYPD=KTjC2_Rt0t8pXzA@mail.gmail.com>
References: <20200324074524.1.Ibc9c511db58caa8a1e4c56d7e7824d7690718aeb@changeid>
 <CAEUhbmXe4nZQrsmonpmeQ6Be=zf+J6SRXGcRGZ7GvSkzycLEqQ@mail.gmail.com>
 <CAPnjgZ0T1yRsuCv9yCysG9ykPA2fF6fYPD=KTjC2_Rt0t8pXzA@mail.gmail.com>
Message-ID: <CAEUhbmXw2u6Xqy+vV+ywXusp8EOSbEUxs0XW8mHtDhbgnVwJBw@mail.gmail.com>
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

Hi Simon,

On Fri, Mar 27, 2020 at 12:20 AM Simon Glass <sjg@chromium.org> wrote:
>
> HI Bin,
>
> On Wed, 25 Mar 2020 at 01:25, Bin Meng <bmeng.cn@gmail.com> wrote:
> >
> > Hi Simon,
> >
> > On Tue, Mar 24, 2020 at 9:45 PM Simon Glass <sjg@chromium.org> wrote:
> > >
> > > At present we query the memory map on boards which don't support it. Fix
> > > this by only doing it on Apollo Lake.
> > >
> >
> > I wonder isn't this check already covered in mrccache_get_region() below:
> >
> > ret = dm_spi_get_mmap(dev, &map_base, &map_size, &offset);
> > if (!ret) {
> > entry->base = map_base;
> > } else {
> > ret = dev_read_u32_array(dev, "memory-map", reg, 2);
> > if (ret)
> > return log_msg_ret("Cannot find memory map\n", ret);
> > entry->base = reg[0];
> > }
>
> Yes it is, so long as dm_spi_get_mmap() returns an error, as it does
> with my patch.

So does ich_get_mmap_bus() returns 0 on chromebook_link?

Regards,
Bin