[PATCH v2 0/2] Add firmware debug logging support for the qcom Iris driver

[PATCH v2 0/2] Add firmware debug logging support for the qcom Iris driver

[Renjiang Han]

The driver currently has no way to configure firmware debug
logging at runtime. This makes firmware-side issues harder to
debug, because changing the firmware debug mask requires a
rebuild or reload.

The patch adds a debugfs fw_level file under the Iris debugfs
directory. The value is stored per Iris core and is normalized
on write, so userspace reads back the same mask that is
programmed to firmware.

The firmware debug configuration is sent during core
initialization and before opening a new session. This allows a
fw_level value written before a session starts to take effect for
that session.

HFI Gen1 and Gen2 paths are added for programming the debug
configuration and reading firmware debug messages from the debug
queue. Firmware error and fatal messages are printed with
dev_err_ratelimited(), while other firmware messages use
dev_dbg().

The default mask enables only firmware error and fatal messages
to avoid enabling verbose firmware logging by default.

Failure to configure firmware debug logging is treated as non-fatal
and does not block normal video operation.

Signed-off-by: Renjiang Han <renjiang.han@oss.qualcomm.com>
---
Changes in v2:
- 1. Increase response timeout for verbose firmware logging.
- 2. Increase runtime PM autosuspend delay for verbose firmware logging.
- 3. Drain message and debug queues until both queues are empty.
- 4. Fix checkpatch alignment warning in iris_hfi_common.c.
- 5. Drop direct iris_hfi_common.h includes from response files.
- 6. Keep bounded firmware log printing with %.*s.
- Link to v1: https://lore.kernel.org/r/20260815-media-qcom-iris-fw-log-v1-1-c8a70eb08849@oss.qualcomm.com

---
Renjiang Han (2):
      media: qcom: iris: add firmware debug logging support
      media: qcom: iris: reduce impact of verbose firmware logging

 drivers/media/platform/qcom/iris/Makefile          |  1 +
 drivers/media/platform/qcom/iris/iris_core.c       |  2 +-
 drivers/media/platform/qcom/iris/iris_core.h       |  9 +++-
 drivers/media/platform/qcom/iris/iris_debugfs.c    | 56 ++++++++++++++++++++++
 drivers/media/platform/qcom/iris/iris_debugfs.h    | 14 ++++++
 drivers/media/platform/qcom/iris/iris_hfi_common.c | 43 +++++++++++++++++
 drivers/media/platform/qcom/iris/iris_hfi_common.h | 11 +++++
 .../platform/qcom/iris/iris_hfi_gen1_command.c     | 30 ++++++++++++
 .../platform/qcom/iris/iris_hfi_gen1_defines.h     |  8 ++++
 .../platform/qcom/iris/iris_hfi_gen1_response.c    | 55 ++++++++++++++++-----
 .../platform/qcom/iris/iris_hfi_gen2_command.c     | 21 ++++++++
 .../platform/qcom/iris/iris_hfi_gen2_defines.h     |  3 ++
 .../platform/qcom/iris/iris_hfi_gen2_packet.c      | 29 +++++++++++
 .../platform/qcom/iris/iris_hfi_gen2_packet.h      |  1 +
 .../platform/qcom/iris/iris_hfi_gen2_response.c    | 36 ++++++++++----
 drivers/media/platform/qcom/iris/iris_probe.c      |  7 +++
 drivers/media/platform/qcom/iris/iris_utils.c      |  4 +-
 drivers/media/platform/qcom/iris/iris_vb2.c        |  7 ++-
 18 files changed, 311 insertions(+), 26 deletions(-)
---
base-commit: 28d012efb4327f9c75d5e042a7c91e9a542efa98
change-id: 20260815-media-qcom-iris-fw-log-3b62b8645b7a

Best regards,
-- 
Renjiang Han <renjiang.han@oss.qualcomm.com>

[PATCH v2 1/2] media: qcom: iris: add firmware debug logging support

[Renjiang Han]

Iris does not provide runtime control for firmware debug logging, which
makes firmware-side failures harder to diagnose without rebuilding or
reloading the driver.

Add a debugfs fw_level file under the Iris debugfs directory so the
firmware debug log mask can be configured at runtime. Keep the default
mask limited to firmware error and fatal messages to avoid enabling
verbose logging by default.

Store the mask per Iris core and normalize values written through
debugfs so userspace reads back the same mask that is sent to firmware.
Program the firmware debug configuration during core initialization,
runtime resume, and before opening a new session. This allows a
fw_level value written before session start to take effect without
requiring a driver reload, and preserves the setting after firmware is
reinitialized.

Add HFI Gen1 and Gen2 support for sending the firmware debug
configuration, and flush firmware debug messages from the debug queue.
Emit firmware error and fatal messages with dev_err_ratelimited(), and
use dev_dbg() for other firmware debug messages. Bound firmware log
printing by the packet size so messages do not need to be NUL
terminated by firmware.

Treat failures to configure firmware debug logging as non-fatal, since
they should not prevent video sessions from running.

Tested-by: Wangao Wang <wangao.wang@oss.qualcomm.com>
Signed-off-by: Renjiang Han <renjiang.han@oss.qualcomm.com>
---
 drivers/media/platform/qcom/iris/Makefile          |  1 +
 drivers/media/platform/qcom/iris/iris_core.h       |  7 +++-
 drivers/media/platform/qcom/iris/iris_debugfs.c    | 42 +++++++++++++++++++++
 drivers/media/platform/qcom/iris/iris_debugfs.h    | 14 +++++++
 drivers/media/platform/qcom/iris/iris_hfi_common.c | 43 ++++++++++++++++++++++
 drivers/media/platform/qcom/iris/iris_hfi_common.h | 10 +++++
 .../platform/qcom/iris/iris_hfi_gen1_command.c     | 30 +++++++++++++++
 .../platform/qcom/iris/iris_hfi_gen1_defines.h     |  8 ++++
 .../platform/qcom/iris/iris_hfi_gen1_response.c    | 31 +++++++++++++---
 .../platform/qcom/iris/iris_hfi_gen2_command.c     | 21 +++++++++++
 .../platform/qcom/iris/iris_hfi_gen2_defines.h     |  3 ++
 .../platform/qcom/iris/iris_hfi_gen2_packet.c      | 29 +++++++++++++++
 .../platform/qcom/iris/iris_hfi_gen2_packet.h      |  1 +
 .../platform/qcom/iris/iris_hfi_gen2_response.c    | 12 ++++--
 drivers/media/platform/qcom/iris/iris_probe.c      |  6 +++
 drivers/media/platform/qcom/iris/iris_vb2.c        |  7 +++-
 16 files changed, 253 insertions(+), 12 deletions(-)

diff --git a/drivers/media/platform/qcom/iris/Makefile b/drivers/media/platform/qcom/iris/Makefile
index 48e415cbc4390bc596f6239fefa2a2ad2cd3a2bb..105a66e1b0af44f010730a899be26cae23d76c73 100644
--- a/drivers/media/platform/qcom/iris/Makefile
+++ b/drivers/media/platform/qcom/iris/Makefile
@@ -2,6 +2,7 @@ qcom-iris-objs += iris_buffer.o \
              iris_common.o \
              iris_core.o \
              iris_ctrls.o \
+             iris_debugfs.o \
              iris_firmware.o \
              iris_hfi_common.o \
              iris_hfi_gen1.o \
diff --git a/drivers/media/platform/qcom/iris/iris_core.h b/drivers/media/platform/qcom/iris/iris_core.h
index 24da60448cf24820af7947b85eb7208555ab7786..a8e34c456ce3f3f178ca66a278d06d1ab50dafc4 100644
--- a/drivers/media/platform/qcom/iris/iris_core.h
+++ b/drivers/media/platform/qcom/iris/iris_core.h
@@ -31,6 +31,7 @@ enum domain_type {
 };
 
 struct qcom_ubwc_cfg_data;
+struct dentry;
 
 /**
  * struct iris_core - holds core parameters valid for all instances
@@ -65,7 +66,9 @@ struct qcom_ubwc_cfg_data;
  * @command_queue: shared interface queue to send commands to firmware
  * @message_queue: shared interface queue to receive responses from firmware
  * @debug_queue: shared interface queue to receive debug info from firmware
- * @lock: a lock for this strucure
+ * @root: debugfs root directory
+ * @fw_debug: firmware debug log mask
+ * @lock: a lock for this structure
  * @response_packet: a pointer to response packet from fw to driver
  * @header_id: id of packet header
  * @packet_id: id of packet
@@ -110,6 +113,8 @@ struct iris_core {
 	struct iris_iface_q_info		command_queue;
 	struct iris_iface_q_info		message_queue;
 	struct iris_iface_q_info		debug_queue;
+	struct dentry				*root;
+	u32					fw_debug;
 	struct mutex				lock; /* lock for core related operations */
 	u8					*response_packet;
 	u32					header_id;
diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.c b/drivers/media/platform/qcom/iris/iris_debugfs.c
new file mode 100644
index 0000000000000000000000000000000000000000..6122b994ad0aa70c0c4f26ea4ce076f1b025f810
--- /dev/null
+++ b/drivers/media/platform/qcom/iris/iris_debugfs.c
@@ -0,0 +1,42 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Copyright (c) Qualcomm Innovation Center, Inc. All rights reserved.
+ */
+
+#include <linux/debugfs.h>
+
+#include "iris_core.h"
+#include "iris_debugfs.h"
+
+static int iris_fw_level_get(void *data, u64 *val)
+{
+	struct iris_core *core = data;
+
+	*val = READ_ONCE(core->fw_debug);
+
+	return 0;
+}
+
+static int iris_fw_level_set(void *data, u64 val)
+{
+	struct iris_core *core = data;
+
+	WRITE_ONCE(core->fw_debug, (u32)val & IRIS_FW_DEBUG_LOGMASK);
+
+	return 0;
+}
+
+DEFINE_DEBUGFS_ATTRIBUTE(iris_fw_level_fops, iris_fw_level_get,
+			 iris_fw_level_set, "0x%08llx\n");
+
+void iris_debugfs_init(struct iris_core *core)
+{
+	core->root = debugfs_create_dir("iris", NULL);
+	debugfs_create_file("fw_level", 0600, core->root, core,
+			    &iris_fw_level_fops);
+}
+
+void iris_debugfs_deinit(struct iris_core *core)
+{
+	debugfs_remove(core->root);
+}
diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.h b/drivers/media/platform/qcom/iris/iris_debugfs.h
new file mode 100644
index 0000000000000000000000000000000000000000..aa3cb97eb2b2e71dfe6a0db6ccfd430256cdb6fa
--- /dev/null
+++ b/drivers/media/platform/qcom/iris/iris_debugfs.h
@@ -0,0 +1,14 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/*
+ * Copyright (c) Qualcomm Innovation Center, Inc. All rights reserved.
+ */
+
+#ifndef __IRIS_DEBUGFS_H__
+#define __IRIS_DEBUGFS_H__
+
+struct iris_core;
+
+void iris_debugfs_init(struct iris_core *core);
+void iris_debugfs_deinit(struct iris_core *core);
+
+#endif
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.c b/drivers/media/platform/qcom/iris/iris_hfi_common.c
index 8769ec61f11769e004945063381d9baddb302b06..706a4d321ee2899aadc9f650d0d9a864fdc6769b 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_common.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_common.c
@@ -87,9 +87,44 @@ int iris_hfi_core_init(struct iris_core *core)
 	if (ret)
 		return ret;
 
+	if (hfi_ops->sys_set_debug) {
+		ret = hfi_ops->sys_set_debug(core);
+		if (ret)
+			dev_warn(core->dev,
+				 "failed to configure firmware debug logging: %d\n",
+				 ret);
+	}
+
 	return hfi_ops->sys_interframe_powercollapse(core);
 }
 
+int iris_hfi_set_debug(struct iris_core *core)
+{
+	const struct iris_hfi_sys_ops *hfi_ops = core->hfi_sys_ops;
+	int ret = 0;
+
+	if (!hfi_ops->sys_set_debug)
+		return 0;
+
+	ret = pm_runtime_resume_and_get(core->dev);
+	if (ret < 0)
+		return ret;
+
+	mutex_lock(&core->lock);
+	if (core->state != IRIS_CORE_INIT) {
+		ret = 0;
+		goto unlock;
+	}
+
+	ret = hfi_ops->sys_set_debug(core);
+
+unlock:
+	mutex_unlock(&core->lock);
+	pm_runtime_put_autosuspend(core->dev);
+
+	return ret;
+}
+
 irqreturn_t iris_hfi_isr(int irq, void *data)
 {
 	disable_irq_nosync(irq);
@@ -163,6 +198,14 @@ int iris_hfi_pm_resume(struct iris_core *core)
 	if (ret)
 		goto err_suspend_hw;
 
+	if (ops->sys_set_debug) {
+		ret = ops->sys_set_debug(core);
+		if (ret)
+			dev_warn(core->dev,
+				 "failed to configure firmware debug logging: %d\n",
+				 ret);
+	}
+
 	ret = ops->sys_interframe_powercollapse(core);
 	if (ret)
 		goto err_suspend_hw;
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.h b/drivers/media/platform/qcom/iris/iris_hfi_common.h
index a27447eb2519962cb958b0e330a6d018310c3450..d8bc0862c54bd229d575113752526d51e863e303 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_common.h
+++ b/drivers/media/platform/qcom/iris/iris_hfi_common.h
@@ -105,9 +105,18 @@ struct iris_hfi_prop_type_handle {
 	int (*handle)(struct iris_inst *inst, u32 plane);
 };
 
+#define IRIS_FW_DEBUG_LOW	0x00000001
+#define IRIS_FW_DEBUG_MEDIUM	0x00000002
+#define IRIS_FW_DEBUG_HIGH	0x00000004
+#define IRIS_FW_DEBUG_ERROR	0x00000008
+#define IRIS_FW_DEBUG_FATAL	0x00000010
+#define IRIS_FW_DEBUG_PERF	0x00000020
+#define IRIS_FW_DEBUG_LOGMASK	0x0fffffff
+
 struct iris_hfi_sys_ops {
 	int (*sys_init)(struct iris_core *core);
 	int (*sys_image_version)(struct iris_core *core);
+	int (*sys_set_debug)(struct iris_core *core);
 	int (*sys_interframe_powercollapse)(struct iris_core *core);
 	int (*sys_pc_prep)(struct iris_core *core);
 
@@ -153,6 +162,7 @@ u32 iris_hfi_get_v4l2_color_primaries(u32 hfi_primaries);
 u32 iris_hfi_get_v4l2_transfer_char(u32 hfi_characterstics);
 u32 iris_hfi_get_v4l2_matrix_coefficients(u32 hfi_coefficients);
 int iris_hfi_core_init(struct iris_core *core);
+int iris_hfi_set_debug(struct iris_core *core);
 int iris_hfi_pm_suspend(struct iris_core *core);
 int iris_hfi_pm_resume(struct iris_core *core);
 
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
index f6e9cbfccd09e4013e842c92c8272ddeb1b166d3..55e2fade62e0450b8ce5e5c928b746dcb6ef8387 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
@@ -53,6 +53,35 @@ static int iris_hfi_gen1_sys_image_version(struct iris_core *core)
 	return iris_hfi_queue_cmd_write_locked(core, &packet, packet.hdr.size);
 }
 
+static int iris_hfi_gen1_sys_set_debug(struct iris_core *core)
+{
+	struct hfi_sys_set_property_pkt *pkt;
+	struct hfi_debug_config *hfi;
+	u32 fw_debug;
+	u32 packet_size;
+	int ret;
+
+	packet_size = struct_size(pkt, data, 1) + sizeof(*hfi);
+	pkt = kzalloc(packet_size, GFP_KERNEL);
+	if (!pkt)
+		return -ENOMEM;
+
+	hfi = (struct hfi_debug_config *)&pkt->data[1];
+
+	pkt->hdr.size = packet_size;
+	pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
+	pkt->num_properties = 1;
+	pkt->data[0] = HFI_PROPERTY_SYS_DEBUG_CONFIG;
+	fw_debug = READ_ONCE(core->fw_debug) & IRIS_FW_DEBUG_LOGMASK;
+	hfi->config = fw_debug;
+	hfi->mode = fw_debug ? HFI_DEBUG_MODE_QUEUE : 0;
+
+	ret = iris_hfi_queue_cmd_write_locked(core, pkt, pkt->hdr.size);
+	kfree(pkt);
+
+	return ret;
+}
+
 static int iris_hfi_gen1_sys_interframe_powercollapse(struct iris_core *core)
 {
 	struct hfi_sys_set_property_pkt *pkt;
@@ -1149,6 +1178,7 @@ static struct iris_inst *iris_hfi_gen1_get_instance(void)
 static const struct iris_hfi_sys_ops iris_hfi_gen1_sys_ops = {
 	.sys_init = iris_hfi_gen1_sys_init,
 	.sys_image_version = iris_hfi_gen1_sys_image_version,
+	.sys_set_debug = iris_hfi_gen1_sys_set_debug,
 	.sys_interframe_powercollapse = iris_hfi_gen1_sys_interframe_powercollapse,
 	.sys_pc_prep = iris_hfi_gen1_sys_pc_prep,
 
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h b/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
index 275f3fea3d2aabe26a2a0498c78c09ec28765b24..887f96c5623809beb1c8e911949b314b9bd029da 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
@@ -80,6 +80,14 @@
 #define HFI_BUFFER_INTERNAL_SCRATCH_1			0x7
 #define HFI_BUFFER_INTERNAL_SCRATCH_2			0x8
 
+#define HFI_DEBUG_MODE_QUEUE				0x01
+
+struct hfi_debug_config {
+	u32 config;
+	u32 mode;
+};
+
+#define HFI_PROPERTY_SYS_DEBUG_CONFIG			0x1
 #define HFI_PROPERTY_SYS_CODEC_POWER_PLANE_CTRL		0x5
 #define HFI_PROPERTY_SYS_IMAGE_VERSION			0x6
 
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
index bfd7495bf44f064091afb3245ec53cd18d323068..903b60a7380559f4711b0907fb1de405fe1707f3 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
@@ -674,16 +674,35 @@ static void iris_hfi_gen1_handle_response(struct iris_core *core, void *response
 
 static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
 {
-	struct hfi_msg_sys_coverage_pkt *pkt;
+	struct hfi_msg_sys_debug_pkt *pkt;
+	struct hfi_pkt_hdr *hdr;
+	u32 log_size;
+	u8 *log;
 
 	while (!iris_hfi_queue_dbg_read(core, packet)) {
-		pkt = (struct hfi_msg_sys_coverage_pkt *)packet;
+		hdr = (struct hfi_pkt_hdr *)packet;
 
-		if (pkt->hdr.pkt_type != HFI_MSG_SYS_COV) {
-			struct hfi_msg_sys_debug_pkt *pkt =
-				(struct hfi_msg_sys_debug_pkt *)packet;
+		if (hdr->size <= sizeof(*hdr))
+			continue;
+
+		if (hdr->size > IFACEQ_CORE_PKT_SIZE)
+			continue;
+
+		if (hdr->pkt_type != HFI_MSG_SYS_COV) {
+			pkt = (struct hfi_msg_sys_debug_pkt *)packet;
+
+			if (hdr->size <= sizeof(*pkt))
+				continue;
+
+			log = pkt->msg_data;
+			log_size = hdr->size - sizeof(*pkt);
+			if (pkt->msg_size < log_size)
+				log_size = pkt->msg_size;
 
-			dev_dbg(core->dev, "%s", pkt->msg_data);
+			if (pkt->msg_type & (IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL))
+				dev_err_ratelimited(core->dev, "%.*s", (int)log_size, log);
+			else
+				dev_dbg(core->dev, "%.*s", (int)log_size, log);
 		}
 	}
 }
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
index 388a36ff2b07b7bcd8db21d4345bc900356b4ec3..cca3c2e8668802d45ff309928a731b04a78a1bcc 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
@@ -17,6 +17,9 @@
 #define SYS_IFPC_PKT_SIZE (sizeof(struct iris_hfi_header) + \
 	sizeof(struct iris_hfi_packet) + sizeof(u32))
 
+#define SYS_DEBUG_PKT_SIZE (sizeof(struct iris_hfi_header) + \
+	2 * (sizeof(struct iris_hfi_packet) + sizeof(u32)))
+
 #define SYS_NO_PAYLOAD_PKT_SIZE (sizeof(struct iris_hfi_header) + \
 	sizeof(struct iris_hfi_packet))
 
@@ -54,6 +57,23 @@ static int iris_hfi_gen2_sys_image_version(struct iris_core *core)
 	return ret;
 }
 
+static int iris_hfi_gen2_sys_set_debug(struct iris_core *core)
+{
+	struct iris_hfi_header *hdr;
+	int ret;
+
+	hdr = kzalloc(SYS_DEBUG_PKT_SIZE, GFP_KERNEL);
+	if (!hdr)
+		return -ENOMEM;
+
+	iris_hfi_gen2_packet_set_debug(core, hdr);
+	ret = iris_hfi_queue_cmd_write_locked(core, hdr, hdr->size);
+
+	kfree(hdr);
+
+	return ret;
+}
+
 static int iris_hfi_gen2_sys_interframe_powercollapse(struct iris_core *core)
 {
 	struct iris_hfi_header *hdr;
@@ -1391,6 +1411,7 @@ static struct iris_inst *iris_hfi_gen2_get_instance(void)
 static const struct iris_hfi_sys_ops iris_hfi_gen2_sys_ops = {
 	.sys_init = iris_hfi_gen2_sys_init,
 	.sys_image_version = iris_hfi_gen2_sys_image_version,
+	.sys_set_debug = iris_hfi_gen2_sys_set_debug,
 	.sys_interframe_powercollapse = iris_hfi_gen2_sys_interframe_powercollapse,
 	.sys_pc_prep = iris_hfi_gen2_sys_pc_prep,
 
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h b/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
index f43aea10090d8f1d6ae5c20547e5f6321b2ca203..1d4ded82ecdac6e4af9fdd11460158682d27fbf3 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
@@ -39,6 +39,9 @@
 #define HFI_PROP_UBWC_BANK_SWZL_LEVEL2		0x03000007
 #define HFI_PROP_UBWC_BANK_SWZL_LEVEL3		0x03000008
 #define HFI_PROP_UBWC_BANK_SPREADING		0x03000009
+#define HFI_PROP_DEBUG_CONFIG			0x0300000a
+#define HFI_PROP_DEBUG_LOG_LEVEL		0x0300000b
+#define HFI_DEBUG_CONFIG_DEFAULT		0x00000000
 #define HFI_PROP_CODEC				0x03000100
 #define HFI_PROP_COLOR_FORMAT			0x03000101
 #define HFI_PROP_BITSTREAM_RESOLUTION		0x03000103
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
index 0d05dd2afc07d830cc8502ab5f28001312991ba8..2dbe57b87fd3c3198d183c32be912797cf36de39 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
@@ -226,6 +226,35 @@ void iris_hfi_gen2_packet_image_version(struct iris_core *core, struct iris_hfi_
 				    NULL, 0);
 }
 
+void iris_hfi_gen2_packet_set_debug(struct iris_core *core, struct iris_hfi_header *hdr)
+{
+	u32 fw_debug;
+	u32 payload;
+
+	iris_hfi_gen2_create_header(hdr, 0, core->header_id++);
+
+	payload = HFI_DEBUG_CONFIG_DEFAULT;
+	iris_hfi_gen2_create_packet(hdr,
+				    HFI_PROP_DEBUG_CONFIG,
+				    HFI_HOST_FLAGS_NONE,
+				    HFI_PAYLOAD_U32_ENUM,
+				    HFI_PORT_NONE,
+				    core->packet_id++,
+				    &payload,
+				    sizeof(u32));
+
+	fw_debug = READ_ONCE(core->fw_debug) & IRIS_FW_DEBUG_LOGMASK;
+	payload = fw_debug;
+	iris_hfi_gen2_create_packet(hdr,
+				    HFI_PROP_DEBUG_LOG_LEVEL,
+				    HFI_HOST_FLAGS_NONE,
+				    HFI_PAYLOAD_U32_ENUM,
+				    HFI_PORT_NONE,
+				    core->packet_id++,
+				    &payload,
+				    sizeof(u32));
+}
+
 void iris_hfi_gen2_packet_session_command(struct iris_inst *inst, u32 pkt_type,
 					  u32 flags, u32 port, u32 session_id,
 					  u32 payload_type, void *payload,
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
index 25b9582349ca1a0ce6efc0b146a3abb798485c45..aff025fbf5eadbedbf0108b41b6f9bf418d4f779 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
@@ -111,6 +111,7 @@ u32 iris_hfi_gen2_get_color_info(u32 matrix_coeff, u32 transfer_char, u32 primar
 
 void iris_hfi_gen2_packet_sys_init(struct iris_core *core, struct iris_hfi_header *hdr);
 void iris_hfi_gen2_packet_image_version(struct iris_core *core, struct iris_hfi_header *hdr);
+void iris_hfi_gen2_packet_set_debug(struct iris_core *core, struct iris_hfi_header *hdr);
 void iris_hfi_gen2_packet_session_command(struct iris_inst *inst, u32 pkt_type,
 					  u32 flags, u32 port, u32 session_id,
 					  u32 payload_type, void *payload,
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
index 8c2644c7f6e85983d7ad7584fc0cb570e4813ae4..e8dc8751ea939bd544699f464a831736fcc10b7f 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
@@ -991,20 +991,24 @@ static int iris_hfi_gen2_handle_response(struct iris_core *core, void *response)
 static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
 {
 	struct hfi_debug_header *pkt;
+	u32 log_size;
 	u8 *log;
 
 	while (!iris_hfi_queue_dbg_read(core, packet)) {
 		pkt = (struct hfi_debug_header *)packet;
 
-		if (pkt->size < sizeof(*pkt))
+		if (pkt->size <= sizeof(*pkt) + 1)
 			continue;
 
-		if (pkt->size >= IFACEQ_CORE_PKT_SIZE)
+		if (pkt->size > IFACEQ_CORE_PKT_SIZE)
 			continue;
 
-		packet[pkt->size] = '\0';
 		log = (u8 *)packet + sizeof(*pkt) + 1;
-		dev_dbg(core->dev, "%s", log);
+		log_size = pkt->size - sizeof(*pkt) - 1;
+		if (pkt->debug_level & (IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL))
+			dev_err_ratelimited(core->dev, "%.*s", (int)log_size, log);
+		else
+			dev_dbg(core->dev, "%.*s", (int)log_size, log);
 	}
 }
 
diff --git a/drivers/media/platform/qcom/iris/iris_probe.c b/drivers/media/platform/qcom/iris/iris_probe.c
index e4acf4a74f944bcae83089ef5489f204d4b0078e..8bc94acc4f58ce7792ddaeaad036475c476d5c36 100644
--- a/drivers/media/platform/qcom/iris/iris_probe.c
+++ b/drivers/media/platform/qcom/iris/iris_probe.c
@@ -12,6 +12,7 @@
 #include <linux/reset.h>
 #include <linux/soc/qcom/ubwc.h>
 
+#include "iris_debugfs.h"
 #include "iris_core.h"
 #include "iris_ctrls.h"
 #include "iris_vidc.h"
@@ -207,6 +208,8 @@ static void iris_remove(struct platform_device *pdev)
 
 	v4l2_device_unregister(&core->v4l2_dev);
 
+	iris_debugfs_deinit(core);
+
 	mutex_destroy(&core->lock);
 }
 
@@ -230,6 +233,7 @@ static int iris_probe(struct platform_device *pdev)
 	if (!core)
 		return -ENOMEM;
 	core->dev = dev;
+	core->fw_debug = IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL;
 
 	core->state = IRIS_CORE_DEINIT;
 	mutex_init(&core->lock);
@@ -298,6 +302,8 @@ static int iris_probe(struct platform_device *pdev)
 	if (ret)
 		goto err_vdev_unreg_enc;
 
+	iris_debugfs_init(core);
+
 	return 0;
 
 err_vdev_unreg_enc:
diff --git a/drivers/media/platform/qcom/iris/iris_vb2.c b/drivers/media/platform/qcom/iris/iris_vb2.c
index a2ea2d67f60d05bc0dceb097f94a4f80199b0618..3761987a62e1daabc02a0fd28342c108eec02a19 100644
--- a/drivers/media/platform/qcom/iris/iris_vb2.c
+++ b/drivers/media/platform/qcom/iris/iris_vb2.c
@@ -127,7 +127,10 @@ int iris_vb2_queue_setup(struct vb2_queue *q,
 		goto unlock;
 
 	if (!inst->once_per_session_set) {
-		inst->once_per_session_set = true;
+		ret = iris_hfi_set_debug(core);
+		if (ret)
+			dev_dbg(core->dev,
+				"failed to set firmware debug level: %d\n", ret);
 
 		ret = inst->hfi_session_ops->session_open(inst);
 		if (ret) {
@@ -136,6 +139,8 @@ int iris_vb2_queue_setup(struct vb2_queue *q,
 			goto unlock;
 		}
 
+		inst->once_per_session_set = true;
+
 		ret = iris_inst_change_state(inst, IRIS_INST_INIT);
 		if (ret)
 			goto unlock;

-- 
2.34.1

[PATCH v2 2/2] media: qcom: iris: reduce impact of verbose firmware logging

[Renjiang Han]

Verbose firmware logging can produce a large number of debug queue
packets. The Iris interrupt thread used to drain the message queue once
and then drain the debug queue. With heavy firmware logging, response
packets that arrive while debug packets are being drained can remain
pending until a later interrupt.

Increase the HFI response timeout and runtime PM autosuspend delay when
firmware logging is enabled beyond the default error and fatal levels.
Restore the default values when the firmware log level is reduced again.

Signed-off-by: Renjiang Han <renjiang.han@oss.qualcomm.com>
---
 drivers/media/platform/qcom/iris/iris_core.c       |  2 +-
 drivers/media/platform/qcom/iris/iris_core.h       |  2 ++
 drivers/media/platform/qcom/iris/iris_debugfs.c    | 16 ++++++++++++++-
 drivers/media/platform/qcom/iris/iris_hfi_common.h |  1 +
 .../platform/qcom/iris/iris_hfi_gen1_response.c    | 24 ++++++++++++++++------
 .../platform/qcom/iris/iris_hfi_gen2_response.c    | 24 ++++++++++++++++------
 drivers/media/platform/qcom/iris/iris_probe.c      |  3 ++-
 drivers/media/platform/qcom/iris/iris_utils.c      |  4 +++-
 8 files changed, 60 insertions(+), 16 deletions(-)

diff --git a/drivers/media/platform/qcom/iris/iris_core.c b/drivers/media/platform/qcom/iris/iris_core.c
index 8c335dbfce166848892c2b323cdcafe43af4a2d9..bbbb7191c4778d842d4fef32fa95a0a9aeeecf7a 100644
--- a/drivers/media/platform/qcom/iris/iris_core.c
+++ b/drivers/media/platform/qcom/iris/iris_core.c
@@ -40,7 +40,7 @@ static int iris_wait_for_system_response(struct iris_core *core)
 		return -EIO;
 
 	ret = wait_for_completion_timeout(&core->core_init_done,
-					  msecs_to_jiffies(HW_RESPONSE_TIMEOUT_VALUE));
+					  msecs_to_jiffies(READ_ONCE(core->hw_response_timeout)));
 	if (!ret) {
 		core->state = IRIS_CORE_ERROR;
 		return -ETIMEDOUT;
diff --git a/drivers/media/platform/qcom/iris/iris_core.h b/drivers/media/platform/qcom/iris/iris_core.h
index a8e34c456ce3f3f178ca66a278d06d1ab50dafc4..5449e6b0e11c3122d1f57aedb6dd374ffa816497 100644
--- a/drivers/media/platform/qcom/iris/iris_core.h
+++ b/drivers/media/platform/qcom/iris/iris_core.h
@@ -68,6 +68,7 @@ struct dentry;
  * @debug_queue: shared interface queue to receive debug info from firmware
  * @root: debugfs root directory
  * @fw_debug: firmware debug log mask
+ * @hw_response_timeout: hardware response timeout
  * @lock: a lock for this structure
  * @response_packet: a pointer to response packet from fw to driver
  * @header_id: id of packet header
@@ -115,6 +116,7 @@ struct iris_core {
 	struct iris_iface_q_info		debug_queue;
 	struct dentry				*root;
 	u32					fw_debug;
+	u32					hw_response_timeout;
 	struct mutex				lock; /* lock for core related operations */
 	u8					*response_packet;
 	u32					header_id;
diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.c b/drivers/media/platform/qcom/iris/iris_debugfs.c
index 6122b994ad0aa70c0c4f26ea4ce076f1b025f810..77726270b11777acb5ea85203223c6943365a5ce 100644
--- a/drivers/media/platform/qcom/iris/iris_debugfs.c
+++ b/drivers/media/platform/qcom/iris/iris_debugfs.c
@@ -4,6 +4,7 @@
  */
 
 #include <linux/debugfs.h>
+#include <linux/pm_runtime.h>
 
 #include "iris_core.h"
 #include "iris_debugfs.h"
@@ -20,8 +21,21 @@ static int iris_fw_level_get(void *data, u64 *val)
 static int iris_fw_level_set(void *data, u64 val)
 {
 	struct iris_core *core = data;
+	bool debug_enabled;
+	u32 fw_debug;
 
-	WRITE_ONCE(core->fw_debug, (u32)val & IRIS_FW_DEBUG_LOGMASK);
+	fw_debug = (u32)val & IRIS_FW_DEBUG_LOGMASK;
+	debug_enabled = fw_debug & ~IRIS_FW_DEBUG_DEFAULT;
+
+	WRITE_ONCE(core->fw_debug, fw_debug);
+
+	if (debug_enabled) {
+		WRITE_ONCE(core->hw_response_timeout, 4 * HW_RESPONSE_TIMEOUT_VALUE);
+		pm_runtime_set_autosuspend_delay(core->dev, 4 * AUTOSUSPEND_DELAY_VALUE);
+	} else {
+		WRITE_ONCE(core->hw_response_timeout, HW_RESPONSE_TIMEOUT_VALUE);
+		pm_runtime_set_autosuspend_delay(core->dev, AUTOSUSPEND_DELAY_VALUE);
+	}
 
 	return 0;
 }
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.h b/drivers/media/platform/qcom/iris/iris_hfi_common.h
index d8bc0862c54bd229d575113752526d51e863e303..18c852d8b818f41bf5e1b0afd35082eeaf5240b4 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_common.h
+++ b/drivers/media/platform/qcom/iris/iris_hfi_common.h
@@ -112,6 +112,7 @@ struct iris_hfi_prop_type_handle {
 #define IRIS_FW_DEBUG_FATAL	0x00000010
 #define IRIS_FW_DEBUG_PERF	0x00000020
 #define IRIS_FW_DEBUG_LOGMASK	0x0fffffff
+#define IRIS_FW_DEBUG_DEFAULT	(IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL)
 
 struct iris_hfi_sys_ops {
 	int (*sys_init)(struct iris_core *core);
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
index 903b60a7380559f4711b0907fb1de405fe1707f3..6b8f952d0d4b54f37596728f1823c9265c637bbe 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
@@ -672,15 +672,17 @@ static void iris_hfi_gen1_handle_response(struct iris_core *core, void *response
 	}
 }
 
-static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
+static int iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
 {
 	struct hfi_msg_sys_debug_pkt *pkt;
 	struct hfi_pkt_hdr *hdr;
+	int num_pkts = 0;
 	u32 log_size;
 	u8 *log;
 
 	while (!iris_hfi_queue_dbg_read(core, packet)) {
 		hdr = (struct hfi_pkt_hdr *)packet;
+		num_pkts++;
 
 		if (hdr->size <= sizeof(*hdr))
 			continue;
@@ -705,15 +707,25 @@ static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
 				dev_dbg(core->dev, "%.*s", (int)log_size, log);
 		}
 	}
+
+	return num_pkts;
 }
 
 void iris_hfi_gen1_response_handler(struct iris_core *core)
 {
-	memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
-	while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
-		iris_hfi_gen1_handle_response(core, core->response_packet);
+	int num_debug_pkts;
+	int num_msg_pkts;
+
+	do {
+		num_msg_pkts = 0;
 		memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
-	}
+		while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
+			iris_hfi_gen1_handle_response(core, core->response_packet);
+			num_msg_pkts++;
+			memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
+		}
 
-	iris_hfi_gen1_flush_debug_queue(core, core->response_packet);
+		num_debug_pkts =
+			iris_hfi_gen1_flush_debug_queue(core, core->response_packet);
+	} while (num_msg_pkts || num_debug_pkts);
 }
diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
index e8dc8751ea939bd544699f464a831736fcc10b7f..1bd4ec0c5ada1ef17de4f07fe19d1fac611db04f 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
@@ -988,14 +988,16 @@ static int iris_hfi_gen2_handle_response(struct iris_core *core, void *response)
 		return iris_hfi_gen2_handle_session_response(core, hdr);
 }
 
-static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
+static int iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
 {
 	struct hfi_debug_header *pkt;
+	int num_pkts = 0;
 	u32 log_size;
 	u8 *log;
 
 	while (!iris_hfi_queue_dbg_read(core, packet)) {
 		pkt = (struct hfi_debug_header *)packet;
+		num_pkts++;
 
 		if (pkt->size <= sizeof(*pkt) + 1)
 			continue;
@@ -1010,10 +1012,15 @@ static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
 		else
 			dev_dbg(core->dev, "%.*s", (int)log_size, log);
 	}
+
+	return num_pkts;
 }
 
 void iris_hfi_gen2_response_handler(struct iris_core *core)
 {
+	int num_debug_pkts;
+	int num_msg_pkts;
+
 	if (iris_vpu_watchdog(core, core->intr_status)) {
 		struct iris_hfi_packet pkt = {.type = HFI_SYS_ERROR_WD_TIMEOUT};
 
@@ -1024,11 +1031,16 @@ void iris_hfi_gen2_response_handler(struct iris_core *core)
 		return;
 	}
 
-	memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
-	while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
-		iris_hfi_gen2_handle_response(core, core->response_packet);
+	do {
+		num_msg_pkts = 0;
 		memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
-	}
+		while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
+			iris_hfi_gen2_handle_response(core, core->response_packet);
+			num_msg_pkts++;
+			memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
+		}
 
-	iris_hfi_gen2_flush_debug_queue(core, core->response_packet);
+		num_debug_pkts =
+			iris_hfi_gen2_flush_debug_queue(core, core->response_packet);
+	} while (num_msg_pkts || num_debug_pkts);
 }
diff --git a/drivers/media/platform/qcom/iris/iris_probe.c b/drivers/media/platform/qcom/iris/iris_probe.c
index 8bc94acc4f58ce7792ddaeaad036475c476d5c36..318a548a7ac606594a41d9dae25e73205de1fd0d 100644
--- a/drivers/media/platform/qcom/iris/iris_probe.c
+++ b/drivers/media/platform/qcom/iris/iris_probe.c
@@ -233,7 +233,8 @@ static int iris_probe(struct platform_device *pdev)
 	if (!core)
 		return -ENOMEM;
 	core->dev = dev;
-	core->fw_debug = IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL;
+	core->fw_debug = IRIS_FW_DEBUG_DEFAULT;
+	core->hw_response_timeout = HW_RESPONSE_TIMEOUT_VALUE;
 
 	core->state = IRIS_CORE_DEINIT;
 	mutex_init(&core->lock);
diff --git a/drivers/media/platform/qcom/iris/iris_utils.c b/drivers/media/platform/qcom/iris/iris_utils.c
index ba5c8dc1280c2e147cd33498b69a815cf7c75683..dbddfd770edce22fba58eff15cb6d45eae5ed638 100644
--- a/drivers/media/platform/qcom/iris/iris_utils.c
+++ b/drivers/media/platform/qcom/iris/iris_utils.c
@@ -70,12 +70,14 @@ void iris_helper_buffers_done(struct iris_inst *inst, unsigned int type,
 int iris_wait_for_session_response(struct iris_inst *inst, bool is_flush)
 {
 	struct completion *done;
+	unsigned int timeout;
 	int ret;
 
 	done = is_flush ? &inst->flush_completion : &inst->completion;
+	timeout = READ_ONCE(inst->core->hw_response_timeout);
 
 	mutex_unlock(&inst->lock);
-	ret = wait_for_completion_timeout(done, msecs_to_jiffies(HW_RESPONSE_TIMEOUT_VALUE));
+	ret = wait_for_completion_timeout(done, msecs_to_jiffies(timeout));
 	mutex_lock(&inst->lock);
 	if (!ret) {
 		iris_inst_change_state(inst, IRIS_INST_ERROR);

-- 
2.34.1

Re: [PATCH v2 1/2] media: qcom: iris: add firmware debug logging support

[Vishnu Reddy]

On 9/11/2026 8:09 AM, Renjiang Han wrote:
> Iris does not provide runtime control for firmware debug logging, which
> makes firmware-side failures harder to diagnose without rebuilding or
> reloading the driver.
>
> Add a debugfs fw_level file under the Iris debugfs directory so the
> firmware debug log mask can be configured at runtime. Keep the default
> mask limited to firmware error and fatal messages to avoid enabling
> verbose logging by default.
>
> Store the mask per Iris core and normalize values written through
> debugfs so userspace reads back the same mask that is sent to firmware.
> Program the firmware debug configuration during core initialization,
> runtime resume, and before opening a new session. This allows a
> fw_level value written before session start to take effect without
> requiring a driver reload, and preserves the setting after firmware is
> reinitialized.
>
> Add HFI Gen1 and Gen2 support for sending the firmware debug
> configuration, and flush firmware debug messages from the debug queue.
> Emit firmware error and fatal messages with dev_err_ratelimited(), and
> use dev_dbg() for other firmware debug messages. Bound firmware log
> printing by the packet size so messages do not need to be NUL
> terminated by firmware.
>
> Treat failures to configure firmware debug logging as non-fatal, since
> they should not prevent video sessions from running.
>
> Tested-by: Wangao Wang <wangao.wang@oss.qualcomm.com>
> Signed-off-by: Renjiang Han <renjiang.han@oss.qualcomm.com>
> ---
>  drivers/media/platform/qcom/iris/Makefile          |  1 +
>  drivers/media/platform/qcom/iris/iris_core.h       |  7 +++-
>  drivers/media/platform/qcom/iris/iris_debugfs.c    | 42 +++++++++++++++++++++
>  drivers/media/platform/qcom/iris/iris_debugfs.h    | 14 +++++++
>  drivers/media/platform/qcom/iris/iris_hfi_common.c | 43 ++++++++++++++++++++++
>  drivers/media/platform/qcom/iris/iris_hfi_common.h | 10 +++++
>  .../platform/qcom/iris/iris_hfi_gen1_command.c     | 30 +++++++++++++++
>  .../platform/qcom/iris/iris_hfi_gen1_defines.h     |  8 ++++
>  .../platform/qcom/iris/iris_hfi_gen1_response.c    | 31 +++++++++++++---
>  .../platform/qcom/iris/iris_hfi_gen2_command.c     | 21 +++++++++++
>  .../platform/qcom/iris/iris_hfi_gen2_defines.h     |  3 ++
>  .../platform/qcom/iris/iris_hfi_gen2_packet.c      | 29 +++++++++++++++
>  .../platform/qcom/iris/iris_hfi_gen2_packet.h      |  1 +
>  .../platform/qcom/iris/iris_hfi_gen2_response.c    | 12 ++++--
>  drivers/media/platform/qcom/iris/iris_probe.c      |  6 +++
>  drivers/media/platform/qcom/iris/iris_vb2.c        |  7 +++-
>  16 files changed, 253 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/media/platform/qcom/iris/Makefile b/drivers/media/platform/qcom/iris/Makefile
> index 48e415cbc4390bc596f6239fefa2a2ad2cd3a2bb..105a66e1b0af44f010730a899be26cae23d76c73 100644
> --- a/drivers/media/platform/qcom/iris/Makefile
> +++ b/drivers/media/platform/qcom/iris/Makefile
> @@ -2,6 +2,7 @@ qcom-iris-objs += iris_buffer.o \
>               iris_common.o \
>               iris_core.o \
>               iris_ctrls.o \
> +             iris_debugfs.o \
>               iris_firmware.o \
>               iris_hfi_common.o \
>               iris_hfi_gen1.o \
> diff --git a/drivers/media/platform/qcom/iris/iris_core.h b/drivers/media/platform/qcom/iris/iris_core.h
> index 24da60448cf24820af7947b85eb7208555ab7786..a8e34c456ce3f3f178ca66a278d06d1ab50dafc4 100644
> --- a/drivers/media/platform/qcom/iris/iris_core.h
> +++ b/drivers/media/platform/qcom/iris/iris_core.h
> @@ -31,6 +31,7 @@ enum domain_type {
>  };
>  
>  struct qcom_ubwc_cfg_data;
> +struct dentry;
>  
>  /**
>   * struct iris_core - holds core parameters valid for all instances
> @@ -65,7 +66,9 @@ struct qcom_ubwc_cfg_data;
>   * @command_queue: shared interface queue to send commands to firmware
>   * @message_queue: shared interface queue to receive responses from firmware
>   * @debug_queue: shared interface queue to receive debug info from firmware
> - * @lock: a lock for this strucure
> + * @root: debugfs root directory
> + * @fw_debug: firmware debug log mask
> + * @lock: a lock for this structure
>   * @response_packet: a pointer to response packet from fw to driver
>   * @header_id: id of packet header
>   * @packet_id: id of packet
> @@ -110,6 +113,8 @@ struct iris_core {
>  	struct iris_iface_q_info		command_queue;
>  	struct iris_iface_q_info		message_queue;
>  	struct iris_iface_q_info		debug_queue;
> +	struct dentry				*root;
> +	u32					fw_debug;
>  	struct mutex				lock; /* lock for core related operations */
>  	u8					*response_packet;
>  	u32					header_id;
> diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.c b/drivers/media/platform/qcom/iris/iris_debugfs.c
> new file mode 100644
> index 0000000000000000000000000000000000000000..6122b994ad0aa70c0c4f26ea4ce076f1b025f810
> --- /dev/null
> +++ b/drivers/media/platform/qcom/iris/iris_debugfs.c
> @@ -0,0 +1,42 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (c) Qualcomm Innovation Center, Inc. All rights reserved.
> + */
> +
> +#include <linux/debugfs.h>
> +
> +#include "iris_core.h"
> +#include "iris_debugfs.h"
> +
> +static int iris_fw_level_get(void *data, u64 *val)
> +{
> +	struct iris_core *core = data;
> +
> +	*val = READ_ONCE(core->fw_debug);
> +
> +	return 0;
> +}
> +
> +static int iris_fw_level_set(void *data, u64 val)
> +{
> +	struct iris_core *core = data;
> +
> +	WRITE_ONCE(core->fw_debug, (u32)val & IRIS_FW_DEBUG_LOGMASK);
> +
> +	return 0;
> +}
> +
> +DEFINE_DEBUGFS_ATTRIBUTE(iris_fw_level_fops, iris_fw_level_get,
> +			 iris_fw_level_set, "0x%08llx\n");
> +
> +void iris_debugfs_init(struct iris_core *core)
> +{
> +	core->root = debugfs_create_dir("iris", NULL);
> +	debugfs_create_file("fw_level", 0600, core->root, core,
> +			    &iris_fw_level_fops);
> +}
> +
> +void iris_debugfs_deinit(struct iris_core *core)
> +{
> +	debugfs_remove(core->root);
> +}
> diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.h b/drivers/media/platform/qcom/iris/iris_debugfs.h
> new file mode 100644
> index 0000000000000000000000000000000000000000..aa3cb97eb2b2e71dfe6a0db6ccfd430256cdb6fa
> --- /dev/null
> +++ b/drivers/media/platform/qcom/iris/iris_debugfs.h
> @@ -0,0 +1,14 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +/*
> + * Copyright (c) Qualcomm Innovation Center, Inc. All rights reserved.
> + */
> +
> +#ifndef __IRIS_DEBUGFS_H__
> +#define __IRIS_DEBUGFS_H__
> +
> +struct iris_core;
> +
> +void iris_debugfs_init(struct iris_core *core);
> +void iris_debugfs_deinit(struct iris_core *core);
> +
> +#endif
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.c b/drivers/media/platform/qcom/iris/iris_hfi_common.c
> index 8769ec61f11769e004945063381d9baddb302b06..706a4d321ee2899aadc9f650d0d9a864fdc6769b 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_common.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_common.c
> @@ -87,9 +87,44 @@ int iris_hfi_core_init(struct iris_core *core)
>  	if (ret)
>  		return ret;
>  
> +	if (hfi_ops->sys_set_debug) {
> +		ret = hfi_ops->sys_set_debug(core);
> +		if (ret)
> +			dev_warn(core->dev,
> +				 "failed to configure firmware debug logging: %d\n",
> +				 ret);
> +	}
> +
>  	return hfi_ops->sys_interframe_powercollapse(core);
>  }
>  
> +int iris_hfi_set_debug(struct iris_core *core)
> +{
> +	const struct iris_hfi_sys_ops *hfi_ops = core->hfi_sys_ops;
> +	int ret = 0;
> +
> +	if (!hfi_ops->sys_set_debug)
> +		return 0;
> +
> +	ret = pm_runtime_resume_and_get(core->dev);
> +	if (ret < 0)
> +		return ret;
> +
> +	mutex_lock(&core->lock);
> +	if (core->state != IRIS_CORE_INIT) {
> +		ret = 0;
> +		goto unlock;
> +	}
> +
> +	ret = hfi_ops->sys_set_debug(core);
> +
> +unlock:
> +	mutex_unlock(&core->lock);
> +	pm_runtime_put_autosuspend(core->dev);
> +
> +	return ret;
> +}
> +
>  irqreturn_t iris_hfi_isr(int irq, void *data)
>  {
>  	disable_irq_nosync(irq);
> @@ -163,6 +198,14 @@ int iris_hfi_pm_resume(struct iris_core *core)
>  	if (ret)
>  		goto err_suspend_hw;
>  
> +	if (ops->sys_set_debug) {
> +		ret = ops->sys_set_debug(core);
> +		if (ret)
> +			dev_warn(core->dev,
> +				 "failed to configure firmware debug logging: %d\n",
> +				 ret);
> +	}
> +
>  	ret = ops->sys_interframe_powercollapse(core);
>  	if (ret)
>  		goto err_suspend_hw;
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.h b/drivers/media/platform/qcom/iris/iris_hfi_common.h
> index a27447eb2519962cb958b0e330a6d018310c3450..d8bc0862c54bd229d575113752526d51e863e303 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_common.h
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_common.h
> @@ -105,9 +105,18 @@ struct iris_hfi_prop_type_handle {
>  	int (*handle)(struct iris_inst *inst, u32 plane);
>  };
>  
> +#define IRIS_FW_DEBUG_LOW	0x00000001
> +#define IRIS_FW_DEBUG_MEDIUM	0x00000002
> +#define IRIS_FW_DEBUG_HIGH	0x00000004
> +#define IRIS_FW_DEBUG_ERROR	0x00000008
> +#define IRIS_FW_DEBUG_FATAL	0x00000010
> +#define IRIS_FW_DEBUG_PERF	0x00000020
> +#define IRIS_FW_DEBUG_LOGMASK	0x0fffffff
> +
>  struct iris_hfi_sys_ops {
>  	int (*sys_init)(struct iris_core *core);
>  	int (*sys_image_version)(struct iris_core *core);
> +	int (*sys_set_debug)(struct iris_core *core);
>  	int (*sys_interframe_powercollapse)(struct iris_core *core);
>  	int (*sys_pc_prep)(struct iris_core *core);
>  
> @@ -153,6 +162,7 @@ u32 iris_hfi_get_v4l2_color_primaries(u32 hfi_primaries);
>  u32 iris_hfi_get_v4l2_transfer_char(u32 hfi_characterstics);
>  u32 iris_hfi_get_v4l2_matrix_coefficients(u32 hfi_coefficients);
>  int iris_hfi_core_init(struct iris_core *core);
> +int iris_hfi_set_debug(struct iris_core *core);
>  int iris_hfi_pm_suspend(struct iris_core *core);
>  int iris_hfi_pm_resume(struct iris_core *core);
>  
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
> index f6e9cbfccd09e4013e842c92c8272ddeb1b166d3..55e2fade62e0450b8ce5e5c928b746dcb6ef8387 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_command.c
> @@ -53,6 +53,35 @@ static int iris_hfi_gen1_sys_image_version(struct iris_core *core)
>  	return iris_hfi_queue_cmd_write_locked(core, &packet, packet.hdr.size);
>  }
>  
> +static int iris_hfi_gen1_sys_set_debug(struct iris_core *core)
> +{
> +	struct hfi_sys_set_property_pkt *pkt;
> +	struct hfi_debug_config *hfi;
> +	u32 fw_debug;
> +	u32 packet_size;
> +	int ret;
> +
> +	packet_size = struct_size(pkt, data, 1) + sizeof(*hfi);
> +	pkt = kzalloc(packet_size, GFP_KERNEL);
> +	if (!pkt)
> +		return -ENOMEM;
> +
> +	hfi = (struct hfi_debug_config *)&pkt->data[1];
> +
> +	pkt->hdr.size = packet_size;
> +	pkt->hdr.pkt_type = HFI_CMD_SYS_SET_PROPERTY;
> +	pkt->num_properties = 1;
> +	pkt->data[0] = HFI_PROPERTY_SYS_DEBUG_CONFIG;
> +	fw_debug = READ_ONCE(core->fw_debug) & IRIS_FW_DEBUG_LOGMASK;
> +	hfi->config = fw_debug;
> +	hfi->mode = fw_debug ? HFI_DEBUG_MODE_QUEUE : 0;
> +
> +	ret = iris_hfi_queue_cmd_write_locked(core, pkt, pkt->hdr.size);
> +	kfree(pkt);
> +
> +	return ret;
> +}
> +
>  static int iris_hfi_gen1_sys_interframe_powercollapse(struct iris_core *core)
>  {
>  	struct hfi_sys_set_property_pkt *pkt;
> @@ -1149,6 +1178,7 @@ static struct iris_inst *iris_hfi_gen1_get_instance(void)
>  static const struct iris_hfi_sys_ops iris_hfi_gen1_sys_ops = {
>  	.sys_init = iris_hfi_gen1_sys_init,
>  	.sys_image_version = iris_hfi_gen1_sys_image_version,
> +	.sys_set_debug = iris_hfi_gen1_sys_set_debug,
>  	.sys_interframe_powercollapse = iris_hfi_gen1_sys_interframe_powercollapse,
>  	.sys_pc_prep = iris_hfi_gen1_sys_pc_prep,
>  
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h b/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
> index 275f3fea3d2aabe26a2a0498c78c09ec28765b24..887f96c5623809beb1c8e911949b314b9bd029da 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_defines.h
> @@ -80,6 +80,14 @@
>  #define HFI_BUFFER_INTERNAL_SCRATCH_1			0x7
>  #define HFI_BUFFER_INTERNAL_SCRATCH_2			0x8
>  
> +#define HFI_DEBUG_MODE_QUEUE				0x01
> +
> +struct hfi_debug_config {
> +	u32 config;
> +	u32 mode;
> +};
> +
> +#define HFI_PROPERTY_SYS_DEBUG_CONFIG			0x1
>  #define HFI_PROPERTY_SYS_CODEC_POWER_PLANE_CTRL		0x5
>  #define HFI_PROPERTY_SYS_IMAGE_VERSION			0x6
>  
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> index bfd7495bf44f064091afb3245ec53cd18d323068..903b60a7380559f4711b0907fb1de405fe1707f3 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> @@ -674,16 +674,35 @@ static void iris_hfi_gen1_handle_response(struct iris_core *core, void *response
>  
>  static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
>  {
> -	struct hfi_msg_sys_coverage_pkt *pkt;
> +	struct hfi_msg_sys_debug_pkt *pkt;
> +	struct hfi_pkt_hdr *hdr;
> +	u32 log_size;
> +	u8 *log;
>  
>  	while (!iris_hfi_queue_dbg_read(core, packet)) {
> -		pkt = (struct hfi_msg_sys_coverage_pkt *)packet;
> +		hdr = (struct hfi_pkt_hdr *)packet;
>  
> -		if (pkt->hdr.pkt_type != HFI_MSG_SYS_COV) {
> -			struct hfi_msg_sys_debug_pkt *pkt =
> -				(struct hfi_msg_sys_debug_pkt *)packet;
> +		if (hdr->size <= sizeof(*hdr))
> +			continue;
> +
> +		if (hdr->size > IFACEQ_CORE_PKT_SIZE)
> +			continue;
> +
> +		if (hdr->pkt_type != HFI_MSG_SYS_COV) {
> +			pkt = (struct hfi_msg_sys_debug_pkt *)packet;
> +
> +			if (hdr->size <= sizeof(*pkt))
> +				continue;
> +
> +			log = pkt->msg_data;
> +			log_size = hdr->size - sizeof(*pkt);
> +			if (pkt->msg_size < log_size)
> +				log_size = pkt->msg_size;
>  
> -			dev_dbg(core->dev, "%s", pkt->msg_data);
> +			if (pkt->msg_type & (IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL))
> +				dev_err_ratelimited(core->dev, "%.*s", (int)log_size, log);
> +			else
> +				dev_dbg(core->dev, "%.*s", (int)log_size, log);
>  		}
>  	}
>  }
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
> index 388a36ff2b07b7bcd8db21d4345bc900356b4ec3..cca3c2e8668802d45ff309928a731b04a78a1bcc 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
> @@ -17,6 +17,9 @@
>  #define SYS_IFPC_PKT_SIZE (sizeof(struct iris_hfi_header) + \
>  	sizeof(struct iris_hfi_packet) + sizeof(u32))
>  
> +#define SYS_DEBUG_PKT_SIZE (sizeof(struct iris_hfi_header) + \
> +	2 * (sizeof(struct iris_hfi_packet) + sizeof(u32)))
> +
>  #define SYS_NO_PAYLOAD_PKT_SIZE (sizeof(struct iris_hfi_header) + \
>  	sizeof(struct iris_hfi_packet))
>  
> @@ -54,6 +57,23 @@ static int iris_hfi_gen2_sys_image_version(struct iris_core *core)
>  	return ret;
>  }
>  
> +static int iris_hfi_gen2_sys_set_debug(struct iris_core *core)
> +{
> +	struct iris_hfi_header *hdr;
> +	int ret;
> +
> +	hdr = kzalloc(SYS_DEBUG_PKT_SIZE, GFP_KERNEL);
> +	if (!hdr)
> +		return -ENOMEM;
> +
> +	iris_hfi_gen2_packet_set_debug(core, hdr);
> +	ret = iris_hfi_queue_cmd_write_locked(core, hdr, hdr->size);
> +
> +	kfree(hdr);
> +
> +	return ret;
> +}
> +
>  static int iris_hfi_gen2_sys_interframe_powercollapse(struct iris_core *core)
>  {
>  	struct iris_hfi_header *hdr;
> @@ -1391,6 +1411,7 @@ static struct iris_inst *iris_hfi_gen2_get_instance(void)
>  static const struct iris_hfi_sys_ops iris_hfi_gen2_sys_ops = {
>  	.sys_init = iris_hfi_gen2_sys_init,
>  	.sys_image_version = iris_hfi_gen2_sys_image_version,
> +	.sys_set_debug = iris_hfi_gen2_sys_set_debug,
>  	.sys_interframe_powercollapse = iris_hfi_gen2_sys_interframe_powercollapse,
>  	.sys_pc_prep = iris_hfi_gen2_sys_pc_prep,
>  
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h b/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
> index f43aea10090d8f1d6ae5c20547e5f6321b2ca203..1d4ded82ecdac6e4af9fdd11460158682d27fbf3 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_defines.h
> @@ -39,6 +39,9 @@
>  #define HFI_PROP_UBWC_BANK_SWZL_LEVEL2		0x03000007
>  #define HFI_PROP_UBWC_BANK_SWZL_LEVEL3		0x03000008
>  #define HFI_PROP_UBWC_BANK_SPREADING		0x03000009
> +#define HFI_PROP_DEBUG_CONFIG			0x0300000a
> +#define HFI_PROP_DEBUG_LOG_LEVEL		0x0300000b
> +#define HFI_DEBUG_CONFIG_DEFAULT		0x00000000
>  #define HFI_PROP_CODEC				0x03000100
>  #define HFI_PROP_COLOR_FORMAT			0x03000101
>  #define HFI_PROP_BITSTREAM_RESOLUTION		0x03000103
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
> index 0d05dd2afc07d830cc8502ab5f28001312991ba8..2dbe57b87fd3c3198d183c32be912797cf36de39 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.c
> @@ -226,6 +226,35 @@ void iris_hfi_gen2_packet_image_version(struct iris_core *core, struct iris_hfi_
>  				    NULL, 0);
>  }
>  
> +void iris_hfi_gen2_packet_set_debug(struct iris_core *core, struct iris_hfi_header *hdr)
> +{
> +	u32 fw_debug;
> +	u32 payload;
> +
> +	iris_hfi_gen2_create_header(hdr, 0, core->header_id++);
> +
> +	payload = HFI_DEBUG_CONFIG_DEFAULT;
> +	iris_hfi_gen2_create_packet(hdr,
> +				    HFI_PROP_DEBUG_CONFIG,
> +				    HFI_HOST_FLAGS_NONE,
> +				    HFI_PAYLOAD_U32_ENUM,
> +				    HFI_PORT_NONE,
> +				    core->packet_id++,
> +				    &payload,
> +				    sizeof(u32));
> +
> +	fw_debug = READ_ONCE(core->fw_debug) & IRIS_FW_DEBUG_LOGMASK;
> +	payload = fw_debug;
> +	iris_hfi_gen2_create_packet(hdr,
> +				    HFI_PROP_DEBUG_LOG_LEVEL,
> +				    HFI_HOST_FLAGS_NONE,
> +				    HFI_PAYLOAD_U32_ENUM,
> +				    HFI_PORT_NONE,
> +				    core->packet_id++,
> +				    &payload,
> +				    sizeof(u32));
> +}
> +
>  void iris_hfi_gen2_packet_session_command(struct iris_inst *inst, u32 pkt_type,
>  					  u32 flags, u32 port, u32 session_id,
>  					  u32 payload_type, void *payload,
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
> index 25b9582349ca1a0ce6efc0b146a3abb798485c45..aff025fbf5eadbedbf0108b41b6f9bf418d4f779 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_packet.h
> @@ -111,6 +111,7 @@ u32 iris_hfi_gen2_get_color_info(u32 matrix_coeff, u32 transfer_char, u32 primar
>  
>  void iris_hfi_gen2_packet_sys_init(struct iris_core *core, struct iris_hfi_header *hdr);
>  void iris_hfi_gen2_packet_image_version(struct iris_core *core, struct iris_hfi_header *hdr);
> +void iris_hfi_gen2_packet_set_debug(struct iris_core *core, struct iris_hfi_header *hdr);
>  void iris_hfi_gen2_packet_session_command(struct iris_inst *inst, u32 pkt_type,
>  					  u32 flags, u32 port, u32 session_id,
>  					  u32 payload_type, void *payload,
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> index 8c2644c7f6e85983d7ad7584fc0cb570e4813ae4..e8dc8751ea939bd544699f464a831736fcc10b7f 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> @@ -991,20 +991,24 @@ static int iris_hfi_gen2_handle_response(struct iris_core *core, void *response)
>  static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
>  {
>  	struct hfi_debug_header *pkt;
> +	u32 log_size;
>  	u8 *log;
>  
>  	while (!iris_hfi_queue_dbg_read(core, packet)) {
>  		pkt = (struct hfi_debug_header *)packet;
>  
> -		if (pkt->size < sizeof(*pkt))
> +		if (pkt->size <= sizeof(*pkt) + 1)
>  			continue;
>  
> -		if (pkt->size >= IFACEQ_CORE_PKT_SIZE)
> +		if (pkt->size > IFACEQ_CORE_PKT_SIZE)
>  			continue;
>  
> -		packet[pkt->size] = '\0';
>  		log = (u8 *)packet + sizeof(*pkt) + 1;
> -		dev_dbg(core->dev, "%s", log);
> +		log_size = pkt->size - sizeof(*pkt) - 1;
> +		if (pkt->debug_level & (IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL))
> +			dev_err_ratelimited(core->dev, "%.*s", (int)log_size, log);
> +		else
> +			dev_dbg(core->dev, "%.*s", (int)log_size, log);
>  	}
>  }
>  
> diff --git a/drivers/media/platform/qcom/iris/iris_probe.c b/drivers/media/platform/qcom/iris/iris_probe.c
> index e4acf4a74f944bcae83089ef5489f204d4b0078e..8bc94acc4f58ce7792ddaeaad036475c476d5c36 100644
> --- a/drivers/media/platform/qcom/iris/iris_probe.c
> +++ b/drivers/media/platform/qcom/iris/iris_probe.c
> @@ -12,6 +12,7 @@
>  #include <linux/reset.h>
>  #include <linux/soc/qcom/ubwc.h>
>  
> +#include "iris_debugfs.h"
>  #include "iris_core.h"
>  #include "iris_ctrls.h"
>  #include "iris_vidc.h"
> @@ -207,6 +208,8 @@ static void iris_remove(struct platform_device *pdev)
>  
>  	v4l2_device_unregister(&core->v4l2_dev);
>  
> +	iris_debugfs_deinit(core);
> +
>  	mutex_destroy(&core->lock);
>  }
>  
> @@ -230,6 +233,7 @@ static int iris_probe(struct platform_device *pdev)
>  	if (!core)
>  		return -ENOMEM;
>  	core->dev = dev;
> +	core->fw_debug = IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL;
>  
>  	core->state = IRIS_CORE_DEINIT;
>  	mutex_init(&core->lock);
> @@ -298,6 +302,8 @@ static int iris_probe(struct platform_device *pdev)
>  	if (ret)
>  		goto err_vdev_unreg_enc;
>  
> +	iris_debugfs_init(core);
> +
>  	return 0;
>  
>  err_vdev_unreg_enc:
> diff --git a/drivers/media/platform/qcom/iris/iris_vb2.c b/drivers/media/platform/qcom/iris/iris_vb2.c
> index a2ea2d67f60d05bc0dceb097f94a4f80199b0618..3761987a62e1daabc02a0fd28342c108eec02a19 100644
> --- a/drivers/media/platform/qcom/iris/iris_vb2.c
> +++ b/drivers/media/platform/qcom/iris/iris_vb2.c
> @@ -127,7 +127,10 @@ int iris_vb2_queue_setup(struct vb2_queue *q,
>  		goto unlock;
>  
>  	if (!inst->once_per_session_set) {
> -		inst->once_per_session_set = true;
> +		ret = iris_hfi_set_debug(core);
> +		if (ret)
> +			dev_dbg(core->dev,
> +				"failed to set firmware debug level: %d\n", ret);
>  
>  		ret = inst->hfi_session_ops->session_open(inst);
>  		if (ret) {
> @@ -136,6 +139,8 @@ int iris_vb2_queue_setup(struct vb2_queue *q,
>  			goto unlock;
>  		}
>  
> +		inst->once_per_session_set = true;
> +
>  		ret = iris_inst_change_state(inst, IRIS_INST_INIT);
>  		if (ret)
>  			goto unlock;

Reviewed-by: Vishnu Reddy <busanna.reddy@oss.qualcomm.com>

Re: [PATCH v2 2/2] media: qcom: iris: reduce impact of verbose firmware logging

[Vishnu Reddy]

On 9/11/2026 8:09 AM, Renjiang Han wrote:
> Verbose firmware logging can produce a large number of debug queue
> packets. The Iris interrupt thread used to drain the message queue once
> and then drain the debug queue. With heavy firmware logging, response
> packets that arrive while debug packets are being drained can remain
> pending until a later interrupt.
>
> Increase the HFI response timeout and runtime PM autosuspend delay when
> firmware logging is enabled beyond the default error and fatal levels.
> Restore the default values when the firmware log level is reduced again.
>
> Signed-off-by: Renjiang Han <renjiang.han@oss.qualcomm.com>
> ---
>  drivers/media/platform/qcom/iris/iris_core.c       |  2 +-
>  drivers/media/platform/qcom/iris/iris_core.h       |  2 ++
>  drivers/media/platform/qcom/iris/iris_debugfs.c    | 16 ++++++++++++++-
>  drivers/media/platform/qcom/iris/iris_hfi_common.h |  1 +
>  .../platform/qcom/iris/iris_hfi_gen1_response.c    | 24 ++++++++++++++++------
>  .../platform/qcom/iris/iris_hfi_gen2_response.c    | 24 ++++++++++++++++------
>  drivers/media/platform/qcom/iris/iris_probe.c      |  3 ++-
>  drivers/media/platform/qcom/iris/iris_utils.c      |  4 +++-
>  8 files changed, 60 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/media/platform/qcom/iris/iris_core.c b/drivers/media/platform/qcom/iris/iris_core.c
> index 8c335dbfce166848892c2b323cdcafe43af4a2d9..bbbb7191c4778d842d4fef32fa95a0a9aeeecf7a 100644
> --- a/drivers/media/platform/qcom/iris/iris_core.c
> +++ b/drivers/media/platform/qcom/iris/iris_core.c
> @@ -40,7 +40,7 @@ static int iris_wait_for_system_response(struct iris_core *core)
>  		return -EIO;
>  
>  	ret = wait_for_completion_timeout(&core->core_init_done,
> -					  msecs_to_jiffies(HW_RESPONSE_TIMEOUT_VALUE));
> +					  msecs_to_jiffies(READ_ONCE(core->hw_response_timeout)));
>  	if (!ret) {
>  		core->state = IRIS_CORE_ERROR;
>  		return -ETIMEDOUT;
> diff --git a/drivers/media/platform/qcom/iris/iris_core.h b/drivers/media/platform/qcom/iris/iris_core.h
> index a8e34c456ce3f3f178ca66a278d06d1ab50dafc4..5449e6b0e11c3122d1f57aedb6dd374ffa816497 100644
> --- a/drivers/media/platform/qcom/iris/iris_core.h
> +++ b/drivers/media/platform/qcom/iris/iris_core.h
> @@ -68,6 +68,7 @@ struct dentry;
>   * @debug_queue: shared interface queue to receive debug info from firmware
>   * @root: debugfs root directory
>   * @fw_debug: firmware debug log mask
> + * @hw_response_timeout: hardware response timeout
>   * @lock: a lock for this structure
>   * @response_packet: a pointer to response packet from fw to driver
>   * @header_id: id of packet header
> @@ -115,6 +116,7 @@ struct iris_core {
>  	struct iris_iface_q_info		debug_queue;
>  	struct dentry				*root;
>  	u32					fw_debug;
> +	u32					hw_response_timeout;
>  	struct mutex				lock; /* lock for core related operations */
>  	u8					*response_packet;
>  	u32					header_id;
> diff --git a/drivers/media/platform/qcom/iris/iris_debugfs.c b/drivers/media/platform/qcom/iris/iris_debugfs.c
> index 6122b994ad0aa70c0c4f26ea4ce076f1b025f810..77726270b11777acb5ea85203223c6943365a5ce 100644
> --- a/drivers/media/platform/qcom/iris/iris_debugfs.c
> +++ b/drivers/media/platform/qcom/iris/iris_debugfs.c
> @@ -4,6 +4,7 @@
>   */
>  
>  #include <linux/debugfs.h>
> +#include <linux/pm_runtime.h>
>  
>  #include "iris_core.h"
>  #include "iris_debugfs.h"
> @@ -20,8 +21,21 @@ static int iris_fw_level_get(void *data, u64 *val)
>  static int iris_fw_level_set(void *data, u64 val)
>  {
>  	struct iris_core *core = data;
> +	bool debug_enabled;
> +	u32 fw_debug;
>  
> -	WRITE_ONCE(core->fw_debug, (u32)val & IRIS_FW_DEBUG_LOGMASK);
> +	fw_debug = (u32)val & IRIS_FW_DEBUG_LOGMASK;
> +	debug_enabled = fw_debug & ~IRIS_FW_DEBUG_DEFAULT;
> +
> +	WRITE_ONCE(core->fw_debug, fw_debug);
> +
> +	if (debug_enabled) {
> +		WRITE_ONCE(core->hw_response_timeout, 4 * HW_RESPONSE_TIMEOUT_VALUE);
> +		pm_runtime_set_autosuspend_delay(core->dev, 4 * AUTOSUSPEND_DELAY_VALUE);
> +	} else {
> +		WRITE_ONCE(core->hw_response_timeout, HW_RESPONSE_TIMEOUT_VALUE);
> +		pm_runtime_set_autosuspend_delay(core->dev, AUTOSUSPEND_DELAY_VALUE);
> +	}
>  
>  	return 0;
>  }
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_common.h b/drivers/media/platform/qcom/iris/iris_hfi_common.h
> index d8bc0862c54bd229d575113752526d51e863e303..18c852d8b818f41bf5e1b0afd35082eeaf5240b4 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_common.h
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_common.h
> @@ -112,6 +112,7 @@ struct iris_hfi_prop_type_handle {
>  #define IRIS_FW_DEBUG_FATAL	0x00000010
>  #define IRIS_FW_DEBUG_PERF	0x00000020
>  #define IRIS_FW_DEBUG_LOGMASK	0x0fffffff
> +#define IRIS_FW_DEBUG_DEFAULT	(IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL)
>  
>  struct iris_hfi_sys_ops {
>  	int (*sys_init)(struct iris_core *core);
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> index 903b60a7380559f4711b0907fb1de405fe1707f3..6b8f952d0d4b54f37596728f1823c9265c637bbe 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen1_response.c
> @@ -672,15 +672,17 @@ static void iris_hfi_gen1_handle_response(struct iris_core *core, void *response
>  	}
>  }
>  
> -static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
> +static int iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
>  {
>  	struct hfi_msg_sys_debug_pkt *pkt;
>  	struct hfi_pkt_hdr *hdr;
> +	int num_pkts = 0;
>  	u32 log_size;
>  	u8 *log;
>  
>  	while (!iris_hfi_queue_dbg_read(core, packet)) {
>  		hdr = (struct hfi_pkt_hdr *)packet;
> +		num_pkts++;
>  
>  		if (hdr->size <= sizeof(*hdr))
>  			continue;
> @@ -705,15 +707,25 @@ static void iris_hfi_gen1_flush_debug_queue(struct iris_core *core, u8 *packet)
>  				dev_dbg(core->dev, "%.*s", (int)log_size, log);
>  		}
>  	}
> +
> +	return num_pkts;
>  }
>  
>  void iris_hfi_gen1_response_handler(struct iris_core *core)
>  {
> -	memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
> -	while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
> -		iris_hfi_gen1_handle_response(core, core->response_packet);
> +	int num_debug_pkts;
> +	int num_msg_pkts;
> +
> +	do {
> +		num_msg_pkts = 0;
>  		memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
> -	}
> +		while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
> +			iris_hfi_gen1_handle_response(core, core->response_packet);
> +			num_msg_pkts++;
> +			memset(core->response_packet, 0, sizeof(struct hfi_pkt_hdr));
> +		}
>  
> -	iris_hfi_gen1_flush_debug_queue(core, core->response_packet);
> +		num_debug_pkts =
> +			iris_hfi_gen1_flush_debug_queue(core, core->response_packet);
> +	} while (num_msg_pkts || num_debug_pkts);
>  }
> diff --git a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> index e8dc8751ea939bd544699f464a831736fcc10b7f..1bd4ec0c5ada1ef17de4f07fe19d1fac611db04f 100644
> --- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> +++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_response.c
> @@ -988,14 +988,16 @@ static int iris_hfi_gen2_handle_response(struct iris_core *core, void *response)
>  		return iris_hfi_gen2_handle_session_response(core, hdr);
>  }
>  
> -static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
> +static int iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
>  {
>  	struct hfi_debug_header *pkt;
> +	int num_pkts = 0;
>  	u32 log_size;
>  	u8 *log;
>  
>  	while (!iris_hfi_queue_dbg_read(core, packet)) {
>  		pkt = (struct hfi_debug_header *)packet;
> +		num_pkts++;
>  
>  		if (pkt->size <= sizeof(*pkt) + 1)
>  			continue;
> @@ -1010,10 +1012,15 @@ static void iris_hfi_gen2_flush_debug_queue(struct iris_core *core, u8 *packet)
>  		else
>  			dev_dbg(core->dev, "%.*s", (int)log_size, log);
>  	}
> +
> +	return num_pkts;
>  }
>  
>  void iris_hfi_gen2_response_handler(struct iris_core *core)
>  {
> +	int num_debug_pkts;
> +	int num_msg_pkts;
> +
>  	if (iris_vpu_watchdog(core, core->intr_status)) {
>  		struct iris_hfi_packet pkt = {.type = HFI_SYS_ERROR_WD_TIMEOUT};
>  
> @@ -1024,11 +1031,16 @@ void iris_hfi_gen2_response_handler(struct iris_core *core)
>  		return;
>  	}
>  
> -	memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
> -	while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
> -		iris_hfi_gen2_handle_response(core, core->response_packet);
> +	do {
> +		num_msg_pkts = 0;
>  		memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
> -	}
> +		while (!iris_hfi_queue_msg_read(core, core->response_packet)) {
> +			iris_hfi_gen2_handle_response(core, core->response_packet);
> +			num_msg_pkts++;
> +			memset(core->response_packet, 0, sizeof(struct iris_hfi_header));
> +		}
>  
> -	iris_hfi_gen2_flush_debug_queue(core, core->response_packet);
> +		num_debug_pkts =
> +			iris_hfi_gen2_flush_debug_queue(core, core->response_packet);
> +	} while (num_msg_pkts || num_debug_pkts);
>  }
> diff --git a/drivers/media/platform/qcom/iris/iris_probe.c b/drivers/media/platform/qcom/iris/iris_probe.c
> index 8bc94acc4f58ce7792ddaeaad036475c476d5c36..318a548a7ac606594a41d9dae25e73205de1fd0d 100644
> --- a/drivers/media/platform/qcom/iris/iris_probe.c
> +++ b/drivers/media/platform/qcom/iris/iris_probe.c
> @@ -233,7 +233,8 @@ static int iris_probe(struct platform_device *pdev)
>  	if (!core)
>  		return -ENOMEM;
>  	core->dev = dev;
> -	core->fw_debug = IRIS_FW_DEBUG_ERROR | IRIS_FW_DEBUG_FATAL;
> +	core->fw_debug = IRIS_FW_DEBUG_DEFAULT;
> +	core->hw_response_timeout = HW_RESPONSE_TIMEOUT_VALUE;
>  
>  	core->state = IRIS_CORE_DEINIT;
>  	mutex_init(&core->lock);
> diff --git a/drivers/media/platform/qcom/iris/iris_utils.c b/drivers/media/platform/qcom/iris/iris_utils.c
> index ba5c8dc1280c2e147cd33498b69a815cf7c75683..dbddfd770edce22fba58eff15cb6d45eae5ed638 100644
> --- a/drivers/media/platform/qcom/iris/iris_utils.c
> +++ b/drivers/media/platform/qcom/iris/iris_utils.c
> @@ -70,12 +70,14 @@ void iris_helper_buffers_done(struct iris_inst *inst, unsigned int type,
>  int iris_wait_for_session_response(struct iris_inst *inst, bool is_flush)
>  {
>  	struct completion *done;
> +	unsigned int timeout;
>  	int ret;
>  
>  	done = is_flush ? &inst->flush_completion : &inst->completion;
> +	timeout = READ_ONCE(inst->core->hw_response_timeout);
>  
>  	mutex_unlock(&inst->lock);
> -	ret = wait_for_completion_timeout(done, msecs_to_jiffies(HW_RESPONSE_TIMEOUT_VALUE));
> +	ret = wait_for_completion_timeout(done, msecs_to_jiffies(timeout));
>  	mutex_lock(&inst->lock);
>  	if (!ret) {
>  		iris_inst_change_state(inst, IRIS_INST_ERROR);

Reviewed-by: Vishnu Reddy <busanna.reddy@oss.qualcomm.com>