Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33196 )
Change subject: siemens/mc_apl5: Change PTN interface settings ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/33196/1/src/mainboard/siemens/mc_apl1/varian... File src/mainboard/siemens/mc_apl1/variants/mc_apl5/ptn3460.c:
https://review.coreboot.org/#/c/33196/1/src/mainboard/siemens/mc_apl1/varian... PS1, Line 79: - No need for the hyphen here. What you want to do here is to drive the clock on the odd bus per default. If there is a need for dual lane, hwinfo data will enable the even bus in addition. So maybe change this comment to: Drive LVDS clock on odd bus per default.
But it is up to you.