[LinuxBIOS] [Patch] Fix processor name string for Rev. F CPUs

Sven Kapferer skapfere at rumms.uni-mannheim.de
Fri May 4 20:56:20 CEST 2007


This patch fixes the processor name string for Rev F. CPUs.
It moves the complete naming functionality to
src/cpu/amd/model_fxx/processor_name.c

Signed-off-by: Sven Kapferer <skapfere at rumms.uni-mannheim.de>
---

The current code sets the processor name string twice for Rev. F CPUs.

In src/cpu/amd/model_fxx/model_fxx_init.c the function
amd_set_name_string_f is called first. Several lines later
init_processor_name is called which doesn't recognize newer CPUs and
actually programs incorrect values, thus overwriting the previously set
CPU name. For example, this resulted in identifying an Opteron 2218 as a
Turion processor.

This patch removes the amd_set_name_string_f function from
src/cpu/amd/model_fxx/model_fxx_init.c and adds support for Rev. F CPUs
to src/cpu/amd/model_fxx/processor_name.c as described in the Revision
Guide for AMD NPT Family 0Fh Processors, AMD Document ID 33610 Rev 3.00,
October 2006.


Regards,
- Sven

-- 
Sven Kapferer
University of Mannheim - Computer Architecture Group
Address : B6 26, 68131 Mannheim, Germany
Web     : http://ra.ti.uni-mannheim.de
Email   : skapfere at rumms.uni-mannheim.de
Phone   : +49 621 181 2720   Fax: +49 621 181 2713
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: processor_name.patch
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070504/e47fee5f/attachment.ksh>


More information about the coreboot mailing list