media: imx: imx7_mipi_csis: Reorganize csi_state structure
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 13 Apr 2021 02:30:10 +0000 (04:30 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sun, 23 May 2021 17:21:32 +0000 (19:21 +0200)
Group the fiels of the csi_state structure logically to improve
readability.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Rui Miguel Silva <rmfrfs@gmail.com>
Tested-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/imx/imx7-mipi-csis.c

index 363aa28fae5711cb860747101b1583545189b7e6..c302b095ce74e24c02080b1ff8507e230c96a7ad 100644 (file)
@@ -292,40 +292,33 @@ static const char * const mipi_csis_clk_id[] = {
 };
 
 struct csi_state {
-       /* lock elements below */
-       struct mutex lock;
-       /* lock for event handler */
-       spinlock_t slock;
        struct device *dev;
-       struct media_pad pads[CSIS_PADS_NUM];
-       struct v4l2_subdev sd;
-       struct v4l2_async_notifier notifier;
-       struct v4l2_subdev *src_sd;
-
-       u8 index;
        void __iomem *regs;
-       u32 state;
-
-       struct dentry *debugfs_root;
-       bool debug;
-
        unsigned int num_clks;
        struct clk_bulk_data *clks;
+       struct reset_control *mrst;
+       struct regulator *mipi_phy_regulator;
+       u8 index;
 
+       struct v4l2_subdev sd;
+       struct media_pad pads[CSIS_PADS_NUM];
+       struct v4l2_async_notifier notifier;
+       struct v4l2_subdev *src_sd;
+
+       struct v4l2_fwnode_bus_mipi_csi2 bus;
        u32 clk_frequency;
        u32 hs_settle;
        u32 clk_settle;
 
-       struct reset_control *mrst;
-
+       struct mutex lock;      /* Protect csis_fmt, format_mbus and state */
        const struct csis_pix_format *csis_fmt;
        struct v4l2_mbus_framefmt format_mbus;
+       u32 state;
 
-       struct v4l2_fwnode_bus_mipi_csi2 bus;
-
+       spinlock_t slock;       /* Protect events */
        struct mipi_csis_event events[MIPI_CSIS_NUM_EVENTS];
-
-       struct regulator *mipi_phy_regulator;
+       struct dentry *debugfs_root;
+       bool debug;
 };
 
 /* -----------------------------------------------------------------------------