Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50141 )
Change subject: soc/intel/broadwell/gma.c: Add missing `break` in switch ......................................................................
soc/intel/broadwell/gma.c: Add missing `break` in switch
Otherwise, the `GT_CDCLK_675` case falls through and exits early.
Change-Id: Icb979f8a980e1a1e1c712c5d9bc8d94c90376b7f Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/50141 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Jacob Garber jgarber1@ualberta.ca Reviewed-by: Nico Huber nico.h@gmx.de --- M src/soc/intel/broadwell/gma.c 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved Jacob Garber: Looks good to me, approved
diff --git a/src/soc/intel/broadwell/gma.c b/src/soc/intel/broadwell/gma.c index 6556dd4..45d1833 100644 --- a/src/soc/intel/broadwell/gma.c +++ b/src/soc/intel/broadwell/gma.c @@ -476,6 +476,7 @@ dpdiv = 338; reg_em4 = 8; reg_em5 = 225; + break; default: return; }