David Schneider has posted comments on this change. ( https://review.coreboot.org/22891 )
Change subject: google/gru: Adjust to incorrect strapping resistors on Kevin ......................................................................
Patch Set 2: Code-Review+1
Patch Set 1:
(1 comment)
Every other bucket is ~60 wide, with 30 on each side...except for bucket 1, which has 30 on the high side and 40 on the low side.
Yeah, but I still don't see the value in changing that. The buckets are skewed but that's the way they were designed. If I put a number in there that's not the arithmetic mean between the target points it's just going to confuse someone later. It's about documentation too... if I didn't split it up I'd still have to put a comment explaining that Kevin is special which takes up about the same amount of space. This is the easiest way to just represent that right in the code.
Okay, this way is fine, then.