[PATCH v3 5/5] firmware: qcom_scm: Add multiple download mode support

From: Mukesh Ojha
Date: Fri Mar 17 2023 - 12:29:11 EST


Currently, scm driver only supports full dump when download
mode is selected. Add support to enable minidump as well both
dump(full dump + minidump).

Signed-off-by: Mukesh Ojha <quic_mojha@xxxxxxxxxxx>
---
drivers/firmware/qcom_scm.c | 21 ++++++++++++++++-----
1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c
index 69f864d..ba3eefc 100644
--- a/drivers/firmware/qcom_scm.c
+++ b/drivers/firmware/qcom_scm.c
@@ -32,6 +32,9 @@ static u32 download_mode;

#define QCOM_DOWNLOAD_MODE_MASK 0x30
#define QCOM_DOWNLOAD_FULLDUMP 0x10
+#define QCOM_DOWNLOAD_MINIDUMP 0x20
+#define QCOM_DOWNLOAD_BOTHDUMP (QCOM_DOWNLOAD_FULLDUMP | QCOM_DOWNLOAD_MINIDUMP)
+
#define QCOM_DOWNLOAD_NODUMP 0x0

struct qcom_scm {
@@ -1421,15 +1424,19 @@ static irqreturn_t qcom_scm_irq_handler(int irq, void *data)
return IRQ_HANDLED;
}

-
static int get_download_mode(char *buffer, const struct kernel_param *kp)
{
- int len;
+ int len = 0;

- if (download_mode == QCOM_DOWNLOAD_FULLDUMP)
+ if (download_mode == QCOM_DOWNLOAD_FULLDUMP) {
len = sysfs_emit(buffer, "full\n");
- else if (download_mode == QCOM_DOWNLOAD_NODUMP)
+ } else if (download_mode == QCOM_DOWNLOAD_MINIDUMP) {
+ len = sysfs_emit(buffer, "mini\n");
+ } else if (download_mode == QCOM_DOWNLOAD_BOTHDUMP) {
+ len = sysfs_emit(buffer, "both\n");
+ } else if (download_mode == QCOM_DOWNLOAD_NODUMP) {
len = sysfs_emit(buffer, "off\n");
+ }

return len;
}
@@ -1438,6 +1445,10 @@ static int set_download_mode(const char *val, const struct kernel_param *kp)
{
if (!strncmp(val, "full", strlen("full"))) {
download_mode = QCOM_DOWNLOAD_FULLDUMP;
+ } else if (!strncmp(val, "mini", strlen("mini"))) {
+ download_mode = QCOM_DOWNLOAD_MINIDUMP;
+ } else if (!strncmp(val, "both", strlen("both"))) {
+ download_mode = QCOM_DOWNLOAD_BOTHDUMP;
} else if (!strncmp(val, "off", strlen("off"))) {
download_mode = QCOM_DOWNLOAD_NODUMP;
} else if (kstrtouint(val, 0, &download_mode) ||
@@ -1459,7 +1470,7 @@ static const struct kernel_param_ops download_mode_param_ops = {

module_param_cb(download_mode, &download_mode_param_ops, NULL, 0644);
MODULE_PARM_DESC(download_mode,
- "Download mode: off/full or 0/1 for existing users");
+ "download mode: off/full/mini/both(full+mini) or 0/1 for existing users");

static int qcom_scm_probe(struct platform_device *pdev)
{
--
2.7.4