[U-Boot] [PATCH v2 21/27] x86: Use consistent name XXX_ADDR for binary blob flash address
Simon Glass
sjg at chromium.org
Fri Dec 12 04:26:18 CET 2014
On 9 December 2014 at 07:50, Bin Meng <bmeng.cn at gmail.com> wrote:
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>
> ---
>
> Changes in v2:
> - New patch to use consistent name XXX_ADDR for binary blobs
>
> Makefile | 2 +-
> arch/x86/cpu/ivybridge/sdram.c | 2 +-
> arch/x86/cpu/queensbay/Kconfig | 4 ++--
> arch/x86/cpu/queensbay/fsp_support.c | 2 +-
> arch/x86/cpu/queensbay/tnc_car.S | 2 +-
> include/configs/chromebook_link.h | 2 +-
> 6 files changed, 7 insertions(+), 7 deletions(-)
Well you could squash most of this into the previous patch, but it
doesn't matter.
Acked-by: Simon Glass <sjg at chromium.org>
>
> diff --git a/Makefile b/Makefile
> index c9ae77b..ff4c4aa 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -973,7 +973,7 @@ IFDTOOL_ME_FLAGS += -i ME:$(srctree)/board/$(BOARDDIR)/me.bin
> endif
>
> ifneq ($(CONFIG_HAVE_MRC),)
> -IFDTOOL_FLAGS += -w $(CONFIG_X86_MRC_START):$(srctree)/board/$(BOARDDIR)/mrc.bin
> +IFDTOOL_FLAGS += -w $(CONFIG_X86_MRC_ADDR):$(srctree)/board/$(BOARDDIR)/mrc.bin
> endif
>
> ifneq ($(CONFIG_X86_OPTION_ROM_ADDR),)
> diff --git a/arch/x86/cpu/ivybridge/sdram.c b/arch/x86/cpu/ivybridge/sdram.c
> index df2b990..b95e781 100644
> --- a/arch/x86/cpu/ivybridge/sdram.c
> +++ b/arch/x86/cpu/ivybridge/sdram.c
> @@ -177,7 +177,7 @@ int sdram_initialise(struct pei_data *pei_data)
>
> debug("PEI data at %p, size %x:\n", pei_data, sizeof(*pei_data));
>
> - data = (char *)CONFIG_X86_MRC_START;
> + data = (char *)CONFIG_X86_MRC_ADDR;
> if (data) {
> int rv;
> int (*func)(struct pei_data *);
> diff --git a/arch/x86/cpu/queensbay/Kconfig b/arch/x86/cpu/queensbay/Kconfig
> index 56fe85c..f6b5201 100644
> --- a/arch/x86/cpu/queensbay/Kconfig
> +++ b/arch/x86/cpu/queensbay/Kconfig
> @@ -29,7 +29,7 @@ config FSP_FILE
> The filename of the file to use as Firmware Support Package binary
> in the board directory.
>
> -config FSP_LOCATION
> +config FSP_ADDR
> hex "Firmware Support Package binary location"
> depends on HAVE_FSP
> default 0xfffc0000
> @@ -65,7 +65,7 @@ config CMC_FILE
> The filename of the file to use as Chipset Micro Code state machine
> binary in the board directory.
>
> -config CMC_LOCATION
> +config CMC_ADDR
> hex "Chipset Micro Code state machine binary location"
> depends on HAVE_CMC
> default 0xfffb0000
> diff --git a/arch/x86/cpu/queensbay/fsp_support.c b/arch/x86/cpu/queensbay/fsp_support.c
> index 58a30de..e85f823 100644
> --- a/arch/x86/cpu/queensbay/fsp_support.c
> +++ b/arch/x86/cpu/queensbay/fsp_support.c
> @@ -64,7 +64,7 @@ UINT32 __attribute__((optimize("O0"))) find_fsp_header(void)
> volatile register UINT8 * fsp asm("eax");
>
> /* Initalize the FSP base */
> - fsp = (UINT8 *)CONFIG_FSP_LOCATION;
> + fsp = (UINT8 *)CONFIG_FSP_ADDR;
>
> /* Check the FV signature, _FVH */
> if (((EFI_FIRMWARE_VOLUME_HEADER *)fsp)->signature == 0x4856465F) {
> diff --git a/arch/x86/cpu/queensbay/tnc_car.S b/arch/x86/cpu/queensbay/tnc_car.S
> index 4f39e42..344597b 100644
> --- a/arch/x86/cpu/queensbay/tnc_car.S
> +++ b/arch/x86/cpu/queensbay/tnc_car.S
> @@ -29,7 +29,7 @@ find_fsp_header_ret:
> mov %eax, %ebp
>
> /* sanity test */
> - cmp $CONFIG_FSP_LOCATION, %eax
> + cmp $CONFIG_FSP_ADDR, %eax
> jb die
>
> /* calculate TempRamInitEntry address */
> diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h
> index 645b31c..c9d84e4 100644
> --- a/include/configs/chromebook_link.h
> +++ b/include/configs/chromebook_link.h
> @@ -25,7 +25,7 @@
>
> #define CONFIG_X86_RESET_VECTOR
> #define CONFIG_NR_DRAM_BANKS 8
> -#define CONFIG_X86_MRC_START 0xfffa0000
> +#define CONFIG_X86_MRC_ADDR 0xfffa0000
> #define CONFIG_CACHE_MRC_SIZE_KB 512
>
> #define CONFIG_COREBOOT_SERIAL
> --
> 1.8.2.1
>
More information about the U-Boot
mailing list