summaryrefslogtreecommitdiff
path: root/util/intelmetool/me_status.c
diff options
context:
space:
mode:
authorMaximilian Schander <maxschander@googlemail.com>2017-10-28 18:33:07 +0200
committerMartin Roth <martinroth@google.com>2017-11-03 23:58:08 +0000
commitdf5b83fe2bd5752ed895fccd40c665a22525dfaf (patch)
treeab8fa9bbf4e10adb27bcae40f74ea0554f13eedd /util/intelmetool/me_status.c
parent181231164529203baf9fde2b82f6994f4d2c1abc (diff)
downloadcoreboot-df5b83fe2bd5752ed895fccd40c665a22525dfaf.tar.xz
intelmetool: Do small cosmetic changes
Refining some of the code indentations and cosmetics to build upon and import some in-review changes. Change-Id: I0038a146bd899f150518c4832258a42792abaabb Signed-off-by: Maximilian Schander <maxschander@googlemail.com> Reviewed-on: https://review.coreboot.org/22216 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Diffstat (limited to 'util/intelmetool/me_status.c')
-rw-r--r--util/intelmetool/me_status.c31
1 files changed, 15 insertions, 16 deletions
diff --git a/util/intelmetool/me_status.c b/util/intelmetool/me_status.c
index 1de51f4f87..0cc670b19d 100644
--- a/util/intelmetool/me_status.c
+++ b/util/intelmetool/me_status.c
@@ -142,42 +142,41 @@ void intel_me_status(uint32_t hfs, uint32_t gmes)
{
/* Check Current States */
printf("ME: FW Partition Table : %s\n",
- ((hfs & 0x20) >> 5) ? "BAD" : "OK");
+ ((hfs & 0x20) >> 5) ? "BAD" : "OK");
printf("ME: Bringup Loader Failure : %s\n",
- ((hfs & 0x400) >> 10) ? "YES" : "NO");
+ ((hfs & 0x400) >> 10) ? "YES" : "NO");
printf("ME: Firmware Init Complete : %s\n",
- ((hfs & 0x200) >> 9) ? "YES" : "NO");
+ ((hfs & 0x200) >> 9) ? "YES" : "NO");
printf("ME: Manufacturing Mode : %s\n",
- ((hfs & 0x10) >> 4) ? "YES" : "NO");
+ ((hfs & 0x10) >> 4) ? "YES" : "NO");
printf("ME: Boot Options Present : %s\n",
- ((hfs & 0x1000000) >> 24) ? "YES" : "NO");
+ ((hfs & 0x1000000) >> 24) ? "YES" : "NO");
printf("ME: Update In Progress : %s\n",
- ((hfs & 0x800) >> 11) ? "YES" : "NO");
+ ((hfs & 0x800) >> 11) ? "YES" : "NO");
printf("ME: Current Working State : %s\n",
- me_cws_values[hfs & 0xf]);
+ me_cws_values[hfs & 0xf]);
printf("ME: Current Operation State : %s\n",
- me_opstate_values[(hfs & 0x1c0) >> 6]);
+ me_opstate_values[(hfs & 0x1c0) >> 6]);
printf("ME: Current Operation Mode : %s\n",
- me_opmode_values[(hfs & 0xf0000) >> 16]);
+ me_opmode_values[(hfs & 0xf0000) >> 16]);
printf("ME: Error Code : %s\n",
- me_error_values[(hfs & 0xf000) >> 12]);
+ me_error_values[(hfs & 0xf000) >> 12]);
printf("ME: Progress Phase : %s\n",
- me_progress_values[(gmes & 0xf0000000) >> 28]);
+ me_progress_values[(gmes & 0xf0000000) >> 28]);
printf("ME: Power Management Event : %s\n",
- me_pmevent_values[(gmes & 0xf000000) >> 24]);
+ me_pmevent_values[(gmes & 0xf000000) >> 24]);
printf("ME: Progress Phase State : ");
switch ((gmes & 0xf0000000) >> 28) {
case ME_GMES_PHASE_ROM: /* ROM Phase */
- printf("%s",
- me_progress_rom_values[(gmes & 0xff0000) >> 16]);
+ printf("%s", me_progress_rom_values[(gmes & 0xff0000) >> 16]);
break;
case ME_GMES_PHASE_BUP: /* Bringup Phase */
if ((gmes & 0xff0000) >> 16 < ARRAY_SIZE(me_progress_bup_values)
&& me_progress_bup_values[(gmes & 0xff0000) >> 16])
printf("%s",
- me_progress_bup_values[(gmes & 0xff0000) >> 16]);
+ me_progress_bup_values[(gmes & 0xff0000) >> 16]);
else
printf("0x%02x", (gmes & 0xff0000) >> 16);
break;
@@ -186,7 +185,7 @@ void intel_me_status(uint32_t hfs, uint32_t gmes)
if ((gmes & 0xff0000) >> 16 < ARRAY_SIZE(me_progress_policy_values)
&& me_progress_policy_values[(gmes & 0xff0000) >> 16])
printf("%s",
- me_progress_policy_values[(gmes & 0xff0000) >> 16]);
+ me_progress_policy_values[(gmes & 0xff0000) >> 16]);
else
printf("0x%02x", (gmes & 0xff0000) >> 16);
break;