media: rkisp1: csi: Rename CSI functions with a common rkisp1_csi prefix
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 14 Jun 2022 19:10:53 +0000 (20:10 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Sun, 17 Jul 2022 11:10:05 +0000 (12:10 +0100)
The CSI-related functions are not named consistently. Fix it by using a
common rkisp1_csi prefix.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Dafna Hirschfeld <dafna@fastmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
drivers/media/platform/rockchip/rkisp1/rkisp1-csi.c
drivers/media/platform/rockchip/rkisp1/rkisp1-csi.h
drivers/media/platform/rockchip/rkisp1/rkisp1-dev.c
drivers/media/platform/rockchip/rkisp1/rkisp1-isp.c

index 4ba30f172c8bfc463ab5dc56e54f335b332f3b6e..d07c5c8e5b0d641cbd6021a6e4ef961cadd49f83 100644 (file)
@@ -541,7 +541,7 @@ void rkisp1_params_disable(struct rkisp1_params *params);
 
 /* irq handlers */
 irqreturn_t rkisp1_isp_isr(int irq, void *ctx);
-irqreturn_t rkisp1_mipi_isr(int irq, void *ctx);
+irqreturn_t rkisp1_csi_isr(int irq, void *ctx);
 irqreturn_t rkisp1_capture_isr(int irq, void *ctx);
 void rkisp1_stats_isr(struct rkisp1_stats *stats, u32 isp_ris);
 void rkisp1_params_isr(struct rkisp1_device *rkisp1);
index 10126da485ed3306e9a8342d5b4e3a723e3c81f8..102deb877aa4d5d20206860b68b04d8807509fb7 100644 (file)
@@ -18,7 +18,7 @@
 #include "rkisp1-common.h"
 #include "rkisp1-csi.h"
 
-static int rkisp1_config_mipi(struct rkisp1_csi *csi)
+static int rkisp1_csi_config(struct rkisp1_csi *csi)
 {
        struct rkisp1_device *rkisp1 = csi->rkisp1;
        const struct rkisp1_mbus_info *sink_fmt = rkisp1->isp.sink_fmt;
@@ -69,7 +69,7 @@ static int rkisp1_config_mipi(struct rkisp1_csi *csi)
        return 0;
 }
 
-static void rkisp1_mipi_start(struct rkisp1_csi *csi)
+static void rkisp1_csi_enable(struct rkisp1_csi *csi)
 {
        struct rkisp1_device *rkisp1 = csi->rkisp1;
        u32 val;
@@ -79,7 +79,7 @@ static void rkisp1_mipi_start(struct rkisp1_csi *csi)
                     val | RKISP1_CIF_MIPI_CTRL_OUTPUT_ENA);
 }
 
-static void rkisp1_mipi_stop(struct rkisp1_csi *csi)
+static void rkisp1_csi_disable(struct rkisp1_csi *csi)
 {
        struct rkisp1_device *rkisp1 = csi->rkisp1;
        u32 val;
@@ -93,8 +93,8 @@ static void rkisp1_mipi_stop(struct rkisp1_csi *csi)
                     val & (~RKISP1_CIF_MIPI_CTRL_OUTPUT_ENA));
 }
 
-int rkisp1_mipi_csi2_start(struct rkisp1_csi *csi,
-                          struct rkisp1_sensor_async *sensor)
+int rkisp1_csi_start(struct rkisp1_csi *csi,
+                    struct rkisp1_sensor_async *sensor)
 {
        struct rkisp1_device *rkisp1 = csi->rkisp1;
        union phy_configure_opts opts;
@@ -102,7 +102,7 @@ int rkisp1_mipi_csi2_start(struct rkisp1_csi *csi,
        s64 pixel_clock;
        int ret;
 
-       ret = rkisp1_config_mipi(csi);
+       ret = rkisp1_csi_config(csi);
        if (ret)
                return ret;
 
@@ -119,19 +119,19 @@ int rkisp1_mipi_csi2_start(struct rkisp1_csi *csi,
        phy_configure(csi->dphy, &opts);
        phy_power_on(csi->dphy);
 
-       rkisp1_mipi_start(csi);
+       rkisp1_csi_enable(csi);
 
        return 0;
 }
 
-void rkisp1_mipi_csi2_stop(struct rkisp1_csi *csi)
+void rkisp1_csi_stop(struct rkisp1_csi *csi)
 {
-       rkisp1_mipi_stop(csi);
+       rkisp1_csi_disable(csi);
 
        phy_power_off(csi->dphy);
 }
 
-irqreturn_t rkisp1_mipi_isr(int irq, void *ctx)
+irqreturn_t rkisp1_csi_isr(int irq, void *ctx)
 {
        struct device *dev = ctx;
        struct rkisp1_device *rkisp1 = dev_get_drvdata(dev);
index 26d8be2ee1789b5fd16eff3e0151e019c551d7a8..180e2f39606e599c4a2a9573b647c9e0daa67047 100644 (file)
@@ -18,8 +18,8 @@ struct rkisp1_sensor_async;
 int rkisp1_csi_init(struct rkisp1_device *rkisp1);
 void rkisp1_csi_cleanup(struct rkisp1_device *rkisp1);
 
-int rkisp1_mipi_csi2_start(struct rkisp1_csi *csi,
-                          struct rkisp1_sensor_async *sensor);
-void rkisp1_mipi_csi2_stop(struct rkisp1_csi *csi);
+int rkisp1_csi_start(struct rkisp1_csi *csi,
+                    struct rkisp1_sensor_async *sensor);
+void rkisp1_csi_stop(struct rkisp1_csi *csi);
 
 #endif /* _RKISP1_CSI_H */
index 2afaa9f26f29101053861853bc3b372386a0af0c..2c441665c017674bea36690be6927e250ce8d354 100644 (file)
@@ -383,7 +383,7 @@ static irqreturn_t rkisp1_isr(int irq, void *ctx)
         */
        rkisp1_capture_isr(irq, ctx);
        rkisp1_isp_isr(irq, ctx);
-       rkisp1_mipi_isr(irq, ctx);
+       rkisp1_csi_isr(irq, ctx);
 
        return IRQ_HANDLED;
 }
@@ -398,7 +398,7 @@ static const char * const px30_isp_clks[] = {
 static const struct rkisp1_isr_data px30_isp_isrs[] = {
        { "isp", rkisp1_isp_isr },
        { "mi", rkisp1_capture_isr },
-       { "mipi", rkisp1_mipi_isr },
+       { "mipi", rkisp1_csi_isr },
 };
 
 static const struct rkisp1_info px30_isp_info = {
index f477368dcec946a394b333c9b8e903176848395f..d2343f166f42a0e2a04bfb82e119c74404d38936 100644 (file)
@@ -763,7 +763,7 @@ static int rkisp1_isp_s_stream(struct v4l2_subdev *sd, int enable)
                v4l2_subdev_call(rkisp1->active_sensor->sd, video, s_stream,
                                 false);
 
-               rkisp1_mipi_csi2_stop(&rkisp1->csi);
+               rkisp1_csi_stop(&rkisp1->csi);
                rkisp1_isp_stop(rkisp1);
 
                return 0;
@@ -789,7 +789,7 @@ static int rkisp1_isp_s_stream(struct v4l2_subdev *sd, int enable)
 
        rkisp1_isp_start(rkisp1);
 
-       ret = rkisp1_mipi_csi2_start(&rkisp1->csi, rkisp1->active_sensor);
+       ret = rkisp1_csi_start(&rkisp1->csi, rkisp1->active_sensor);
        if (ret) {
                rkisp1_isp_stop(rkisp1);
                goto mutex_unlock;
@@ -799,7 +799,7 @@ static int rkisp1_isp_s_stream(struct v4l2_subdev *sd, int enable)
                               true);
        if (ret) {
                rkisp1_isp_stop(rkisp1);
-               rkisp1_mipi_csi2_stop(&rkisp1->csi);
+               rkisp1_csi_stop(&rkisp1->csi);
                goto mutex_unlock;
        }