[coreboot-gerrit] New patch to review for coreboot: soc/intel/apollolake: Implement reset_prepare()

[Andrey Petrov (andrey.petrov@intel.com)]

Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15721

-gerrit

commit 2104bf29cb55a24281e257da5474223b968cc3ef
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Fri Jul 15 14:44:48 2016 -0700

    soc/intel/apollolake: Implement reset_prepare()
    
    At first boot CSE spends long time preparing media for use. As result
    it may not be able to deal with a CPU reset. Add reset_prepare()
    callback that polls CSE readiness.
    
    BUG=chrome-os-partner:55055
    TEST=build with release version of fsp, reboot, observe polling for
    CSE, then proper reboot happening
    
    Change-Id: I639ef900b97132f1a7f269bb864d70009df9fdfe
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/soc/intel/apollolake/reset.c | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/src/soc/intel/apollolake/reset.c b/src/soc/intel/apollolake/reset.c
index 644d88d..ab74732 100644
--- a/src/soc/intel/apollolake/reset.c
+++ b/src/soc/intel/apollolake/reset.c
@@ -13,11 +13,37 @@
  * GNU General Public License for more details.
  */
 
+#include <console/console.h>
+#include <delay.h>
 #include <reset.h>
+#include <soc/heci.h>
 #include <soc/pm.h>
 
+#define CSE_WAIT_MAX_MS							1000
+
 void global_reset(void)
 {
 	global_reset_enable(1);
 	hard_reset();
 }
+
+void reset_prepare(void)
+{
+	if (!heci_cse_normal()) {
+		printk(BIOS_DEBUG, "CSE is not in normal state, resetting\n");
+		return;
+	}
+
+	if (!heci_cse_done()) {
+		uint32_t msec = 0;
+		printk(BIOS_SPEW, "CSE is not done, spinning\n");
+		while (!heci_cse_done()) {
+			mdelay(1);
+			if (msec++ > CSE_WAIT_MAX_MS) {
+				printk(BIOS_SPEW, "still busy. resetting\n");
+				return;
+			}
+		}
+		printk(BIOS_DEBUG, "CSE is ready, took %u ms\n", msec);
+	}
+}