On Mon Nov 10 12:54:29 2025 +0530, Vishnu Reddy wrote:
> The message for attempting to set already configured input or output
> parameter subscribe change properties is not an error condition.
> Move it from error to debug level and make it more descriptive by
> indicating which specific input or output psc was already set.
> 
> Signed-off-by: Vishnu Reddy <[email protected]>
> Reviewed-by: Vikash Garodia <[email protected]>
> Signed-off-by: Bryan O'Donoghue <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

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 df31f452b69b..5db00f8a5e06 100644
--- a/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
+++ b/drivers/media/platform/qcom/iris/iris_hfi_gen2_command.c
@@ -813,7 +813,7 @@ static int iris_hfi_gen2_subscribe_change_param(struct 
iris_inst *inst, u32 plan
 
        if ((V4L2_TYPE_IS_OUTPUT(plane) && inst_hfi_gen2->ipsc_properties_set) 
||
            (V4L2_TYPE_IS_CAPTURE(plane) && 
inst_hfi_gen2->opsc_properties_set)) {
-               dev_err(core->dev, "invalid plane\n");
+               dev_dbg(core->dev, "%cPSC already set\n", 
V4L2_TYPE_IS_OUTPUT(plane) ? 'I' : 'O');
                return 0;
        }
 
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to