Extract the minidump definitions into a header such that the
definitions can be reused by other drivers.
No other change, purely moving the definitions.

Signed-off-by: Eugen Hristev <[email protected]>
---
 drivers/remoteproc/qcom_common.c  | 56 +------------------------
 include/linux/soc/qcom/minidump.h | 68 +++++++++++++++++++++++++++++++
 2 files changed, 69 insertions(+), 55 deletions(-)
 create mode 100644 include/linux/soc/qcom/minidump.h

diff --git a/drivers/remoteproc/qcom_common.c b/drivers/remoteproc/qcom_common.c
index 8c8688f99f0a..4f1c8d005c97 100644
--- a/drivers/remoteproc/qcom_common.c
+++ b/drivers/remoteproc/qcom_common.c
@@ -18,6 +18,7 @@
 #include <linux/rpmsg/qcom_smd.h>
 #include <linux/slab.h>
 #include <linux/soc/qcom/mdt_loader.h>
+#include <linux/soc/qcom/minidump.h>
 #include <linux/soc/qcom/smem.h>
 
 #include "remoteproc_internal.h"
@@ -28,61 +29,6 @@
 #define to_ssr_subdev(d) container_of(d, struct qcom_rproc_ssr, subdev)
 #define to_pdm_subdev(d) container_of(d, struct qcom_rproc_pdm, subdev)
 
-#define MAX_NUM_OF_SS           10
-#define MAX_REGION_NAME_LENGTH  16
-#define SBL_MINIDUMP_SMEM_ID   602
-#define MINIDUMP_REGION_VALID          ('V' << 24 | 'A' << 16 | 'L' << 8 | 'I' 
<< 0)
-#define MINIDUMP_SS_ENCR_DONE          ('D' << 24 | 'O' << 16 | 'N' << 8 | 'E' 
<< 0)
-#define MINIDUMP_SS_ENABLED            ('E' << 24 | 'N' << 16 | 'B' << 8 | 'L' 
<< 0)
-
-/**
- * struct minidump_region - Minidump region
- * @name               : Name of the region to be dumped
- * @seq_num:           : Use to differentiate regions with same name.
- * @valid              : This entry to be dumped (if set to 1)
- * @address            : Physical address of region to be dumped
- * @size               : Size of the region
- */
-struct minidump_region {
-       char    name[MAX_REGION_NAME_LENGTH];
-       __le32  seq_num;
-       __le32  valid;
-       __le64  address;
-       __le64  size;
-};
-
-/**
- * struct minidump_subsystem - Subsystem's SMEM Table of content
- * @status : Subsystem toc init status
- * @enabled : if set to 1, this region would be copied during coredump
- * @encryption_status: Encryption status for this subsystem
- * @encryption_required : Decides to encrypt the subsystem regions or not
- * @region_count : Number of regions added in this subsystem toc
- * @regions_baseptr : regions base pointer of the subsystem
- */
-struct minidump_subsystem {
-       __le32  status;
-       __le32  enabled;
-       __le32  encryption_status;
-       __le32  encryption_required;
-       __le32  region_count;
-       __le64  regions_baseptr;
-};
-
-/**
- * struct minidump_global_toc - Global Table of Content
- * @status : Global Minidump init status
- * @md_revision : Minidump revision
- * @enabled : Minidump enable status
- * @subsystems : Array of subsystems toc
- */
-struct minidump_global_toc {
-       __le32                          status;
-       __le32                          md_revision;
-       __le32                          enabled;
-       struct minidump_subsystem       subsystems[MAX_NUM_OF_SS];
-};
-
 struct qcom_ssr_subsystem {
        const char *name;
        struct srcu_notifier_head notifier_list;
diff --git a/include/linux/soc/qcom/minidump.h 
b/include/linux/soc/qcom/minidump.h
new file mode 100644
index 000000000000..25247a6216e2
--- /dev/null
+++ b/include/linux/soc/qcom/minidump.h
@@ -0,0 +1,68 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Qualcomm Minidump definitions
+ *
+ * Copyright (C) 2016 Linaro Ltd
+ * Copyright (C) 2015 Sony Mobile Communications Inc
+ * Copyright (c) 2012-2013, The Linux Foundation. All rights reserved.
+ */
+
+#ifndef __QCOM_MINIDUMP_H__
+#define __QCOM_MINIDUMP_H__
+
+#define MAX_NUM_OF_SS           10
+#define MAX_REGION_NAME_LENGTH  16
+#define SBL_MINIDUMP_SMEM_ID   602
+#define MINIDUMP_REGION_VALID          ('V' << 24 | 'A' << 16 | 'L' << 8 | 'I' 
<< 0)
+#define MINIDUMP_SS_ENCR_DONE          ('D' << 24 | 'O' << 16 | 'N' << 8 | 'E' 
<< 0)
+#define MINIDUMP_SS_ENABLED            ('E' << 24 | 'N' << 16 | 'B' << 8 | 'L' 
<< 0)
+
+/**
+ * struct minidump_region - Minidump region
+ * @name               : Name of the region to be dumped
+ * @seq_num:           : Use to differentiate regions with same name.
+ * @valid              : This entry to be dumped (if set to 1)
+ * @address            : Physical address of region to be dumped
+ * @size               : Size of the region
+ */
+struct minidump_region {
+       char    name[MAX_REGION_NAME_LENGTH];
+       __le32  seq_num;
+       __le32  valid;
+       __le64  address;
+       __le64  size;
+};
+
+/**
+ * struct minidump_subsystem - Subsystem's SMEM Table of content
+ * @status : Subsystem toc init status
+ * @enabled : if set to 1, this region would be copied during coredump
+ * @encryption_status: Encryption status for this subsystem
+ * @encryption_required : Decides to encrypt the subsystem regions or not
+ * @region_count : Number of regions added in this subsystem toc
+ * @regions_baseptr : regions base pointer of the subsystem
+ */
+struct minidump_subsystem {
+       __le32  status;
+       __le32  enabled;
+       __le32  encryption_status;
+       __le32  encryption_required;
+       __le32  region_count;
+       __le64  regions_baseptr;
+};
+
+/**
+ * struct minidump_global_toc - Global Table of Content
+ * @status : Global Minidump init status
+ * @md_revision : Minidump revision
+ * @enabled : Minidump enable status
+ * @subsystems : Array of subsystems toc
+ */
+struct minidump_global_toc {
+       __le32                          status;
+       __le32                          md_revision;
+       __le32                          enabled;
+       struct minidump_subsystem       subsystems[MAX_NUM_OF_SS];
+};
+
+#endif
-- 
2.43.0


Reply via email to