Change in coreboot[master]: soc/intel/broadwell: Drop unused `cdclk` chip field - coreboot-gerrit

31 Jan 2021


      Attention is currently required from: Angel Pons, Arthur Heymans, Patrick Rudolph.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50135 )
Change subject: soc/intel/broadwell: Drop unused `cdclk` chip field
......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/broadwell/gma.c:
https://review.coreboot.org/c/coreboot/+/50135/comment/2788b058_b6daefbd 
PS1, Line 377: with possible override for 450MHz
...
Why is it stale?
Because the code would not allow it anymore?
But, yeah, it's still possible via fuse. Moving the comment around
would fix it.
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/50135
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I70d69ce2d16b704be207b4a23777f174952ffa04
Gerrit-Change-Number: 50135
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons th3fanbus@gmail.com
Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Angel Pons th3fanbus@gmail.com
Gerrit-Attention: Arthur Heymans arthur@aheymans.xyz
Gerrit-Attention: Patrick Rudolph siro@das-labor.org
Gerrit-Comment-Date: Sun, 31 Jan 2021 20:29:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nico Huber nico.h@gmx.de
Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com
Gerrit-MessageType: comment