[coreboot-gerrit] Change in coreboot[master]: sb/intel/bd82x6x: Support ME Soft Temporary Disable Mode

[Evgeny Zinoviev (Code Review)]

Evgeny Zinoviev has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37115 )

Change subject: sb/intel/bd82x6x: Support ME Soft Temporary Disable Mode
......................................................................

Patch Set 14:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37115/11/src/southbridge/intel/bd82... 
File src/southbridge/intel/bd82x6x/me_8.x.c:

https://review.coreboot.org/c/coreboot/+/37115/11/src/southbridge/intel/bd82... 
PS11, Line 574: for (i = 0; i < 400; i++) {
              : 		mdelay(50);
              : 		pci_read_dword_ptr(dev, &hfs, PCI_ME_HFS);
              : 		printk(BIOS_SPEW, "ME: hfs.fw_init_complete=%d after %dms\n",
              : 			hfs.fw_init_complete, (i+1)*50);
              : 
              : 		if (hfs.fw_init_complete)
              : 			break;
              : 	}
              : 
              : 	if (!hfs.fw_init_complete)
              : 		printk(BIOS_ERR, "ME: giving up on waiting for fw_init_complete\n");
> Please use the stopwatch framework, and print the total time in the end.

Done