Re: [PATCH] media: mali-c55: Disable colour space conversion for RGB outputs

[Jacopo Mondi]

Hi David

On Wed, Oct 07, 2026 at 12:42:27AM +0100, David Carlier wrote:
> The colour space conversion block is only programmed for YUV formats,
> and its configuration is kept across streaming sessions. An RGB stream
> started after a YUV one thus still converts its output to YUV.
>
> Clear the conversion configuration for non-YUV formats.
>
> Fixes: d5f281f3dd29 ("media: mali-c55: Add Mali-C55 ISP driver")
> Cc: stable@xxxxxxxxxxxxxxx
> Assisted-by: LLM
> Signed-off-by: David Carlier <devnexen@xxxxxxxxx>
> ---
>  drivers/media/platform/arm/mali-c55/mali-c55-capture.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-capture.c b/drivers/media/platform/arm/mali-c55/mali-c55-capture.c
> index ff01553026fb..2425a069a939 100644
> --- a/drivers/media/platform/arm/mali-c55/mali-c55-capture.c
> +++ b/drivers/media/platform/arm/mali-c55/mali-c55-capture.c
> @@ -577,6 +577,10 @@ static void mali_c55_cap_dev_format_configure(struct mali_c55_cap_dev *cap_dev)
>  				MALI_C55_REG_CS_CONV_CONFIG,
>  				MALI_C55_CS_CONV_FILTER_MASK,
>  				MALI_C55_CS_CONV_FILTER_ENABLE);
> +	} else {
> +		/* Don't inherit the conversion set up by a previous YUV stream. */
> +		mali_c55_cap_dev_write(cap_dev, MALI_C55_REG_CS_CONV_CONFIG,
> +				       0);

nit: fits on one line. I'll adjust it when applying.

The patch seems correct to me
Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx>

Thanks
   j

>  	}
>  }
>
> --
> 2.55.0
>