[U-Boot] [PATCH 1/2] Revert "board_f: Use static print_cpuinfo if CONFIG_CPU is active"

Bin Meng bmeng.cn at gmail.com
Mon Nov 12 04:55:43 UTC 2018


Hi Simon,

On Mon, Nov 12, 2018 at 11:25 AM Simon Glass <sjg at chromium.org> wrote:
>
> Hi Bin,
>
> On 7 November 2018 at 17:24, Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > Hi Simon,
> >
> > On Thu, Nov 8, 2018 at 1:09 AM Simon Glass <sjg at chromium.org> wrote:
> > >
> > > On 7 November 2018 at 04:50, Bin Meng <bmeng.cn at gmail.com> wrote:
> > > >
> > > > This reverts commit c0434407b595f785fc7401237896c48c791b45fd.
> > > >
> > > > It turns out commit c0434407b595 broke some boards which have DM CPU
> > > > driver with CONFIG_DISPLAY_CPUINFO option on. These boards just fail
> > > > to boot when print_cpuinfo() is called during boot.
> > > >
> > > > Fixes are already sent to ML and in u-boot-dm/next, however since
> > > > we are getting close to the v2018.11 release, it's safer we revert
> > > > the original commit.
> > > >
> > > > This commit should be reverted after v2018.11 release.
> > > >
> > > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > > > ---
> > > >
> > > >  common/board_f.c | 28 ----------------------------
> > > >  include/init.h   |  7 -------
> > > >  2 files changed, 35 deletions(-)
> > >
> > > Thanks Bin. I'd like to get u-boot-dm/next set up ready for the merge
> > > window, so I presume in that we need to un-revert these changes and
> > > then pick up your two series?
> > >
> >
> > That is correct. I can help test u-boot-dm/next.
>
> OK thank you. It is ready there now.
>

Do you mean u-boot-dm/next? It looks to me the repo was updated 7 days ago.

Regards,
Bin


More information about the U-Boot mailing list