Re: [coreboot] r813 - coreboot-v3/lib

[Carl-Daniel Hailfinger]

On 24.08.2008 13:29, [e-mail] wrote:
> Author: stepan
> Date: 2008-08-24 13:29:11 +0200 (Sun, 24 Aug 2008)
> New Revision: 813
>
> Modified:
>    coreboot-v3/lib/lar.c
> Log:
> needs to be const, or the segment checker will bail out. (trivial)
> Signed-off-by: Stefan Reinauer 
> Acked-by: Stefan Reinauer
> Modified: coreboot-v3/lib/lar.c
>
> --- coreboot-v3/lib/lar.c	2008-08-24 07:08:17 UTC (rev 812)
> +++ coreboot-v3/lib/lar.c	2008-08-24 11:29:11 UTC (rev 813)
> @@ -31,7 +31,7 @@
>  #define ntohl(x) (x)
>  #endif
>
> -static const char *algo_name[] = {
> +static const char * const algo_name[] = {
>    "none",
>    "lzma",
>    "nrv2b",

Interesting. The checker didn't complain here. Which gcc are you using?
I'm using gcc 4.2.1.

By the way, can we officially require at least gcc 4.1? Otherwise,
compiling RAMinit won't work reliably if at all.

Regards,
Carl-Daniel