[coreboot] [commit] r6380 - in trunk/src: cpu/amd/model_fxx northbridge/amd/amdk8

Stefan Reinauer stefan.reinauer at coreboot.org
Thu Feb 24 22:19:43 CET 2011


* repository service <svn at coreboot.org> [110224 15:35]:
> Modified: trunk/src/cpu/amd/model_fxx/model_fxx_init.c
> ==============================================================================
> --- trunk/src/cpu/amd/model_fxx/model_fxx_init.c	Thu Feb 24 14:54:10 2011	(r6379)
> +++ trunk/src/cpu/amd/model_fxx/model_fxx_init.c	Thu Feb 24 15:35:42 2011	(r6380)
> @@ -264,7 +264,10 @@
>  
>  	/* See if we scrubbing should be enabled */
>  	enable_scrubbing = 1;
> -	get_option(&enable_scrubbing, "hw_scrubber");
> +	if( get_option(&enable_scrubbing, "hw_scrubber") < 0 ) 
> +	{
> +		enable_scrubbing = CONFIG_HW_SCRUBBER;
> +	}
 
small style note: if() wants the curly brackets on the same line, new
functions don't.

Also, does it make sense to initialize enable_scrubbing like this if
it's overwritten anyways?


Stefan




More information about the coreboot mailing list