build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22604 )
Change subject: cpu/intel/speedstep: Add Netburst CPUs
......................................................................
Patch Set 36:
(2 comments)
https://review.coreboot.org/#/c/22604/36/src/cpu/intel/common/fsb.c
File src/cpu/intel/common/fsb.c:
https://review.coreboot.org/#/c/22604/36/src/cpu/intel/common/fsb.c@48
PS36, Line 48: case 0xf: /* Netburst */
Possible switch case/default not preceded by break or fallthrough comment
https://review.coreboot.org/#/c/22604/36/src/cpu/intel/common/fsb.c@62
PS36, Line 62: default:
Possible switch case/default not preceded by break or fallthrough comment
--
To view, visit
https://review.coreboot.org/22604
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I06e162d3260dedeb3b16583460633507fbcbd52a
Gerrit-Change-Number: 22604
Gerrit-PatchSet: 36
Gerrit-Owner: Elyes HAOUAS
ehaouas@noos.fr
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Elyes HAOUAS
ehaouas@noos.fr
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Patrick Georgi
pgeorgi@google.com
Gerrit-Comment-Date: Fri, 09 Nov 2018 18:55:24 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No