[U-Boot] [PATCH v2 06/18] x86: Rename e820entry to e820_entry

Christian Gmeiner christian.gmeiner at gmail.com
Thu Apr 12 07:28:03 UTC 2018


2018-04-12 7:02 GMT+02:00 Bin Meng <bmeng.cn at gmail.com>:
> This changes 'struct e820entry' to 'struct e820_entry' to conform
> with the coding style.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>

Reviewed-by: Christian Gmeiner <christian.gmeiner at gmail.com>

> ---
>
> Changes in v2: None
>
>  arch/x86/cpu/coreboot/sdram.c    | 2 +-
>  arch/x86/cpu/qemu/e820.c         | 2 +-
>  arch/x86/cpu/tangier/sdram.c     | 4 ++--
>  arch/x86/include/asm/bootparam.h | 2 +-
>  arch/x86/include/asm/e820.h      | 4 ++--
>  arch/x86/lib/coreboot_table.c    | 2 +-
>  arch/x86/lib/e820.c              | 2 +-
>  arch/x86/lib/fsp/fsp_dram.c      | 2 +-
>  cmd/elf.c                        | 6 +++---
>  9 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/arch/x86/cpu/coreboot/sdram.c b/arch/x86/cpu/coreboot/sdram.c
> index 05918bc..885fc6f 100644
> --- a/arch/x86/cpu/coreboot/sdram.c
> +++ b/arch/x86/cpu/coreboot/sdram.c
> @@ -13,7 +13,7 @@
>  DECLARE_GLOBAL_DATA_PTR;
>
>  unsigned int install_e820_map(unsigned int max_entries,
> -                             struct e820entry *entries)
> +                             struct e820_entry *entries)
>  {
>         unsigned int num_entries;
>         int i;
> diff --git a/arch/x86/cpu/qemu/e820.c b/arch/x86/cpu/qemu/e820.c
> index 11409dd..ec733cb 100644
> --- a/arch/x86/cpu/qemu/e820.c
> +++ b/arch/x86/cpu/qemu/e820.c
> @@ -10,7 +10,7 @@
>  DECLARE_GLOBAL_DATA_PTR;
>
>  unsigned int install_e820_map(unsigned int max_entries,
> -                             struct e820entry *entries)
> +                             struct e820_entry *entries)
>  {
>         entries[0].addr = 0;
>         entries[0].size = ISA_START_ADDRESS;
> diff --git a/arch/x86/cpu/tangier/sdram.c b/arch/x86/cpu/tangier/sdram.c
> index daa9909..f5f412a 100644
> --- a/arch/x86/cpu/tangier/sdram.c
> +++ b/arch/x86/cpu/tangier/sdram.c
> @@ -100,7 +100,7 @@ static struct sfi_table_simple *sfi_search_mmap(void)
>              i++, mentry++)                                             \
>
>  static unsigned int sfi_setup_e820(unsigned int max_entries,
> -                                  struct e820entry *entries)
> +                                  struct e820_entry *entries)
>  {
>         struct sfi_table_simple *sb;
>         struct sfi_mem_entry *mentry;
> @@ -190,7 +190,7 @@ static phys_size_t sfi_get_ram_size(void)
>  }
>
>  unsigned int install_e820_map(unsigned int max_entries,
> -                             struct e820entry *entries)
> +                             struct e820_entry *entries)
>  {
>         return sfi_setup_e820(max_entries, entries);
>  }
> diff --git a/arch/x86/include/asm/bootparam.h b/arch/x86/include/asm/bootparam.h
> index 6aba614..0386cbe 100644
> --- a/arch/x86/include/asm/bootparam.h
> +++ b/arch/x86/include/asm/bootparam.h
> @@ -111,7 +111,7 @@ struct boot_params {
>         struct setup_header hdr;    /* setup header */  /* 0x1f1 */
>         __u8  _pad7[0x290-0x1f1-sizeof(struct setup_header)];
>         __u32 edd_mbr_sig_buffer[EDD_MBR_SIG_MAX];      /* 0x290 */
> -       struct e820entry e820_map[E820MAX];             /* 0x2d0 */
> +       struct e820_entry e820_map[E820MAX];            /* 0x2d0 */
>         __u8  _pad8[48];                                /* 0xcd0 */
>         struct edd_info eddbuf[EDDMAXNR];               /* 0xd00 */
>         __u8  _pad9[276];                               /* 0xeec */
> diff --git a/arch/x86/include/asm/e820.h b/arch/x86/include/asm/e820.h
> index 8355c34..9d29f82 100644
> --- a/arch/x86/include/asm/e820.h
> +++ b/arch/x86/include/asm/e820.h
> @@ -12,7 +12,7 @@
>  #ifndef __ASSEMBLY__
>  #include <linux/types.h>
>
> -struct e820entry {
> +struct e820_entry {
>         __u64 addr;     /* start of memory segment */
>         __u64 size;     /* size of memory segment */
>         __u32 type;     /* type of memory segment */
> @@ -25,6 +25,6 @@ struct e820entry {
>
>  /* Implementation defined function to install an e820 map */
>  unsigned int install_e820_map(unsigned int max_entries,
> -                             struct e820entry *);
> +                             struct e820_entry *);
>
>  #endif /* _ASM_X86_E820_H */
> diff --git a/arch/x86/lib/coreboot_table.c b/arch/x86/lib/coreboot_table.c
> index b1b4cd9..5e0edd3 100644
> --- a/arch/x86/lib/coreboot_table.c
> +++ b/arch/x86/lib/coreboot_table.c
> @@ -100,7 +100,7 @@ void write_coreboot_table(u32 addr, struct memory_area *cfg_tables)
>         struct cb_record *cbr;
>         struct cb_memory *mem;
>         struct cb_memory_range *map;
> -       struct e820entry e820[32];
> +       struct e820_entry e820[32];
>         struct cb_framebuffer *fb;
>         struct vesa_mode_info *vesa;
>         int i, num;
> diff --git a/arch/x86/lib/e820.c b/arch/x86/lib/e820.c
> index c2595b1..84c8fab 100644
> --- a/arch/x86/lib/e820.c
> +++ b/arch/x86/lib/e820.c
> @@ -18,7 +18,7 @@ DECLARE_GLOBAL_DATA_PTR;
>   *     CONFIG_PCIE_ECAM_BASE   PCIe ECAM
>   */
>  __weak unsigned int install_e820_map(unsigned int max_entries,
> -                                    struct e820entry *entries)
> +                                    struct e820_entry *entries)
>  {
>         entries[0].addr = 0;
>         entries[0].size = ISA_START_ADDRESS;
> diff --git a/arch/x86/lib/fsp/fsp_dram.c b/arch/x86/lib/fsp/fsp_dram.c
> index c1c6c54..662d4ac 100644
> --- a/arch/x86/lib/fsp/fsp_dram.c
> +++ b/arch/x86/lib/fsp/fsp_dram.c
> @@ -63,7 +63,7 @@ ulong board_get_usable_ram_top(ulong total_size)
>  }
>
>  unsigned int install_e820_map(unsigned int max_entries,
> -                             struct e820entry *entries)
> +                             struct e820_entry *entries)
>  {
>         unsigned int num_entries = 0;
>         const struct hob_header *hdr;
> diff --git a/cmd/elf.c b/cmd/elf.c
> index 0717feb..824f88e 100644
> --- a/cmd/elf.c
> +++ b/cmd/elf.c
> @@ -210,7 +210,7 @@ int do_bootvx(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>  #ifdef CONFIG_X86
>         ulong base;
>         struct e820_info *info;
> -       struct e820entry *data;
> +       struct e820_entry *data;
>  #endif
>
>         /*
> @@ -339,13 +339,13 @@ int do_bootvx(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>          * if we don't specify it in the environment, use a default one.
>          */
>         base = env_get_hex("vx_phys_mem_base", VXWORKS_PHYS_MEM_BASE);
> -       data = (struct e820entry *)(base + E820_DATA_OFFSET);
> +       data = (struct e820_entry *)(base + E820_DATA_OFFSET);
>         info = (struct e820_info *)(base + E820_INFO_OFFSET);
>
>         memset(info, 0, sizeof(struct e820_info));
>         info->sign = E820_SIGNATURE;
>         info->entries = install_e820_map(E820MAX, data);
> -       info->addr = (info->entries - 1) * sizeof(struct e820entry) +
> +       info->addr = (info->entries - 1) * sizeof(struct e820_entry) +
>                      E820_DATA_OFFSET;
>
>         /*
> --
> 2.7.4
>



-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info


More information about the U-Boot mailing list