vc/dasharo/options.c: Temporarily remove HMRFPO enabling in FUM

Uncomment the HMRFPO enabling code, as it issues a global reset, which
corrupts capsule update data. Consider re-enabling this code once
capsule-on-disk is implemented.

Upstream-Status: Inappropriate [Dasharo downstream]
Change-Id: I2dc0384360c1028d547c572f47f8a767abc61fe2
Signed-off-by: Michał Kopeć <[email protected]>

Author: mkopec

src/vendorcode/dasharo/options.c

@@ -223,45 +223,45 @@ bool dma_protection_enabled(void)
 }
 
 #if CONFIG(SOC_INTEL_COMMON_BLOCK_CSE)
-static void disable_me_hmrfpo(uint8_t *var)
-{
-	int hmrfpo_sts = cse_hmrfpo_get_status();
-
-	if (hmrfpo_sts == -1) {
-		printk (BIOS_DEBUG, "ME HMRFPO Get Status failed\n");
-		goto hmrfpo_error;
-	}
-
-	printk (BIOS_DEBUG, "ME HMRFPO status: ");
-	switch (hmrfpo_sts) {
-	case 0:
-		printk (BIOS_DEBUG, "DISABLED\n");
-		if (cse_hmrfpo_enable() != CB_SUCCESS)
-			goto hmrfpo_error;
-		else
-			do_global_reset(); /* No return */
-		break;
-	case 1:
-		printk (BIOS_DEBUG, "LOCKED\n");
-		goto hmrfpo_error;
-	case 2:
-		printk (BIOS_DEBUG, "ENABLED\n");
-		/* Override ME disable method to not switch ME mode */
-		*var =  ME_MODE_DISABLE_HMRFPO;
-		/* TODO: disable HMRFPO here if not in FUM? Reboot disables HMRFPO. */
-		break;
-	default:
-		printk (BIOS_DEBUG, "UNKNOWN\n");
-		goto hmrfpo_error;
-	}
-
-	return;
-
-hmrfpo_error:
-	/* Try other available methods if something goes wrong */
-	*var = CONFIG(HAVE_INTEL_ME_HAP) ? ME_MODE_DISABLE_HAP
-					 : ME_MODE_DISABLE_HECI;
-}
+//static void disable_me_hmrfpo(uint8_t *var)
+//{
+//	int hmrfpo_sts = cse_hmrfpo_get_status();
+//
+//	if (hmrfpo_sts == -1) {
+//		printk (BIOS_DEBUG, "ME HMRFPO Get Status failed\n");
+//		goto hmrfpo_error;
+//	}
+//
+//	printk (BIOS_DEBUG, "ME HMRFPO status: ");
+//	switch (hmrfpo_sts) {
+//	case 0:
+//		printk (BIOS_DEBUG, "DISABLED\n");
+//		if (cse_hmrfpo_enable() != CB_SUCCESS)
+//			goto hmrfpo_error;
+//		else
+//			do_global_reset(); /* No return */
+//		break;
+//	case 1:
+//		printk (BIOS_DEBUG, "LOCKED\n");
+//		goto hmrfpo_error;
+//	case 2:
+//		printk (BIOS_DEBUG, "ENABLED\n");
+//		/* Override ME disable method to not switch ME mode */
+//		*var =  ME_MODE_DISABLE_HMRFPO;
+//		/* TODO: disable HMRFPO here if not in FUM? Reboot disables HMRFPO. */
+//		break;
+//	default:
+//		printk (BIOS_DEBUG, "UNKNOWN\n");
+//		goto hmrfpo_error;
+//	}
+//
+//	return;
+//
+//hmrfpo_error:
+//	/* Try other available methods if something goes wrong */
+//	*var = CONFIG(HAVE_INTEL_ME_HAP) ? ME_MODE_DISABLE_HAP
+//					 : ME_MODE_DISABLE_HECI;
+//}
 
 uint8_t cse_get_me_disable_mode(void)
 {
@@ -274,13 +274,13 @@ uint8_t cse_get_me_disable_mode(void)
 	}
 
 	/* Disable ME via HMRPFO if in Firmware Update Mode */
-	if (CONFIG(DASHARO_FIRMWARE_UPDATE_MODE) && fum) {
-		/* Check if already in HMRFPO mode */
-		if (cse_is_hfs1_com_secover_mei_msg())
-			return ME_MODE_DISABLE_HMRFPO;
-		else
-			disable_me_hmrfpo(&var);
-	}
+	//if (CONFIG(DASHARO_FIRMWARE_UPDATE_MODE) && fum) {
+	//	/* Check if already in HMRFPO mode */
+	//	if (cse_is_hfs1_com_secover_mei_msg())
+	//		return ME_MODE_DISABLE_HMRFPO;
+	//	else
+	//		disable_me_hmrfpo(&var);
+	//}
 
 	return var;
 }