Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-19 Thread James Hogan
On 12/11/13 19:42, Geert Uytterhoeven wrote: > The different architectures used their own (and different) declarations: > > extern struct boot_param_header __dtb_start; > extern u32 __dtb_start[]; > extern char __dtb_start[]; > > Consolidate them using the first variant in . > This re

Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-13 Thread Geert Uytterhoeven
On Wed, Nov 13, 2013 at 6:34 PM, Rob Herring wrote: >>> + if (!initial_boot_params || >>> + (be32_to_cpu(initial_boot_params->magic) != OF_DT_HEADER)) >>> + initial_boot_params = &__dtb_start; >>> + >>> /* check device tree validity */ >>> - if (be32_to_

Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-13 Thread Rob Herring
On 11/13/2013 11:20 AM, Geert Uytterhoeven wrote: > On Wed, Nov 13, 2013 at 4:51 PM, Rob Herring wrote: >> On Tue, Nov 12, 2013 at 1:42 PM, Geert Uytterhoeven >> wrote: >>> The different architectures used their own (and different) declarations: >>> >>> extern struct boot_param_header __dtb_s

Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-13 Thread Geert Uytterhoeven
On Wed, Nov 13, 2013 at 4:51 PM, Rob Herring wrote: > On Tue, Nov 12, 2013 at 1:42 PM, Geert Uytterhoeven > wrote: >> The different architectures used their own (and different) declarations: >> >> extern struct boot_param_header __dtb_start; >> extern u32 __dtb_start[]; >> extern char

Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-13 Thread Rob Herring
On Tue, Nov 12, 2013 at 1:42 PM, Geert Uytterhoeven wrote: > The different architectures used their own (and different) declarations: > > extern struct boot_param_header __dtb_start; > extern u32 __dtb_start[]; > extern char __dtb_start[]; > > Consolidate them using the first variant i

Re: [PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-12 Thread Vineet Gupta
On 11/13/2013 01:12 AM, Geert Uytterhoeven wrote: > The different architectures used their own (and different) declarations: > > extern struct boot_param_header __dtb_start; > extern u32 __dtb_start[]; > extern char __dtb_start[]; > > Consolidate them using the first variant in . > Th

[PATCH 14/17] dt: Consolidate __dtb_start declarations in

2013-11-12 Thread Geert Uytterhoeven
The different architectures used their own (and different) declarations: extern struct boot_param_header __dtb_start; extern u32 __dtb_start[]; extern char __dtb_start[]; Consolidate them using the first variant in . This requires adding a few "address of" operators on architectures w