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

Bin Meng bmeng.cn at gmail.com
Tue Sep 1 08:15:32 CEST 2020


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?

> diff --git a/board/emulation/qemu-x86/start.S b/board/emulation/qemu-x86/start.S
> deleted file mode 100644
> index e4bde561b20d..000000000000

Regards,
Bin


More information about the U-Boot mailing list