On Tue, 2014-04-22 at 20:18 -0500, Rob Herring wrote:
> From: Rob Herring <r...@kernel.org>
> 
> Now powerpc is the only user of struct boot_param_header and FDT defines,
> so they can be moved into the powerpc architecture code.
> 
> Signed-off-by: Rob Herring <r...@kernel.org>
> Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org>
> Cc: Paul Mackerras <pau...@samba.org>
> Cc: linuxppc-dev@lists.ozlabs.org
> ---

I assume we want to get rid of that too eventually ? :-)

We should be able to get the definitions from libfdt too....

In the meantime:

Acked-by: Benjamin Herrenschmidt <b...@kernel.crashing.org>
--

> v2: no change
> 
>  arch/powerpc/include/asm/prom.h | 39 +++++++++++++++++++++++++++++++++++++++
>  include/linux/of_fdt.h          | 37 -------------------------------------
>  2 files changed, 39 insertions(+), 37 deletions(-)
> 
> diff --git a/arch/powerpc/include/asm/prom.h b/arch/powerpc/include/asm/prom.h
> index d977b9b..74b79f0 100644
> --- a/arch/powerpc/include/asm/prom.h
> +++ b/arch/powerpc/include/asm/prom.h
> @@ -26,6 +26,45 @@
>  #include <linux/of_irq.h>
>  #include <linux/platform_device.h>
>  
> +#define OF_DT_BEGIN_NODE     0x1             /* Start of node, full name */
> +#define OF_DT_END_NODE               0x2             /* End node */
> +#define OF_DT_PROP           0x3             /* Property: name off, size,
> +                                              * content */
> +#define OF_DT_NOP            0x4             /* nop */
> +#define OF_DT_END            0x9
> +
> +#define OF_DT_VERSION                0x10
> +
> +/*
> + * This is what gets passed to the kernel by prom_init or kexec
> + *
> + * The dt struct contains the device tree structure, full pathes and
> + * property contents. The dt strings contain a separate block with just
> + * the strings for the property names, and is fully page aligned and
> + * self contained in a page, so that it can be kept around by the kernel,
> + * each property name appears only once in this page (cheap compression)
> + *
> + * the mem_rsvmap contains a map of reserved ranges of physical memory,
> + * passing it here instead of in the device-tree itself greatly simplifies
> + * the job of everybody. It's just a list of u64 pairs (base/size) that
> + * ends when size is 0
> + */
> +struct boot_param_header {
> +     __be32  magic;                  /* magic word OF_DT_HEADER */
> +     __be32  totalsize;              /* total size of DT block */
> +     __be32  off_dt_struct;          /* offset to structure */
> +     __be32  off_dt_strings;         /* offset to strings */
> +     __be32  off_mem_rsvmap;         /* offset to memory reserve map */
> +     __be32  version;                /* format version */
> +     __be32  last_comp_version;      /* last compatible version */
> +     /* version 2 fields below */
> +     __be32  boot_cpuid_phys;        /* Physical CPU id we're booting on */
> +     /* version 3 fields below */
> +     __be32  dt_strings_size;        /* size of the DT strings block */
> +     /* version 17 fields below */
> +     __be32  dt_struct_size;         /* size of the DT structure block */
> +};
> +
>  /*
>   * OF address retreival & translation
>   */
> diff --git a/include/linux/of_fdt.h b/include/linux/of_fdt.h
> index 1f882e1..5c0ab05 100644
> --- a/include/linux/of_fdt.h
> +++ b/include/linux/of_fdt.h
> @@ -17,45 +17,8 @@
>  
>  /* Definitions used by the flattened device tree */
>  #define OF_DT_HEADER         0xd00dfeed      /* marker */
> -#define OF_DT_BEGIN_NODE     0x1             /* Start of node, full name */
> -#define OF_DT_END_NODE               0x2             /* End node */
> -#define OF_DT_PROP           0x3             /* Property: name off, size,
> -                                              * content */
> -#define OF_DT_NOP            0x4             /* nop */
> -#define OF_DT_END            0x9
> -
> -#define OF_DT_VERSION                0x10
>  
>  #ifndef __ASSEMBLY__
> -/*
> - * This is what gets passed to the kernel by prom_init or kexec
> - *
> - * The dt struct contains the device tree structure, full pathes and
> - * property contents. The dt strings contain a separate block with just
> - * the strings for the property names, and is fully page aligned and
> - * self contained in a page, so that it can be kept around by the kernel,
> - * each property name appears only once in this page (cheap compression)
> - *
> - * the mem_rsvmap contains a map of reserved ranges of physical memory,
> - * passing it here instead of in the device-tree itself greatly simplifies
> - * the job of everybody. It's just a list of u64 pairs (base/size) that
> - * ends when size is 0
> - */
> -struct boot_param_header {
> -     __be32  magic;                  /* magic word OF_DT_HEADER */
> -     __be32  totalsize;              /* total size of DT block */
> -     __be32  off_dt_struct;          /* offset to structure */
> -     __be32  off_dt_strings;         /* offset to strings */
> -     __be32  off_mem_rsvmap;         /* offset to memory reserve map */
> -     __be32  version;                /* format version */
> -     __be32  last_comp_version;      /* last compatible version */
> -     /* version 2 fields below */
> -     __be32  boot_cpuid_phys;        /* Physical CPU id we're booting on */
> -     /* version 3 fields below */
> -     __be32  dt_strings_size;        /* size of the DT strings block */
> -     /* version 17 fields below */
> -     __be32  dt_struct_size;         /* size of the DT structure block */
> -};
>  
>  #if defined(CONFIG_OF_FLATTREE)
>  


_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to