On Thu Feb 12 11:03:09 2026 +0000, Daniel Scally wrote:
> The Mali C55 Iridix block has a digital gain function and tone mapping
> function, whose enablement is controlled by two different bits
> in the BYPASS_3 register.
> 
> Unfortunately, the "Gain" and "Tonemap" bypass bit definitions are the
> wrong way around. Swap them.
> 
> Cc: [email protected]
> Fixes: d5f281f3dd29 ("media: mali-c55: Add Mali-C55 ISP driver")
> Signed-off-by: Daniel Scally <[email protected]>
> Reviewed-by: Barnabás Pőcze <[email protected]>
> Signed-off-by: Jacopo Mondi <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/arm/mali-c55/mali-c55-registers.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-registers.h 
b/drivers/media/platform/arm/mali-c55/mali-c55-registers.h
index f5a148add1c8..f098effde7b4 100644
--- a/drivers/media/platform/arm/mali-c55/mali-c55-registers.h
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-registers.h
@@ -128,8 +128,8 @@ enum mali_c55_interrupts {
 #define MALI_C55_REG_BYPASS_3_SENSOR_OFFSET_PRE_SH     BIT(1)
 #define MALI_C55_REG_BYPASS_3_MESH_SHADING             BIT(3)
 #define MALI_C55_REG_BYPASS_3_WHITE_BALANCE            BIT(4)
-#define MALI_C55_REG_BYPASS_3_IRIDIX                   BIT(5)
-#define MALI_C55_REG_BYPASS_3_IRIDIX_GAIN              BIT(6)
+#define MALI_C55_REG_BYPASS_3_IRIDIX_GAIN              BIT(5)
+#define MALI_C55_REG_BYPASS_3_IRIDIX                   BIT(6)
 #define MALI_C55_REG_BYPASS_4                          0x18ec0
 #define MALI_C55_REG_BYPASS_4_DEMOSAIC_RGB             BIT(1)
 #define MALI_C55_REG_BYPASS_4_PF_CORRECTION            BIT(3)
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to