[coreboot] r813 - coreboot-v3/lib

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sun Aug 24 13:44:41 CEST 2008


On 24.08.2008 13:29, svn at coreboot.org 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 <stepan at coresystems.de>
> Acked-by: Stefan Reinauer <stepan at coresystems.de>
>
>
>
> 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




More information about the coreboot mailing list