Attention is currently required from: Sathyanarayana Nujella, Varshit B Pandya, Subrata Banik, Furquan Shaikh, Maulik V Vaghela, Paul Menzel, Angel Pons, Ronak Kanabar, Jairaj Arava, Patrick Rudolph.
View Change
1 comment:
Commit Message:
Patch Set #6, Line 9: These UPD values are automatically set by the FSP.
I can't claim to understand this completely, but based on the limited information I have seen this s […]
Not quite sure I understand the rational in "..common code should set UPDs as follows.." .There are no UPD settings in common code that I know of, rather they are done in fsp_param.c, which are always specific to each SOC, not common.
Since there is no "common" UPD setting, I don't see the benefit of introducing a new config here.
Additionally, the suggestion to set the default PchHdaIDispLinkTmode -> 4T also won't work. For TGL it was the default value, and may have worked. But for ADL we had to switch to 8T for it to work properly. Please see the first set of this CL.
For me, it still makes sense to use the devicetree in mainboard to set the proper register values which are passed to UPDs in fsp_param.c (as originally done in patchset 1 & 2), and drop the subsequent changes (in patchset 3 onward)
To view, visit change 51353. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9d0bf7dc76348f7e184e8496f042badc30bf3211
Gerrit-Change-Number: 51353
Gerrit-PatchSet: 7
Gerrit-Owner: Francois Toguo Fotso <francois.toguo.fotso@intel.com>
Gerrit-Reviewer: Jairaj Arava <jairaj.arava@intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela@intel.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar@intel.com>
Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella@intel.com>
Gerrit-Reviewer: Subrata Banik <subi.banik@gmail.com>
Gerrit-Reviewer: Varshit B Pandya <varshit.b.pandya@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-CC: Furquan Shaikh <furquan@google.com>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Selma Bensaid <selma.bensaid@intel.com>
Gerrit-Attention: Sathyanarayana Nujella <sathyanarayana.nujella@intel.com>
Gerrit-Attention: Varshit B Pandya <varshit.b.pandya@intel.com>
Gerrit-Attention: Subrata Banik <subi.banik@gmail.com>
Gerrit-Attention: Furquan Shaikh <furquan@google.com>
Gerrit-Attention: Maulik V Vaghela <maulik.v.vaghela@intel.com>
Gerrit-Attention: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Ronak Kanabar <ronak.kanabar@intel.com>
Gerrit-Attention: Jairaj Arava <jairaj.arava@intel.com>
Gerrit-Attention: Patrick Rudolph <siro@das-labor.org>
Gerrit-Comment-Date: Wed, 31 Mar 2021 23:49:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan@google.com>
Comment-In-Reply-To: Francois Toguo Fotso <francois.toguo.fotso@intel.com>
Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com>
Gerrit-MessageType: comment