media: atmel: atmel-isc: extend pipeline with extra modules
authorEugen Hristev <eugen.hristev@microchip.com>
Tue, 13 Apr 2021 10:57:16 +0000 (12:57 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 8 Jun 2021 12:14:20 +0000 (14:14 +0200)
Newer ISC pipelines have the additional modules of
Defective Pixel Correction -> DPC itself,
Defective Pixel Correction -> Green Disparity Correction (DPC_GDC)
Defective Pixel Correction -> Black Level Correction (DPC_BLC)
Vertical and Horizontal Scaler -> VHXS

Some products have this full pipeline (sama7g5), other products do not (sama5d2)

Add the modules to the isc base, and also extend the register range to include
the modules.

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>
drivers/media/platform/atmel/atmel-isc-base.c
drivers/media/platform/atmel/atmel-isc.h

index cfe60b2..a6b62e0 100644 (file)
@@ -2324,8 +2324,14 @@ int isc_pipeline_init(struct isc_device *isc)
        struct regmap_field *regs;
        unsigned int i;
 
-       /* WB-->CFA-->CC-->GAM-->CSC-->CBC-->SUB422-->SUB420 */
+       /*
+        * DPCEN-->GDCEN-->BLCEN-->WB-->CFA-->CC-->
+        * GAM-->VHXS-->CSC-->CBC-->SUB422-->SUB420
+        */
        const struct reg_field regfields[ISC_PIPE_LINE_NODE_NUM] = {
+               REG_FIELD(ISC_DPC_CTRL, 0, 0),
+               REG_FIELD(ISC_DPC_CTRL, 1, 1),
+               REG_FIELD(ISC_DPC_CTRL, 2, 2),
                REG_FIELD(ISC_WB_CTRL, 0, 0),
                REG_FIELD(ISC_CFA_CTRL, 0, 0),
                REG_FIELD(ISC_CC_CTRL, 0, 0),
@@ -2333,6 +2339,7 @@ int isc_pipeline_init(struct isc_device *isc)
                REG_FIELD(ISC_GAM_CTRL, 1, 1),
                REG_FIELD(ISC_GAM_CTRL, 2, 2),
                REG_FIELD(ISC_GAM_CTRL, 3, 3),
+               REG_FIELD(ISC_VHXS_CTRL, 0, 0),
                REG_FIELD(ISC_CSC_CTRL + isc->offsets.csc, 0, 0),
                REG_FIELD(ISC_CBC_CTRL + isc->offsets.cbc, 0, 0),
                REG_FIELD(ISC_SUB422_CTRL + isc->offsets.sub422, 0, 0),
@@ -2351,7 +2358,7 @@ int isc_pipeline_init(struct isc_device *isc)
 }
 
 /* regmap configuration */
-#define ATMEL_ISC_REG_MAX    0xbfc
+#define ATMEL_ISC_REG_MAX    0xd5c
 const struct regmap_config isc_regmap_config = {
        .reg_bits       = 32,
        .reg_stride     = 4,
index c5d956f..f574bcc 100644 (file)
@@ -68,17 +68,21 @@ struct isc_format {
 };
 
 /* Pipeline bitmap */
-#define WB_ENABLE      BIT(0)
-#define CFA_ENABLE     BIT(1)
-#define CC_ENABLE      BIT(2)
-#define GAM_ENABLE     BIT(3)
-#define GAM_BENABLE    BIT(4)
-#define GAM_GENABLE    BIT(5)
-#define GAM_RENABLE    BIT(6)
-#define CSC_ENABLE     BIT(7)
-#define CBC_ENABLE     BIT(8)
-#define SUB422_ENABLE  BIT(9)
-#define SUB420_ENABLE  BIT(10)
+#define DPC_DPCENABLE  BIT(0)
+#define DPC_GDCENABLE  BIT(1)
+#define DPC_BLCENABLE  BIT(2)
+#define WB_ENABLE      BIT(3)
+#define CFA_ENABLE     BIT(4)
+#define CC_ENABLE      BIT(5)
+#define GAM_ENABLE     BIT(6)
+#define GAM_BENABLE    BIT(7)
+#define GAM_GENABLE    BIT(8)
+#define GAM_RENABLE    BIT(9)
+#define VHXS_ENABLE    BIT(10)
+#define CSC_ENABLE     BIT(11)
+#define CBC_ENABLE     BIT(12)
+#define SUB422_ENABLE  BIT(13)
+#define SUB420_ENABLE  BIT(14)
 
 #define GAM_ENABLES    (GAM_RENABLE | GAM_GENABLE | GAM_BENABLE | GAM_ENABLE)
 
@@ -142,7 +146,7 @@ struct isc_ctrls {
        u32 hist_minmax[HIST_BAYER][2];
 };
 
-#define ISC_PIPE_LINE_NODE_NUM 11
+#define ISC_PIPE_LINE_NODE_NUM 15
 
 /*
  * struct isc_reg_offsets - ISC device register offsets