Re: [coreboot] how much more const could it be?

[Myles Watson]

On Fri, Nov 14, 2008 at 9:24 AM, ron minnich  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