Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33913 )
Change subject: src/mainboard/asus/am1i-a/buildOpts.c: reorder lines for comparison convenience ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33913/1/src/mainboard/asus/am1i-a/buildOpts.... File src/mainboard/asus/am1i-a/buildOpts.c:
https://review.coreboot.org/#/c/33913/1/src/mainboard/asus/am1i-a/buildOpts.... PS1, Line 31: :
supported socket are at the same 'bloc'
I brought it closer to #ifdef's using this #define, but could return it back if you would like. Sadly can't do it immediately because of hard drive problems :(
https://review.coreboot.org/#/c/33913/1/src/mainboard/asus/am1i-a/buildOpts.... PS1, Line 143:
please, why this is removed?
Other board didn't have it and omitting "ul" does not cause any difference. Perhaps I should've separated it in its' own commit, but this is so tiny... Should I do this?