[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