* 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
