[coreboot-gerrit] Patch set updated for coreboot: soc/intel/apollolake: Add basic HECI support

[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/15713

-gerrit

commit 0d5844c1f1a27f3f4c9626cf26d466ac2b510307
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Thu Jul 14 17:16:35 2016 -0700

    soc/intel/apollolake: Add basic HECI support
    
    Add functions to read Host Firmware Status register and a helper
    function to determine if CSE is ready.
    
    BUG=chrome-os-partner:55055
    
    Change-Id: If511a51c04f7e59427d7952fa67b61060e2be404
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/soc/intel/apollolake/Makefile.inc           |  4 +++
 src/soc/intel/apollolake/heci.c                 | 35 +++++++++++++++++++++++
 src/soc/intel/apollolake/include/soc/heci.h     | 37 +++++++++++++++++++++++++
 src/soc/intel/apollolake/include/soc/pci_devs.h |  3 ++
 4 files changed, 79 insertions(+)

diff --git a/src/soc/intel/apollolake/Makefile.inc b/src/soc/intel/apollolake/Makefile.inc
index c44fc46..fda3523 100644
--- a/src/soc/intel/apollolake/Makefile.inc
+++ b/src/soc/intel/apollolake/Makefile.inc
@@ -13,6 +13,7 @@ bootblock-y += bootblock/cache_as_ram.S
 bootblock-y += bootblock/bootblock.c
 bootblock-y += car.c
 bootblock-y += gpio.c
+bootblock-y += heci.c
 bootblock-y += itss.c
 bootblock-y += lpc_lib.c
 bootblock-y += mmap_boot.c
@@ -24,6 +25,7 @@ bootblock-$(CONFIG_SOC_UART_DEBUG) += uart_early.c
 romstage-y += car.c
 romstage-$(CONFIG_PLATFORM_USES_FSP2_0) += romstage.c
 romstage-y += gpio.c
+romstage-y += heci.c
 romstage-y += i2c_early.c
 romstage-y += itss.c
 romstage-$(CONFIG_SOC_UART_DEBUG) += uart_early.c
@@ -49,6 +51,7 @@ ramstage-y += chip.c
 ramstage-y += dsp.c
 ramstage-y += gpio.c
 ramstage-y += graphics.c
+ramstage-y += heci.c
 ramstage-y += i2c.c
 ramstage-y += itss.c
 ramstage-$(CONFIG_SOC_UART_DEBUG) += uart_early.c
@@ -77,6 +80,7 @@ postcar-y += tsc_freq.c
 
 verstage-y += car.c
 verstage-y += i2c_early.c
+verstage-y += heci.c
 verstage-y += memmap.c
 verstage-y += mmap_boot.c
 verstage-$(CONFIG_SOC_UART_DEBUG) += uart_early.c
diff --git a/src/soc/intel/apollolake/heci.c b/src/soc/intel/apollolake/heci.c
new file mode 100644
index 0000000..007f184
--- /dev/null
+++ b/src/soc/intel/apollolake/heci.c
@@ -0,0 +1,35 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016 Intel Corp.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <device/device.h>
+#include <device/pci_def.h>
+#include <device/pci_ops.h>
+#include <soc/heci.h>
+#include <soc/pci_devs.h>
+
+enum sec_status heci_fw_sts(void)
+{
+	device_t dev = CSE_DEV;
+	enum sec_status sts;
+
+	sts = pci_read_config32(dev, REG_SEC_FW_STS0) & 0xf;
+	return sts;
+}
+
+bool heci_cse_ready(void)
+{
+	return (heci_fw_sts() == SEC_STATE_NORMAL);
+}
diff --git a/src/soc/intel/apollolake/include/soc/heci.h b/src/soc/intel/apollolake/include/soc/heci.h
new file mode 100644
index 0000000..17867cf
--- /dev/null
+++ b/src/soc/intel/apollolake/include/soc/heci.h
@@ -0,0 +1,37 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016 Intel Corp.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _SOC_APOLLOLAKE_HECI_H_
+#define _SOC_APOLLOLAKE_HECI_H_
+
+enum sec_status {
+	SEC_STATE_RESET = 0,
+	SEC_STATE_INIT,
+	SEC_STATE_RECOVERY,
+	SEC_STATE_UNKNOWN0,
+	SEC_STATE_UNKNOWN1,
+	SEC_STATE_NORMAL,
+	SEC_STATE_DISABLE_WAIT,
+	SEC_STATE_TRANSITION,
+	SEC_STATE_INVALID_CPU
+};
+
+#define REG_SEC_FW_STS0					0x40
+
+enum sec_status heci_fw_sts(void);
+bool heci_cse_ready(void);
+
+#endif
diff --git a/src/soc/intel/apollolake/include/soc/pci_devs.h b/src/soc/intel/apollolake/include/soc/pci_devs.h
index f29784d..b7519ed 100644
--- a/src/soc/intel/apollolake/include/soc/pci_devs.h
+++ b/src/soc/intel/apollolake/include/soc/pci_devs.h
@@ -46,6 +46,9 @@
 #define HDA_DEV				_PCI_DEV(0xe, 0)
 #define HDA_DEVFN			_PCI_DEVFN(0xe, 0)
 
+#define CSE_DEV				_PCI_DEV(0xf, 0)
+#define CSE_DEVFN			_PCI_DEVFN(0xf, 0)
+
 #define ISH_DEV				_PCI_DEV(0x11, 0)
 #define ISH_DEVFN			_PCI_DEVFN(0x11, 0)