On 03.10.2010 21:18, Patrick Georgi wrote: > Am 03.10.2010 20:47, schrieb Uwe Hermann: > >> - movl $0xff, %edx /* (1 << (CONFIG_CPU_ADDR_BITS - 32)) - 1 for K8 >> (CONFIG_CPU_ADDR_BITS = 40) */ >> - jmp_if_k8(wbcache_post_fam10_setup) >> - movl $0xffff, %edx /* (1 << (CONFIG_CPU_ADDR_BITS - 32)) - 1 for >> FAM10 (CONFIG_CPU_ADDR_BITS = 48) */ >> -wbcache_post_fam10_setup: >> + movl $CONFIG_CPU_ADDR_BITS_MASK, %edx >> > This was to allow a single image to run through CAR with both K8 and > Fam10. I suppose that's not necessary anymore, given that the bootblock > doesn't use CAR, and any chip selection can and should happen there. >
I want to be able to use my socket AM2+ board with K8 and Fam10 processors. If I also want fallback/normal images, this change forces me to use 4 images instead of 2. And yes, people who use mainboards actually may expect to be able to use processors from different generations if the "normal" BIOS can handle that as well. Due to that, I'd say Nack, but if you have strong technical reasons (instead of cosmetic reasons) which result in a net benefit to end users even if you factor in the feature loss, I will retract the Nack. Regards, Carl-Daniel -- http://www.hailfinger.org/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
