media: mali-c55: Bypass Purple Fringe Correction

The Purple Fringe Correction module is enabled by default with a POR
configuration which might not be correct for the lens/sensor combination
in use.

If the Purple Fringe Correction module has to be used, it needs to be
supported in the driver's uAPI and properly configured by userspace.

As the driver doesn't support it at the moment, bypass it by default.

Reviewed-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Daniel Scally <dan.scally@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>

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 b9fc904a38445a0bd9ab7ae07e013d69c04f852b..de0e9d898db7b936610fd801ab3b1e384acc44d0 100644 (file)
--- a/drivers/media/platform/arm/mali-c55/mali-c55-params.c
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-params.c
@@ -830,9 +830,10 @@ void mali_c55_params_init_isp_config(struct mali_c55 *mali_c55,
                                 MALI_C55_REG_BYPASS_3_IRIDIX,
                                 MALI_C55_REG_BYPASS_3_IRIDIX);
 
-       /* Bypass the colour noise reduction  */
+       /* Bypass the colour noise reduction and the PF modules  */
        mali_c55_ctx_write(mali_c55, MALI_C55_REG_BYPASS_4,
-                          MALI_C55_REG_BYPASS_4_CNR);
+                          MALI_C55_REG_BYPASS_4_CNR |
+                          MALI_C55_REG_BYPASS_4_PF_CORRECTION);
 
        /* Disable the sinter module */
        mali_c55_ctx_update_bits(mali_c55, MALI_C55_REG_SINTER_CONFIG,