media: atmel: atmel-isc: add CC initialization function

The CC submodule is a part of the atmel-isc pipeline, and stands for
Color Correction. It is used to apply gains and offsets to the
chroma (U, V) components of the YUV elements.
Implement the CC submodule initialization, as a product
specific function, which currently configures the neutral point in color
correction.

[hverkuil: made isc_sama5d2_config_cc static]

Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/media/platform/atmel/atmel-isc-base.c b/drivers/media/platform/atmel/atmel-isc-base.c
index a6b62e009c383125220b5c22c34afde11bda8ba3..ffce8de2cf4d8b3bc0317ae8b8d9c45e84bef445 100644 (file)
--- a/drivers/media/platform/atmel/atmel-isc-base.c
+++ b/drivers/media/platform/atmel/atmel-isc-base.c
@@ -661,6 +661,7 @@ static void isc_set_pipeline(struct isc_device *isc, u32 pipeline)
 
        isc->config_csc(isc);
        isc->config_cbc(isc);
+       isc->config_cc(isc);
 }
 
 static int isc_update_profile(struct isc_device *isc)

diff --git a/drivers/media/platform/atmel/atmel-isc.h b/drivers/media/platform/atmel/atmel-isc.h
index f574bcc3ba67a7930ae7608f0378074e377dc169..a5f8d500138115d4bffacb07f40a7bb8c9746cdb 100644 (file)
--- a/drivers/media/platform/atmel/atmel-isc.h
+++ b/drivers/media/platform/atmel/atmel-isc.h
@@ -225,6 +225,8 @@ struct isc_reg_offsets {
  *                     specific CSC module
  * @config_cbc:                pointer to a function that initializes product
  *                     specific CBC module
+ * @config_cc:         pointer to a function that initializes product
+ *                     specific CC module
  *
  * @offsets:           struct holding the product specific register offsets
  */
@@ -298,6 +300,7 @@ struct isc_device {
        struct {
                void (*config_csc)(struct isc_device *isc);
                void (*config_cbc)(struct isc_device *isc);
+               void (*config_cc)(struct isc_device *isc);
        };
 
        struct isc_reg_offsets          offsets;

diff --git a/drivers/media/platform/atmel/atmel-sama5d2-isc.c b/drivers/media/platform/atmel/atmel-sama5d2-isc.c
index 72cb91b4b8baa583d942ae450c8c8297a75ae751..3d2e6e68015aa13d16bdc207f8a8711e5b8b1caa 100644 (file)
--- a/drivers/media/platform/atmel/atmel-sama5d2-isc.c
+++ b/drivers/media/platform/atmel/atmel-sama5d2-isc.c
@@ -83,6 +83,19 @@ static void isc_sama5d2_config_cbc(struct isc_device *isc)
                     isc->ctrls.contrast);
 }
 
+static void isc_sama5d2_config_cc(struct isc_device *isc)
+{
+       struct regmap *regmap = isc->regmap;
+
+       /* Configure each register at the neutral fixed point 1.0 or 0.0 */
+       regmap_write(regmap, ISC_CC_RR_RG, (1 << 8));
+       regmap_write(regmap, ISC_CC_RB_OR, 0);
+       regmap_write(regmap, ISC_CC_GR_GG, (1 << 8) << 16);
+       regmap_write(regmap, ISC_CC_GB_OG, 0);
+       regmap_write(regmap, ISC_CC_BR_BG, 0);
+       regmap_write(regmap, ISC_CC_BB_OB, (1 << 8));
+}
+
 /* Gamma table with gamma 1/2.2 */
 static const u32 isc_sama5d2_gamma_table[][GAMMA_ENTRIES] = {
        /* 0 --> gamma 1/1.8 */
@@ -230,6 +243,7 @@ static int atmel_isc_probe(struct platform_device *pdev)
 
        isc->config_csc = isc_sama5d2_config_csc;
        isc->config_cbc = isc_sama5d2_config_cbc;
+       isc->config_cc = isc_sama5d2_config_cc;
 
        isc->offsets.csc = ISC_SAMA5D2_CSC_OFFSET;
        isc->offsets.cbc = ISC_SAMA5D2_CBC_OFFSET;