HI,
On Fri, Aug 19, 2011 at 04:24:00PM +0200, Cousson, Benoit wrote:
> Hi Grant,
>
> The current devicetree/test branch is broken due to the introduction
> of the __initconst for dt_compat table.
>
> My original basic fix is the following one, but Felipe figured out the
> root cause. So it can be fixed by adding an extra const to make the
> whole table const:
> - const char **dt_compat; /* array of device tree
> + const char * const *dt_compat; /* array of device tree
>
> That fix will require fixing all the current users of dt_compat with
> something like that:
>
> -static const char *omap3_beagle_dt_match[] __initconst = {
> +static const char * const omap3_beagle_dt_match[] __initconst = {
>
> A couple of of_XXX APIs will have to updated as well, that's why I was
> wondering if you wanted to go that way.
>
> Any thought?the funny thing is that this was not happening with CodeSourcery 2011.03 when I sent the patch. Maybe it's either a compiler bug on one of the toolchains or newer GCCs are smart enough to create another section ?? But then again, 2011.03 isn't compiling sleep34xx.S: arch/arm/mach-omap2/sleep34xx.S: Assembler messages: arch/arm/mach-omap2/sleep34xx.S:150: Error: selected processor does not support ARM mode `smc #1' arch/arm/mach-omap2/sleep34xx.S:418: Error: selected processor does not support ARM mode `smc #1' arch/arm/mach-omap2/sleep34xx.S:429: Error: selected processor does not support ARM mode `smc #1' arch/arm/mach-omap2/sleep34xx.S:454: Error: selected processor does not support ARM mode `smc #0' arch/arm/mach-omap2/sleep34xx.S:460: Error: selected processor does not support ARM mode `smc #0' arch/arm/mach-omap2/sleep34xx.S:465: Error: selected processor does not support ARM mode `smc #0' make[1]: *** [arch/arm/mach-omap2/sleep34xx.o] Error 1 so it might be better to keep 2010.09 working. -- balbi
signature.asc
Description: Digital signature
