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

2018-04-16 Thread Bin Meng
On Thu, Apr 12, 2018 at 3:28 PM, Christian Gmeiner
 wrote:
> 2018-04-12 7:02 GMT+02:00 Bin Meng :
>> This changes 'struct e820entry' to 'struct e820_entry' to conform
>> with the coding style.
>>
>> Signed-off-by: Bin Meng 
>
> Reviewed-by: Christian Gmeiner 
>
>> ---
>>
>> 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(-)
>>

applied to u-boot-x86, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2018-04-12 Thread Christian Gmeiner
2018-04-12 7:02 GMT+02:00 Bin Meng :
> This changes 'struct e820entry' to 'struct e820_entry' to conform
> with the coding style.
>
> Signed-off-by: Bin Meng 

Reviewed-by: Christian Gmeiner 

> ---
>
> 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 
>
> -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,
> -