* repository service <s...@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

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to