[coreboot] how much more const could it be?

[Myles Watson]

On Fri, Nov 14, 2008 at 9:24 AM, ron minnich <rminnich at gmail.com> wrote:

>        static const u32 const * const dual_channel_slew_group_lookup[] = {
>        static const u32 const * const single_channel_slew_group_lookup[] =
> {
> /home/rminnich/coreboot-v3/build/coreboot.initram_partiallylinked.o:
> section .data.rel.ro.local: dual_channel_slew_group_lookup.3240
> single_channel_slew_group_lookup.3241
>
> what's it looking for here?


I don't really have an answer, but I have a question:

Is it a types problem?

static const u32 nc[] = {...};
static const u32 const * const dual_channel_slew_group_lookup[] = { nc, nc
};

I think maybe it doesn't like having unconstrained pointers with this many
levels.  Could you try a typedef or something like that?  I was looking at
southbridge/amd/cs5536/cs5536.c and they don't need nearly as many consts as
you had.

I could be way off :)

Thanks,
Myles



>
>
> ron
> p.s. const, const, const, const ==> spam, spam, spam, spam, spam,
> spam, spam, spam
>
> --
> coreboot mailing list: coreboot at coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081114/b32ee816/attachment.html>