drm/i915/uc: Move uc firmware layout definitions to dedicated file
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Thu, 25 Jul 2019 14:13:08 +0000 (14:13 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 25 Jul 2019 20:23:15 +0000 (21:23 +0100)
Generic uc firmware layout definitions are unlikely to change and
are separate to other GuC specific definitions.

v2: reordered

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190725141308.24660-3-michal.wajdeczko@intel.com
Documentation/gpu/i915.rst
drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h
drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h [new file with mode: 0644]

index 3e233f9d675f95c69048c3cd58e0655a1995efb0..0e322688be5c9c71572fbfa861e15b710dd6b925 100644 (file)
@@ -433,7 +433,7 @@ GuC
 Firmware Layout
 -------------------
 
-.. kernel-doc:: drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h
+.. kernel-doc:: drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h
    :doc: Firmware Layout
 
 GuC-specific firmware loader
index 108b386c52ec1a9e0721016969dec57e7aec305f..06a9bdfb0faf759883f3dc25db8bfb37915fed73 100644 (file)
 
 #define GUC_CTL_MAX_DWORDS             (SOFT_SCRATCH_COUNT - 2) /* [1..14] */
 
-/**
- * DOC: Firmware Layout
- *
- * The GuC/HuC firmware layout looks like this::
- *
- *      +======================================================================+
- *      |  Firmware blob                                                       |
- *      +===============+===============+============+============+============+
- *      |  CSS header   |     uCode     |  RSA key   |  modulus   |  exponent  |
- *      +===============+===============+============+============+============+
- *       <-header size->                 <---header size continued ----------->
- *       <--- size ----------------------------------------------------------->
- *                                       <-key size->
- *                                                    <-mod size->
- *                                                                 <-exp size->
- *
- * The firmware may or may not have modulus key and exponent data. The header,
- * uCode and RSA signature are must-have components that will be used by driver.
- * Length of each components, which is all in dwords, can be found in header.
- * In the case that modulus and exponent are not present in fw, a.k.a truncated
- * image, the length value still appears in header.
- *
- * Driver will do some basic fw size validation based on the following rules:
- *
- * 1. Header, uCode and RSA are must-have components.
- * 2. All firmware components, if they present, are in the sequence illustrated
- *    in the layout table above.
- * 3. Length info of each component can be found in header, in dwords.
- * 4. Modulus and exponent key are not required by driver. They may not appear
- *    in fw. So driver will load a truncated firmware in this case.
- *
- * The only difference between GuC and HuC firmwares is how the version
- * information is saved.
- */
-
-struct uc_css_header {
-       u32 module_type;
-       /* header_size includes all non-uCode bits, including css_header, rsa
-        * key, modulus key and exponent data. */
-       u32 header_size_dw;
-       u32 header_version;
-       u32 module_id;
-       u32 module_vendor;
-       u32 date;
-#define CSS_DATE_DAY                   (0xFF << 0)
-#define CSS_DATE_MONTH                 (0xFF << 8)
-#define CSS_DATE_YEAR                  (0xFFFF << 16)
-       u32 size_dw; /* uCode plus header_size_dw */
-       u32 key_size_dw;
-       u32 modulus_size_dw;
-       u32 exponent_size_dw;
-       u32 time;
-#define CSS_TIME_HOUR                  (0xFF << 0)
-#define CSS_DATE_MIN                   (0xFF << 8)
-#define CSS_DATE_SEC                   (0xFFFF << 16)
-       char username[8];
-       char buildnumber[12];
-       u32 sw_version;
-#define CSS_SW_VERSION_GUC_MAJOR       (0xFF << 16)
-#define CSS_SW_VERSION_GUC_MINOR       (0xFF << 8)
-#define CSS_SW_VERSION_GUC_PATCH       (0xFF << 0)
-#define CSS_SW_VERSION_HUC_MAJOR       (0xFFFF << 16)
-#define CSS_SW_VERSION_HUC_MINOR       (0xFFFF << 0)
-       u32 reserved[14];
-       u32 header_info;
-} __packed;
-
 /* Work item for submitting workloads into work queue of GuC. */
 struct guc_wq_item {
        u32 header;
index 789b3d7228a4a2f608952846f3676ba42103c123..168d368bcd3e67f583dde49d42a633f6dcb2a90a 100644 (file)
@@ -27,6 +27,7 @@
 #include <drm/drm_print.h>
 
 #include "intel_uc_fw.h"
+#include "intel_uc_fw_abi.h"
 #include "i915_drv.h"
 
 /*
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw_abi.h
new file mode 100644 (file)
index 0000000..545e86c
--- /dev/null
@@ -0,0 +1,80 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Copyright © 2019 Intel Corporation
+ */
+
+#ifndef _INTEL_UC_FW_ABI_H
+#define _INTEL_UC_FW_ABI_H
+
+#include <linux/types.h>
+
+/**
+ * DOC: Firmware Layout
+ *
+ * The GuC/HuC firmware layout looks like this::
+ *
+ *      +======================================================================+
+ *      |  Firmware blob                                                       |
+ *      +===============+===============+============+============+============+
+ *      |  CSS header   |     uCode     |  RSA key   |  modulus   |  exponent  |
+ *      +===============+===============+============+============+============+
+ *       <-header size->                 <---header size continued ----------->
+ *       <--- size ----------------------------------------------------------->
+ *                                       <-key size->
+ *                                                    <-mod size->
+ *                                                                 <-exp size->
+ *
+ * The firmware may or may not have modulus key and exponent data. The header,
+ * uCode and RSA signature are must-have components that will be used by driver.
+ * Length of each components, which is all in dwords, can be found in header.
+ * In the case that modulus and exponent are not present in fw, a.k.a truncated
+ * image, the length value still appears in header.
+ *
+ * Driver will do some basic fw size validation based on the following rules:
+ *
+ * 1. Header, uCode and RSA are must-have components.
+ * 2. All firmware components, if they present, are in the sequence illustrated
+ *    in the layout table above.
+ * 3. Length info of each component can be found in header, in dwords.
+ * 4. Modulus and exponent key are not required by driver. They may not appear
+ *    in fw. So driver will load a truncated firmware in this case.
+ *
+ * The only difference between GuC and HuC firmwares is how the version
+ * information is saved.
+ */
+
+struct uc_css_header {
+       u32 module_type;
+       /*
+        * header_size includes all non-uCode bits, including css_header, rsa
+        * key, modulus key and exponent data.
+        */
+       u32 header_size_dw;
+       u32 header_version;
+       u32 module_id;
+       u32 module_vendor;
+       u32 date;
+#define CSS_DATE_DAY                   (0xFF << 0)
+#define CSS_DATE_MONTH                 (0xFF << 8)
+#define CSS_DATE_YEAR                  (0xFFFF << 16)
+       u32 size_dw; /* uCode plus header_size_dw */
+       u32 key_size_dw;
+       u32 modulus_size_dw;
+       u32 exponent_size_dw;
+       u32 time;
+#define CSS_TIME_HOUR                  (0xFF << 0)
+#define CSS_DATE_MIN                   (0xFF << 8)
+#define CSS_DATE_SEC                   (0xFFFF << 16)
+       char username[8];
+       char buildnumber[12];
+       u32 sw_version;
+#define CSS_SW_VERSION_GUC_MAJOR       (0xFF << 16)
+#define CSS_SW_VERSION_GUC_MINOR       (0xFF << 8)
+#define CSS_SW_VERSION_GUC_PATCH       (0xFF << 0)
+#define CSS_SW_VERSION_HUC_MAJOR       (0xFFFF << 16)
+#define CSS_SW_VERSION_HUC_MINOR       (0xFFFF << 0)
+       u32 reserved[14];
+       u32 header_info;
+} __packed;
+
+#endif /* _INTEL_UC_FW_ABI_H */