Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/27204 )
Change subject: nb/intel/i945: Fix C1DRBs values when channel B is empty ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/#/c/27204/5//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/27204/5//COMMIT_MSG@9 PS5, Line 9: C1DRBs must be programmed with the same value as C0DRB3 : when the channel B is empty. : If channels are interleaved, C0DRB* is equal to C1DRB*. The code and the comments suggest this is untrue so what makes you believe it is?