[PATCH v2 15/15] x86: qemu: Remove dead code

Andy Shevchenko andriy.shevchenko at linux.intel.com
Tue Sep 1 09:05:00 CEST 2020


On Tue, Sep 01, 2020 at 02:15:32PM +0800, Bin Meng wrote:
> Hi Andy,
> 
> On Thu, Aug 20, 2020 at 6:02 PM Andy Shevchenko
> <andriy.shevchenko at linux.intel.com> wrote:
> >
> > start.S does nothing and can be safely removed. Makefile is still being used
> > by the build system, so simply drop the rule from it.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > Reviewed-by: Simon Glass <sjg at chromium.org>
> > ---
> > v2: added Rb tag(s)
> >
> >  board/emulation/qemu-x86/Makefile | 2 --
> >  board/emulation/qemu-x86/start.S  | 8 --------
> >  2 files changed, 10 deletions(-)
> >  delete mode 100644 board/emulation/qemu-x86/start.S
> >
> > diff --git a/board/emulation/qemu-x86/Makefile b/board/emulation/qemu-x86/Makefile
> > index 782e298b74ce..5af8d31d5840 100644
> > --- a/board/emulation/qemu-x86/Makefile
> > +++ b/board/emulation/qemu-x86/Makefile
> > @@ -1,5 +1,3 @@
> >  # SPDX-License-Identifier: GPL-2.0+
> >  #
> >  # Copyright (C) 2015, Bin Meng <bmeng.cn at gmail.com>
> > -
> > -obj-y  += start.o
> 
> Unfortunately this does not build.
> 
> We possibly need to fix the top-level Makefile and promote the Kconfig
> option for all boards?

Hmm... I believe that I left Makefile in order to make it build.
Which configuration had you used to reproduce?

Okay, I see linker error now. Let me look into the issue.

-- 
With Best Regards,
Andy Shevchenko




More information about the U-Boot mailing list