[coreboot] [patch][v2] AMD Fam10 HT cleanup
Marc Jones
Marc.Jones at amd.com
Thu Jul 17 01:51:47 CEST 2008
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.
>
> Regards,
> Carl-Daniel
>
The example code that this is based on has to build with a pre-c99
compiler. c99 initializers would have been the way to go if that were
not the case. I wouldn't nack a change but it does make it easier to
track changes if the code stays similar.
Marc
--
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors
More information about the coreboot
mailing list