All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Gmeiner <christian.gmeiner@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 06/18] x86: Rename e820entry to e820_entry
Date: Thu, 12 Apr 2018 09:28:03 +0200	[thread overview]
Message-ID: <CAH9NwWe+78uHBdY4M5Ayqpee=apMCs-6OojmR1G9YL0X6eubZA@mail.gmail.com> (raw)
In-Reply-To: <1523509343-18123-7-git-send-email-bmeng.cn@gmail.com>

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

Reviewed-by: Christian Gmeiner <christian.gmeiner@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

  reply	other threads:[~2018-04-12  7:28 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-12  5:02 [U-Boot] [PATCH v2 00/18] bootvx: Various enhancements to booting VxWorks x86 kernels Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 01/18] doc: vxworks: Minor update for clarity Bin Meng
2018-04-16  9:15   ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 02/18] bootvx: x86: Prepare e820 related stuff from the given kernel memory base address Bin Meng
2018-04-12  7:25   ` Christian Gmeiner
2018-04-16  9:15     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 03/18] bootvx: x86: Explicitly clear the bootloader image size Bin Meng
2018-04-12 16:41   ` Simon Glass
2018-04-16  9:15     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 04/18] vxworks: x86: Rename e820info to e820_info Bin Meng
2018-04-12  7:26   ` Christian Gmeiner
2018-04-16  9:15     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 05/18] x86: Use 'unsigned int' in install_e820_map() functions Bin Meng
2018-04-12  7:27   ` Christian Gmeiner
2018-04-16  9:15     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 06/18] x86: Rename e820entry to e820_entry Bin Meng
2018-04-12  7:28   ` Christian Gmeiner [this message]
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 07/18] elf: Clean up the ELF header file Bin Meng
2018-04-12  7:29   ` Christian Gmeiner
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 08/18] elf: Add ELF64 related structure defines Bin Meng
2018-04-12 16:41   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 09/18] elf: Add a very simple ELF64 loader Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-13  3:30     ` Bin Meng
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 10/18] bios: vesa: Guard setting vesa mode with CONFIG_FRAMEBUFFER_SET_VESA_MODE Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-13  3:30     ` Bin Meng
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 11/18] video: vesa: Change default FRAMEBUFFER_VESA_MODE Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 12/18] x86: Change default FRAMEBUFFER_VESA_MODE of some boards Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 13/18] pci: video: Only print out when everything is OK Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 14/18] bootvx: x86: Make VxWorks EFI console driver happy Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 15/18] bootvx: Refactor the bootline copy codes a little bit Bin Meng
2018-04-12 16:42   ` Simon Glass
2018-04-16  9:16     ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 16/18] bootvx: Exit if bootline address is not specified Bin Meng
2018-04-16  9:17   ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 17/18] bootvx: x86: Assign bootaddr based on kernel memory base Bin Meng
2018-04-16  9:17   ` Bin Meng
2018-04-12  5:02 ` [U-Boot] [PATCH v2 18/18] doc: vxworks: Update x86 specific instructions Bin Meng
2018-04-16  9:17   ` Bin Meng

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAH9NwWe+78uHBdY4M5Ayqpee=apMCs-6OojmR1G9YL0X6eubZA@mail.gmail.com' \
    --to=christian.gmeiner@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.