Re: [PATCH] media: qcom: iris: add firmware debug logging support

[Vishnu Reddy]

On 8/15/2026 9:55 PM, 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 firmware 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. Use READ_ONCE() and WRITE_ONCE() for lockless
> debugfs access and HFI packet construction.
>
> Program the firmware debug configuration during core
> initialization and before opening a new session. This lets a
> fw_level value written before session start take effect for that
> session without requiring a driver reload.
>
> Add HFI Gen1 and Gen2 support for sending the firmware debug
> configuration to firmware, and flush firmware debug messages
> from the debug queue. Firmware error and fatal messages are
> emitted through dev_err_ratelimited(), while other firmware debug
> messages use dev_dbg().
>
> 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@xxxxxxxxxxxxxxxx>
> Signed-off-by: Renjiang Han <renjiang.han@xxxxxxxxxxxxxxxx>
> ---
> 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.
> ---
>  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 | 35 ++++++++++++++++++
>  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     |  7 ++++
>  .../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    | 10 ++++--
>  drivers/media/platform/qcom/iris/iris_probe.c      |  6 ++++
>  drivers/media/platform/qcom/iris/iris_vb2.c        |  6 +++-
>  16 files changed, 242 insertions(+), 11 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..d54394770740016852d5f46905fe3b476ce094ad 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) {

Do we need hfi_ops->sys_set_debug NULL check here, It won't be NULL now in any
platform with this patch right?

> +		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);
> 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..c892e28058641a82ade1a0eb52102e133429ef7d 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,13 @@
>  #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..a446c1930fe58480eedb441be6612ba42a083093 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..c845e5e6de50b75237cb829d35beb988f3b3d367 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)
>  			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..e4b874a7119fd05669e49e66efc3c0e3a0feb314 100644
> --- a/drivers/media/platform/qcom/iris/iris_vb2.c
> +++ b/drivers/media/platform/qcom/iris/iris_vb2.c
> @@ -127,7 +127,9 @@ 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_warn(core->dev, "failed to set firmware debug level: %d\n", ret);
>  
>  		ret = inst->hfi_session_ops->session_open(inst);
>  		if (ret) {
> @@ -136,6 +138,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;
>
> ---
> base-commit: 28d012efb4327f9c75d5e042a7c91e9a542efa98
> change-id: 20260815-media-qcom-iris-fw-log-3b62b8645b7a
>
> Best regards,