Change in coreboot[master]: soc/intel/broadwell/igd.c: Add missing "break"

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45165 )

Change subject: soc/intel/broadwell/igd.c: Add missing "break"
......................................................................

soc/intel/broadwell/igd.c: Add missing "break"

Change-Id: I29dd96a7e00ddf73bb281e6fbbf256e439bce2f1
Signed-off-by: Elyes HAOUAS 
---
M src/soc/intel/broadwell/igd.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/45165/1

diff --git a/src/soc/intel/broadwell/igd.c b/src/soc/intel/broadwell/igd.c
index 41167b1..39c551d 100644
--- a/src/soc/intel/broadwell/igd.c
+++ b/src/soc/intel/broadwell/igd.c
@@ -467,6 +467,7 @@
    	dpdiv = 338;
    	reg_em4 = 8;
    	reg_em5 = 225;
+		break;
    default:
    	return;
    }

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45165 )

Change subject: soc/intel/broadwell/igd.c: Add missing "break"
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45165/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/45165/1//COMMIT_MSG@7 
PS1, Line 7: Add missing "break"
Maybe:
> Avoid premature return for GT_CDCLK_675

Change in coreboot[master]: soc/intel/broadwell/igd.c: Avoid premature return for GT_CDCLK_675

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Arthur Heymans, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/45165

to look at the new patch set (#2).

Change subject: soc/intel/broadwell/igd.c: Avoid premature return for GT_CDCLK_675
......................................................................

soc/intel/broadwell/igd.c: Avoid premature return for GT_CDCLK_675

Change-Id: I29dd96a7e00ddf73bb281e6fbbf256e439bce2f1
Signed-off-by: Elyes HAOUAS 
---
M src/soc/intel/broadwell/igd.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/45165/2

Change in coreboot[master]: soc/intel/broadwell/igd.c: Avoid premature return for GT_CDCLK_675

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45165 )

Change subject: soc/intel/broadwell/igd.c: Avoid premature return for GT_CDCLK_675
......................................................................

Patch Set 2:

(1 comment)

Thx

https://review.coreboot.org/c/coreboot/+/45165/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/45165/1//COMMIT_MSG@7 
PS1, Line 7: Add missing "break"
> Maybe: […]

Done

Change in coreboot[master]: soc/intel/broadwell/gma.c: Avoid premature return for GT_CDCLK_675

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Arthur Heymans, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/45165

to look at the new patch set (#3).

Change subject: soc/intel/broadwell/gma.c: Avoid premature return for GT_CDCLK_675
......................................................................

soc/intel/broadwell/gma.c: Avoid premature return for GT_CDCLK_675

Change-Id: I29dd96a7e00ddf73bb281e6fbbf256e439bce2f1
Signed-off-by: Elyes HAOUAS 
---
M src/soc/intel/broadwell/gma.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/45165/3