[U-Boot] [PATCH 2/5] x86: Forward declare gd_t
Graeme Russ
graeme.russ at gmail.com
Wed Nov 28 02:43:33 CET 2012
Hi Simon,
Was there a reason to repost these patches? I notice you did a couple of
tweaks and added your SoB, but for something so trivial, I don't think
there was a need to repost
Regards,
Graeme
On Wed, Nov 28, 2012 at 12:38 PM, Simon Glass <sjg at chromium.org> wrote:
> From: Graeme Russ <graeme.russ at gmail.com>
>
> So it can be used as a type in struct global_data and remove an ugly
> typecast
>
> Signed-off-by: Graeme Russ <graeme.russ at gmail.com>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Acked-by: Marek Vasut <marex at denx.de>
> ---
> arch/x86/cpu/cpu.c | 2 +-
> arch/x86/include/asm/global_data.h | 4 +++-
> arch/x86/lib/init_helpers.c | 2 +-
> 3 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/arch/x86/cpu/cpu.c b/arch/x86/cpu/cpu.c
> index e9bb0d7..67de6bc 100644
> --- a/arch/x86/cpu/cpu.c
> +++ b/arch/x86/cpu/cpu.c
> @@ -92,7 +92,7 @@ static void load_gdt(const u64 *boot_gdt, u16
> num_entries)
>
> void init_gd(gd_t *id, u64 *gdt_addr)
> {
> - id->gd_addr = (ulong)id;
> + id->gd_addr = id;
> setup_gdt(id, gdt_addr);
> }
>
> diff --git a/arch/x86/include/asm/global_data.h
> b/arch/x86/include/asm/global_data.h
> index bce999f..d471367 100644
> --- a/arch/x86/include/asm/global_data.h
> +++ b/arch/x86/include/asm/global_data.h
> @@ -33,9 +33,11 @@
>
> #ifndef __ASSEMBLY__
>
> +typedef struct global_data gd_t;
> +
> typedef struct global_data {
> /* NOTE: gd_addr MUST be first member of struct global_data! */
> - unsigned long gd_addr; /* Location of Global Data */
> + gd_t *gd_addr; /* Location of Global Data */
> bd_t *bd;
> unsigned long flags;
> unsigned int baudrate;
> diff --git a/arch/x86/lib/init_helpers.c b/arch/x86/lib/init_helpers.c
> index 9ec34ff..2f718d7 100644
> --- a/arch/x86/lib/init_helpers.c
> +++ b/arch/x86/lib/init_helpers.c
> @@ -126,7 +126,7 @@ int copy_gd_to_ram_f_r(void)
> * in-RAM copy of Global Data (calculate_relocation_address()
> * has already calculated the in-RAM location of the GDT)
> */
> - ram_gd->gd_addr = (ulong)ram_gd;
> + ram_gd->gd_addr = ram_gd;
> init_gd(ram_gd, (u64 *)gd->gdt_addr);
>
> return 0;
> --
> 1.7.7.3
>
>
More information about the U-Boot
mailing list