[U-Boot] [PATCH 11/23] x86: Move common FSP code into a common location

Bin Meng bmeng.cn at gmail.com
Tue Jan 27 11:52:51 CET 2015


Hi Simon,

On Tue, Jan 27, 2015 at 9:23 AM, Simon Glass <sjg at chromium.org> wrote:
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>

Nice to see the existing FSP support codes are common for new platforms.

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Tested-by: Bin Meng <bmeng.cn at gmail.com>

>  arch/x86/cpu/queensbay/Makefile                                | 2 +-
>  arch/x86/cpu/queensbay/fsp_configs.c                           | 2 +-
>  arch/x86/cpu/queensbay/tnc.c                                   | 2 +-
>  arch/x86/cpu/queensbay/tnc_dram.c                              | 2 +-
>  arch/x86/cpu/queensbay/tnc_pci.c                               | 2 +-
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_api.h        | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_bootmode.h   | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_ffs.h        | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_fv.h         | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_hob.h        | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_infoheader.h | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_platform.h   | 0
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_support.h    | 2 +-
>  arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_types.h      | 0
>  arch/x86/lib/Makefile                                          | 1 +
>  arch/x86/lib/cmd_hob.c                                         | 2 +-
>  arch/x86/lib/fsp/Makefile                                      | 7 +++++++
>  arch/x86/{cpu/queensbay => lib/fsp}/fsp_support.c              | 2 +-
>  18 files changed, 16 insertions(+), 8 deletions(-)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_api.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_bootmode.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_ffs.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_fv.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_hob.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_infoheader.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_platform.h (100%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_support.h (99%)
>  rename arch/x86/include/asm/{arch-queensbay => }/fsp/fsp_types.h (100%)
>  create mode 100644 arch/x86/lib/fsp/Makefile
>  rename arch/x86/{cpu/queensbay => lib/fsp}/fsp_support.c (99%)
>
> diff --git a/arch/x86/cpu/queensbay/Makefile b/arch/x86/cpu/queensbay/Makefile
> index 2c2ec01..b3e7d87 100644
> --- a/arch/x86/cpu/queensbay/Makefile
> +++ b/arch/x86/cpu/queensbay/Makefile
> @@ -5,5 +5,5 @@
>  #
>
>  obj-y += tnc_car.o tnc_dram.o tnc.o topcliff.o
> -obj-y += fsp_configs.o fsp_support.o
> +obj-y += fsp_configs.o
>  obj-$(CONFIG_PCI) += tnc_pci.o
> diff --git a/arch/x86/cpu/queensbay/fsp_configs.c b/arch/x86/cpu/queensbay/fsp_configs.c
> index af28e45..78bc966 100644
> --- a/arch/x86/cpu/queensbay/fsp_configs.c
> +++ b/arch/x86/cpu/queensbay/fsp_configs.c
> @@ -6,7 +6,7 @@
>   */
>
>  #include <common.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>
>  void update_fsp_upd(struct upd_region *fsp_upd)
>  {
> diff --git a/arch/x86/cpu/queensbay/tnc.c b/arch/x86/cpu/queensbay/tnc.c
> index 8637cdc..f9b3bfa 100644
> --- a/arch/x86/cpu/queensbay/tnc.c
> +++ b/arch/x86/cpu/queensbay/tnc.c
> @@ -9,7 +9,7 @@
>  #include <asm/pci.h>
>  #include <asm/post.h>
>  #include <asm/arch/tnc.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>  #include <asm/processor.h>
>
>  static void unprotect_spi_flash(void)
> diff --git a/arch/x86/cpu/queensbay/tnc_dram.c b/arch/x86/cpu/queensbay/tnc_dram.c
> index df79a39..4c0a7c8 100644
> --- a/arch/x86/cpu/queensbay/tnc_dram.c
> +++ b/arch/x86/cpu/queensbay/tnc_dram.c
> @@ -5,7 +5,7 @@
>   */
>
>  #include <common.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>  #include <asm/e820.h>
>  #include <asm/post.h>
>
> diff --git a/arch/x86/cpu/queensbay/tnc_pci.c b/arch/x86/cpu/queensbay/tnc_pci.c
> index 39bff49..9b0b725 100644
> --- a/arch/x86/cpu/queensbay/tnc_pci.c
> +++ b/arch/x86/cpu/queensbay/tnc_pci.c
> @@ -7,7 +7,7 @@
>  #include <common.h>
>  #include <pci.h>
>  #include <asm/pci.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>
>  DECLARE_GLOBAL_DATA_PTR;
>
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_api.h b/arch/x86/include/asm/fsp/fsp_api.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_api.h
> rename to arch/x86/include/asm/fsp/fsp_api.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_bootmode.h b/arch/x86/include/asm/fsp/fsp_bootmode.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_bootmode.h
> rename to arch/x86/include/asm/fsp/fsp_bootmode.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_ffs.h b/arch/x86/include/asm/fsp/fsp_ffs.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_ffs.h
> rename to arch/x86/include/asm/fsp/fsp_ffs.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_fv.h b/arch/x86/include/asm/fsp/fsp_fv.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_fv.h
> rename to arch/x86/include/asm/fsp/fsp_fv.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_hob.h b/arch/x86/include/asm/fsp/fsp_hob.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_hob.h
> rename to arch/x86/include/asm/fsp/fsp_hob.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_infoheader.h b/arch/x86/include/asm/fsp/fsp_infoheader.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_infoheader.h
> rename to arch/x86/include/asm/fsp/fsp_infoheader.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_platform.h b/arch/x86/include/asm/fsp/fsp_platform.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_platform.h
> rename to arch/x86/include/asm/fsp/fsp_platform.h
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_support.h b/arch/x86/include/asm/fsp/fsp_support.h
> similarity index 99%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_support.h
> rename to arch/x86/include/asm/fsp/fsp_support.h
> index ebdbd03..6329cfe 100644
> --- a/arch/x86/include/asm/arch-queensbay/fsp/fsp_support.h
> +++ b/arch/x86/include/asm/fsp/fsp_support.h
> @@ -16,7 +16,7 @@
>  #include "fsp_platform.h"
>  #include "fsp_infoheader.h"
>  #include "fsp_bootmode.h"
> -#include "fsp_vpd.h"
> +#include <asm/arch/fsp/fsp_vpd.h>
>
>  struct shared_data {
>         struct fsp_header       *fsp_hdr;
> diff --git a/arch/x86/include/asm/arch-queensbay/fsp/fsp_types.h b/arch/x86/include/asm/fsp/fsp_types.h
> similarity index 100%
> rename from arch/x86/include/asm/arch-queensbay/fsp/fsp_types.h
> rename to arch/x86/include/asm/fsp/fsp_types.h
> diff --git a/arch/x86/lib/Makefile b/arch/x86/lib/Makefile
> index 32d7b98..c17f7f0 100644
> --- a/arch/x86/lib/Makefile
> +++ b/arch/x86/lib/Makefile
> @@ -25,6 +25,7 @@ obj-y += string.o
>  obj-$(CONFIG_SYS_X86_TSC_TIMER)        += tsc_timer.o
>  obj-$(CONFIG_VIDEO_VGA)        += video.o
>  obj-$(CONFIG_CMD_ZBOOT)        += zimage.o
> +obj-$(CONFIG_HAVE_FSP) += fsp/
>
>  extra-$(CONFIG_USE_PRIVATE_LIBGCC) := lib.a
>
> diff --git a/arch/x86/lib/cmd_hob.c b/arch/x86/lib/cmd_hob.c
> index a0ef037..915746a 100644
> --- a/arch/x86/lib/cmd_hob.c
> +++ b/arch/x86/lib/cmd_hob.c
> @@ -7,7 +7,7 @@
>  #include <common.h>
>  #include <command.h>
>  #include <linux/compiler.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>
>  DECLARE_GLOBAL_DATA_PTR;
>
> diff --git a/arch/x86/lib/fsp/Makefile b/arch/x86/lib/fsp/Makefile
> new file mode 100644
> index 0000000..1b9b0e2
> --- /dev/null
> +++ b/arch/x86/lib/fsp/Makefile
> @@ -0,0 +1,7 @@
> +#
> +# Copyright (C) 2015 Google, Inc
> +#
> +# SPDX-License-Identifier:     GPL-2.0+
> +#
> +
> +obj-y += fsp_support.o
> diff --git a/arch/x86/cpu/queensbay/fsp_support.c b/arch/x86/lib/fsp/fsp_support.c
> similarity index 99%
> rename from arch/x86/cpu/queensbay/fsp_support.c
> rename to arch/x86/lib/fsp/fsp_support.c
> index aed3e2b..f6ae85a 100644
> --- a/arch/x86/cpu/queensbay/fsp_support.c
> +++ b/arch/x86/lib/fsp/fsp_support.c
> @@ -6,7 +6,7 @@
>   */
>
>  #include <common.h>
> -#include <asm/arch/fsp/fsp_support.h>
> +#include <asm/fsp/fsp_support.h>
>  #include <asm/post.h>
>
>  /**
> --

Regards,
Bin


More information about the U-Boot mailing list