GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nb/intel/i945: Remove irrelevant conditional statement 

After a {break,return}, "else" is generally not needed.

Change-Id: Id55af179f63316f7218e93978628cbe05e94e0aa
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/29266
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Elyes HAOUAS 2018-10-25 10:41:27 +02:00 committed by Patrick Georgi
parent 07065edd5e
commit 2a1c4302d1
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/northbridge/intel/i945/gma.c
View File

@ -588,16 +588,15 @@ static u32 get_cdclk(struct device *const dev)
{ {
u16 gcfgc = pci_read_config16(dev, GCFGC); u16 gcfgc = pci_read_config16(dev, GCFGC);
if (gcfgc & GC_LOW_FREQUENCY_ENABLE) { if (gcfgc & GC_LOW_FREQUENCY_ENABLE)
return 133333333; return 133333333;
} else {
switch (gcfgc & GC_DISPLAY_CLOCK_MASK) { switch (gcfgc & GC_DISPLAY_CLOCK_MASK) {
case GC_DISPLAY_CLOCK_333_320_MHZ: case GC_DISPLAY_CLOCK_333_320_MHZ:
return 320000000; return 320000000;
default: default:
case GC_DISPLAY_CLOCK_190_200_MHZ: case GC_DISPLAY_CLOCK_190_200_MHZ:
return 200000000; return 200000000;
}
} }
} }