* 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