Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40531 )
Change subject: mb/google/dedede: Remove pad termination for RAM_STRAP_4 ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40531/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40531/1//COMMIT_MSG@10 PS1, Line 10: Remove the internal pull-up.
All the current board versions have both channels populated. […]
Sorry I have to take a step back.
Does that mean that all the boards from current hardware revision have external PD stuffed?
External PD stuffed -> Single channel configuration External PU stuffed -> Dual channel configuration
Currently there are 3 revisions of board. 1) No resistor strapping stuffed - Dual channel configuration 2) External PU stuffed - Dual channel configuration 3) External PD stuffed - Single Channel configuration