On Mon Dec 15 13:11:06 2025 +0100, Jacopo Mondi wrote:
> The Mali C55 driver uses the v4l2-isp framework, which performs
> validation of the parameters buffer versioning in the
> v4l2_isp_params_validate_buffer() function.
> 
> It is not necessary to replicate the validation of the parameters buffer
> versioning in the platform-specific implementation.
> 
> Remove it.
> 
> Signed-off-by: Jacopo Mondi <[email protected]>
> Reviewed-by: Daniel Scally <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/arm/mali-c55/mali-c55-params.c | 7 -------
 1 file changed, 7 deletions(-)

---

diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-params.c 
b/drivers/media/platform/arm/mali-c55/mali-c55-params.c
index 082cda4f4f63..be0e909bcf29 100644
--- a/drivers/media/platform/arm/mali-c55/mali-c55-params.c
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-params.c
@@ -582,13 +582,6 @@ static int mali_c55_params_buf_prepare(struct vb2_buffer 
*vb)
        struct mali_c55 *mali_c55 = params->mali_c55;
        int ret;
 
-       if (config->version != MALI_C55_PARAM_BUFFER_V1) {
-               dev_dbg(mali_c55->dev,
-                       "Unsupported extensible format version: %u\n",
-                       config->version);
-               return -EINVAL;
-       }
-
        ret = v4l2_isp_params_validate_buffer_size(mali_c55->dev, vb,
                        v4l2_isp_params_buffer_size(MALI_C55_PARAMS_MAX_SIZE));
        if (ret)
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to