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

[Renjiang Han]

On 9/2/2026 12:15 PM, Vishnu Reddy wrote:
> On 9/2/2026 9:32 AM, Renjiang Han wrote:
> > On 9/1/2026 12:13 PM, Vishnu Reddy wrote:
> > > Setting the firmware log level to 0x0000003f may cause hardware response
> > > timeouts due to excessive serial logging. HW_RESPONSE_TIMEOUT_VALUE should
> > > be increased accordingly.
> > Yes, but this patch is for fw log feature, response timeout value is
> > performance issue.
> > I think we'd better use another patch to fix it. What do you think of it?
> It's better to include it in this patch, since this change can directly trigger
> response timeouts when high FW log levels are enabled.
If this level is not enabled, it will not introduce any performance
impact. Even when enabled, any performance overhead caused by the
logging at this level would only occur during debugging.

However, without the current patch, the functionality of the Iris
driver is incomplete. When a firmware issue occurs, there is essentially
no way to perform root cause analysis or identify the problem. Therefore,
this patch is necessary to ensure proper debugging and issue localization
when firmware-related problems are encountered.

--
Best regards,
Renjiang Han