[coreboot] AMD Fam10 HT cleanup
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Thu Jul 17 01:55:11 CEST 2008
On 17.07.2008 01:39, Jordan Crouse wrote:
> On 17/07/08 01:30 +0200, Carl-Daniel Hailfinger wrote:
>
>> On 17.07.2008 00:06, Marc Jones wrote:
>>
>>> Some HT cleanup. Shouldn't be anything major.
>>>
>>> Clean up AMD FAM10 HT variable initialization. The structure init is cleaner,
>>> avoid compiler warnings,
>>> and matches the AMD example code more closely.
>>>
>>> Signed-off-by: Marc Jones <marc.jones at amd.com>
>>>
>>>
>> While I do like most parts of the cleanup, I'm not so totally sure about
>> the structure inits. They are easier to read now, but I wonder whether
>> C99 initializers would have been the way to go.
>>
>
> I don't think that Stefan is going to make the repository read-only
> any time soon. There is no prejudice against future cleanups.
>
Point taken.
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list