[PATCH 36/49] staging: hikey9xx/gpu: solve most coding style issues

Mauro Carvalho Chehab mchehab+huawei at kernel.org
Wed Aug 19 11:46:04 UTC 2020


There are lots of coding style issues on this driver, as
reported by checkpatch.

Address most of them.

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei at kernel.org>
---
 .../staging/hikey9xx/gpu/kirin960_dpe_reg.h   |  340 +++--
 .../staging/hikey9xx/gpu/kirin970_dpe_reg.h   | 1199 ++++++++---------
 .../hikey9xx/gpu/kirin9xx_drm_dpe_utils.c     |  285 ++--
 .../hikey9xx/gpu/kirin9xx_drm_dpe_utils.h     |    4 +-
 .../staging/hikey9xx/gpu/kirin9xx_drm_drv.c   |    2 +-
 .../staging/hikey9xx/gpu/kirin9xx_drm_drv.h   |    4 +-
 .../staging/hikey9xx/gpu/kirin9xx_drm_dss.c   |   62 +-
 .../hikey9xx/gpu/kirin9xx_drm_overlay_utils.c |  116 +-
 .../hikey9xx/gpu/kirin9xx_dw_drm_dsi.c        |  226 ++--
 .../staging/hikey9xx/gpu/kirin9xx_fb_panel.h  |   18 +-
 drivers/staging/hikey9xx/gpu/kirin9xx_pwm.c   |   26 +-
 11 files changed, 1140 insertions(+), 1142 deletions(-)

diff --git a/drivers/staging/hikey9xx/gpu/kirin960_dpe_reg.h b/drivers/staging/hikey9xx/gpu/kirin960_dpe_reg.h
index 131bb80d9bd1..651b3b172033 100644
--- a/drivers/staging/hikey9xx/gpu/kirin960_dpe_reg.h
+++ b/drivers/staging/hikey9xx/gpu/kirin960_dpe_reg.h
@@ -28,11 +28,11 @@
 #include <drm/drm_plane.h>
 #include <drm/drm_crtc.h>
 
-#define FB_ACCEL_HI62xx 	0x1
-#define FB_ACCEL_HI363x 	0x2
-#define FB_ACCEL_HI365x 	0x4
-#define FB_ACCEL_HI625x 	0x8
-#define FB_ACCEL_HI366x 	0x10
+#define FB_ACCEL_HI62xx	0x1
+#define FB_ACCEL_HI363x	0x2
+#define FB_ACCEL_HI365x	0x4
+#define FB_ACCEL_HI625x	0x8
+#define FB_ACCEL_HI366x	0x10
 #define FB_ACCEL_KIRIN970_ES  0x20
 #define FB_ACCEL_KIRIN970  0x40
 #define FB_ACCEL_KIRIN660  0x80
@@ -40,9 +40,9 @@
 #define FB_ACCEL_KIRIN980  0x200
 #define FB_ACCEL_PLATFORM_TYPE_FPGA     0x10000000   //FPGA
 #define FB_ACCEL_PLATFORM_TYPE_ASIC     0x20000000   //ASIC
-/*******************************************************************************
-**
-*/
+
+/******************************************************************************/
+
 enum dss_chn_idx {
 	DSS_RCHN_NONE = -1,
 	DSS_RCHN_D2 = 0,
@@ -104,32 +104,32 @@ enum dss_ovl_idx {
 #define DSS_WCH_MAX  (2)
 
 typedef struct dss_img {
-	uint32_t format;
-	uint32_t width;
-	uint32_t height;
-	uint32_t bpp;		/* bytes per pixel */
-	uint32_t buf_size;
-	uint32_t stride;
-	uint32_t stride_plane1;
-	uint32_t stride_plane2;
-	uint64_t phy_addr;
-	uint64_t vir_addr;
-	uint32_t offset_plane1;
-	uint32_t offset_plane2;
+	u32 format;
+	u32 width;
+	u32 height;
+	u32 bpp;		/* bytes per pixel */
+	u32 buf_size;
+	u32 stride;
+	u32 stride_plane1;
+	u32 stride_plane2;
+	u64 phy_addr;
+	u64 vir_addr;
+	u32 offset_plane1;
+	u32 offset_plane2;
 
-	uint64_t afbc_header_addr;
-	uint64_t afbc_payload_addr;
-	uint32_t afbc_header_stride;
-	uint32_t afbc_payload_stride;
-	uint32_t afbc_scramble_mode;
-	uint32_t mmbuf_base;
-	uint32_t mmbuf_size;
+	u64 afbc_header_addr;
+	u64 afbc_payload_addr;
+	u32 afbc_header_stride;
+	u32 afbc_payload_stride;
+	u32 afbc_scramble_mode;
+	u32 mmbuf_base;
+	u32 mmbuf_size;
 
-	uint32_t mmu_enable;
-	uint32_t csc_mode;
-	uint32_t secure_mode;
-	int32_t shared_fd;
-	uint32_t reserved0;
+	u32 mmu_enable;
+	u32 csc_mode;
+	u32 secure_mode;
+	s32 shared_fd;
+	u32 reserved0;
 } dss_img_t;
 
 typedef struct drm_dss_layer {
@@ -137,20 +137,18 @@ typedef struct drm_dss_layer {
 	dss_rect_t src_rect;
 	dss_rect_t src_rect_mask;
 	dss_rect_t dst_rect;
-	uint32_t transform;
-	int32_t blending;
-	uint32_t glb_alpha;
-	uint32_t color;		/* background color or dim color */
-	int32_t layer_idx;
-	int32_t chn_idx;
-	uint32_t need_cap;
-	int32_t acquire_fence;
+	u32 transform;
+	s32 blending;
+	u32 glb_alpha;
+	u32 color;		/* background color or dim color */
+	s32 layer_idx;
+	s32 chn_idx;
+	u32 need_cap;
+	s32 acquire_fence;
 } drm_dss_layer_t;
 
+/******************************************************************************/
 
-/*******************************************************************************
-**
-*/
 #define DEFAULT_MIPI_CLK_RATE	(192 * 100000L)
 #define DEFAULT_PCLK_DSI_RATE	(120 * 1000000L)
 
@@ -178,9 +176,8 @@ typedef struct drm_dss_layer {
 #define GPIO_PG_SEL_B (76)
 #define GPIO_TX_RX_B (78)
 
-/*******************************************************************************
- **
- */
+/******************************************************************************/
+
 #define CRGPERI_PLL0_CLK_RATE	(1600000000UL)
 #define CRGPERI_PLL2_CLK_RATE	(960000000UL)
 #define CRGPERI_PLL3_CLK_RATE	(1600000000UL)
@@ -195,10 +192,10 @@ typedef struct drm_dss_layer {
 #define DSS_MAX_PXL0_CLK_288M (288000000UL)
 
 /*dss clk power off */
-#define DEFAULT_DSS_CORE_CLK_RATE_POWER_OFF   	(277000000UL)
-#define DEFAULT_DSS_PXL0_CLK_RATE_POWER_OFF    	(277000000UL)
-#define DEFAULT_DSS_MMBUF_CLK_RATE_POWER_OFF 	(238000000UL)
-#define DEFAULT_DSS_PXL1_CLK_RATE_POWER_OFF 	(238000000UL)
+#define DEFAULT_DSS_CORE_CLK_RATE_POWER_OFF	(277000000UL)
+#define DEFAULT_DSS_PXL0_CLK_RATE_POWER_OFF	(277000000UL)
+#define DEFAULT_DSS_MMBUF_CLK_RATE_POWER_OFF	(238000000UL)
+#define DEFAULT_DSS_PXL1_CLK_RATE_POWER_OFF	(238000000UL)
 
 #define MMBUF_SIZE_MAX	(288 * 1024)
 #define HISI_DSS_CMDLIST_MAX	(16)
@@ -234,7 +231,7 @@ typedef struct drm_dss_layer {
 #define DEFAULT_ACLK_DPCTRL_RATE_CS	207000000UL
 #define DEFAULT_MIDIA_PPLL7_CLOCK_FREQ	1782000000UL
 
-#define KIRIN970_VCO_MIN_FREQ_OUPUT         1000000 /*Boston: 1000 * 1000*/
+#define KIRIN970_VCO_MIN_FREQ_OUTPUT         1000000 /*Boston: 1000 * 1000*/
 #define KIRIN970_SYS_19M2   19200 /*Boston: 19.2f * 1000 */
 
 #define MIDIA_PPLL7_CTRL0	0x50c
@@ -249,7 +246,7 @@ typedef struct drm_dss_layer {
 
 /*
  * DSS Registers
-*/
+ */
 
 /* MACROS */
 #define DSS_WIDTH(width)	((width) - 1)
@@ -470,9 +467,7 @@ enum dss_rdma_idx {
 	DSS_RDMA_MAX,
 };
 
-/*******************************************************************************
- **
- */
+/*****************************************************************************/
 
 #define PEREN0	(0x000)
 #define PERDIS0	(0x004)
@@ -539,9 +534,8 @@ enum dss_rdma_idx {
 #define PCTRL_DPHYTX_CTRL1	BIT(1)
 #define PCTRL_DPHYTX_CTRL0	BIT(0)
 
-/*******************************************************************************
- **
- */
+/*****************************************************************************/
+
 #define BIT_DSS_GLB_INTS	BIT(30)
 #define BIT_MMU_IRPT_S	BIT(29)
 #define BIT_MMU_IRPT_NS	BIT(28)
@@ -706,8 +700,8 @@ enum dss_rdma_idx {
 #define BIT_CE_HIST1_RW_COLLIDE_IND	BIT(1)
 #define BIT_CE_HIST0_RW_COLLIDE_IND	BIT(0)
 
-/*******************************************************************************
- ** MODULE BASE ADDRESS
+/*
+ * MODULE BASE ADDRESS
  */
 
 #define DSS_MIPI_DSI0_OFFSET	(0x00001000)
@@ -884,8 +878,8 @@ enum dss_rdma_idx {
 #define DSS_DSC_OFFSET	(0x7DC00)
 #define DSS_LDI1_OFFSET	(0x7E000)
 
-/*******************************************************************************
- ** GLB
+/*
+ * GLB
  */
 #define GLB_DSS_TAG	 (DSS_GLB0_OFFSET + 0x0000)
 
@@ -947,8 +941,8 @@ enum dss_rdma_idx {
 #define GLB_DSS_MEM_CTRL	(DSS_GLB0_OFFSET + 0x0600)
 #define GLB_DSS_PM_CTRL	(DSS_GLB0_OFFSET + 0x0604)
 
-/*******************************************************************************
- ** DBG
+/*
+ * DBG
  */
 #define DBG_CRC_DBG_OV0	(0x0000)
 #define DBG_CRC_DBG_OV1	(0x0004)
@@ -991,8 +985,8 @@ enum dss_rdma_idx {
 #define DBG_RCH8_INTS	(0x02A4)
 #define DBG_RCH8_INT_MSK	(0x02A8)
 
-/*******************************************************************************
- ** CMDLIST
+/*
+ * CMDLIST
  */
 
 #define CMDLIST_CH0_PENDING_CLR	(0x0000)
@@ -1054,8 +1048,8 @@ enum dss_rdma_idx {
 #define BIT_CMDLIST_CH1_INTS	BIT(1)
 #define BIT_CMDLIST_CH0_INTS	BIT(0)
 
-/*******************************************************************************
- ** AIF
+/*
+ * AIF
  */
 #define AIF0_CH0_OFFSET	(DSS_VBIF0_AIF + 0x00)
 #define AIF0_CH0_ADD_OFFSET	(DSS_VBIF0_AIF + 0x04)
@@ -1178,8 +1172,8 @@ typedef struct dss_aif_bw {
 	u8 is_used;
 } dss_aif_bw_t;
 
-/*******************************************************************************
- ** MIF
+/*
+ * MIF
  */
 #define MIF_ENABLE	(0x0000)
 #define MIF_MEM_CTRL	(0x0004)
@@ -1227,8 +1221,8 @@ typedef struct dss_mif {
 } dss_mif_t;
 
 /*
- ** stretch blt, linear/tile, rotation, pixel format
- ** 0 0 000
+ * stretch blt, linear/tile, rotation, pixel format
+ * 0 0 000
  */
 enum dss_mmu_tlb_tag_org {
 	MMU_TLB_TAG_ORG_0x0 = 0x0,
@@ -1260,8 +1254,8 @@ enum dss_mmu_tlb_tag_org {
 	MMU_TLB_TAG_ORG_0x1F = 0x1F,
 };
 
-/*******************************************************************************
- **SMMU
+/*
+ * SMMU
  */
 #define SMMU_SCR	(0x0000)
 #define SMMU_MEMCTRL	(0x0004)
@@ -1383,8 +1377,8 @@ typedef struct dss_smmu {
 	u8 smmu_smrx_ns_used[DSS_CHN_MAX_DEFINE];
 } dss_smmu_t;
 
-/*******************************************************************************
- ** RDMA
+/*
+ * RDMA
  */
 
 #define DMA_OFT_X0	(0x0000)
@@ -1447,8 +1441,8 @@ typedef struct dss_smmu {
 #define CH_CLK_SEL	(0x00E0)
 #define CH_CLK_EN	(0x00E4)
 
-/*******************************************************************************
- ** DFC
+/*
+ * DFC
  */
 #define DFC_DISP_SIZE	(0x0000)
 #define DFC_PIX_IN_NUM	(0x0004)
@@ -1473,8 +1467,8 @@ typedef struct dss_dfc {
 	u32 padding_ctl;
 } dss_dfc_t;
 
-/*******************************************************************************
- ** SCF
+/*
+ * SCF
  */
 #define DSS_SCF_H0_Y_COEF_OFFSET	(0x0000)
 #define DSS_SCF_Y_COEF_OFFSET	(0x2000)
@@ -1506,7 +1500,7 @@ typedef struct dss_dfc {
 #define SCF_CLK_SEL	(0x00F8)
 #define SCF_CLK_EN	(0x00FC)
 #define WCH_SCF_COEF_MEM_CTRL (0x0218)
-#define WCH_SCF_LB_MEM_CTRL 	(0x290)
+#define WCH_SCF_LB_MEM_CTRL	(0x290)
 
 /* MACROS */
 #define SCF_MIN_INPUT	(16)
@@ -1514,7 +1508,7 @@ typedef struct dss_dfc {
 
 /* Threshold for SCF Stretch and SCF filter */
 #define RDMA_STRETCH_THRESHOLD	(2)
-#define SCF_INC_FACTOR	(1 << 18)
+#define SCF_INC_FACTOR	BIT(18)
 #define SCF_UPSCALE_MAX	(60)
 #define SCF_DOWNSCALE_MAX	  (60)
 #define SCF_EDGE_FACTOR (3)
@@ -1630,8 +1624,8 @@ typedef struct dss_arsr2p {
 	u32 ivbottom1;
 } dss_arsr2p_t;
 
-/*******************************************************************************
- ** POST_CLIP  v g
+/*
+ * POST_CLIP  v g
  */
 #define POST_CLIP_DISP_SIZE	(0x0000)
 #define POST_CLIP_CTL_HRZ	(0x0010)
@@ -1645,8 +1639,8 @@ typedef struct dss_post_clip {
 	u32 ctl_clip_en;
 } dss_post_clip_t;
 
-/*******************************************************************************
- ** PCSC v
+/*
+ * PCSC v
  */
 #define PCSC_IDC0	(0x0000)
 #define PCSC_IDC2	(0x0004)
@@ -1664,9 +1658,10 @@ typedef struct dss_pcsc {
 	u32 pcsc_idc0;
 } dss_pcsc_t;
 
-/*******************************************************************************
- ** CSC
+/*
+ * CSC
  */
+
 #define CSC_IDC0	(0x0000)
 #define CSC_IDC2	(0x0004)
 #define CSC_ODC0	(0x0008)
@@ -1693,19 +1688,19 @@ typedef struct dss_csc {
 	u32 mprec;
 } dss_csc_t;
 
-/*******************************************************************************
- **channel DEBUG
+/*
+ * channel DEBUG
  */
 #define CH_DEBUG_SEL (0x600)
 
-/*******************************************************************************
- ** VPP
+/*
+ * VPP
  */
 #define VPP_CTRL (0x700)
 #define VPP_MEM_CTRL (0x704)
 
-/*******************************************************************************
- **DMA BUF
+/*
+ * DMA BUF
  */
 #define DMA_BUF_CTRL	(0x800)
 #define DMA_BUF_SIZE  (0x850)
@@ -1733,15 +1728,15 @@ typedef struct dss_csc {
 #define AFBCD_MONITOR_REG2_OFFSET	(0x94C)
 #define AFBCD_MONITOR_REG3_OFFSET	(0x950)
 #define AFBCD_DEBUG_REG0_OFFSET	(0x954)
-#define AFBCD_CREG_FBCD_CTRL_MODE  	(0x960)
-#define AFBCD_HREG_HDR_PTR_L1 		(0x964)
-#define AFBCD_HREG_PLD_PTR_L1 		(0x968)
-#define AFBCD_HEADER_SRTIDE_1 		(0x96C)
-#define AFBCD_PAYLOAD_SRTIDE_1	 	(0x970)
-#define AFBCD_HREG_HDR_PTR_L1 		(0x964)
-#define AFBCD_HREG_PLD_PTR_L1 		(0x968)
-#define AFBCD_HEADER_SRTIDE_1 		(0x96C)
-#define AFBCD_PAYLOAD_SRTIDE_1 		(0x970)
+#define AFBCD_CREG_FBCD_CTRL_MODE	(0x960)
+#define AFBCD_HREG_HDR_PTR_L1		(0x964)
+#define AFBCD_HREG_PLD_PTR_L1		(0x968)
+#define AFBCD_HEADER_SRTIDE_1		(0x96C)
+#define AFBCD_PAYLOAD_SRTIDE_1		(0x970)
+#define AFBCD_HREG_HDR_PTR_L1		(0x964)
+#define AFBCD_HREG_PLD_PTR_L1		(0x968)
+#define AFBCD_HEADER_SRTIDE_1		(0x96C)
+#define AFBCD_PAYLOAD_SRTIDE_1		(0x970)
 #define AFBCD_BLOCK_TYPE				(0x974)
 #define AFBCD_MM_BASE_1				(0x978)
 #define AFBCD_MM_BASE_2				(0x97C)
@@ -1763,12 +1758,12 @@ typedef struct dss_csc {
 #define AFBCE_THRESHOLD	(0x92C)
 #define AFBCE_SCRAMBLE_MODE	(0x930)
 #define AFBCE_HEADER_POINTER_OFFSET	(0x934)
-#define AFBCE_CREG_FBCE_CTRL_MODE  	(0x950)
-#define AFBCE_HREG_HDR_PTR_L1 		(0x954)
-#define AFBCE_HREG_PLD_PTR_L1 		(0x958)
-#define AFBCE_HEADER_SRTIDE_1 			(0x95C)
-#define AFBCE_PAYLOAD_SRTIDE_1 		(0x960)
-#define AFBCE_MEM_CTRL_1 				(0x968)
+#define AFBCE_CREG_FBCE_CTRL_MODE	(0x950)
+#define AFBCE_HREG_HDR_PTR_L1		(0x954)
+#define AFBCE_HREG_PLD_PTR_L1		(0x958)
+#define AFBCE_HEADER_SRTIDE_1			(0x95C)
+#define AFBCE_PAYLOAD_SRTIDE_1		(0x960)
+#define AFBCE_MEM_CTRL_1				(0x968)
 #define FBCD_CREG_FBCD_CTRL_GATE		(0x98C)
 
 #define ROT_FIRST_LNS	(0x530)
@@ -1954,8 +1949,8 @@ typedef struct dss_wdma {
 	u8 rot_used;
 } dss_wdma_t;
 
-/*******************************************************************************
- ** MCTL  MUTEX0 1 2 3 4 5
+/*
+ * MCTL  MUTEX0 1 2 3 4 5
  */
 #define MCTL_CTL_EN	(0x0000)
 #define MCTL_CTL_MUTEX	(0x0004)
@@ -1990,8 +1985,8 @@ typedef struct dss_wdma {
 #define MCTL_CTL_CLK_EN	(0x0084)
 #define MCTL_CTL_DBG	(0x00E0)
 
-/*******************************************************************************
- ** MCTL  SYS
+/*
+ * MCTL  SYS
  */
 #define MCTL_CTL_SECU_CFG	(0x0000)
 #define MCTL_PAY_SECU_FLUSH_EN  (0x0018)
@@ -2157,8 +2152,8 @@ typedef struct dss_mctl_sys {
 	u8 wch_ov_sel_used[DSS_WCH_MAX];
 } dss_mctl_sys_t;
 
-/*******************************************************************************
- ** OVL
+/*
+ * OVL
  */
 #define OVL_SIZE	(0x0000)
 #define OVL_BG_COLOR	(0x4)
@@ -2320,45 +2315,45 @@ typedef struct dss_mctl_sys {
 #define OVL_6LAYER_NUM		(6)
 #define OVL_2LAYER_NUM		(2)
 
-/*******************************************************************************
-** OVL
-*/
+/*
+ * OVL
+ */
 #define OV_SIZE						(0x000)
-#define OV_BG_COLOR_RGB 			(0x004)
-#define OV_BG_COLOR_A 				(0x008)
-#define OV_DST_STARTPOS 			(0x00C)
-#define OV_DST_ENDPOS 				(0x010)
-#define OV_GCFG 					(0x014)
-#define OV_LAYER0_POS 				(0x030)
-#define OV_LAYER0_SIZE 				(0x034)
+#define OV_BG_COLOR_RGB			(0x004)
+#define OV_BG_COLOR_A				(0x008)
+#define OV_DST_STARTPOS			(0x00C)
+#define OV_DST_ENDPOS				(0x010)
+#define OV_GCFG					(0x014)
+#define OV_LAYER0_POS				(0x030)
+#define OV_LAYER0_SIZE				(0x034)
 #define OV_LAYER0_SRCLOKEY		(0x038)
-#define OV_LAYER0_SRCHIKEY 		(0x03C)
-#define OV_LAYER0_DSTLOKEY 		(0x040)
-#define OV_LAYER0_DSTHIKEY 		(0x044)
-#define OV_LAYER0_PATTERN_RGB 	(0x048)
-#define OV_LAYER0_PATTERN_A 		(0x04C)
-#define OV_LAYER0_ALPHA_MODE 		(0x050)
-#define OV_LAYER0_ALPHA_A 			(0x054)
-#define OV_LAYER0_CFG 				(0x058)
-#define OV_LAYER0_PSPOS 			(0x05C)
-#define OV_LAYER0_PEPOS 			(0x060)
-#define OV_LAYER0_INFO_ALPHA 		(0x064)
-#define OV_LAYER0_INFO_SRCCOLOR 	(0x068)
-#define OV_LAYER0_DBG_INFO 		(0x06C)
+#define OV_LAYER0_SRCHIKEY		(0x03C)
+#define OV_LAYER0_DSTLOKEY		(0x040)
+#define OV_LAYER0_DSTHIKEY		(0x044)
+#define OV_LAYER0_PATTERN_RGB	(0x048)
+#define OV_LAYER0_PATTERN_A		(0x04C)
+#define OV_LAYER0_ALPHA_MODE		(0x050)
+#define OV_LAYER0_ALPHA_A			(0x054)
+#define OV_LAYER0_CFG				(0x058)
+#define OV_LAYER0_PSPOS			(0x05C)
+#define OV_LAYER0_PEPOS			(0x060)
+#define OV_LAYER0_INFO_ALPHA		(0x064)
+#define OV_LAYER0_INFO_SRCCOLOR	(0x068)
+#define OV_LAYER0_DBG_INFO		(0x06C)
 #define OV8_BASE_DBG_INFO			(0x340)
 #define OV8_RD_SHADOW_SEL			(0x344)
 #define OV8_CLK_SEL					(0x348)
-#define OV8_CLK_EN 					(0x34C)
-#define OV8_BLOCK_SIZE 				(0x350)
-#define OV8_BLOCK_DBG 				(0x354)
-#define OV8_REG_DEFAULT 			(0x358)
+#define OV8_CLK_EN					(0x34C)
+#define OV8_BLOCK_SIZE				(0x350)
+#define OV8_BLOCK_DBG				(0x354)
+#define OV8_REG_DEFAULT			(0x358)
 #define OV2_BASE_DBG_INFO			(0x200)
 #define OV2_RD_SHADOW_SEL			(0x204)
 #define OV2_CLK_SEL					(0x208)
-#define OV2_CLK_EN 					(0x20C)
-#define OV2_BLOCK_SIZE 				(0x210)
-#define OV2_BLOCK_DBG 				(0x214)
-#define OV2_REG_DEFAULT 			(0x218)
+#define OV2_CLK_EN					(0x20C)
+#define OV2_BLOCK_SIZE				(0x210)
+#define OV2_BLOCK_DBG				(0x214)
+#define OV2_REG_DEFAULT			(0x218)
 
 #define OV_8LAYER_NUM				(8)
 typedef struct dss_ovl_layer {
@@ -2405,8 +2400,8 @@ typedef struct dss_ovl_alpha {
 	u32 fix_mode;
 } dss_ovl_alpha_t;
 
-/*******************************************************************************
- ** DBUF
+/*
+ * DBUF
  */
 #define DBUF_FRM_SIZE	(0x0000)
 #define DBUF_FRM_HSIZE	(0x0004)
@@ -2440,8 +2435,8 @@ typedef struct dss_ovl_alpha {
 #define DBUF_DFS_RAM_MANAGE  (0x00A8)
 #define DBUF_DFS_DATA_FILL_OUT  (0x00AC)
 
-/*******************************************************************************
- ** DPP
+/*
+ * DPP
  */
 #define DPP_RD_SHADOW_SEL	(0x000)
 #define DPP_DEFAULT	(0x004)
@@ -2576,11 +2571,11 @@ typedef struct dss_ovl_alpha {
 #define ACE_Y_EXT	(0x03C)
 #define ACE_U_EXT	(0x040)
 #define ACE_V_EXT	(0x044)
-#define ACE_Y_ATTENU 	(0x048)
+#define ACE_Y_ATTENU	(0x048)
 #define ACE_U_ATTENU	(0x04C)
 #define ACE_V_ATTENU	(0x050)
 #define ACE_ROTA	(0x054)
-#define ACE_ROTB 	(0x058)
+#define ACE_ROTB	(0x058)
 #define ACE_Y_CORE	(0x05C)
 #define ACE_U_CORE	(0x060)
 #define ACE_V_CORE	(0x064)
@@ -2683,8 +2678,8 @@ typedef struct dss_arsr1p {
 #define ARSR1P_LSC_CFG3		(0x080)
 #define ARSR1P_FORCE_CLK_ON_CFG		(0x084)
 
-/*******************************************************************************
- ** BIT EXT
+/*
+ * BIT EXT
  */
 #define BIT_EXT0_CTL (0x000)
 
@@ -2768,8 +2763,8 @@ typedef struct dss_arsr1p {
 #define HIACE_GAMMA_RAM_B_CFG_PM_CTRL (0x0144)
 #define HIACE_LHIST_RAM_CFG_PM_CTRL (0x0148)
 
-/*******************************************************************************
- ** IFBC
+/*
+ * IFBC
  */
 #define IFBC_SIZE	(0x0000)
 #define IFBC_CTRL	(0x0004)
@@ -2793,8 +2788,8 @@ typedef struct dss_arsr1p {
 #define IFBC_PAD	(0x004C)
 #define IFBC_REG_DEFAULT	(0x0050)
 
-/*******************************************************************************
- ** DSC
+/*
+ * DSC
  */
 #define DSC_VERSION	(0x0000)
 #define DSC_PPS_IDENTIFIER	(0x0004)
@@ -2844,8 +2839,8 @@ typedef struct dss_arsr1p {
 #define DSC_RD_SHADOW_SEL	(0x00B4)
 #define DSC_REG_DEFAULT	(0x00B8)
 
-/*******************************************************************************
- ** LDI
+/*
+ * LDI
  */
 #define LDI_DPI0_HRZ_CTRL0	(0x0000)
 #define LDI_DPI0_HRZ_CTRL1	(0x0004)
@@ -2902,8 +2897,8 @@ typedef struct dss_arsr1p {
 #define LDI_MODULE_CLK_SEL (0x0258)
 #define LDI_MODULE_CLK_EN (0x025C)
 
-/*******************************************************************************
- ** MIPI DSI
+/*
+ * MIPI DSI
  */
 #define MIPIDSI_VERSION_OFFSET	(0x0000)
 #define MIPIDSI_PWR_UP_OFFSET	(0x0004)
@@ -2986,8 +2981,8 @@ typedef struct dss_arsr1p {
 #define VID_VACTIVE_LINES_ACT	(0x0160)
 #define SDF_3D_ACT	(0x0190)
 
-/*******************************************************************************
- ** MMBUF
+/*
+ * MMBUF
  */
 #define SMC_LOCK	(0x0000)
 #define SMC_MEM_LP	(0x0004)
@@ -3094,12 +3089,12 @@ struct dss_hw_ctx {
 };
 
 typedef struct dss_clk_rate {
-	uint64_t dss_pri_clk_rate;
-	uint64_t dss_pclk_dss_rate;
-	uint64_t dss_pclk_pctrl_rate;
-	uint64_t dss_mmbuf_rate;
-	uint32_t dss_voltage_value; //0:0.7v, 2:0.8v
-	uint32_t reserved;
+	u64 dss_pri_clk_rate;
+	u64 dss_pclk_dss_rate;
+	u64 dss_pclk_pctrl_rate;
+	u64 dss_mmbuf_rate;
+	u32 dss_voltage_value; //0:0.7v, 2:0.8v
+	u32 reserved;
 } dss_clk_rate_t;
 
 struct dss_crtc {
@@ -3221,9 +3216,8 @@ typedef struct mipi_ifbc_division {
 	u32 pxl0_dsi_gt_en;
 } mipi_ifbc_division_t;
 
-/*******************************************************************************
-**
-*/
+/*****************************************************************************/
+
 #define outp32(addr, val) writel(val, addr)
 #define outp16(addr, val) writew(val, addr)
 #define outp8(addr, val) writeb(val, addr)
diff --git a/drivers/staging/hikey9xx/gpu/kirin970_dpe_reg.h b/drivers/staging/hikey9xx/gpu/kirin970_dpe_reg.h
index 00bbad95ee3d..9c5009389f00 100644
--- a/drivers/staging/hikey9xx/gpu/kirin970_dpe_reg.h
+++ b/drivers/staging/hikey9xx/gpu/kirin970_dpe_reg.h
@@ -28,11 +28,11 @@
 #include <drm/drm_plane.h>
 #include <drm/drm_crtc.h>
 
-#define FB_ACCEL_HI62xx 	0x1
-#define FB_ACCEL_HI363x 	0x2
-#define FB_ACCEL_HI365x 	0x4
-#define FB_ACCEL_HI625x 	0x8
-#define FB_ACCEL_HI366x 	0x10
+#define FB_ACCEL_HI62xx	0x1
+#define FB_ACCEL_HI363x	0x2
+#define FB_ACCEL_HI365x	0x4
+#define FB_ACCEL_HI625x	0x8
+#define FB_ACCEL_HI366x	0x10
 #define FB_ACCEL_KIRIN970_ES  0x20
 #define FB_ACCEL_KIRIN970  0x40
 #define FB_ACCEL_KIRIN660  0x80
@@ -44,9 +44,8 @@
 /* vcc name */
 #define REGULATOR_PDP_NAME	"ldo3"
 
-/*******************************************************************************
-**
-*/
+/*****************************************************************************/
+
 enum dss_chn_idx {
 	DSS_RCHN_NONE = -1,
 	DSS_RCHN_D2 = 0,
@@ -116,13 +115,13 @@ typedef struct dss_img {
 	u32 stride;
 	u32 stride_plane1;
 	u32 stride_plane2;
-	uint64_t phy_addr;
-	uint64_t vir_addr;
+	u64 phy_addr;
+	u64 vir_addr;
 	u32 offset_plane1;
 	u32 offset_plane2;
 
-	uint64_t afbc_header_addr;
-	uint64_t afbc_payload_addr;
+	u64 afbc_header_addr;
+	u64 afbc_payload_addr;
 	u32 afbc_header_stride;
 	u32 afbc_payload_stride;
 	u32 afbc_scramble_mode;
@@ -132,7 +131,7 @@ typedef struct dss_img {
 	u32 mmu_enable;
 	u32 csc_mode;
 	u32 secure_mode;
-	int32_t shared_fd;
+	s32 shared_fd;
 	u32 reserved0;
 } dss_img_t;
 
@@ -142,19 +141,17 @@ typedef struct drm_dss_layer {
 	dss_rect_t src_rect_mask;
 	dss_rect_t dst_rect;
 	u32 transform;
-	int32_t blending;
+	s32 blending;
 	u32 glb_alpha;
 	u32 color;		/* background color or dim color */
-	int32_t layer_idx;
-	int32_t chn_idx;
+	s32 layer_idx;
+	s32 chn_idx;
 	u32 need_cap;
-	int32_t acquire_fence;
+	s32 acquire_fence;
 } drm_dss_layer_t;
 
+/*****************************************************************************/
 
-/*******************************************************************************
-**
-*/
 #define DEFAULT_MIPI_CLK_RATE	(192 * 100000L)
 #define DEFAULT_PCLK_DSI_RATE	(120 * 1000000L)
 
@@ -182,10 +179,8 @@ typedef struct drm_dss_layer {
 #define GPIO_PG_SEL_B (76)
 #define GPIO_TX_RX_B (78)
 
+/*****************************************************************************/
 
-/*******************************************************************************
-**
-*/
 #define CRGPERI_PLL0_CLK_RATE	(1660000000UL)
 #define CRGPERI_PLL2_CLK_RATE	(1920000000UL)
 #define CRGPERI_PLL3_CLK_RATE	(1200000000UL)
@@ -204,7 +199,7 @@ typedef struct drm_dss_layer {
 #define DEFAULT_DSS_PXL0_CLK_RATE_L1	(300000000UL)
 
 /*mmbuf_clk: 0.65v-237.14M, 0.75-332M, 0.8-480M*/
-#define DEFAULT_DSS_MMBUF_CLK_RATE_L3 	(480000000UL)
+#define DEFAULT_DSS_MMBUF_CLK_RATE_L3	(480000000UL)
 #define DEFAULT_DSS_MMBUF_CLK_RATE_L2	(332000000UL)
 #define DEFAULT_DSS_MMBUF_CLK_RATE_L1	(238000000UL)
 
@@ -219,10 +214,10 @@ typedef struct drm_dss_layer {
 #define DEFAULT_MDC_CORE_CLK_RATE_L1	(240000000UL)
 
 /*dss clk power off */
-#define DEFAULT_DSS_CORE_CLK_RATE_POWER_OFF   	(277000000UL)
-#define DEFAULT_DSS_PXL0_CLK_RATE_POWER_OFF    	(238000000UL)
-#define DEFAULT_DSS_MMBUF_CLK_RATE_POWER_OFF 	(208000000UL)
-#define DEFAULT_DSS_PXL1_CLK_RATE_POWER_OFF 	(238000000UL)
+#define DEFAULT_DSS_CORE_CLK_RATE_POWER_OFF	(277000000UL)
+#define DEFAULT_DSS_PXL0_CLK_RATE_POWER_OFF	(238000000UL)
+#define DEFAULT_DSS_MMBUF_CLK_RATE_POWER_OFF	(208000000UL)
+#define DEFAULT_DSS_PXL1_CLK_RATE_POWER_OFF	(238000000UL)
 
 #define DEFAULT_PCLK_DSS_RATE	(114000000UL)
 #define DEFAULT_PCLK_PCTRL_RATE	(80000000UL)
@@ -265,7 +260,7 @@ typedef struct drm_dss_layer {
 #define DEFAULT_ACLK_DPCTRL_RATE_CS	207000000UL
 #define DEFAULT_MIDIA_PPLL7_CLOCK_FREQ	1782000000UL
 
-#define KIRIN970_VCO_MIN_FREQ_OUPUT         1000000 /*Boston: 1000 * 1000*/
+#define KIRIN970_VCO_MIN_FREQ_OUTPUT         1000000 /*Boston: 1000 * 1000*/
 #define KIRIN970_SYS_19M2   19200 /*Boston: 19.2f * 1000 */
 
 #define MIDIA_PPLL7_CTRL0	0x50c
@@ -280,7 +275,7 @@ typedef struct drm_dss_layer {
 
 /*
  * DSS Registers
-*/
+ */
 
 /* MACROS */
 #define DSS_WIDTH(width)	((width) - 1)
@@ -298,6 +293,7 @@ typedef struct drm_dss_layer {
 #define DFC_MAX_CLIP_NUM	(31)
 
 /* for DFS */
+
 /* 1480 * 144bits */
 #define DFS_TIME	(80)
 #define DFS_TIME_MIN	(50)
@@ -502,9 +498,7 @@ enum dss_rdma_idx {
 	DSS_RDMA_MAX,
 };
 
-/*******************************************************************************
- **
- */
+/*****************************************************************************/
 
 #define PEREN0	(0x000)
 #define PERDIS0	(0x004)
@@ -556,18 +550,17 @@ enum dss_rdma_idx {
 #define NOC_POWER_IDLE	(0x388)
 
 //SYSCTRL
-#define SCISODIS 	(0x044)
+#define SCISODIS	(0x044)
 #define SCPERCLKEN1 (0x048)
-#define SCPWREN 	(0x060)
-#define SCPEREN1 	(0x170)
-#define SCPERDIS1  	(0x174)
-#define SCPEREN4 	(0x1B0)
-#define SCPERDIS4  	(0x1B4)
+#define SCPWREN	(0x060)
+#define SCPEREN1	(0x170)
+#define SCPERDIS1	(0x174)
+#define SCPEREN4	(0x1B0)
+#define SCPERDIS4	(0x1B4)
 #define SCPERRSTDIS1	(0x210)
 #define SCCLKDIV2	(0x258)
 #define SCCLKDIV4	(0x260)
 
-
 //PCTRL
 #define PERI_CTRL23	(0x060)
 #define PERI_CTRL29	(0x078)
@@ -584,9 +577,8 @@ enum dss_rdma_idx {
 #define PCTRL_DPHYTX_CTRL1	BIT(1)
 #define PCTRL_DPHYTX_CTRL0	BIT(0)
 
-/*******************************************************************************
- **
- */
+/*****************************************************************************/
+
 #define BIT_DSS_GLB_INTS	BIT(30)
 #define BIT_MMU_IRPT_S	BIT(29)
 #define BIT_MMU_IRPT_NS	BIT(28)
@@ -618,7 +610,6 @@ enum dss_rdma_idx {
 #define BIT_CMDLIST1	BIT(2)
 #define BIT_CMDLIST0	BIT(1)
 
-
 // CPU_SDP_INTS		0x22C
 // CPU_SDP_INT_MSK	0x230
 #define BIT_SDP_DSS_GLB_INTS	BIT(29)
@@ -652,7 +643,6 @@ enum dss_rdma_idx {
 #define BIT_SDP_CMDLIST0	BIT(1)
 #define BIT_SDP_RCH_CE_INTS	BIT(0)
 
-
 // CPU_OFF_INTS	0x234
 // CPU_OFF_INT_MASK	0x238
 #define BIT_OFF_DSS_GLB_INTS	BIT(31)
@@ -757,9 +747,10 @@ enum dss_rdma_idx {
 #define BIT_CE_HIST1_RW_COLLIDE_IND	BIT(1)
 #define BIT_CE_HIST0_RW_COLLIDE_IND	BIT(0)
 
-/*******************************************************************************
-** MODULE BASE ADDRESS
-*/
+/*
+ * MODULE BASE ADDRESS
+ */
+
 //DSI0 DSI1
 #define DSS_MIPI_DSI0_OFFSET	(0x00001000)
 #define DSS_MIPI_DSI1_OFFSET	(0x00001400)
@@ -794,7 +785,7 @@ enum dss_rdma_idx {
 
 // RCH_V
 #define DSS_RCH_VG0_DMA_OFFSET			(0x20000)
-#define DSS_RCH_VG0_DFC_OFFSET 			(0x20100)
+#define DSS_RCH_VG0_DFC_OFFSET			(0x20100)
 #define DSS_RCH_VG0_SCL_OFFSET			(0x20200)
 #define DSS_RCH_VG0_ARSR_OFFSET			(0x20300)
 #define DSS_RCH_VG0_POST_CLIP_OFFSET_ES	(0x203A0)
@@ -810,7 +801,7 @@ enum dss_rdma_idx {
 #define DSS_RCH_VG0_ARSR_LUT_OFFSET		(0x25000)
 
 #define DSS_RCH_VG1_DMA_OFFSET			(0x28000)
-#define DSS_RCH_VG1_DFC_OFFSET 			(0x28100)
+#define DSS_RCH_VG1_DFC_OFFSET			(0x28100)
 #define DSS_RCH_VG1_SCL_OFFSET			(0x28200)
 #define DSS_RCH_VG1_POST_CLIP_OFFSET_ES	(0x283A0)
 #define DSS_RCH_VG1_POST_CLIP_OFFSET	(0x28480)
@@ -823,7 +814,7 @@ enum dss_rdma_idx {
 #define DSS_RCH_VG1_SCL_LUT_OFFSET		(0x29000)
 
 #define DSS_RCH_VG2_DMA_OFFSET			(0x30000)
-#define DSS_RCH_VG2_DFC_OFFSET 			(0x30100)
+#define DSS_RCH_VG2_DFC_OFFSET			(0x30100)
 #define DSS_RCH_VG2_SCL_OFFSET			(0x30200)
 #define DSS_RCH_VG2_POST_CLIP_OFFSET_ES	(0x303A0)
 #define DSS_RCH_VG2_POST_CLIP_OFFSET	(0x30480)
@@ -839,23 +830,23 @@ enum dss_rdma_idx {
 #define DSS_RCH_G0_DFC_OFFSET				(0x38100)
 #define DSS_RCH_G0_SCL_OFFSET				(0x38200)
 #define DSS_RCH_G0_POST_CLIP_OFFSET_ES		(0x383A0)
-#define DSS_RCH_G0_POST_CLIP_OFFSET 		(0x38480)
-#define DSS_RCH_G0_CSC_OFFSET 			(0x38500)
-#define DSS_RCH_G0_DEBUG_OFFSET 			(0x38600)
-#define DSS_RCH_G0_DMA_BUF_OFFSET 		(0x38800)
-#define DSS_RCH_G0_AFBCD_OFFSET 			(0x38900)
-#define DSS_RCH_G0_REG_DEFAULT_OFFSET 	(0x38A00)
+#define DSS_RCH_G0_POST_CLIP_OFFSET		(0x38480)
+#define DSS_RCH_G0_CSC_OFFSET			(0x38500)
+#define DSS_RCH_G0_DEBUG_OFFSET			(0x38600)
+#define DSS_RCH_G0_DMA_BUF_OFFSET		(0x38800)
+#define DSS_RCH_G0_AFBCD_OFFSET			(0x38900)
+#define DSS_RCH_G0_REG_DEFAULT_OFFSET	(0x38A00)
 
 #define DSS_RCH_G1_DMA_OFFSET				(0x40000)
 #define DSS_RCH_G1_DFC_OFFSET				(0x40100)
 #define DSS_RCH_G1_SCL_OFFSET				(0x40200)
 #define DSS_RCH_G1_POST_CLIP_OFFSET_ES	(0x403A0)
 #define DSS_RCH_G1_POST_CLIP_OFFSET		(0x40480)
-#define DSS_RCH_G1_CSC_OFFSET 			(0x40500)
-#define DSS_RCH_G1_DEBUG_OFFSET 			(0x40600)
-#define DSS_RCH_G1_DMA_BUF_OFFSET 		(0x40800)
-#define DSS_RCH_G1_AFBCD_OFFSET 			(0x40900)
-#define DSS_RCH_G1_REG_DEFAULT_OFFSET 	(0x40A00)
+#define DSS_RCH_G1_CSC_OFFSET			(0x40500)
+#define DSS_RCH_G1_DEBUG_OFFSET			(0x40600)
+#define DSS_RCH_G1_DMA_BUF_OFFSET		(0x40800)
+#define DSS_RCH_G1_AFBCD_OFFSET			(0x40900)
+#define DSS_RCH_G1_REG_DEFAULT_OFFSET	(0x40A00)
 
 // RCH_D
 #define DSS_RCH_D2_DMA_OFFSET			(0x50000)
@@ -886,7 +877,7 @@ enum dss_rdma_idx {
 // WCH
 #define DSS_WCH0_DMA_OFFSET			(0x5A000)
 #define DSS_WCH0_DFC_OFFSET			(0x5A100)
-#define DSS_WCH0_BITEXT_OFFSET 		(0x5A140)
+#define DSS_WCH0_BITEXT_OFFSET		(0x5A140)
 #define DSS_WCH0_DITHER_OFFSET            (0x5A1D0)
 #define DSS_WCH0_PCSC_OFFSET			(0x5A400)
 #define DSS_WCH0_CSC_OFFSET			(0x5A500)
@@ -898,16 +889,16 @@ enum dss_rdma_idx {
 
 #define DSS_WCH1_DMA_OFFSET			(0x5C000)
 #define DSS_WCH1_DFC_OFFSET			(0x5C100)
-#define DSS_WCH1_BITEXT_OFFSET 		(0x5C140)
+#define DSS_WCH1_BITEXT_OFFSET		(0x5C140)
 #define DSS_WCH1_DITHER_OFFSET            (0x5C1D0)
-#define DSS_WCH1_SCL_OFFSET          		(0x5C200)
+#define DSS_WCH1_SCL_OFFSET			(0x5C200)
 #define DSS_WCH1_PCSC_OFFSET			(0x5C400)
 #define DSS_WCH1_CSC_OFFSET			(0x5C500)
 #define DSS_WCH1_ROT_OFFSET			(0x5C530)
 #define DSS_WCH1_DEBUG_OFFSET		(0x5C600)
 #define DSS_WCH1_DMA_BUFFER_OFFSET	(0x5C800)
 #define DSS_WCH1_AFBCE_OFFSET			(0x5C900)
-#define DSS_WCH1_FBCE_CREG_CTRL_GATE 	(0x5C964)
+#define DSS_WCH1_FBCE_CREG_CTRL_GATE	(0x5C964)
 
 #define DSS_WCH2_DMA_OFFSET			(0x5E000)
 #define DSS_WCH2_DFC_OFFSET			(0x5E100)
@@ -917,8 +908,6 @@ enum dss_rdma_idx {
 #define DSS_WCH2_DMA_BUFFER_OFFSET	(0x5E800)
 #define DSS_WCH2_AFBCE_OFFSET			(0x5E900)
 
-
-
 // OVL
 #define DSS_OVL0_OFFSET	(0x60000)
 #define DSS_OVL1_OFFSET	(0x60400)
@@ -977,9 +966,10 @@ enum dss_rdma_idx {
 #define DSS_DSC_OFFSET	(0x7DC00)
 #define DSS_LDI1_OFFSET	(0x7E000)
 
-/*******************************************************************************
-** GLB
-*/
+/*
+ * GLB
+ */
+
 #define GLB_DSS_TAG	 (DSS_GLB0_OFFSET + 0x0000)
 //APB
 #define GLB_APB_CTL	 (DSS_GLB0_OFFSET + 0x0004)
@@ -1042,9 +1032,8 @@ enum dss_rdma_idx {
 #define GLB_DSS_MEM_CTRL	(DSS_GLB0_OFFSET + 0x0600)
 #define GLB_DSS_PM_CTRL	(DSS_GLB0_OFFSET + 0x0604)
 
-/*******************************************************************************
-** DBG
-*/
+/*****************************************************************************/
+
 #define DBG_CRC_DBG_OV0	(0x0000)
 #define DBG_CRC_DBG_OV1	(0x0004)
 #define DBG_CRC_DBG_SUM	(0x0008)
@@ -1086,9 +1075,9 @@ enum dss_rdma_idx {
 #define DBG_RCH8_INTS	(0x02A4)
 #define DBG_RCH8_INT_MSK	(0x02A8)
 
-/*******************************************************************************
-** CMDLIST
-*/
+/*
+ * CMDLIST
+ */
 //DSS_CMD_OFFSET +  CMDLIST_CH0_* + 0x40 * i
 #define CMDLIST_CH0_PENDING_CLR	(0x0000)
 #define CMDLIST_CH0_CTRL	(0x0004)
@@ -1150,9 +1139,9 @@ enum dss_rdma_idx {
 #define BIT_CMDLIST_CH1_INTS	BIT(1)
 #define BIT_CMDLIST_CH0_INTS	BIT(0)
 
-/*******************************************************************************
-** AIF
-*/
+/*
+ * AIF
+ */
 #define AIF0_CH0_OFFSET		(DSS_VBIF0_AIF + 0x00)
 #define AIF0_CH1_OFFSET		(DSS_VBIF0_AIF + 0x20)
 #define AIF0_CH2_OFFSET		(DSS_VBIF0_AIF + 0x40)
@@ -1185,11 +1174,11 @@ enum dss_rdma_idx {
 //(0x0000+0x20*n)
 #define AIF_CH_CTL	(0x0000)
 //(0x0004+0x20*n)  //ES
-#define AIF_CH_CTL_ADD 	(0x0004)
+#define AIF_CH_CTL_ADD	(0x0004)
 //(0x0004+0x20*n)
-#define AIF_CH_HS 	(0x0004)
+#define AIF_CH_HS	(0x0004)
 //(0x0008+0x20*n)
-#define AIF_CH_LS 	(0x0008)
+#define AIF_CH_LS	(0x0008)
 
 /* aif common */
 #define AXI0_RID_MSK0	(0x0800)
@@ -1241,22 +1230,22 @@ enum dss_rdma_idx {
 #define AIF_MODULE_CLK_EN	(0x0A08)
 
 typedef struct dss_aif {
-	uint32_t aif_ch_ctl;
-	uint32_t aif_ch_ctl_add;  //ES
-	uint32_t aif_ch_hs;
-	uint32_t aif_ch_ls;
+	u32 aif_ch_ctl;
+	u32 aif_ch_ctl_add;  //ES
+	u32 aif_ch_hs;
+	u32 aif_ch_ls;
 } dss_aif_t;
 
 typedef struct dss_aif_bw {
-	uint64_t bw;
-	uint8_t chn_idx;
-	int8_t axi_sel;
-	uint8_t is_used;
+	u64 bw;
+	u8 chn_idx;
+	s8 axi_sel;
+	u8 is_used;
 } dss_aif_bw_t;
 
-/*******************************************************************************
-** MIF
-*/
+/*
+ * MIF
+ */
 #define MIF_ENABLE	(0x0000)
 #define MIF_MEM_CTRL	(0x0004)
 
@@ -1277,30 +1266,29 @@ typedef struct dss_aif_bw {
 #define MIF_STAT2	(0x0608)
 
 #define MIF_CTRL_OFFSET	(0x20)
-#define MIF_CH0_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*1)
-#define MIF_CH1_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*2)
-#define MIF_CH2_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*3)
-#define MIF_CH3_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*4)
-#define MIF_CH4_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*5)
-#define MIF_CH5_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*6)
-#define MIF_CH6_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*7)
-#define MIF_CH7_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*8)
-#define MIF_CH8_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*9)
-#define MIF_CH9_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*10)
-#define MIF_CH10_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*11)
-#define MIF_CH11_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET*12)
+#define MIF_CH0_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 1)
+#define MIF_CH1_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 2)
+#define MIF_CH2_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 3)
+#define MIF_CH3_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 4)
+#define MIF_CH4_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 5)
+#define MIF_CH5_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 6)
+#define MIF_CH6_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 7)
+#define MIF_CH7_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 8)
+#define MIF_CH8_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 9)
+#define MIF_CH9_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 10)
+#define MIF_CH10_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 11)
+#define MIF_CH11_OFFSET	(DSS_MIF_OFFSET + MIF_CTRL_OFFSET * 12)
 #define MIF_CTRL_NUM	(12)
 
-
 #define LITTLE_LAYER_BUF_SIZE	(256 * 1024)
 #define MIF_STRIDE_UNIT (4 * 1024)
 
 typedef struct dss_mif {
-	uint32_t mif_ctrl1;
-	uint32_t mif_ctrl2;
-	uint32_t mif_ctrl3;
-	uint32_t mif_ctrl4;
-	uint32_t mif_ctrl5;
+	u32 mif_ctrl1;
+	u32 mif_ctrl2;
+	u32 mif_ctrl3;
+	u32 mif_ctrl4;
+	u32 mif_ctrl5;
 } dss_mif_t;
 
 /*
@@ -1337,9 +1325,9 @@ enum dss_mmu_tlb_tag_org {
 	MMU_TLB_TAG_ORG_0x1F = 0x1F,
 };
 
-/*******************************************************************************
-**SMMU
-*/
+/*
+ * SMMU
+ */
 #define SMMU_SCR	(0x0000)
 #define SMMU_MEMCTRL	(0x0004)
 #define SMMU_LP_CTRL	(0x0008)
@@ -1415,68 +1403,68 @@ enum dss_mmu_tlb_tag_org {
 #define SMMU_SID_NUM	(64)
 
 typedef struct dss_smmu {
-	uint32_t smmu_scr;
-	uint32_t smmu_memctrl;
-	uint32_t smmu_lp_ctrl;
-	uint32_t smmu_press_remap;
-	uint32_t smmu_intmask_ns;
-	uint32_t smmu_intraw_ns;
-	uint32_t smmu_intstat_ns;
-	uint32_t smmu_intclr_ns;
-	uint32_t smmu_smrx_ns[SMMU_SID_NUM];
-	uint32_t smmu_rld_en0_ns;
-	uint32_t smmu_rld_en1_ns;
-	uint32_t smmu_rld_en2_ns;
-	uint32_t smmu_cb_sctrl;
-	uint32_t smmu_cb_ttbr0;
-	uint32_t smmu_cb_ttbr1;
-	uint32_t smmu_cb_ttbcr;
-	uint32_t smmu_offset_addr_ns;
-	uint32_t smmu_scachei_all;
-	uint32_t smmu_scachei_l1;
-	uint32_t smmu_scachei_l2l3;
-	uint32_t smmu_fama_ctrl0_ns;
-	uint32_t smmu_fama_ctrl1_ns;
-	uint32_t smmu_addr_msb;
-	uint32_t smmu_err_rdaddr;
-	uint32_t smmu_err_wraddr;
-	uint32_t smmu_fault_addr_tcu;
-	uint32_t smmu_fault_id_tcu;
-	uint32_t smmu_fault_addr_tbux;
-	uint32_t smmu_fault_id_tbux;
-	uint32_t smmu_fault_infox;
-	uint32_t smmu_dbgrptr_tlb;
-	uint32_t smmu_dbgrdata_tlb;
-	uint32_t smmu_dbgrptr_cache;
-	uint32_t smmu_dbgrdata0_cache;
-	uint32_t smmu_dbgrdata1_cache;
-	uint32_t smmu_dbgaxi_ctrl;
-	uint32_t smmu_ova_addr;
-	uint32_t smmu_opa_addr;
-	uint32_t smmu_ova_ctrl;
-	uint32_t smmu_opref_addr;
-	uint32_t smmu_opref_ctrl;
-	uint32_t smmu_opref_cnt;
-	uint32_t smmu_smrx_s[SMMU_SID_NUM];
-	uint32_t smmu_rld_en0_s;
-	uint32_t smmu_rld_en1_s;
-	uint32_t smmu_rld_en2_s;
-	uint32_t smmu_intmas_s;
-	uint32_t smmu_intraw_s;
-	uint32_t smmu_intstat_s;
-	uint32_t smmu_intclr_s;
-	uint32_t smmu_scr_s;
-	uint32_t smmu_scb_sctrl;
-	uint32_t smmu_scb_ttbr;
-	uint32_t smmu_scb_ttbcr;
-	uint32_t smmu_offset_addr_s;
+	u32 smmu_scr;
+	u32 smmu_memctrl;
+	u32 smmu_lp_ctrl;
+	u32 smmu_press_remap;
+	u32 smmu_intmask_ns;
+	u32 smmu_intraw_ns;
+	u32 smmu_intstat_ns;
+	u32 smmu_intclr_ns;
+	u32 smmu_smrx_ns[SMMU_SID_NUM];
+	u32 smmu_rld_en0_ns;
+	u32 smmu_rld_en1_ns;
+	u32 smmu_rld_en2_ns;
+	u32 smmu_cb_sctrl;
+	u32 smmu_cb_ttbr0;
+	u32 smmu_cb_ttbr1;
+	u32 smmu_cb_ttbcr;
+	u32 smmu_offset_addr_ns;
+	u32 smmu_scachei_all;
+	u32 smmu_scachei_l1;
+	u32 smmu_scachei_l2l3;
+	u32 smmu_fama_ctrl0_ns;
+	u32 smmu_fama_ctrl1_ns;
+	u32 smmu_addr_msb;
+	u32 smmu_err_rdaddr;
+	u32 smmu_err_wraddr;
+	u32 smmu_fault_addr_tcu;
+	u32 smmu_fault_id_tcu;
+	u32 smmu_fault_addr_tbux;
+	u32 smmu_fault_id_tbux;
+	u32 smmu_fault_infox;
+	u32 smmu_dbgrptr_tlb;
+	u32 smmu_dbgrdata_tlb;
+	u32 smmu_dbgrptr_cache;
+	u32 smmu_dbgrdata0_cache;
+	u32 smmu_dbgrdata1_cache;
+	u32 smmu_dbgaxi_ctrl;
+	u32 smmu_ova_addr;
+	u32 smmu_opa_addr;
+	u32 smmu_ova_ctrl;
+	u32 smmu_opref_addr;
+	u32 smmu_opref_ctrl;
+	u32 smmu_opref_cnt;
+	u32 smmu_smrx_s[SMMU_SID_NUM];
+	u32 smmu_rld_en0_s;
+	u32 smmu_rld_en1_s;
+	u32 smmu_rld_en2_s;
+	u32 smmu_intmas_s;
+	u32 smmu_intraw_s;
+	u32 smmu_intstat_s;
+	u32 smmu_intclr_s;
+	u32 smmu_scr_s;
+	u32 smmu_scb_sctrl;
+	u32 smmu_scb_ttbr;
+	u32 smmu_scb_ttbcr;
+	u32 smmu_offset_addr_s;
 
-	uint8_t smmu_smrx_ns_used[DSS_CHN_MAX_DEFINE];
+	u8 smmu_smrx_ns_used[DSS_CHN_MAX_DEFINE];
 } dss_smmu_t;
 
-/*******************************************************************************
-** RDMA
-*/
+/*
+ * RDMA
+ */
 
 //DMA_CMN
 #define DMA_OFT_X0	(0x0000)
@@ -1544,9 +1532,9 @@ typedef struct dss_smmu {
 #define CH_CLK_SEL	(0x00E0)
 #define CH_CLK_EN	(0x00E4)
 
-/*******************************************************************************
-** DFC
-*/
+/*
+ * DFC
+ */
 #define DFC_DISP_SIZE		(0x0000)
 #define DFC_PIX_IN_NUM		(0x0004)
 #define DFC_GLB_ALPHA01		(0x0008)
@@ -1562,21 +1550,21 @@ typedef struct dss_smmu {
 #define DFC_DITHER_CTL1      (0x00D0)
 
 typedef struct dss_dfc {
-	uint32_t disp_size;
-	uint32_t pix_in_num;
-	uint32_t disp_fmt;
-	uint32_t clip_ctl_hrz;
-	uint32_t clip_ctl_vrz;
-	uint32_t ctl_clip_en;
-	uint32_t icg_module;
-	uint32_t dither_enable;
-	uint32_t padding_ctl;
-	uint32_t bitext_ctl;
+	u32 disp_size;
+	u32 pix_in_num;
+	u32 disp_fmt;
+	u32 clip_ctl_hrz;
+	u32 clip_ctl_vrz;
+	u32 ctl_clip_en;
+	u32 icg_module;
+	u32 dither_enable;
+	u32 padding_ctl;
+	u32 bitext_ctl;
 } dss_dfc_t;
 
-/*******************************************************************************
-** SCF
-*/
+/*
+ * SCF
+ */
 #define DSS_SCF_H0_Y_COEF_OFFSET	(0x0000)
 #define DSS_SCF_Y_COEF_OFFSET	(0x2000)
 #define DSS_SCF_UV_COEF_OFFSET	(0x2800)
@@ -1607,7 +1595,7 @@ typedef struct dss_dfc {
 #define SCF_CLK_SEL	(0x00F8)
 #define SCF_CLK_EN	(0x00FC)
 #define WCH_SCF_COEF_MEM_CTRL (0x0218)
-#define WCH_SCF_LB_MEM_CTRL 	(0x290)
+#define WCH_SCF_LB_MEM_CTRL	(0x290)
 
 /* MACROS */
 #define SCF_MIN_INPUT	(16) //SCF min input pix 16x16
@@ -1615,26 +1603,26 @@ typedef struct dss_dfc {
 
 /* Threshold for SCF Stretch and SCF filter */
 #define RDMA_STRETCH_THRESHOLD	(2)
-#define SCF_INC_FACTOR	(1 << 18) //(262144)
+#define SCF_INC_FACTOR	BIT(18) //(262144)
 #define SCF_UPSCALE_MAX	(60)
 #define SCF_DOWNSCALE_MAX	  (60)
 #define SCF_EDGE_FACTOR (3)
 #define ARSR2P_INC_FACTOR (65536)
 
 typedef struct dss_scl {
-	uint32_t en_hscl_str;
-	uint32_t en_vscl_str;
-	uint32_t h_v_order;
-	uint32_t input_width_height;
-	uint32_t output_width_height;
-	uint32_t en_hscl;
-	uint32_t en_vscl;
-	uint32_t acc_hscl;
-	uint32_t inc_hscl;
-	uint32_t inc_vscl;
-	uint32_t en_mmp;
-	uint32_t scf_ch_core_gt;
-	uint32_t fmt;
+	u32 en_hscl_str;
+	u32 en_vscl_str;
+	u32 h_v_order;
+	u32 input_width_height;
+	u32 output_width_height;
+	u32 en_hscl;
+	u32 en_vscl;
+	u32 acc_hscl;
+	u32 inc_hscl;
+	u32 inc_vscl;
+	u32 en_mmp;
+	u32 scf_ch_core_gt;
+	u32 fmt;
 } dss_scl_t;
 
 enum scl_coef_lut_idx {
@@ -1644,9 +1632,9 @@ enum scl_coef_lut_idx {
 	SCL_COEF_IDX_MAX = 2,
 };
 
-/*******************************************************************************
-** ARSR2P ES  v0
-*/
+/*
+ * ARSR2P ES  v0
+ */
 #define ARSR2P_INPUT_WIDTH_HEIGHT_ES	(0x000)
 #define ARSR2P_OUTPUT_WIDTH_HEIGHT_ES	(0x004)
 #define ARSR2P_IHLEFT_ES	(0x008)
@@ -1692,10 +1680,9 @@ enum scl_coef_lut_idx {
 #define ARSR2P_LUT_COEFUV_V_OFFSET_ES	(0x0600)
 #define ARSR2P_LUT_COEFUV_H_OFFSET_ES	(0x0700)
 
-
-/*******************************************************************************
-** ARSR2P  v0
-*/
+/*
+ * ARSR2P  v0
+ */
 #define ARSR2P_INPUT_WIDTH_HEIGHT (0x000)
 #define ARSR2P_OUTPUT_WIDTH_HEIGHT (0x004)
 #define ARSR2P_IHLEFT (0x008)
@@ -1745,9 +1732,9 @@ enum scl_coef_lut_idx {
 #define ARSR2P_LUT_COEFUV_V_OFFSET (0x0600)
 #define ARSR2P_LUT_COEFUV_H_OFFSET (0x0700)
 
-/*******************************************************************************
-** POST_CLIP  v g
-*/
+/*
+ * POST_CLIP  v g
+ */
 #define POST_CLIP_DISP_SIZE			(0x0000)
 #define POST_CLIP_CTL_HRZ			(0x0004)
 #define POST_CLIP_CTL_VRZ			(0x0008)
@@ -1758,16 +1745,16 @@ enum scl_coef_lut_idx {
 #define POST_CLIP_CTL_VRZ_ES		(0x0014)
 #define POST_CLIP_EN_ES			(0x0018)
 
-typedef struct dss_post_clip{
-	uint32_t disp_size;
-	uint32_t clip_ctl_hrz;
-	uint32_t clip_ctl_vrz;
-	uint32_t ctl_clip_en;
+typedef struct dss_post_clip {
+	u32 disp_size;
+	u32 clip_ctl_hrz;
+	u32 clip_ctl_vrz;
+	u32 ctl_clip_en;
 } dss_post_clip_t;
 
-/*******************************************************************************
-** PCSC v
-*/
+/*
+ * PCSC v
+ */
 #define PCSC_IDC0	(0x0000)
 #define PCSC_IDC2	(0x0004)
 #define PCSC_ODC0	(0x0008)
@@ -1780,13 +1767,13 @@ typedef struct dss_post_clip{
 #define PCSC_ICG_MODULE	(0x0024)
 #define PCSC_MPREC	(0x0028)
 
-typedef struct dss_pcsc{
-	uint32_t pcsc_idc0;
+typedef struct dss_pcsc {
+	u32 pcsc_idc0;
 } dss_pcsc_t;
 
-/*******************************************************************************
-** CSC
-*/
+/*
+ * CSC
+ */
 #define CSC_IDC0			(0x0000)
 #define CSC_IDC2			(0x0004)
 #define CSC_ODC0			(0x0008)
@@ -1810,50 +1797,49 @@ typedef struct dss_pcsc{
 #define CSC_ICG_MODULE		(0x0034)
 
 typedef struct dss_csc {
-	uint32_t idc0;
-	uint32_t idc2;
-	uint32_t odc0;
-	uint32_t odc2;
-	uint32_t p0;
-	uint32_t p1;
-	uint32_t p2;
-	uint32_t p3;
-	uint32_t p4;
-	uint32_t icg_module_es;
-	uint32_t mprec;
-	uint32_t p00;
-	uint32_t p01;
-	uint32_t p02;
-	uint32_t p10;
-	uint32_t p11;
-	uint32_t p12;
-	uint32_t p20;
-	uint32_t p21;
-	uint32_t p22;
-	uint32_t icg_module;
+	u32 idc0;
+	u32 idc2;
+	u32 odc0;
+	u32 odc2;
+	u32 p0;
+	u32 p1;
+	u32 p2;
+	u32 p3;
+	u32 p4;
+	u32 icg_module_es;
+	u32 mprec;
+	u32 p00;
+	u32 p01;
+	u32 p02;
+	u32 p10;
+	u32 p11;
+	u32 p12;
+	u32 p20;
+	u32 p21;
+	u32 p22;
+	u32 icg_module;
 } dss_csc_t;
 
-/*******************************************************************************
-**channel DEBUG
-*/
+/*
+ * channel DEBUG
+ */
 #define CH_DEBUG_SEL (0x600)
 
-/*******************************************************************************
-** VPP
-*/
+/*
+ * VPP
+ */
 #define VPP_CTRL (0x700)
 #define VPP_MEM_CTRL (0x704)
 
-/*******************************************************************************
-**DMA BUF
-*/
+/*
+ * DMA BUF
+ */
 #define DMA_BUF_CTRL	(0x800)
 #define DMA_BUF_SIZE  (0x850)
 #define DMA_BUF_MEM_CTRL	(0x854)
 #define DMA_BUF_DBG0 (0x0838)
 #define DMA_BUF_DBG1 (0x083c)
 
-
 //AFBCD
 #define AFBCD_HREG_HDR_PTR_LO		(0x900)
 #define AFBCD_HREG_PIC_WIDTH			(0x904)
@@ -1875,15 +1861,15 @@ typedef struct dss_csc {
 #define AFBCD_MONITOR_REG2_OFFSET	(0x94C)
 #define AFBCD_MONITOR_REG3_OFFSET	(0x950)
 #define AFBCD_DEBUG_REG0_OFFSET		(0x954)
-#define AFBCD_CREG_FBCD_CTRL_MODE  	(0x960)
-#define AFBCD_HREG_HDR_PTR_L1 		(0x964)
-#define AFBCD_HREG_PLD_PTR_L1 		(0x968)
-#define AFBCD_HEADER_SRTIDE_1 		(0x96C)
-#define AFBCD_PAYLOAD_SRTIDE_1	 	(0x970)
-#define AFBCD_HREG_HDR_PTR_L1 		(0x964)
-#define AFBCD_HREG_PLD_PTR_L1 		(0x968)
-#define AFBCD_HEADER_SRTIDE_1 		(0x96C)
-#define AFBCD_PAYLOAD_SRTIDE_1 		(0x970)
+#define AFBCD_CREG_FBCD_CTRL_MODE	(0x960)
+#define AFBCD_HREG_HDR_PTR_L1		(0x964)
+#define AFBCD_HREG_PLD_PTR_L1		(0x968)
+#define AFBCD_HEADER_SRTIDE_1		(0x96C)
+#define AFBCD_PAYLOAD_SRTIDE_1		(0x970)
+#define AFBCD_HREG_HDR_PTR_L1		(0x964)
+#define AFBCD_HREG_PLD_PTR_L1		(0x968)
+#define AFBCD_HEADER_SRTIDE_1		(0x96C)
+#define AFBCD_PAYLOAD_SRTIDE_1		(0x970)
 #define AFBCD_BLOCK_TYPE				(0x974)
 #define AFBCD_MM_BASE_1				(0x978)
 #define AFBCD_MM_BASE_2				(0x97C)
@@ -1894,8 +1880,8 @@ typedef struct dss_csc {
 //AFBCE
 #define AFBCE_HREG_PIC_BLKS			(0x900)
 #define AFBCE_HREG_FORMAT				(0x904)
-#define AFBCE_HREG_HDR_PTR_L0 		(0x908)
-#define AFBCE_HREG_PLD_PTR_L0 		(0x90C)
+#define AFBCE_HREG_HDR_PTR_L0		(0x908)
+#define AFBCE_HREG_PLD_PTR_L0		(0x90C)
 #define AFBCE_PICTURE_SIZE				(0x910)
 #define AFBCE_CTL						(0x914)
 #define AFBCE_HEADER_SRTIDE			(0x918)
@@ -1906,19 +1892,19 @@ typedef struct dss_csc {
 #define AFBCE_THRESHOLD				(0x92C)
 #define AFBCE_SCRAMBLE_MODE			(0x930)
 #define AFBCE_HEADER_POINTER_OFFSET	(0x934)
-#define AFBCE_CREG_FBCE_CTRL_MODE  	(0x950)
-#define AFBCE_HREG_HDR_PTR_L1 		(0x954)
-#define AFBCE_HREG_PLD_PTR_L1 		(0x958)
-#define AFBCE_HEADER_SRTIDE_1 			(0x95C)
-#define AFBCE_PAYLOAD_SRTIDE_1 		(0x960)
-#define AFBCE_MEM_CTRL_1 				(0x968)
+#define AFBCE_CREG_FBCE_CTRL_MODE	(0x950)
+#define AFBCE_HREG_HDR_PTR_L1		(0x954)
+#define AFBCE_HREG_PLD_PTR_L1		(0x958)
+#define AFBCE_HEADER_SRTIDE_1			(0x95C)
+#define AFBCE_PAYLOAD_SRTIDE_1		(0x960)
+#define AFBCE_MEM_CTRL_1				(0x968)
 #define FBCD_CREG_FBCD_CTRL_GATE		(0x98C)
 
 //ROT
 #define ROT_FIRST_LNS			(0x530)
 #define ROT_STATE				(0x534)
 #define ROT_MEM_CTRL_ES		(0x538)
-#define ROT_SIZE_ES 			(0x53C)
+#define ROT_SIZE_ES			(0x53C)
 #define ROT_CPU_CTL0			(0x540)
 #define ROT_CPU_START0			(0x544)
 #define ROT_CPU_ADDR0			(0x548)
@@ -1934,8 +1920,8 @@ typedef struct dss_csc {
 #define ROT_CPU_WDATA2		(0x570)
 #define ROT_CPU_WDATA3		(0x574)
 
-#define ROT_MEM_CTRL		   	(0x588)
-#define ROT_SIZE 				(0x58C)
+#define ROT_MEM_CTRL			(0x588)
+#define ROT_SIZE				(0x58C)
 #define ROT_422_MODE			(0x590)
 
 //REG_DEFAULT
@@ -1955,7 +1941,6 @@ typedef struct dss_csc {
 #define DMA_IN_WIDTH_MAX	(2048)
 #define DMA_IN_HEIGHT_MAX	(8192)
 
-
 #define AFBC_PIC_WIDTH_MIN	(16)
 #define AFBC_PIC_WIDTH_MAX	(8192)
 #define AFBC_PIC_HEIGHT_MIN	(16)
@@ -1999,7 +1984,7 @@ typedef struct dss_csc {
 #define HFBC_PIC_HEIGHT_ROT_MIN	(32)
 #define HFBC_PIC_HEIGHT_MAX	(8196)
 #define HFBC_PIC_HEIGHT_ROT_MAX	(2160)
-#define HFBC_BLOCK0_WIDTH_ALIGN 	(64)
+#define HFBC_BLOCK0_WIDTH_ALIGN	(64)
 #define HFBC_BLOCK0_HEIGHT_ALIGN     (8)
 #define HFBC_BLOCK1_WIDTH_ALIGN	 (32)
 #define HFBC_BLOCK1_HEIGHT_ALIGN   (16)
@@ -2022,154 +2007,154 @@ enum DSS_AFBC_HALF_BLOCK_MODE {
 };
 
 typedef struct dss_rdma {
-	uint32_t oft_x0;
-	uint32_t oft_y0;
-	uint32_t oft_x1;
-	uint32_t oft_y1;
-	uint32_t mask0;
-	uint32_t mask1;
-	uint32_t stretch_size_vrt;
-	uint32_t ctrl;
-	uint32_t tile_scram;
+	u32 oft_x0;
+	u32 oft_y0;
+	u32 oft_x1;
+	u32 oft_y1;
+	u32 mask0;
+	u32 mask1;
+	u32 stretch_size_vrt;
+	u32 ctrl;
+	u32 tile_scram;
 
-	uint32_t data_addr0;
-	uint32_t stride0;
-	uint32_t stretch_stride0;
-	uint32_t data_num0;
+	u32 data_addr0;
+	u32 stride0;
+	u32 stretch_stride0;
+	u32 data_num0;
 
-	uint32_t data_addr1;
-	uint32_t stride1;
-	uint32_t stretch_stride1;
-	uint32_t data_num1;
+	u32 data_addr1;
+	u32 stride1;
+	u32 stretch_stride1;
+	u32 data_num1;
 
-	uint32_t data_addr2;
-	uint32_t stride2;
-	uint32_t stretch_stride2;
-	uint32_t data_num2;
+	u32 data_addr2;
+	u32 stride2;
+	u32 stretch_stride2;
+	u32 data_num2;
 
-	uint32_t ch_rd_shadow;
-	uint32_t ch_ctl;
+	u32 ch_rd_shadow;
+	u32 ch_ctl;
 
-	uint32_t dma_buf_ctrl;
+	u32 dma_buf_ctrl;
 
-	uint32_t vpp_ctrl;
-	uint32_t vpp_mem_ctrl;
+	u32 vpp_ctrl;
+	u32 vpp_mem_ctrl;
 
-	uint32_t afbcd_hreg_hdr_ptr_lo;
-	uint32_t afbcd_hreg_pic_width;
-	uint32_t afbcd_hreg_pic_height;
-	uint32_t afbcd_hreg_format;
-	uint32_t afbcd_ctl;
-	uint32_t afbcd_str;
-	uint32_t afbcd_line_crop;
-	uint32_t afbcd_input_header_stride;
-	uint32_t afbcd_payload_stride;
-	uint32_t afbcd_mm_base_0;
+	u32 afbcd_hreg_hdr_ptr_lo;
+	u32 afbcd_hreg_pic_width;
+	u32 afbcd_hreg_pic_height;
+	u32 afbcd_hreg_format;
+	u32 afbcd_ctl;
+	u32 afbcd_str;
+	u32 afbcd_line_crop;
+	u32 afbcd_input_header_stride;
+	u32 afbcd_payload_stride;
+	u32 afbcd_mm_base_0;
 	//uint32_t afbcd_mm_base_1;
-	uint32_t afbcd_afbcd_payload_pointer;
-	uint32_t afbcd_height_bf_str;
-	uint32_t afbcd_os_cfg;
-	uint32_t afbcd_mem_ctrl;
-	uint32_t afbcd_scramble_mode;
-	uint32_t afbcd_header_pointer_offset;
+	u32 afbcd_afbcd_payload_pointer;
+	u32 afbcd_height_bf_str;
+	u32 afbcd_os_cfg;
+	u32 afbcd_mem_ctrl;
+	u32 afbcd_scramble_mode;
+	u32 afbcd_header_pointer_offset;
 
-	uint32_t hfbcd_hreg_hdr_ptr_l0;
-	uint32_t hfbcd_hreg_pic_width;
-	uint32_t hfbcd_hreg_pic_height;
-	uint32_t hfbcd_line_crop;
-	uint32_t hfbcd_input_header_stride0;
-	uint32_t hfbcd_payload_stride0;
-	uint32_t hfbcd_payload_pointer; //hfbcd_hreg_pld_ptr_l0;
-	uint32_t hfbcd_scramble_mode;
-	uint32_t hfbcd_creg_fbcd_ctrl_mode;
-	uint32_t hfbcd_hreg_hdr_ptr_l1;
-	uint32_t hfbcd_hreg_pld_ptr_l1;
-	uint32_t hfbcd_header_stride1;
-	uint32_t hfbcd_payload_stride1;
-	uint32_t hfbcd_block_type;
-	uint32_t hfbcd_mm_base0_y8;
-	uint32_t hfbcd_mm_base1_c8;
-	uint32_t hfbcd_mm_base2_y2;
-	uint32_t hfbcd_mm_base3_c2;
+	u32 hfbcd_hreg_hdr_ptr_l0;
+	u32 hfbcd_hreg_pic_width;
+	u32 hfbcd_hreg_pic_height;
+	u32 hfbcd_line_crop;
+	u32 hfbcd_input_header_stride0;
+	u32 hfbcd_payload_stride0;
+	u32 hfbcd_payload_pointer; //hfbcd_hreg_pld_ptr_l0;
+	u32 hfbcd_scramble_mode;
+	u32 hfbcd_creg_fbcd_ctrl_mode;
+	u32 hfbcd_hreg_hdr_ptr_l1;
+	u32 hfbcd_hreg_pld_ptr_l1;
+	u32 hfbcd_header_stride1;
+	u32 hfbcd_payload_stride1;
+	u32 hfbcd_block_type;
+	u32 hfbcd_mm_base0_y8;
+	u32 hfbcd_mm_base1_c8;
+	u32 hfbcd_mm_base2_y2;
+	u32 hfbcd_mm_base3_c2;
 
-	uint8_t vpp_used;
-	uint8_t afbc_used;
-	uint8_t hfbcd_used;
+	u8 vpp_used;
+	u8 afbc_used;
+	u8 hfbcd_used;
 } dss_rdma_t;
 
 typedef struct dss_wdma {
-	uint32_t oft_x0;
-	uint32_t oft_y0;
-	uint32_t oft_x1;
-	uint32_t oft_y1;
+	u32 oft_x0;
+	u32 oft_y0;
+	u32 oft_x1;
+	u32 oft_y1;
 
-	uint32_t mask0;
-	uint32_t mask1;
-	uint32_t stretch_size_vrt;
-	uint32_t ctrl;
-	uint32_t tile_scram;
+	u32 mask0;
+	u32 mask1;
+	u32 stretch_size_vrt;
+	u32 ctrl;
+	u32 tile_scram;
 
-	uint32_t sw_mask_en;
-	uint32_t start_mask0;
-	uint32_t end_mask0;
-	uint32_t start_mask1;
-	uint32_t end_mask1;
+	u32 sw_mask_en;
+	u32 start_mask0;
+	u32 end_mask0;
+	u32 start_mask1;
+	u32 end_mask1;
 
-	uint32_t data_addr;
-	uint32_t stride0;
-	uint32_t data1_addr;
-	uint32_t stride1;
+	u32 data_addr;
+	u32 stride0;
+	u32 data1_addr;
+	u32 stride1;
 
-	uint32_t stretch_stride;
-	uint32_t data_num;
+	u32 stretch_stride;
+	u32 data_num;
 
-	uint32_t ch_rd_shadow;
-	uint32_t ch_ctl;
-	uint32_t ch_secu_en;
-	uint32_t ch_sw_end_req;
+	u32 ch_rd_shadow;
+	u32 ch_ctl;
+	u32 ch_secu_en;
+	u32 ch_sw_end_req;
 
-	uint32_t dma_buf_ctrl;
-	uint32_t dma_buf_size;
+	u32 dma_buf_ctrl;
+	u32 dma_buf_size;
 
-	uint32_t rot_size;
+	u32 rot_size;
 
-	uint32_t afbce_hreg_pic_blks;
-	uint32_t afbce_hreg_format;
-	uint32_t afbce_hreg_hdr_ptr_l0;
-	uint32_t afbce_hreg_pld_ptr_l0;
-	uint32_t afbce_picture_size;
-	uint32_t afbce_ctl;
-	uint32_t afbce_header_srtide;
-	uint32_t afbce_payload_stride;
-	uint32_t afbce_enc_os_cfg;
-	uint32_t afbce_mem_ctrl;
-	uint32_t afbce_qos_cfg;
-	uint32_t afbce_threshold;
-	uint32_t afbce_scramble_mode;
-	uint32_t afbce_header_pointer_offset;
+	u32 afbce_hreg_pic_blks;
+	u32 afbce_hreg_format;
+	u32 afbce_hreg_hdr_ptr_l0;
+	u32 afbce_hreg_pld_ptr_l0;
+	u32 afbce_picture_size;
+	u32 afbce_ctl;
+	u32 afbce_header_srtide;
+	u32 afbce_payload_stride;
+	u32 afbce_enc_os_cfg;
+	u32 afbce_mem_ctrl;
+	u32 afbce_qos_cfg;
+	u32 afbce_threshold;
+	u32 afbce_scramble_mode;
+	u32 afbce_header_pointer_offset;
 
-	uint32_t hfbce_hreg_pic_blks;
-	uint32_t hfbce_hreg_hdr_ptr_l0;
-	uint32_t hfbce_hreg_pld_ptr_l0;
-	uint32_t hfbce_picture_size;
-	uint32_t hfbce_scramble_mode;
-	uint32_t hfbce_header_stride0;
-	uint32_t hfbce_payload_stride0;
-	uint32_t hfbce_header_pointer_offset;
-	uint32_t fbce_creg_fbce_ctrl_mode;
-	uint32_t hfbce_hreg_hdr_ptr_l1;
-	uint32_t hfbce_hreg_pld_ptr_l1;
-	uint32_t hfbce_header_stride1;
-	uint32_t hfbce_payload_stride1;
+	u32 hfbce_hreg_pic_blks;
+	u32 hfbce_hreg_hdr_ptr_l0;
+	u32 hfbce_hreg_pld_ptr_l0;
+	u32 hfbce_picture_size;
+	u32 hfbce_scramble_mode;
+	u32 hfbce_header_stride0;
+	u32 hfbce_payload_stride0;
+	u32 hfbce_header_pointer_offset;
+	u32 fbce_creg_fbce_ctrl_mode;
+	u32 hfbce_hreg_hdr_ptr_l1;
+	u32 hfbce_hreg_pld_ptr_l1;
+	u32 hfbce_header_stride1;
+	u32 hfbce_payload_stride1;
 
-	uint8_t afbc_used;
-	uint8_t hfbce_used;
-	uint8_t rot_used;
+	u8 afbc_used;
+	u8 hfbce_used;
+	u8 rot_used;
 } dss_wdma_t;
 
-/*******************************************************************************
-** MCTL  MUTEX0 1 2 3 4 5
-*/
+/*
+ * MCTL  MUTEX0 1 2 3 4 5
+ */
 #define MCTL_CTL_EN	(0x0000)
 #define MCTL_CTL_MUTEX	(0x0004)
 #define MCTL_CTL_MUTEX_STATUS	(0x0008)
@@ -2203,12 +2188,12 @@ typedef struct dss_wdma {
 #define MCTL_CTL_CLK_EN	(0x0084)
 #define MCTL_CTL_DBG	(0x00E0)
 
-/*******************************************************************************
-** MCTL  SYS
-*/
+/*
+ * MCTL  SYS
+ */
 //SECU
 #define MCTL_CTL_SECU_CFG			(0x0000)
-#define MCTL_PAY_SECU_FLUSH_EN  	(0x0018)
+#define MCTL_PAY_SECU_FLUSH_EN	(0x0018)
 #define MCTL_CTL_SECU_GATE0    (0x0080)
 #define MCTL_CTL_SECU_GATE1    (0x0084)
 #define MCTL_CTL_SECU_GATE2    (0x0088)
@@ -2361,33 +2346,33 @@ enum dss_mctl_idx {
 };
 
 typedef struct dss_mctl {
-	uint32_t ctl_mutex_itf;
-	uint32_t ctl_mutex_dbuf;
-	uint32_t ctl_mutex_scf;
-	uint32_t ctl_mutex_ov;
+	u32 ctl_mutex_itf;
+	u32 ctl_mutex_dbuf;
+	u32 ctl_mutex_scf;
+	u32 ctl_mutex_ov;
 } dss_mctl_t;
 
 typedef struct dss_mctl_ch {
-	uint32_t chn_mutex;
-	uint32_t chn_flush_en;
-	uint32_t chn_ov_oen;
-	uint32_t chn_starty;
-	uint32_t chn_mod_dbg;
+	u32 chn_mutex;
+	u32 chn_flush_en;
+	u32 chn_ov_oen;
+	u32 chn_starty;
+	u32 chn_mod_dbg;
 } dss_mctl_ch_t;
 
 typedef struct dss_mctl_sys {
-	uint32_t ov_flush_en[DSS_OVL_IDX_MAX];
-	uint32_t chn_ov_sel[DSS_OVL_IDX_MAX];
-	uint32_t chn_ov_sel1[DSS_OVL_IDX_MAX];
-	uint32_t wchn_ov_sel[DSS_WCH_MAX];
-	uint8_t ov_flush_en_used[DSS_OVL_IDX_MAX];
-	uint8_t chn_ov_sel_used[DSS_OVL_IDX_MAX];
-	uint8_t wch_ov_sel_used[DSS_WCH_MAX];
+	u32 ov_flush_en[DSS_OVL_IDX_MAX];
+	u32 chn_ov_sel[DSS_OVL_IDX_MAX];
+	u32 chn_ov_sel1[DSS_OVL_IDX_MAX];
+	u32 wchn_ov_sel[DSS_WCH_MAX];
+	u8 ov_flush_en_used[DSS_OVL_IDX_MAX];
+	u8 chn_ov_sel_used[DSS_OVL_IDX_MAX];
+	u8 wch_ov_sel_used[DSS_WCH_MAX];
 } dss_mctl_sys_t;
 
-/*******************************************************************************
-** OVL ES
-*/
+/*
+ * OVL ES
+ */
 #define OVL_SIZE	(0x0000)
 #define OVL_BG_COLOR	(0x4)
 #define OVL_DST_STARTPOS	(0x8)
@@ -2534,7 +2519,6 @@ typedef struct dss_mctl_sys {
 #define OVL2_BLOCK_DBG	(0xB4)
 #define OVL2_REG_DEFAULT	(0xB8)
 
-
 /* LAYER0_CFG */
 #define BIT_OVL_LAYER_SRC_CFG	BIT(8)
 #define BIT_OVL_LAYER_ENABLE	BIT(0)
@@ -2546,101 +2530,99 @@ typedef struct dss_mctl_sys {
 /* LAYER0_INFO_SRCCOLOR */
 #define BIT_OVL_LAYER_SRCCOLOR_FLAG	BIT(0)
 
-
 #define OVL_6LAYER_NUM		(6)
 #define OVL_2LAYER_NUM		(2)
 
-/*******************************************************************************
-** OVL
-*/
+/*
+ * OVL
+ */
 #define OV_SIZE						(0x000)
-#define OV_BG_COLOR_RGB 			(0x004)
-#define OV_BG_COLOR_A 				(0x008)
-#define OV_DST_STARTPOS 			(0x00C)
-#define OV_DST_ENDPOS 				(0x010)
-#define OV_GCFG 					(0x014)
-#define OV_LAYER0_POS 				(0x030)
-#define OV_LAYER0_SIZE 				(0x034)
+#define OV_BG_COLOR_RGB			(0x004)
+#define OV_BG_COLOR_A				(0x008)
+#define OV_DST_STARTPOS			(0x00C)
+#define OV_DST_ENDPOS				(0x010)
+#define OV_GCFG					(0x014)
+#define OV_LAYER0_POS				(0x030)
+#define OV_LAYER0_SIZE				(0x034)
 #define OV_LAYER0_SRCLOKEY		(0x038)
-#define OV_LAYER0_SRCHIKEY 		(0x03C)
-#define OV_LAYER0_DSTLOKEY 		(0x040)
-#define OV_LAYER0_DSTHIKEY 		(0x044)
-#define OV_LAYER0_PATTERN_RGB 	(0x048)
-#define OV_LAYER0_PATTERN_A 		(0x04C)
-#define OV_LAYER0_ALPHA_MODE 		(0x050)
-#define OV_LAYER0_ALPHA_A 			(0x054)
-#define OV_LAYER0_CFG 				(0x058)
-#define OV_LAYER0_PSPOS 			(0x05C)
-#define OV_LAYER0_PEPOS 			(0x060)
-#define OV_LAYER0_INFO_ALPHA 		(0x064)
-#define OV_LAYER0_INFO_SRCCOLOR 	(0x068)
-#define OV_LAYER0_DBG_INFO 		(0x06C)
+#define OV_LAYER0_SRCHIKEY		(0x03C)
+#define OV_LAYER0_DSTLOKEY		(0x040)
+#define OV_LAYER0_DSTHIKEY		(0x044)
+#define OV_LAYER0_PATTERN_RGB	(0x048)
+#define OV_LAYER0_PATTERN_A		(0x04C)
+#define OV_LAYER0_ALPHA_MODE		(0x050)
+#define OV_LAYER0_ALPHA_A			(0x054)
+#define OV_LAYER0_CFG				(0x058)
+#define OV_LAYER0_PSPOS			(0x05C)
+#define OV_LAYER0_PEPOS			(0x060)
+#define OV_LAYER0_INFO_ALPHA		(0x064)
+#define OV_LAYER0_INFO_SRCCOLOR	(0x068)
+#define OV_LAYER0_DBG_INFO		(0x06C)
 #define OV8_BASE_DBG_INFO			(0x340)
 #define OV8_RD_SHADOW_SEL			(0x344)
 #define OV8_CLK_SEL					(0x348)
-#define OV8_CLK_EN 					(0x34C)
-#define OV8_BLOCK_SIZE 				(0x350)
-#define OV8_BLOCK_DBG 				(0x354)
-#define OV8_REG_DEFAULT 			(0x358)
+#define OV8_CLK_EN					(0x34C)
+#define OV8_BLOCK_SIZE				(0x350)
+#define OV8_BLOCK_DBG				(0x354)
+#define OV8_REG_DEFAULT			(0x358)
 #define OV2_BASE_DBG_INFO			(0x200)
 #define OV2_RD_SHADOW_SEL			(0x204)
 #define OV2_CLK_SEL					(0x208)
-#define OV2_CLK_EN 					(0x20C)
-#define OV2_BLOCK_SIZE 				(0x210)
-#define OV2_BLOCK_DBG 				(0x214)
-#define OV2_REG_DEFAULT 			(0x218)
+#define OV2_CLK_EN					(0x20C)
+#define OV2_BLOCK_SIZE				(0x210)
+#define OV2_BLOCK_DBG				(0x214)
+#define OV2_REG_DEFAULT			(0x218)
 
 #define OV_8LAYER_NUM				(8)
 
 typedef struct dss_ovl_layer {
-	uint32_t layer_pos;
-	uint32_t layer_size;
-	uint32_t layer_pattern;
-	uint32_t layer_pattern_alpha;
-	uint32_t layer_alpha_a;
-	uint32_t layer_alpha;
-	uint32_t layer_cfg;
+	u32 layer_pos;
+	u32 layer_size;
+	u32 layer_pattern;
+	u32 layer_pattern_alpha;
+	u32 layer_alpha_a;
+	u32 layer_alpha;
+	u32 layer_cfg;
 } dss_ovl_layer_t;
 
 typedef struct dss_ovl_layer_pos {
-	uint32_t layer_pspos;
-	uint32_t layer_pepos;
+	u32 layer_pspos;
+	u32 layer_pepos;
 } dss_ovl_layer_pos_t;
 
 typedef struct dss_ovl {
-	uint32_t ovl_size;
-	uint32_t ovl_bg_color;
-	uint32_t ovl_bg_color_alpha;
-	uint32_t ovl_dst_startpos;
-	uint32_t ovl_dst_endpos;
-	uint32_t ovl_gcfg;
-	uint32_t ovl_block_size;
+	u32 ovl_size;
+	u32 ovl_bg_color;
+	u32 ovl_bg_color_alpha;
+	u32 ovl_dst_startpos;
+	u32 ovl_dst_endpos;
+	u32 ovl_gcfg;
+	u32 ovl_block_size;
 	dss_ovl_layer_t ovl_layer[OV_8LAYER_NUM];
 	dss_ovl_layer_pos_t ovl_layer_pos[OV_8LAYER_NUM];
-	uint8_t ovl_layer_used[OV_8LAYER_NUM];
+	u8 ovl_layer_used[OV_8LAYER_NUM];
 } dss_ovl_t;
 
 typedef struct dss_ovl_alpha {
-	uint32_t src_amode;
-	uint32_t src_gmode;
-	uint32_t alpha_offsrc;
-	uint32_t src_lmode;
-	uint32_t src_pmode;
+	u32 src_amode;
+	u32 src_gmode;
+	u32 alpha_offsrc;
+	u32 src_lmode;
+	u32 src_pmode;
 
-	uint32_t alpha_smode;
+	u32 alpha_smode;
 
-	uint32_t dst_amode;
-	uint32_t dst_gmode;
-	uint32_t alpha_offdst;
-	uint32_t dst_pmode;
+	u32 dst_amode;
+	u32 dst_gmode;
+	u32 alpha_offdst;
+	u32 dst_pmode;
 
-	uint32_t fix_mode;
+	u32 fix_mode;
 } dss_ovl_alpha_t;
 
-
-/*******************************************************************************
-** DBUF
-*/
+/*
+ * DBUF
+ */
 #define DBUF_FRM_SIZE	(0x0000)
 #define DBUF_FRM_HSIZE	(0x0004)
 #define DBUF_SRAM_VALID_NUM	(0x0008)
@@ -2665,7 +2647,7 @@ typedef struct dss_ovl_alpha {
 #define DBUF_MEM_CTRL	(0x0054)
 #define DBUF_PM_CTRL	(0x0058)
 #define DBUF_CLK_SEL	(0x005C)
-#define DBUF_CLK_EN  	(0x0060)
+#define DBUF_CLK_EN	(0x0060)
 #define DBUF_THD_FLUX_REQ_AFT (0x0064)
 #define DBUF_THD_DFS_OK (0x0068)
 #define DBUF_FLUX_REQ_CTRL (0x006C)
@@ -2673,9 +2655,9 @@ typedef struct dss_ovl_alpha {
 #define DBUF_DFS_RAM_MANAGE  (0x00A8)
 #define DBUF_DFS_DATA_FILL_OUT  (0x00AC)
 
-/*******************************************************************************
-** SBL
-*/
+/*
+ * SBL
+ */
 //SBL FOR ES
 #define SBL_REG_FRMT_MODE_ES                          (0x0000)
 #define SBL_REG_FRMT_DBUF_CTRL_ES                     (0x0008)
@@ -3008,9 +2990,9 @@ typedef struct dss_sbl {
 #define SBL_VC_ANTI_FLCKR_AL_ANTI_FLCKR_T_DURATION         (0x03a8)
 #define SBL_VC_ANTI_FLCKR_ALPHA                            (0x03ac)
 
-/*******************************************************************************
-** DPP
-*/
+/*
+ * DPP
+ */
 //DPP TOP
 #define DPP_RD_SHADOW_SEL	(0x000)
 #define DPP_DEFAULT	(0x004)
@@ -3034,7 +3016,6 @@ typedef struct dss_sbl {
 //#define DPP_ARSR1P	(0x048)
 #define DPP_DBG_CNT  DPP_DBG1_CNT
 
-
 //COLORBAR
 #define DPP_CLRBAR_CTRL (0x100)
 #define DPP_CLRBAR_1ST_CLR (0x104)
@@ -3120,8 +3101,6 @@ typedef struct dss_sbl {
 #define DITHER_DBG1_ES (0x034)
 #define DITHER_DBG2_ES (0x038)
 
-
-
 //CSC_RGB2YUV_10bits  CSC_YUV2RGB_10bits
 #define CSC10B_IDC0	(0x000)
 #define CSC10B_IDC1	(0x004)
@@ -3194,7 +3173,6 @@ typedef struct dss_sbl {
 #define ACM_DEBUG_CFG_ES     (0x0A8)
 #define ACM_DEBUG_W_ES       (0x0AC)
 
-
 //ACM
 #define ACM_EN	(0x000)
 #define ACM_SATA_OFFSET	(0x004)
@@ -3319,11 +3297,11 @@ typedef struct dss_sbl {
 #define ACE_Y_EXT	(0x03C)
 #define ACE_U_EXT	(0x040)
 #define ACE_V_EXT	(0x044)
-#define ACE_Y_ATTENU 	(0x048)
+#define ACE_Y_ATTENU	(0x048)
 #define ACE_U_ATTENU	(0x04C)
 #define ACE_V_ATTENU	(0x050)
 #define ACE_ROTA	(0x054)
-#define ACE_ROTB 	(0x058)
+#define ACE_ROTB	(0x058)
 #define ACE_Y_CORE	(0x05C)
 #define ACE_U_CORE	(0x060)
 #define ACE_V_CORE	(0x064)
@@ -3379,99 +3357,98 @@ typedef struct dss_sbl {
 #define GMP_DBG_R2 (0x018)
 
 //ARSR1P ES
-#define ARSR1P_IHLEFT_ES             	(0x000)
+#define ARSR1P_IHLEFT_ES		(0x000)
 #define ARSR1P_IHRIGHT_ES          (0x004)
 #define ARSR1P_IHLEFT1_ES          (0x008)
 #define ARSR1P_IHRIGHT1_ES         (0x00C)
 #define ARSR1P_IVTOP_ES            (0x010)
 #define ARSR1P_IVBOTTOM_ES         (0x014)
-#define ARSR1P_UV_OFFSET_ES          	(0x018)
+#define ARSR1P_UV_OFFSET_ES		(0x018)
 #define ARSR1P_IHINC_ES            (0x01C)
 #define ARSR1P_IVINC_ES            (0x020)
-#define ARSR1P_MODE_ES               	(0x024)
+#define ARSR1P_MODE_ES		(0x024)
 #define ARSR1P_FORMAT_ES           (0x028)
-#define ARSR1P_SKIN_THRES_Y_ES       	(0x02C)
-#define ARSR1P_SKIN_THRES_U_ES       	(0x030)
-#define ARSR1P_SKIN_THRES_V_ES       	(0x034)
+#define ARSR1P_SKIN_THRES_Y_ES	(0x02C)
+#define ARSR1P_SKIN_THRES_U_ES	(0x030)
+#define ARSR1P_SKIN_THRES_V_ES	(0x034)
 #define ARSR1P_SKIN_EXPECTED_ES    (0x038)
-#define ARSR1P_SKIN_CFG_ES           	(0x03C)
-#define ARSR1P_SHOOT_CFG1_ES         	(0x040)
-#define ARSR1P_SHOOT_CFG2_ES         	(0x044)
-#define ARSR1P_SHARP_CFG1_ES         	(0x048)
-#define ARSR1P_SHARP_CFG2_ES         	(0x04C)
-#define ARSR1P_SHARP_CFG3_ES         	(0x050)
-#define ARSR1P_SHARP_CFG4_ES         	(0x054)
-#define ARSR1P_SHARP_CFG5_ES         	(0x058)
-#define ARSR1P_SHARP_CFG6_ES         	(0x05C)
-#define ARSR1P_SHARP_CFG7_ES         	(0x060)
-#define ARSR1P_SHARP_CFG8_ES         	(0x064)
-#define ARSR1P_SHARP_CFG9_ES         	(0x068)
-#define ARSR1P_SHARP_CFG10_ES        	(0x06C)
-#define ARSR1P_SHARP_CFG11_ES        	(0x070)
-#define ARSR1P_DIFF_CTRL_ES          	(0x074)
+#define ARSR1P_SKIN_CFG_ES		(0x03C)
+#define ARSR1P_SHOOT_CFG1_ES		(0x040)
+#define ARSR1P_SHOOT_CFG2_ES		(0x044)
+#define ARSR1P_SHARP_CFG1_ES		(0x048)
+#define ARSR1P_SHARP_CFG2_ES		(0x04C)
+#define ARSR1P_SHARP_CFG3_ES		(0x050)
+#define ARSR1P_SHARP_CFG4_ES		(0x054)
+#define ARSR1P_SHARP_CFG5_ES		(0x058)
+#define ARSR1P_SHARP_CFG6_ES		(0x05C)
+#define ARSR1P_SHARP_CFG7_ES		(0x060)
+#define ARSR1P_SHARP_CFG8_ES		(0x064)
+#define ARSR1P_SHARP_CFG9_ES		(0x068)
+#define ARSR1P_SHARP_CFG10_ES		(0x06C)
+#define ARSR1P_SHARP_CFG11_ES		(0x070)
+#define ARSR1P_DIFF_CTRL_ES		(0x074)
 #define ARSR1P_LSC_CFG1_ES         (0x078)
 #define ARSR1P_LSC_CFG2_ES         (0x07C)
 #define ARSR1P_LSC_CFG3_ES         (0x080)
-#define ARSR1P_FORCE_CLK_ON_CFG_ES   	(0x084)
-
+#define ARSR1P_FORCE_CLK_ON_CFG_ES	(0x084)
 
 //ARSR1P
 typedef struct dss_arsr1p {
-	uint32_t ihleft;
-	uint32_t ihright;
-	uint32_t ihleft1;
-	uint32_t ihright1;
-	uint32_t ivtop;
-	uint32_t ivbottom;
-	uint32_t uv_offset;
-	uint32_t ihinc;
-	uint32_t ivinc;
-	uint32_t mode;
-	uint32_t format;
+	u32 ihleft;
+	u32 ihright;
+	u32 ihleft1;
+	u32 ihright1;
+	u32 ivtop;
+	u32 ivbottom;
+	u32 uv_offset;
+	u32 ihinc;
+	u32 ivinc;
+	u32 mode;
+	u32 format;
 
-	uint32_t skin_thres_y;
-	uint32_t skin_thres_u;
-	uint32_t skin_thres_v;
-	uint32_t skin_expected;
-	uint32_t skin_cfg;
-	uint32_t shoot_cfg1;
-	uint32_t shoot_cfg2;
-	uint32_t shoot_cfg3;
-	uint32_t sharp_cfg1_h;
-	uint32_t sharp_cfg1_l;
-	uint32_t sharp_cfg2_h;
-	uint32_t sharp_cfg2_l;
-	uint32_t sharp_cfg3;
-	uint32_t sharp_cfg4;
-	uint32_t sharp_cfg5;
-	uint32_t sharp_cfg6;
-	uint32_t sharp_cfg6_cut;
-	uint32_t sharp_cfg7;
-	uint32_t sharp_cfg7_ratio;
-	uint32_t sharp_cfg8;
-	uint32_t sharp_cfg9;
-	uint32_t sharp_cfg10;
-	uint32_t sharp_cfg11;
-	uint32_t diff_ctrl;
-	uint32_t skin_slop_y;
-	uint32_t skin_slop_u;
-	uint32_t skin_slop_v;
-	uint32_t force_clk_on_cfg;
+	u32 skin_thres_y;
+	u32 skin_thres_u;
+	u32 skin_thres_v;
+	u32 skin_expected;
+	u32 skin_cfg;
+	u32 shoot_cfg1;
+	u32 shoot_cfg2;
+	u32 shoot_cfg3;
+	u32 sharp_cfg1_h;
+	u32 sharp_cfg1_l;
+	u32 sharp_cfg2_h;
+	u32 sharp_cfg2_l;
+	u32 sharp_cfg3;
+	u32 sharp_cfg4;
+	u32 sharp_cfg5;
+	u32 sharp_cfg6;
+	u32 sharp_cfg6_cut;
+	u32 sharp_cfg7;
+	u32 sharp_cfg7_ratio;
+	u32 sharp_cfg8;
+	u32 sharp_cfg9;
+	u32 sharp_cfg10;
+	u32 sharp_cfg11;
+	u32 diff_ctrl;
+	u32 skin_slop_y;
+	u32 skin_slop_u;
+	u32 skin_slop_v;
+	u32 force_clk_on_cfg;
 
-	uint32_t dbuf_frm_size;
-	uint32_t dbuf_frm_hsize;
-	uint32_t dbuf_used;
+	u32 dbuf_frm_size;
+	u32 dbuf_frm_hsize;
+	u32 dbuf_used;
 
-	uint32_t dpp_img_size_bef_sr;
-	uint32_t dpp_img_size_aft_sr;
-	uint32_t dpp_used;
+	u32 dpp_img_size_bef_sr;
+	u32 dpp_img_size_aft_sr;
+	u32 dpp_used;
 
 	//for ES
-	uint32_t sharp_cfg1;
-	uint32_t sharp_cfg2;
-	uint32_t lsc_cfg1;
-	uint32_t lsc_cfg2;
-	uint32_t lsc_cfg3;
+	u32 sharp_cfg1;
+	u32 sharp_cfg2;
+	u32 lsc_cfg1;
+	u32 lsc_cfg2;
+	u32 lsc_cfg3;
 
 } dss_arsr1p_t;
 
@@ -3523,10 +3500,9 @@ typedef struct dss_arsr1p {
 #define ARSR_POST_DEBUG_RO_1 (0x0AC)
 #define ARSR_POST_DEBUG_RO_2 (0x0B0)
 
-
-/*******************************************************************************
-** BIT EXT
-*/
+/*
+ * BIT EXT
+ */
 //#define BIT_EXT0_CTL (0x000)
 
 //GAMA LUT
@@ -3750,10 +3726,9 @@ typedef struct dss_arsr1p {
 #define DPE_NR_RAM_A_CFG_MEM_CTRL (0x0498)
 #define DPE_NR_RAM_A_CFG_PM_CTRL (0x049c)
 
-
-/*******************************************************************************
-** IFBC
-*/
+/*
+ * IFBC
+ */
 #define IFBC_SIZE	(0x0000)
 #define IFBC_CTRL	(0x0004)
 #define IFBC_HIMAX_CTRL0	(0x0008)
@@ -3776,10 +3751,9 @@ typedef struct dss_arsr1p {
 #define IFBC_PAD	(0x004C)
 #define IFBC_REG_DEFAULT	(0x0050)
 
-
-/*******************************************************************************
-** DSC
-*/
+/*
+ * DSC
+ */
 #define DSC_VERSION	(0x0000)
 #define DSC_PPS_IDENTIFIER	(0x0004)
 #define DSC_EN	(0x0008)
@@ -3828,10 +3802,9 @@ typedef struct dss_arsr1p {
 #define DSC_RD_SHADOW_SEL	(0x00B4)
 #define DSC_REG_DEFAULT	(0x00B8)
 
-
-/*******************************************************************************
-** LDI
-*/
+/*
+ * LDI
+ */
 #define LDI_DPI0_HRZ_CTRL0	(0x0000)
 #define LDI_DPI0_HRZ_CTRL1	(0x0004)
 #define LDI_DPI0_HRZ_CTRL2	(0x0008)
@@ -3858,7 +3831,7 @@ typedef struct dss_arsr1p {
 #define LDI_CMD_EVENT_SEL	(0x0060)
 #define LDI_SRAM_LP_CTRL	(0x0064)
 #define LDI_ITF_RD_SHADOW	(0x006C)
-#define LDI_DP_DSI_SEL 		(0x0080)
+#define LDI_DP_DSI_SEL		(0x0080)
 #define LDI_DPI1_HRZ_CTRL0	(0x00F0)
 #define LDI_DPI1_HRZ_CTRL1	(0x00F4)
 #define LDI_DPI1_HRZ_CTRL2	(0x00F8)
@@ -3888,10 +3861,9 @@ typedef struct dss_arsr1p {
 #define LDI_MODULE_CLK_SEL (0x0258)
 #define LDI_MODULE_CLK_EN (0x025C)
 
-
-/*******************************************************************************
-** MIPI DSI
-*/
+/*
+ * MIPI DSI
+ */
 #define MIPIDSI_VERSION_OFFSET	(0x0000)
 #define MIPIDSI_PWR_UP_OFFSET	(0x0004)
 #define MIPIDSI_CLKMGR_CFG_OFFSET	(0x0008)
@@ -3955,9 +3927,9 @@ typedef struct dss_arsr1p {
 #define MIPIDSI_INT_MSK1_OFFSET	(0x00c8)
 #define INT_FORCE0	(0x00D8)
 #define INT_FORCE1	(0x00DC)
-#define AUTO_ULPS_MODE 	(0x00E0)
-#define AUTO_ULPS_ENTER_DELAY 	(0x00E4)
-#define AUTO_ULPS_WAKEUP_TIME 	(0x00E8)
+#define AUTO_ULPS_MODE	(0x00E0)
+#define AUTO_ULPS_ENTER_DELAY	(0x00E4)
+#define AUTO_ULPS_WAKEUP_TIME	(0x00E8)
 #define MIPIDSI_DSC_PARAMETER_OFFSET	(0x00F0)
 #define MIPIDSI_PHY_TMR_RD_CFG_OFFSET	(0x00F4)
 #define AUTO_ULPS_MIN_TIME  (0xF8)
@@ -3982,9 +3954,9 @@ typedef struct dss_arsr1p {
 #define DSI_PM_CTRL  (0x0198)
 #define DSI_DEBUG  (0x019C)
 
-/*******************************************************************************
-** MMBUF
-*/
+/*
+ * MMBUF
+ */
 #define SMC_LOCK	(0x0000)
 #define SMC_MEM_LP	(0x0004)
 #define SMC_GCLK_CS	(0x000C)
@@ -4106,12 +4078,12 @@ struct dss_hw_ctx {
 };
 
 typedef struct dss_clk_rate {
-	uint64_t dss_pri_clk_rate;
-	uint64_t dss_pclk_dss_rate;
-	uint64_t dss_pclk_pctrl_rate;
-	uint64_t dss_mmbuf_rate;
-	uint32_t dss_voltage_value; //0:0.7v, 2:0.8v
-	uint32_t reserved;
+	u64 dss_pri_clk_rate;
+	u64 dss_pclk_dss_rate;
+	u64 dss_pclk_pctrl_rate;
+	u64 dss_mmbuf_rate;
+	u32 dss_voltage_value; //0:0.7v, 2:0.8v
+	u32 reserved;
 } dss_clk_rate_t;
 
 struct dss_crtc {
@@ -4233,9 +4205,8 @@ typedef struct mipi_ifbc_division {
 	u32 pxl0_dsi_gt_en;
 } mipi_ifbc_division_t;
 
-/*******************************************************************************
-**
-*/
+/*****************************************************************************/
+
 #define outp32(addr, val) writel(val, addr)
 #define outp16(addr, val) writew(val, addr)
 #define outp8(addr, val) writeb(val, addr)
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.c b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.c
index fa317be188e0..8f07fabeee8c 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.c
@@ -17,79 +17,103 @@
 
 #include "kirin9xx_drm_dpe_utils.h"
 
-int g_debug_set_reg_val = 0;
+static int g_debug_set_reg_val;
 
 DEFINE_SEMAPHORE(hisi_fb_dss_regulator_sem);
 
-extern u32 g_dss_module_ovl_base[DSS_MCTL_IDX_MAX][MODULE_OVL_MAX];
-
-mipi_ifbc_division_t g_mipi_ifbc_division[MIPI_DPHY_NUM][IFBC_TYPE_MAX] = {
+struct mipi_ifbc_division g_mipi_ifbc_division[MIPI_DPHY_NUM][IFBC_TYPE_MAX] = {
 	/*single mipi*/
 	{
-		/*none*/
-		{XRES_DIV_1, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_0, PXL0_DSI_GT_EN_1},
-		/*orise2x*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*orise3x*/
-		{XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_1, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3},
-		/*himax2x*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_2, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*rsp2x*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_3, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*rsp3x  [NOTE]reality: xres_div = 1.5, yres_div = 2, amended in "mipi_ifbc_get_rect" function*/
-		{XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_4, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3},
-		/*vesa2x_1pipe*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*vesa3x_1pipe*/
-		{XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3},
-		/*vesa2x_2pipe*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*vesa3x_2pipe*/
-		{XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3}
-	},
+			/*none*/
+		{
+			XRES_DIV_1, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_0, PXL0_DSI_GT_EN_1
+		}, {
+			/*orise2x*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*orise3x*/
+			XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_1, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3
+		}, {
+			/*himax2x*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_2, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*rsp2x*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_3, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*
+			 * rsp3x
+			 * NOTE: in reality: xres_div = 1.5, yres_div = 2,
+			 * amended in "mipi_ifbc_get_rect" function
+			 */
+			XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_4, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa2x_1pipe*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa3x_1pipe*/
+			XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa2x_2pipe*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa3x_2pipe*/
+			XRES_DIV_3, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_2, PXL0_DSI_GT_EN_3
+		}
 
 	/*dual mipi*/
-	{
-		/*none*/
-		{XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3},
-		/*orise2x*/
-		{XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3},
-		/*orise3x*/
-		{XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_1, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3},
-		/*himax2x*/
-		{XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_2, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3},
-		/*rsp2x*/
-		{XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_3, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3},
-		/*rsp3x*/
-		{XRES_DIV_3, YRES_DIV_2, IFBC_COMP_MODE_4, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3},
-		/*vesa2x_1pipe*/
-		{XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3},
-		/*vesa3x_1pipe*/
-		{XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3},
-		/*vesa2x_2pipe*/
-		{XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3},
-		/*vesa3x_2pipe*/
-		{XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
-			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, 3} }
+	}, {
+		{
+			/*none*/
+			XRES_DIV_2, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_1, PXL0_DSI_GT_EN_3
+		}, {
+			/*orise2x*/
+			XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_0, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3
+		}, {
+			/*orise3x*/
+			XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_1, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3
+		}, {
+			/*himax2x*/
+			XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_2, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3
+		}, {
+			/*rsp2x*/
+			XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_3, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3
+		}, {
+			/*rsp3x*/
+			XRES_DIV_3, YRES_DIV_2, IFBC_COMP_MODE_4, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_OPEN, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa2x_1pipe*/
+			XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa3x_1pipe*/
+			XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_5, PXL0_DIV2_GT_EN_CLOSE,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa2x_2pipe*/
+			XRES_DIV_4, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_3, PXL0_DSI_GT_EN_3
+		}, {
+			/*vesa3x_2pipe*/
+			XRES_DIV_6, YRES_DIV_1, IFBC_COMP_MODE_6, PXL0_DIV2_GT_EN_OPEN,
+			PXL0_DIV4_GT_EN_CLOSE, PXL0_DIVCFG_5, 3
+		}
+	}
 };
 
 void set_reg(char __iomem *addr, uint32_t val, uint8_t bw, uint8_t bs)
@@ -104,14 +128,14 @@ void set_reg(char __iomem *addr, uint32_t val, uint8_t bw, uint8_t bs)
 
 	if (g_debug_set_reg_val) {
 		printk(KERN_INFO "writel: [%p] = 0x%x\n", addr,
-			     tmp | ((val & mask) << bs));
+		       tmp | ((val & mask) << bs));
 	}
 }
 
-uint32_t set_bits32(uint32_t old_val, uint32_t val, uint8_t bw, uint8_t bs)
+u32 set_bits32(u32 old_val, uint32_t val, uint8_t bw, uint8_t bs)
 {
-	uint32_t mask = (1UL << bw) - 1UL;
-	uint32_t tmp = 0;
+	u32 mask = (1UL << bw) - 1UL;
+	u32 tmp = 0;
 
 	tmp = old_val;
 	tmp &= ~(mask << bs);
@@ -139,9 +163,10 @@ static int mipi_ifbc_get_rect(struct dss_rect *rect)
 		DRM_ERROR("yres(%d) is not division_v(%d) pixel aligned!\n", rect->h, yres_div);
 
 	/*
-	** [NOTE] rsp3x && single_mipi CMD mode amended xres_div = 1.5, yres_div = 2 ,
-	** VIDEO mode amended xres_div = 3, yres_div = 1
-	*/
+	 * NOTE: rsp3x && single_mipi CMD mode amended xres_div = 1.5,
+	 *  yres_div = 2,
+	 * VIDEO mode amended xres_div = 3, yres_div = 1
+	 */
 	rect->w /= xres_div;
 	rect->h /= yres_div;
 
@@ -225,7 +250,7 @@ void init_ldi(struct dss_crtc *acrtc)
 	ctx = acrtc->ctx;
 	if (!ctx) {
 		DRM_ERROR("ctx is NULL!\n");
-		return ;
+		return;
 	}
 
 	mode = &acrtc->base.state->mode;
@@ -249,16 +274,16 @@ void init_ldi(struct dss_crtc *acrtc)
 	init_ldi_pxl_div(acrtc);
 
 	outp32(ldi_base + LDI_DPI0_HRZ_CTRL0,
-		hfp | ((hbp + DSS_WIDTH(hsw)) << 16));
+	       hfp | ((hbp + DSS_WIDTH(hsw)) << 16));
 	outp32(ldi_base + LDI_DPI0_HRZ_CTRL1, 0);
 	outp32(ldi_base + LDI_DPI0_HRZ_CTRL2, DSS_WIDTH(rect.w));
 	outp32(ldi_base + LDI_VRT_CTRL0,
-		vfp | (vbp << 16));
+	       vfp | (vbp << 16));
 	outp32(ldi_base + LDI_VRT_CTRL1, DSS_HEIGHT(vsw));
 	outp32(ldi_base + LDI_VRT_CTRL2, DSS_HEIGHT(rect.h));
 
 	outp32(ldi_base + LDI_PLR_CTRL,
-		vsync_plr | (hsync_plr << 1) |
+	       vsync_plr | (hsync_plr << 1) |
 		(pixelclk_plr << 2) | (data_en_plr << 3));
 
 	/* bpp*/
@@ -292,7 +317,7 @@ void deinit_ldi(struct dss_crtc *acrtc)
 	ctx = acrtc->ctx;
 	if (!ctx) {
 		DRM_ERROR("ctx is NULL!\n");
-		return ;
+		return;
 	}
 
 	ldi_base = ctx->base + DSS_LDI0_OFFSET;
@@ -389,12 +414,12 @@ void init_dbuf(struct dss_crtc *acrtc)
 		mode->vdisplay);
 
 	/*
-	** int K = 0;
-	** int Tp = 1000000  / adj_mode->clock;
-	** K = (hsw + hbp + mode->hdisplay +
-	**	hfp) / mode->hdisplay;
-	** thd_cg_out = dfs_time / (Tp * K * 6);
-	*/
+	 * int K = 0;
+	 * int Tp = 1000000  / adj_mode->clock;
+	 * K = (hsw + hbp + mode->hdisplay +
+	 *	hfp) / mode->hdisplay;
+	 * thd_cg_out = dfs_time / (Tp * K * 6);
+	 */
 	thd_cg_out = (dfs_time * adj_mode->clock * 1000UL * mode->hdisplay) /
 		(((hsw + hbp + hfp) + mode->hdisplay) * 6 * 1000000UL);
 
@@ -457,9 +482,8 @@ void init_dbuf(struct dss_crtc *acrtc)
 	outp32(dbuf_base + DBUF_FLUX_REQ_CTRL, (dfs_ok_mask << 1) | thd_flux_req_sw_en);
 
 	outp32(dbuf_base + DBUF_DFS_LP_CTRL, 0x1);
-	if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
+	if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970)
 		outp32(dbuf_base + DBUF_DFS_RAM_MANAGE, dfs_ram);
-	}
 }
 
 void init_dpp(struct dss_crtc *acrtc)
@@ -483,9 +507,9 @@ void init_dpp(struct dss_crtc *acrtc)
 	mctl_sys_base = ctx->base + DSS_MCTRL_SYS_OFFSET;
 
 	outp32(dpp_base + DPP_IMG_SIZE_BEF_SR,
-		(DSS_HEIGHT(mode->vdisplay) << 16) | DSS_WIDTH(mode->hdisplay));
+	       (DSS_HEIGHT(mode->vdisplay) << 16) | DSS_WIDTH(mode->hdisplay));
 	outp32(dpp_base + DPP_IMG_SIZE_AFT_SR,
-		(DSS_HEIGHT(mode->vdisplay) << 16) | DSS_WIDTH(mode->hdisplay));
+	       (DSS_HEIGHT(mode->vdisplay) << 16) | DSS_WIDTH(mode->hdisplay));
 }
 
 void enable_ldi(struct dss_crtc *acrtc)
@@ -577,7 +601,6 @@ void dpe_interrupt_unmask(struct dss_crtc *acrtc)
 	unmask &= ~(BIT_VSYNC | BIT_VACTIVE0_END | BIT_LDI_UNFLOW);
 
 	outp32(dss_base + DSS_LDI0_OFFSET + LDI_CPU_ITF_INT_MSK, unmask);
-
 }
 
 void dpe_interrupt_mask(struct dss_crtc *acrtc)
@@ -589,7 +612,7 @@ void dpe_interrupt_mask(struct dss_crtc *acrtc)
 	ctx = acrtc->ctx;
 	if (!ctx) {
 		DRM_ERROR("ctx is NULL!\n");
-		return ;
+		return;
 	}
 
 	dss_base = ctx->base;
@@ -660,7 +683,7 @@ void dpe_check_itf_status(struct dss_crtc *acrtc)
 	ctx = acrtc->ctx;
 	if (!ctx) {
 		DRM_ERROR("ctx is NULL!\n");
-		return ;
+		return;
 	}
 
 	itf_idx = 0;
@@ -672,15 +695,13 @@ void dpe_check_itf_status(struct dss_crtc *acrtc)
 			is_timeout = (delay_count > 100) ? true : false;
 			delay_count = 0;
 			break;
-		} else {
-			mdelay(1);
-			++delay_count;
 		}
+		mdelay(1);
+		++delay_count;
 	}
 
-	if (is_timeout) {
+	if (is_timeout)
 		DRM_DEBUG_DRIVER("mctl_itf%d not in idle status,ints=0x%x !\n", itf_idx, tmp);
-	}
 }
 
 void dss_inner_clk_pdp_disable(struct dss_hw_ctx *ctx)
@@ -708,9 +729,9 @@ void dss_inner_clk_common_enable(struct dss_hw_ctx *ctx)
 {
 	char __iomem *dss_base;
 
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL Pointer!\n");
-		return ;
+		return;
 	}
 
 	dss_base = ctx->base;
@@ -815,7 +836,7 @@ int dpe_common_clk_enable(struct dss_hw_ctx *ctx)
 	int ret = 0;
 	struct clk *clk_tmp = NULL;
 
-	if (ctx == NULL) {
+	if (!ctx) {
 		DRM_ERROR("ctx is NULL point!\n");
 		return -EINVAL;
 	}
@@ -872,7 +893,7 @@ int dpe_common_clk_disable(struct dss_hw_ctx *ctx)
 {
 	struct clk *clk_tmp = NULL;
 
-	if (ctx == NULL) {
+	if (!ctx) {
 		DRM_ERROR("ctx is NULL point!\n");
 		return -EINVAL;
 	}
@@ -903,7 +924,7 @@ int dpe_inner_clk_enable(struct dss_hw_ctx *ctx)
 	int ret = 0;
 	struct clk *clk_tmp = NULL;
 
-	if (ctx == NULL) {
+	if (!ctx) {
 		DRM_ERROR("ctx is NULL point!\n");
 		return -EINVAL;
 	}
@@ -945,7 +966,7 @@ int dpe_inner_clk_disable(struct dss_hw_ctx *ctx)
 {
 	struct clk *clk_tmp = NULL;
 
-	if (ctx == NULL) {
+	if (!ctx) {
 		DRM_ERROR("ctx is NULL point!\n");
 		return -EINVAL;
 	}
@@ -970,7 +991,7 @@ int dpe_regulator_enable(struct dss_hw_ctx *ctx)
 	int ret = 0;
 
 	DRM_INFO("enabling DPE regulator\n");
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL ptr.\n");
 		return -EINVAL;
 	}
@@ -981,7 +1002,7 @@ int dpe_regulator_enable(struct dss_hw_ctx *ctx)
 		return -EINVAL;
 	}
 
-	DRM_INFO("-. \n");
+	DRM_INFO("-.\n");
 
 	return ret;
 }
@@ -990,12 +1011,12 @@ int dpe_regulator_disable(struct dss_hw_ctx *ctx)
 {
 	int ret = 0;
 
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL ptr.\n");
 		return -EINVAL;
 	}
 
-	#if defined (CONFIG_DRM_HISI_KIRIN970)
+	#if defined(CONFIG_DRM_HISI_KIRIN970)
 		dpe_set_pixel_clk_rate_on_pll0(ctx);
 		dpe_set_common_clk_rate_on_pll0(ctx);
 	#endif
@@ -1013,15 +1034,14 @@ int mediacrg_regulator_enable(struct dss_hw_ctx *ctx)
 {
 	int ret = 0;
 
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL ptr.\n");
 		return -EINVAL;
 	}
 
 	//ret = regulator_enable(ctx->mediacrg_regulator);
-	if (ret) {
+	if (ret)
 		DRM_ERROR("mediacrg regulator_enable failed, error=%d!\n", ret);
-	}
 
 	return ret;
 }
@@ -1030,7 +1050,7 @@ int mediacrg_regulator_disable(struct dss_hw_ctx *ctx)
 {
 	int ret = 0;
 
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL ptr.\n");
 		return -EINVAL;
 	}
@@ -1046,10 +1066,10 @@ int mediacrg_regulator_disable(struct dss_hw_ctx *ctx)
 
 int dpe_set_clk_rate(struct dss_hw_ctx *ctx)
 {
-	uint64_t clk_rate;
+	u64 clk_rate;
 	int ret = 0;
 
-	if (NULL == ctx) {
+	if (!ctx) {
 		DRM_ERROR("NULL Pointer!\n");
 		return -EINVAL;
 	}
@@ -1061,20 +1081,19 @@ int dpe_set_clk_rate(struct dss_hw_ctx *ctx)
 		return -EINVAL;
 	}
 	DRM_INFO("dss_pri_clk:[%llu]->[%llu].\n",
-		clk_rate, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
+		 clk_rate, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
 
 #if 0 /* it will be set on dss_ldi_set_mode func */
 	ret = clk_set_rate(ctx->dss_pxl0_clk, pinfo->pxl_clk_rate);
 	if (ret < 0) {
 		DRM_ERROR("fb%d dss_pxl0_clk clk_set_rate(%llu) failed, error=%d!\n",
-			ctx->index, pinfo->pxl_clk_rate, ret);
-		if (g_fpga_flag == 0) {
+			  ctx->index, pinfo->pxl_clk_rate, ret);
+		if (g_fpga_flag == 0)
 			return -EINVAL;
-		}
 	}
 
 	DRM_INFO("dss_pxl0_clk:[%llu]->[%llu].\n",
-			pinfo->pxl_clk_rate, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
+		 pinfo->pxl_clk_rate, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
 #endif
 
 	clk_rate = DEFAULT_DSS_MMBUF_CLK_RATE_L1;
@@ -1085,7 +1104,7 @@ int dpe_set_clk_rate(struct dss_hw_ctx *ctx)
 	}
 
 	DRM_INFO("dss_mmbuf_clk:[%llu]->[%llu].\n",
-		clk_rate, (uint64_t)clk_get_rate(ctx->dss_mmbuf_clk));
+		 clk_rate, (uint64_t)clk_get_rate(ctx->dss_mmbuf_clk));
 
 	return ret;
 }
@@ -1093,10 +1112,10 @@ int dpe_set_clk_rate(struct dss_hw_ctx *ctx)
 int dpe_set_pixel_clk_rate_on_pll0(struct dss_hw_ctx *ctx)
 {
 	int ret;
-	uint64_t clk_rate;
+	u64 clk_rate;
 
-	DRM_INFO("+. \n");
-	if (NULL == ctx) {
+	DRM_INFO("+.\n");
+	if (!ctx) {
 		DRM_ERROR("NULL Pointer!\n");
 		return -EINVAL;
 	}
@@ -1104,10 +1123,12 @@ int dpe_set_pixel_clk_rate_on_pll0(struct dss_hw_ctx *ctx)
 	clk_rate = DEFAULT_DSS_PXL0_CLK_RATE_POWER_OFF;
 	ret = clk_set_rate(ctx->dss_pxl0_clk, clk_rate);
 	if (ret < 0) {
-		DRM_ERROR("dss_pxl0_clk clk_set_rate(%llu) failed, error=%d!\n", clk_rate, ret);
+		DRM_ERROR("dss_pxl0_clk clk_set_rate(%llu) failed, error=%d!\n",
+			  clk_rate, ret);
 		return -EINVAL;
 	}
-	DRM_INFO("dss_pxl0_clk:[%llu]->[%llu].\n", clk_rate, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
+	DRM_INFO("dss_pxl0_clk:[%llu]->[%llu].\n",
+		 clk_rate, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
 
 	return ret;
 }
@@ -1115,10 +1136,10 @@ int dpe_set_pixel_clk_rate_on_pll0(struct dss_hw_ctx *ctx)
 int dpe_set_common_clk_rate_on_pll0(struct dss_hw_ctx *ctx)
 {
 	int ret;
-	uint64_t clk_rate;
+	u64 clk_rate;
 
-	DRM_INFO("+. \n");
-	if (NULL == ctx) {
+	DRM_INFO("+.\n");
+	if (!ctx) {
 		DRM_ERROR("NULL Pointer!\n");
 		return -EINVAL;
 	}
@@ -1126,18 +1147,22 @@ int dpe_set_common_clk_rate_on_pll0(struct dss_hw_ctx *ctx)
 	clk_rate = DEFAULT_DSS_MMBUF_CLK_RATE_POWER_OFF;
 	ret = clk_set_rate(ctx->dss_mmbuf_clk, clk_rate);
 	if (ret < 0) {
-		DRM_ERROR("dss_mmbuf clk_set_rate(%llu) failed, error=%d!\n", clk_rate, ret);
+		DRM_ERROR("dss_mmbuf clk_set_rate(%llu) failed, error=%d!\n",
+			  clk_rate, ret);
 		return -EINVAL;
 	}
-	DRM_INFO("dss_mmbuf_clk:[%llu]->[%llu].\n", clk_rate, (uint64_t)clk_get_rate(ctx->dss_mmbuf_clk));
+	DRM_INFO("dss_mmbuf_clk:[%llu]->[%llu].\n",
+		 clk_rate, (uint64_t)clk_get_rate(ctx->dss_mmbuf_clk));
 
 	clk_rate = DEFAULT_DSS_CORE_CLK_RATE_POWER_OFF;
 	ret = clk_set_rate(ctx->dss_pri_clk, clk_rate);
 	if (ret < 0) {
-		DRM_ERROR("dss_pri_clk clk_set_rate(%llu) failed, error=%d!\n", clk_rate, ret);
+		DRM_ERROR("dss_pri_clk clk_set_rate(%llu) failed, error=%d!\n",
+			  clk_rate, ret);
 		return -EINVAL;
 	}
-	DRM_INFO("dss_pri_clk:[%llu]->[%llu].\n", clk_rate, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
+	DRM_INFO("dss_pri_clk:[%llu]->[%llu].\n",
+		 clk_rate, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
 
 	return ret;
 }
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.h b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.h
index e3681c26f7f4..1ab504d940a0 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.h
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dpe_utils.h
@@ -16,7 +16,7 @@
 #ifndef KIRIN_DRM_DPE_UTILS_H
 #define KIRIN_DRM_DPE_UTILS_H
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 #include "kirin970_dpe_reg.h"
 #else
 #include "kirin960_dpe_reg.h"
@@ -24,7 +24,7 @@
 #include "kirin9xx_drm_drv.h"
 
 void set_reg(char __iomem *addr, uint32_t val, uint8_t bw, uint8_t bs);
-uint32_t set_bits32(uint32_t old_val, uint32_t val, uint8_t bw, uint8_t bs);
+u32 set_bits32(u32 old_val, uint32_t val, uint8_t bw, uint8_t bs);
 
 void init_dbuf(struct dss_crtc *acrtc);
 void init_dpp(struct dss_crtc *acrtc);
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.c b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.c
index 12668646c2d3..acb8420e332a 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.c
@@ -312,7 +312,7 @@ static int kirin_drm_platform_suspend(struct platform_device *pdev, pm_message_t
 
 	dc_ops = of_device_get_match_data(dev);
 
-	DRM_INFO("+. pdev->name is %s, m_message is %d \n", pdev->name, state.event);
+	DRM_INFO("+. pdev->name is %s, m_message is %d\n", pdev->name, state.event);
 	if (!dc_ops) {
 		DRM_ERROR("dc_ops is NULL\n");
 		return -EINVAL;
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.h b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.h
index 8322abc0752c..232e88441bd1 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.h
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_drv.h
@@ -48,9 +48,9 @@ struct kirin_fbdev {
 };
 
 extern const struct kirin_dc_ops dss_dc_ops;
-extern void dsi_set_output_client(struct drm_device *dev);
+void dsi_set_output_client(struct drm_device *dev);
 
 struct drm_framebuffer *kirin_framebuffer_init(struct drm_device *dev,
-		struct drm_mode_fb_cmd2 *mode_cmd);
+					       struct drm_mode_fb_cmd2 *mode_cmd);
 
 #endif /* __KIRIN_DRM_DRV_H__ */
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dss.c b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dss.c
index 10e62bdb9161..e3bb0a32dddf 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dss.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_dss.c
@@ -40,7 +40,7 @@
 
 #include "kirin9xx_drm_drv.h"
 #include "kirin9xx_drm_dpe_utils.h"
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 #include "kirin970_dpe_reg.h"
 #else
 #include "kirin960_dpe_reg.h"
@@ -48,7 +48,7 @@
 
 //#define DSS_POWER_UP_ON_UEFI
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 #define DTS_COMP_DSS_NAME "hisilicon,kirin970-dpe"
 #else
 #define DTS_COMP_DSS_NAME "hisilicon,hi3660-dpe"
@@ -107,21 +107,20 @@ u32 dss_get_format(u32 pixel_format)
  **
  */
 
-int hdmi_ceil(uint64_t a, uint64_t b)
+int hdmi_ceil(u64 a, uint64_t b)
 {
 	if (b == 0)
 		return -1;
 
-	if (a%b != 0) {
-		return a/b + 1;
-	} else {
-		return a/b;
-	}
+	if (a % b != 0)
+		return a / b + 1;
+	else
+		return a / b;
 }
 
 int hdmi_pxl_ppll7_init(struct dss_hw_ctx *ctx, u64 pixel_clock)
 {
-	u64 vco_min_freq_output = KIRIN970_VCO_MIN_FREQ_OUPUT;
+	u64 vco_min_freq_output = KIRIN970_VCO_MIN_FREQ_OUTPUT;
 	u64 refdiv, fbdiv, frac, postdiv1 = 0, postdiv2 = 0;
 	u64 dss_pxl0_clk = 7 * 144000000UL;
 	u64 sys_clock_fref = KIRIN970_SYS_19M2;
@@ -323,7 +322,7 @@ static int dss_power_up(struct dss_crtc *acrtc)
 	struct dss_hw_ctx *ctx = acrtc->ctx;
 	int ret = 0;
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	mediacrg_regulator_enable(ctx);
 	dpe_common_clk_enable(ctx);
 	dpe_inner_clk_enable(ctx);
@@ -388,7 +387,7 @@ static void dss_power_down(struct dss_crtc *acrtc)
 	dpe_check_itf_status(acrtc);
 	dss_inner_clk_pdp_disable(ctx);
 
-	if (ctx->g_dss_version_tag & FB_ACCEL_KIRIN970 ) {
+	if (ctx->g_dss_version_tag & FB_ACCEL_KIRIN970) {
 		dpe_regulator_disable(ctx);
 		dpe_inner_clk_disable(ctx);
 		dpe_common_clk_disable(ctx);
@@ -609,7 +608,6 @@ static void dss_crtc_atomic_flush(struct drm_crtc *crtc,
 			drm_crtc_send_vblank_event(crtc, event);
 		spin_unlock_irq(&crtc->dev->event_lock);
 	}
-
 }
 
 static const struct drm_crtc_helper_funcs dss_crtc_helper_funcs = {
@@ -797,15 +795,15 @@ static int dss_dts_parse(struct platform_device *pdev, struct dss_hw_ctx *ctx)
 	np = of_find_compatible_node(NULL, NULL, DTS_COMP_DSS_NAME);
 	if (!np) {
 		DRM_ERROR("NOT FOUND device node %s!\n",
-			DTS_COMP_DSS_NAME);
+			  DTS_COMP_DSS_NAME);
 		return -ENXIO;
 	}
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	ret = of_property_read_u32(np, "dss_version_tag", &dss_version_tag);
-	if (ret) {
+	if (ret)
 		DRM_ERROR("failed to get dss_version_tag.\n");
-	}
+
 	ctx->g_dss_version_tag = dss_version_tag;
 	DRM_INFO("dss_version_tag=0x%x.\n", ctx->g_dss_version_tag);
 #else
@@ -815,52 +813,52 @@ static int dss_dts_parse(struct platform_device *pdev, struct dss_hw_ctx *ctx)
 
 	ctx->base = of_iomap(np, 0);
 	if (!(ctx->base)) {
-		DRM_ERROR ("failed to get dss base resource.\n");
+		DRM_ERROR("failed to get dss base resource.\n");
 		return -ENXIO;
 	}
 
 	ctx->peri_crg_base  = of_iomap(np, 1);
 	if (!(ctx->peri_crg_base)) {
-		DRM_ERROR ("failed to get dss peri_crg_base resource.\n");
+		DRM_ERROR("failed to get dss peri_crg_base resource.\n");
 		return -ENXIO;
 	}
 
 	ctx->sctrl_base  = of_iomap(np, 2);
 	if (!(ctx->sctrl_base)) {
-		DRM_ERROR ("failed to get dss sctrl_base resource.\n");
+		DRM_ERROR("failed to get dss sctrl_base resource.\n");
 		return -ENXIO;
 	}
 
 	if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
 		ctx->pctrl_base = of_iomap(np, 3);
 		if (!(ctx->pctrl_base)) {
-			DRM_ERROR ("failed to get dss pctrl_base resource.\n");
+			DRM_ERROR("failed to get dss pctrl_base resource.\n");
 			return -ENXIO;
 		}
 	} else {
 		ctx->pmc_base = of_iomap(np, 3);
 		if (!(ctx->pmc_base)) {
-			DRM_ERROR ("failed to get dss pmc_base resource.\n");
+			DRM_ERROR("failed to get dss pmc_base resource.\n");
 			return -ENXIO;
 		}
 	}
 
 	ctx->noc_dss_base = of_iomap(np, 4);
 	if (!(ctx->noc_dss_base)) {
-		DRM_ERROR ("failed to get noc_dss_base resource.\n");
+		DRM_ERROR("failed to get noc_dss_base resource.\n");
 		return -ENXIO;
 	}
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	ctx->pmctrl_base = of_iomap(np, 5);
 	if (!(ctx->pmctrl_base)) {
-		DRM_ERROR ("failed to get dss pmctrl_base resource.\n");
+		DRM_ERROR("failed to get dss pmctrl_base resource.\n");
 		return -ENXIO;
 	}
 
 	ctx->media_crg_base = of_iomap(np, 6);
 	if (!(ctx->media_crg_base)) {
-		DRM_ERROR ("failed to get dss media_crg_base resource.\n");
+		DRM_ERROR("failed to get dss media_crg_base resource.\n");
 		return -ENXIO;
 	}
 #endif
@@ -872,9 +870,9 @@ static int dss_dts_parse(struct platform_device *pdev, struct dss_hw_ctx *ctx)
 		return -ENXIO;
 	}
 
-	DRM_INFO("dss irq = %d. \n", ctx->irq);
+	DRM_INFO("dss irq = %d.\n", ctx->irq);
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	ctx->dpe_regulator = devm_regulator_get(dev, REGULATOR_PDP_NAME);
 	if (!ctx->dpe_regulator) {
 		DRM_ERROR("failed to get dpe_regulator resource! ret=%d.\n", ret);
@@ -903,19 +901,19 @@ static int dss_dts_parse(struct platform_device *pdev, struct dss_hw_ctx *ctx)
 	ctx->dss_pri_clk = devm_clk_get(dev, "clk_edc0");
 	if (!ctx->dss_pri_clk) {
 		DRM_ERROR("failed to parse dss_pri_clk\n");
-	    return -ENODEV;
+	return -ENODEV;
 	}
 
 	if (ctx->g_dss_version_tag != FB_ACCEL_KIRIN970) {
 		ret = clk_set_rate(ctx->dss_pri_clk, DEFAULT_DSS_CORE_CLK_07V_RATE);
 		if (ret < 0) {
 			DRM_ERROR("dss_pri_clk clk_set_rate(%lu) failed, error=%d!\n",
-				DEFAULT_DSS_CORE_CLK_07V_RATE, ret);
+				  DEFAULT_DSS_CORE_CLK_07V_RATE, ret);
 			return -EINVAL;
 		}
 
 		DRM_INFO("dss_pri_clk:[%lu]->[%llu].\n",
-			DEFAULT_DSS_CORE_CLK_07V_RATE, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
+			 DEFAULT_DSS_CORE_CLK_07V_RATE, (uint64_t)clk_get_rate(ctx->dss_pri_clk));
 	}
 
 	ctx->dss_pxl0_clk = devm_clk_get(dev, "clk_ldi0");
@@ -928,12 +926,12 @@ static int dss_dts_parse(struct platform_device *pdev, struct dss_hw_ctx *ctx)
 		ret = clk_set_rate(ctx->dss_pxl0_clk, DSS_MAX_PXL0_CLK_144M);
 		if (ret < 0) {
 			DRM_ERROR("dss_pxl0_clk clk_set_rate(%lu) failed, error=%d!\n",
-				DSS_MAX_PXL0_CLK_144M, ret);
+				  DSS_MAX_PXL0_CLK_144M, ret);
 			return -EINVAL;
 		}
 
 		DRM_INFO("dss_pxl0_clk:[%lu]->[%llu].\n",
-			DSS_MAX_PXL0_CLK_144M, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
+			 DSS_MAX_PXL0_CLK_144M, (uint64_t)clk_get_rate(ctx->dss_pxl0_clk));
 	}
 
 	/* regulator enable */
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_overlay_utils.c b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_overlay_utils.c
index 128d63d74168..9113937478f5 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_drm_overlay_utils.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_drm_overlay_utils.c
@@ -25,10 +25,12 @@
 #include "kirin9xx_drm_dpe_utils.h"
 #include "kirin9xx_drm_drv.h"
 
-static int mid_array[DSS_CHN_MAX_DEFINE] = {0xb, 0xa, 0x9, 0x8, 0x7, 0x6, 0x5, 0x4, 0x2, 0x1, 0x3, 0x0};
+static const int mid_array[DSS_CHN_MAX_DEFINE] = {
+	0xb, 0xa, 0x9, 0x8, 0x7, 0x6, 0x5, 0x4, 0x2, 0x1, 0x3, 0x0
+};
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
-uint32_t g_dss_module_base[DSS_CHN_MAX_DEFINE][MODULE_CHN_MAX] = {
+#if defined(CONFIG_DRM_HISI_KIRIN970)
+static const u32 g_dss_module_base[DSS_CHN_MAX_DEFINE][MODULE_CHN_MAX] = {
 	// D0
 	{
 	MIF_CH0_OFFSET,   //MODULE_MIF_CHN
@@ -293,7 +295,7 @@ uint32_t g_dss_module_base[DSS_CHN_MAX_DEFINE][MODULE_CHN_MAX] = {
 	},
 };
 
-uint32_t g_dss_module_ovl_base[DSS_MCTL_IDX_MAX][MODULE_OVL_MAX] = {
+static const u32 g_dss_module_ovl_base[DSS_MCTL_IDX_MAX][MODULE_OVL_MAX] = {
 	{DSS_OVL0_OFFSET,
 	DSS_MCTRL_CTL0_OFFSET},
 
@@ -316,7 +318,7 @@ uint32_t g_dss_module_ovl_base[DSS_MCTL_IDX_MAX][MODULE_OVL_MAX] = {
 //SCF_LUT_CHN coef_idx
 int g_scf_lut_chn_coef_idx[DSS_CHN_MAX_DEFINE] = {-1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1};
 
-uint32_t g_dss_module_cap[DSS_CHN_MAX_DEFINE][MODULE_CAP_MAX] = {
+u32 g_dss_module_cap[DSS_CHN_MAX_DEFINE][MODULE_CAP_MAX] = {
 	/* D2 */
 	{0, 0, 1, 0, 0, 0, 1, 0, 0, 0, 1},
 	/* D3 */
@@ -346,22 +348,22 @@ uint32_t g_dss_module_cap[DSS_CHN_MAX_DEFINE][MODULE_CAP_MAX] = {
 };
 
 /* number of smrx idx for each channel */
-uint32_t g_dss_chn_sid_num[DSS_CHN_MAX_DEFINE] = {
-    4, 1, 4, 4, 4, 4, 1, 1, 3, 4, 3, 3
+u32 g_dss_chn_sid_num[DSS_CHN_MAX_DEFINE] = {
+	4, 1, 4, 4, 4, 4, 1, 1, 3, 4, 3, 3
 };
 
 /* start idx of each channel */
 /* smrx_idx = g_dss_smmu_smrx_idx[chn_idx] + (0 ~ g_dss_chn_sid_num[chn_idx]) */
-uint32_t g_dss_smmu_smrx_idx[DSS_CHN_MAX_DEFINE] = {
-    0, 4, 5, 9, 13, 17, 21, 22, 26, 29, 23, 36
+u32 g_dss_smmu_smrx_idx[DSS_CHN_MAX_DEFINE] = {
+	0, 4, 5, 9, 13, 17, 21, 22, 26, 29, 23, 36
 };
 #else
 /*
-** dss_chn_idx
-** DSS_RCHN_D2 = 0,	DSS_RCHN_D3,	DSS_RCHN_V0,	DSS_RCHN_G0,	DSS_RCHN_V1,
-** DSS_RCHN_G1,	DSS_RCHN_D0,	DSS_RCHN_D1,	DSS_WCHN_W0,	DSS_WCHN_W1,
-** DSS_RCHN_V2,   DSS_WCHN_W2,
-*/
+ * dss_chn_idx
+ * DSS_RCHN_D2 = 0,	DSS_RCHN_D3,	DSS_RCHN_V0,	DSS_RCHN_G0,	DSS_RCHN_V1,
+ * DSS_RCHN_G1,	DSS_RCHN_D0,	DSS_RCHN_D1,	DSS_WCHN_W0,	DSS_WCHN_W1,
+ * DSS_RCHN_V2,   DSS_WCHN_W2,
+ */
 u32 g_dss_module_base[DSS_CHN_MAX_DEFINE][MODULE_CHN_MAX] = {
 	/* D0 */
 	{
@@ -676,6 +678,7 @@ u32 g_dss_chn_sid_num[DSS_CHN_MAX_DEFINE] = {
 u32 g_dss_smmu_smrx_idx[DSS_CHN_MAX_DEFINE] = {
 	0, 4, 5, 9, 13, 17, 21, 22, 26, 29, 23, 32
 };
+
 u32 g_dss_mif_sid_map[DSS_CHN_MAX] = {
 	0, 0, 0, 0, 0, 0, 0, 0, 0, 0
 };
@@ -684,7 +687,7 @@ static int hisi_pixel_format_hal2dma(int format)
 {
 	int ret = 0;
 
-	switch(format) {
+	switch (format) {
 	case HISI_FB_PIXEL_FORMAT_RGB_565:
 	case HISI_FB_PIXEL_FORMAT_BGR_565:
 		ret = DMA_PIXEL_FORMAT_RGB_565;
@@ -902,11 +905,10 @@ static int hisi_dss_mif_config(struct dss_hw_ctx *ctx, int chn_idx, bool mmu_ena
 	mif_ch_base = ctx->base +
 		g_dss_module_base[chn_idx][MODULE_MIF_CHN];
 
-	if (!mmu_enable) {
+	if (!mmu_enable)
 		set_reg(mif_ch_base + MIF_CTRL1, 0x1, 1, 5);
-	} else  {
+	else
 		set_reg(mif_ch_base + MIF_CTRL1, 0x00080000, 32, 0);
-	}
 
 	return 0;
 }
@@ -992,11 +994,10 @@ static int hisi_dss_mctl_sys_config(struct dss_hw_ctx *ctx, int chn_idx)
 	set_reg(mctl_sys_base + mctl_rch_ov_oen_offset,
 		((1 << (layer_idx + 1)) | (0x100 << DSS_OVL0)), 32, 0);
 
-	if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
+	if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970)
 		set_reg(mctl_sys_base + MCTL_RCH_OV0_SEL, 0xe, 4, 0);
-	} else {
+	else
 		set_reg(mctl_sys_base + MCTL_RCH_OV0_SEL, 0x8, 4, 0);
-	}
 
 	set_reg(mctl_sys_base + MCTL_RCH_OV0_SEL, chn_idx, 4, (layer_idx + 1) * 4);
 
@@ -1007,7 +1008,7 @@ static int hisi_dss_mctl_sys_config(struct dss_hw_ctx *ctx, int chn_idx)
 }
 
 static int hisi_dss_rdma_config(struct dss_hw_ctx *ctx,
-	const dss_rect_ltrb_t *rect, u32 display_addr, u32 hal_format,
+				const dss_rect_ltrb_t *rect, u32 display_addr, u32 hal_format,
 	u32 bpp, int chn_idx, bool afbcd, bool mmu_enable)
 {
 	void __iomem *rdma_base;
@@ -1036,13 +1037,12 @@ static int hisi_dss_rdma_config(struct dss_hw_ctx *ctx,
 		return -1;
 	}
 
-	if (bpp == 4) {
+	if (bpp == 4)
 		rdma_bpp = 0x5;
-	} else if (bpp == 2) {
+	else if (bpp == 2)
 		rdma_bpp = 0x0;
-	} else {
+	else
 		rdma_bpp = 0x0;
-	}
 
 	rdma_base = ctx->base +
 		g_dss_module_base[chn_idx][MODULE_DMA];
@@ -1066,18 +1066,16 @@ static int hisi_dss_rdma_config(struct dss_hw_ctx *ctx,
 		mm_base_1 = ALIGN_UP(mm_base_1, MMBUF_ADDR_ALIGN);
 
 		if ((((rect->right - rect->left) + 1) & (AFBC_HEADER_ADDR_ALIGN - 1)) ||
-				(((rect->bottom - rect->top) + 1) & (AFBC_BLOCK_ALIGN - 1))) {
-			DRM_ERROR("img width(%d) is not %d bytes aligned, or "
-					"img heigh(%d) is not %d bytes aligned!\n",
-					((rect->right - rect->left) + 1), AFBC_HEADER_ADDR_ALIGN,
-					((rect->bottom - rect->top) + 1), AFBC_BLOCK_ALIGN);
+		    (((rect->bottom - rect->top) + 1) & (AFBC_BLOCK_ALIGN - 1))) {
+			DRM_ERROR("img width(%d) is not %d bytes aligned, or img heigh(%d) is not %d bytes aligned!\n",
+				  ((rect->right - rect->left) + 1), AFBC_HEADER_ADDR_ALIGN,
+				  ((rect->bottom - rect->top) + 1), AFBC_BLOCK_ALIGN);
 		}
 
 		if ((mm_base_0 & (MMBUF_ADDR_ALIGN - 1)) || (mm_base_1 & (MMBUF_ADDR_ALIGN - 1))) {
-			DRM_ERROR("mm_base_0(0x%x) is not %d bytes aligned, or "
-					"mm_base_1(0x%x) is not %d bytes aligned!\n",
-					mm_base_0, MMBUF_ADDR_ALIGN,
-					mm_base_1, MMBUF_ADDR_ALIGN);
+			DRM_ERROR("mm_base_0(0x%x) is not %d bytes aligned, or mm_base_1(0x%x) is not %d bytes aligned!\n",
+				  mm_base_0, MMBUF_ADDR_ALIGN,
+				  mm_base_1, MMBUF_ADDR_ALIGN);
 		}
 		/*header*/
 		afbcd_header_stride = (((rect->right - rect->left) + 1) / AFBC_BLOCK_ALIGN) * AFBC_HEADER_STRIDE_BLOCK;
@@ -1148,7 +1146,8 @@ static int hisi_dss_rdma_config(struct dss_hw_ctx *ctx,
 }
 
 static int hisi_dss_rdfc_config(struct dss_hw_ctx *ctx,
-	const dss_rect_ltrb_t *rect, u32 hal_format, u32 bpp, int chn_idx)
+				const struct dss_rect_ltrb *rect,
+				u32 hal_format, u32 bpp, int chn_idx)
 {
 	void __iomem *rdfc_base;
 
@@ -1235,7 +1234,7 @@ int hisi_dss_ovl_base_config(struct dss_hw_ctx *ctx, u32 xres, u32 yres)
 }
 
 static int hisi_dss_ovl_config(struct dss_hw_ctx *ctx,
-	const dss_rect_ltrb_t *rect, u32 xres, u32 yres)
+			       const dss_rect_ltrb_t *rect, u32 xres, u32 yres)
 {
 	void __iomem *ovl0_base;
 
@@ -1338,8 +1337,8 @@ void hisi_dss_smmu_on(struct dss_hw_ctx *ctx)
 {
 	void __iomem *smmu_base;
 	struct iommu_domain_data *domain_data = NULL;
-	uint32_t phy_pgd_base = 0;
-	uint64_t fama_phy_pgd_base;
+	u32 phy_pgd_base = 0;
+	u64 fama_phy_pgd_base;
 
 	if (!ctx) {
 		DRM_ERROR("ctx is NULL!\n");
@@ -1444,9 +1443,8 @@ void hisifb_mctl_sw_clr(struct dss_crtc *acrtc)
 	mctl_base = ctx->base +
 		g_dss_module_ovl_base[DSS_MCTL0][MODULE_MCTL_BASE];
 
-	if (mctl_base) {
+	if (mctl_base)
 		set_reg(mctl_base + MCTL_CTL_CLEAR, 0x1, 1, 0);
-	}
 
 	while (1) {
 		mctl_status = inp32(mctl_base + MCTL_CTL_STATUS);
@@ -1454,15 +1452,14 @@ void hisifb_mctl_sw_clr(struct dss_crtc *acrtc)
 			is_timeout = (delay_count > 100) ? true : false;
 			delay_count = 0;
 			break;
-		} else {
-			udelay(1);
-			++delay_count;
 		}
+
+		udelay(1);
+		++delay_count;
 	}
 
-	if (is_timeout) {
+	if (is_timeout)
 		DRM_ERROR("mctl_status =0x%x !\n", mctl_status);
-	}
 
 	enable_ldi(acrtc);
 	DRM_INFO("-.\n");
@@ -1479,7 +1476,7 @@ static int hisi_dss_wait_for_complete(struct dss_crtc *acrtc)
 
 REDO:
 	ret = wait_event_interruptible_timeout(ctx->vactive0_end_wq,
-		(prev_vactive0_end != ctx->vactive0_end_flag),
+					       (prev_vactive0_end != ctx->vactive0_end_flag),
 		msecs_to_jiffies(300));
 	if (ret == -ERESTARTSYS) {
 		if (times < 50) {
@@ -1549,13 +1546,12 @@ void hisi_fb_pan_display(struct drm_plane *plane)
 	rect.bottom = src_h - 1;
 	hal_fmt = HISI_FB_PIXEL_FORMAT_BGRA_8888;//dss_get_format(fb->pixel_format);
 
-	DRM_DEBUG_DRIVER("channel%d: src:(%d,%d, %dx%d) crtc:(%d,%d, %dx%d), rect(%d,%d,%d,%d),"
-		"fb:%dx%d, pixel_format=%d, stride=%d, paddr=0x%x, bpp=%d.\n",
-		chn_idx, src_x, src_y, src_w, src_h,
-		crtc_x, crtc_y, crtc_w, crtc_h,
-		rect.left, rect.top, rect.right, rect.bottom,
-		fb->width, fb->height, hal_fmt,
-		stride, display_addr, bpp);
+	DRM_DEBUG_DRIVER("channel%d: src:(%d,%d, %dx%d) crtc:(%d,%d, %dx%d), rect(%d,%d,%d,%d),fb:%dx%d, pixel_format=%d, stride=%d, paddr=0x%x, bpp=%d.\n",
+			 chn_idx, src_x, src_y, src_w, src_h,
+			 crtc_x, crtc_y, crtc_w, crtc_h,
+			 rect.left, rect.top, rect.right, rect.bottom,
+			 fb->width, fb->height, hal_fmt,
+			 stride, display_addr, bpp);
 
 	hfp = mode->hsync_start - mode->hdisplay;
 	hbp = mode->htotal - mode->hsync_end;
@@ -1582,7 +1578,8 @@ void hisi_fb_pan_display(struct drm_plane *plane)
 	hisi_dss_wait_for_complete(acrtc);
 }
 
-void hisi_dss_online_play(struct kirin_fbdev *fbdev, struct drm_plane *plane, drm_dss_layer_t *layer)
+void hisi_dss_online_play(struct kirin_fbdev *fbdev, struct drm_plane *plane,
+			  struct drm_dss_layer *layer)
 {
 	struct drm_plane_state *state = plane->state;
 	struct drm_display_mode *mode;
@@ -1620,11 +1617,10 @@ void hisi_dss_online_play(struct kirin_fbdev *fbdev, struct drm_plane *plane, dr
 	rect.top = 0;
 	rect.bottom = src_h - 1;
 
-	DRM_DEBUG("channel%d: src:(%dx%d) rect(%d,%d,%d,%d),"
-		"pixel_format=%d, stride=%d, paddr=0x%x, bpp=%d.\n",
-		chn_idx, src_w, src_h,
-		rect.left, rect.top, rect.right, rect.bottom,
-		hal_fmt, stride, display_addr, bpp);
+	DRM_DEBUG("channel%d: src:(%dx%d) rect(%d,%d,%d,%d),pixel_format=%d, stride=%d, paddr=0x%x, bpp=%d.\n",
+		  chn_idx, src_w, src_h,
+		  rect.left, rect.top, rect.right, rect.bottom,
+		  hal_fmt, stride, display_addr, bpp);
 
 	hfp = mode->hsync_start - mode->hdisplay;
 	hbp = mode->htotal - mode->hsync_end;
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c
index 0612ca149c4b..5411113f148c 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c
@@ -31,16 +31,15 @@
 #include <drm/drm_panel.h>
 #include <drm/drm_probe_helper.h>
 
-
 #include "kirin9xx_dw_dsi_reg.h"
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 #include "kirin970_dpe_reg.h"
 #else
 #include "kirin960_dpe_reg.h"
 #endif
 #include "kirin9xx_drm_drv.h"
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 #define DTS_COMP_DSI_NAME "hisilicon,kirin970-dsi"
 #else
 #define DTS_COMP_DSI_NAME "hisilicon,kirin960-dsi"
@@ -123,18 +122,18 @@ struct mipi_phy_params {
 	u32 load_command;
 
 	// for CDPHY
-	uint32_t rg_cphy_div;	//Q
-	uint32_t rg_div;		//M 0x4A[7:0]
-	uint32_t rg_pre_div;	//N 0x49[0]
-	uint32_t rg_320m;		//0x48[2]
-	uint32_t rg_2p5g;		//0x48[1]
-	uint32_t rg_0p8v;		//0x48[0]
-	uint32_t rg_lpf_r;		//0x46[5:4]
-	uint32_t rg_cp;			//0x46[3:0]
-	uint32_t t_prepare;
-	uint32_t t_lpx;
-	uint32_t t_prebegin;
-	uint32_t t_post;
+	u32 rg_cphy_div;	//Q
+	u32 rg_div;		//M 0x4A[7:0]
+	u32 rg_pre_div;	//N 0x49[0]
+	u32 rg_320m;		//0x48[2]
+	u32 rg_2p5g;		//0x48[1]
+	u32 rg_0p8v;		//0x48[0]
+	u32 rg_lpf_r;		//0x46[5:4]
+	u32 rg_cp;			//0x46[3:0]
+	u32 t_prepare;
+	u32 t_lpx;
+	u32 t_prebegin;
+	u32 t_post;
 };
 
 struct dsi_hw_ctx {
@@ -208,9 +207,9 @@ struct ldi_panel_info {
 	u32 h_pulse_width;
 
 	/*
-	** note: vbp > 8 if used overlay compose,
-	** also lcd vbp > 8 in lcd power on sequence
-	*/
+	 * note: vbp > 8 if used overlay compose,
+	 * also lcd vbp > 8 in lcd power on sequence
+	 */
 	u32 v_back_porch;
 	u32 v_front_porch;
 	u32 v_pulse_width;
@@ -328,7 +327,7 @@ void dsi_set_output_client(struct drm_device *dev)
 }
 EXPORT_SYMBOL(dsi_set_output_client);
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 			      struct mipi_phy_params *phy_ctrl, u32 id)
 {
@@ -374,9 +373,9 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 		return;
 
 	if (mode->clock > 80000)
-	    dsi->client[id].lanes = 4;
+		dsi->client[id].lanes = 4;
 	else
-	    dsi->client[id].lanes = 3;
+		dsi->client[id].lanes = 3;
 
 	if (dsi->client[id].phy_clock)
 		dphy_req_kHz = dsi->client[id].phy_clock;
@@ -390,14 +389,15 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 	//chip spec :
 	//If the output data rate is below 320 Mbps, RG_BNAD_SEL should be set to 1.
 	//At this mode a post divider of 1/4 will be applied to VCO.
-	if ((320 <= lane_clock) && (lane_clock <= 2500)) {
+	if ((lane_clock >= 320) && (lane_clock <= 2500)) {
 		phy_ctrl->rg_band_sel = 0;
 		vco_div = 1;
-	} else if ((80 <= lane_clock) && (lane_clock < 320)) {
+	} else if ((lane_clock >= 80) && (lane_clock < 320)) {
 		phy_ctrl->rg_band_sel = 1;
 		vco_div = 4;
 	} else {
-		DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n", lane_clock);
+		DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n",
+			  lane_clock);
 	}
 
 	m_n_int = lane_clock * vco_div * 1000000UL / DEFAULT_MIPI_CLK_RATE;
@@ -408,13 +408,13 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 	m_pll = (u32)(lane_clock * vco_div * n_pll * 1000000UL / DEFAULT_MIPI_CLK_RATE);
 
 	lane_clock = m_pll * (DEFAULT_MIPI_CLK_RATE / n_pll) / vco_div;
-	if (lane_clock > 750000000) {
+	if (lane_clock > 750000000)
 		phy_ctrl->rg_cp = 3;
-	} else if ((80000000 <= lane_clock) && (lane_clock <= 750000000)) {
+	else if ((lane_clock >= 80000000) && (lane_clock <= 750000000))
 		phy_ctrl->rg_cp = 1;
-	} else {
-		DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n", lane_clock);
-	}
+	else
+		DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n",
+			  lane_clock);
 
 	//chip spec :
 	phy_ctrl->rg_pre_div = n_pll - 1;
@@ -463,7 +463,7 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 
 	// D-PHY Specification : 40ns + 4*UI <= data_t_hs_prepare <= 85ns + 6*UI
 	// clocked by TXBYTECLKHS
-	data_t_hs_prepare = 400 * accuracy + 4*ui;
+	data_t_hs_prepare = 400 * accuracy + 4 * ui;
 	// D-PHY chip spec : clk_t_lpx + clk_t_hs_prepare > 200ns
 	// D-PHY Specification : clk_t_lpx >= 50ns
 	// clocked by TXBYTECLKHS
@@ -497,7 +497,7 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 
 	phy_ctrl->clk_post_delay = phy_ctrl->data_t_hs_trial + ROUND1(clk_post, unit_tx_byte_clk_hs);
 	phy_ctrl->data_pre_delay = phy_ctrl->clk_pre_delay + 2 + phy_ctrl->clk_t_lpx +
-		phy_ctrl->clk_t_hs_prepare + phy_ctrl->clk_t_hs_zero + 8 + ROUND1(clk_pre, unit_tx_byte_clk_hs) ;
+		phy_ctrl->clk_t_hs_prepare + phy_ctrl->clk_t_hs_zero + 8 + ROUND1(clk_pre, unit_tx_byte_clk_hs);
 
 	phy_ctrl->clk_lane_lp2hs_time = phy_ctrl->clk_pre_delay + phy_ctrl->clk_t_lpx + phy_ctrl->clk_t_hs_prepare +
 		phy_ctrl->clk_t_hs_zero + 5 + 7;
@@ -514,7 +514,7 @@ static void get_dsi_dphy_ctrl(struct dw_dsi *dsi,
 		(uint32_t)(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk + 1) :
 		(uint32_t)(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk);
 
-	DRM_DEBUG("DPHY clock_lane and data_lane config : \n"
+	DRM_DEBUG("DPHY clock_lane and data_lane config :\n"
 		"lane_clock = %llu, n_pll=%d, m_pll=%d\n"
 		"rg_cp=%d\n"
 		"rg_band_sel=%d\n"
@@ -608,10 +608,12 @@ static void get_dsi_phy_ctrl(struct dw_dsi *dsi,
 	bpp = mipi_dsi_pixel_format_to_bpp(dsi->client[id].format);
 	if (bpp < 0)
 		return;
+
 	if (mode->clock > 80000)
-	    dsi->client[id].lanes = 4;
+		dsi->client[id].lanes = 4;
 	else
-	    dsi->client[id].lanes = 3;
+		dsi->client[id].lanes = 3;
+
 	if (dsi->client[id].phy_clock)
 		dphy_req_kHz = dsi->client[id].phy_clock;
 	else
@@ -621,20 +623,22 @@ static void get_dsi_phy_ctrl(struct dw_dsi *dsi,
 	DRM_INFO("Expected : lane_clock = %llu M\n", lane_clock);
 
 	/************************  PLL parameters config  *********************/
-	/*chip spec :
-		If the output data rate is below 320 Mbps,
-		RG_BNAD_SEL should be set to 1.
-		At this mode a post divider of 1/4 will be applied to VCO.
-	*/
-	if ((320 <= lane_clock) && (lane_clock <= 2500)) {
+
+	/*
+	 * chip spec :
+	 *	If the output data rate is below 320 Mbps,
+	 *	RG_BNAD_SEL should be set to 1.
+	 *	At this mode a post divider of 1/4 will be applied to VCO.
+	 */
+	if ((lane_clock >= 320) && (lane_clock <= 2500)) {
 		phy_ctrl->rg_band_sel = 0;	/*0x1E[2]*/
 		vco_div = 1;
-	} else if ((80 <= lane_clock) && (lane_clock < 320)) {
+	} else if ((lane_clock >= 80) && (lane_clock < 320)) {
 		phy_ctrl->rg_band_sel = 1;
 		vco_div = 4;
 	} else {
 		DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M\n",
-			lane_clock);
+			  lane_clock);
 	}
 
 	m_n_int = lane_clock * vco_div * 1000000UL / DEFAULT_MIPI_CLK_RATE;
@@ -824,7 +828,7 @@ static void get_dsi_phy_ctrl(struct dw_dsi *dsi,
 
 	phy_ctrl->clk_post_delay = phy_ctrl->data_t_hs_trial + ROUND1(clk_post, unit_tx_byte_clk_hs);
 	phy_ctrl->data_pre_delay = clk_pre_delay_reality + phy_ctrl->clk_t_lpx +
-		phy_ctrl->clk_t_hs_prepare + clk_t_hs_zero_reality + ROUND1(clk_pre, unit_tx_byte_clk_hs) ;
+		phy_ctrl->clk_t_hs_prepare + clk_t_hs_zero_reality + ROUND1(clk_pre, unit_tx_byte_clk_hs);
 
 	clk_post_delay_reality = phy_ctrl->clk_post_delay + 4;
 	data_pre_delay_reality = phy_ctrl->data_pre_delay + 2;
@@ -844,7 +848,7 @@ static void get_dsi_phy_ctrl(struct dw_dsi *dsi,
 		(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk + 1) :
 		(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk);
 
-	DRM_DEBUG("PHY clock_lane and data_lane config : \n"
+	DRM_DEBUG("PHY clock_lane and data_lane config :\n"
 		"rg_vrefsel_vcm=%u\n"
 		"clk_pre_delay=%u\n"
 		"clk_post_delay=%u\n"
@@ -949,8 +953,8 @@ static void mipi_config_dphy_spec1v2_parameter(struct dw_dsi *dsi,
 					       char __iomem *mipi_dsi_base,
 					       u32 id)
 {
-	uint32_t i;
-	uint32_t addr = 0;
+	u32 i;
+	u32 addr = 0;
 	u32 lanes;
 
 	lanes =  dsi->client[id].lanes - 1;
@@ -979,7 +983,7 @@ static void mipi_config_dphy_spec1v2_parameter(struct dw_dsi *dsi,
 	dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_TRAIL, DSS_REDUCE(dsi->phy.clk_t_hs_trial));
 
 	for (i = 0; i <= 4; i++) {
-		if (lanes == 2 && i == 1) /*init mipi dsi 3 lanes shoud skip lane3*/
+		if (lanes == 2 && i == 1) /*init mipi dsi 3 lanes should skip lane3*/
 			i++;
 
 		if (i == 2) /* skip clock lane*/
@@ -1009,7 +1013,7 @@ static void mipi_config_dphy_spec1v2_parameter(struct dw_dsi *dsi,
 		addr = MIPIDSI_PHY_TST_DATA_TRAIL + (i << 5);
 		dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_t_hs_trial));
 
-		DRM_DEBUG("DPHY spec1v2 config : \n"
+		DRM_DEBUG("DPHY spec1v2 config :\n"
 			"addr=0x%x\n"
 			"clk_pre_delay=%u\n"
 			"clk_t_hs_trial=%u\n"
@@ -1039,15 +1043,14 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 	dss_rect_t rect;
 	u32 cmp_stopstate_val = 0;
 	u32 lanes;
-#if !defined (CONFIG_DRM_HISI_KIRIN970)
+#if !defined(CONFIG_DRM_HISI_KIRIN970)
 	int i = 0;
 #endif
 
 	WARN_ON(!dsi);
 	WARN_ON(!mipi_dsi_base);
 
-	DRM_INFO("dsi_mipi_init, id=%d\n", id);
-
+	DRM_INFO("%s: id=%d\n", __func__, id);
 
 	mipi = &dsi->mipi;
 
@@ -1058,7 +1061,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 
 	memset(&dsi->phy, 0, sizeof(struct mipi_phy_params));
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	get_dsi_dphy_ctrl(dsi, &dsi->phy, id);
 #else
 	get_dsi_phy_ctrl(dsi, &dsi->phy, id);
@@ -1081,7 +1084,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 	outp32(mipi_dsi_base + MIPIDSI_PHY_TST_CTRL0_OFFSET, 0x00000001);
 	outp32(mipi_dsi_base + MIPIDSI_PHY_TST_CTRL0_OFFSET, 0x00000000);
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	dsi_phy_tst_set(mipi_dsi_base, 0x0042, 0x21);
 	//PLL configuration I
 	dsi_phy_tst_set(mipi_dsi_base, 0x0046, dsi->phy.rg_cp + (dsi->phy.rg_lpf_r << 4));
@@ -1113,7 +1116,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 #else
 	/* physical configuration PLL I*/
 	dsi_phy_tst_set(mipi_dsi_base, 0x14,
-		(dsi->phy.rg_pll_fbd_s << 4) + (dsi->phy.rg_pll_enswc << 3) +
+			(dsi->phy.rg_pll_fbd_s << 4) + (dsi->phy.rg_pll_enswc << 3) +
 		(dsi->phy.rg_pll_enbwt << 2) + dsi->phy.rg_pll_chp);
 
 	/* physical configuration PLL II, M*/
@@ -1121,7 +1124,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 
 	/* physical configuration PLL III*/
 	dsi_phy_tst_set(mipi_dsi_base, 0x16,
-		(dsi->phy.rg_pll_cp << 5) + (dsi->phy.rg_pll_lpf_cs << 4) +
+			(dsi->phy.rg_pll_cp << 5) + (dsi->phy.rg_pll_lpf_cs << 4) +
 		dsi->phy.rg_pll_refsel);
 
 	/* physical configuration PLL IV, N*/
@@ -1132,7 +1135,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 
 	/* MISC AFE Configuration*/
 	dsi_phy_tst_set(mipi_dsi_base, 0x1E,
-		(dsi->phy.rg_pll_cp_p << 5) + (dsi->phy.reload_sel << 4) +
+			(dsi->phy.rg_pll_cp_p << 5) + (dsi->phy.reload_sel << 4) +
 		(dsi->phy.rg_phase_gen_en << 3) + (dsi->phy.rg_band_sel << 2) +
 		(dsi->phy.pll_power_down << 1) + dsi->phy.pll_register_override);
 
@@ -1205,7 +1208,7 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 
 	if (!is_ready) {
 		DRM_INFO("phylock is not ready!MIPIDSI_PHY_STATUS_OFFSET=0x%x.\n",
-			tmp);
+			 tmp);
 	}
 
 	if (lanes >= DSI_4_LANES)
@@ -1229,19 +1232,20 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 
 	if (!is_ready) {
 		DRM_INFO("phystopstateclklane is not ready! MIPIDSI_PHY_STATUS_OFFSET=0x%x.\n",
-			tmp);
+			 tmp);
 	}
 
 	/*************************Configure the DPHY end*************************/
 
-	/* phy_stop_wait_time*/
+	/* phy_stop_wait_time */
 	set_reg(mipi_dsi_base + MIPIDSI_PHY_IF_CFG_OFFSET, dsi->phy.phy_stop_wait_time, 8, 8);
 
 	/*--------------configuring the DPI packet transmission----------------*/
+
 	/*
-	** 2. Configure the DPI Interface:
-	** This defines how the DPI interface interacts with the controller.
-	*/
+	 * 2. Configure the DPI Interface:
+	 * This defines how the DPI interface interacts with the controller.
+	 */
 	set_reg(mipi_dsi_base + MIPIDSI_DPI_VCID_OFFSET, mipi->vc, 2, 0);
 	set_reg(mipi_dsi_base + MIPIDSI_DPI_COLOR_CODING_OFFSET, mipi->color_mode, 4, 0);
 
@@ -1252,10 +1256,10 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 	set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, 0x0, 1, 4);
 
 	/*
-	** 3. Select the Video Transmission Mode:
-	** This defines how the processor requires the video line to be
-	** transported through the DSI link.
-	*/
+	 * 3. Select the Video Transmission Mode:
+	 * This defines how the processor requires the video line to be
+	 * transported through the DSI link.
+	 */
 	/* video mode: low power mode*/
 	set_reg(mipi_dsi_base + MIPIDSI_VID_MODE_CFG_OFFSET, 0x3f, 6, 8);
 	/* set_reg(mipi_dsi_base + MIPIDSI_VID_MODE_CFG_OFFSET, 0x0, 1, 14); */
@@ -1273,12 +1277,12 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 	set_reg(mipi_dsi_base + MIPIDSI_PCKHDL_CFG_OFFSET, 0x1, 1, 2);
 
 	/*
-	** 4. Define the DPI Horizontal timing configuration:
-	**
-	** Hsa_time = HSA*(PCLK period/Clk Lane Byte Period);
-	** Hbp_time = HBP*(PCLK period/Clk Lane Byte Period);
-	** Hline_time = (HSA+HBP+HACT+HFP)*(PCLK period/Clk Lane Byte Period);
-	*/
+	 * 4. Define the DPI Horizontal timing configuration:
+	 *
+	 * Hsa_time = HSA*(PCLK period/Clk Lane Byte Period);
+	 * Hbp_time = HBP*(PCLK period/Clk Lane Byte Period);
+	 * Hline_time = (HSA+HBP+HACT+HFP)*(PCLK period/Clk Lane Byte Period);
+	 */
 	pixel_clk = dsi->cur_mode.clock * 1000;
 	/*htot = dsi->cur_mode.htotal;*/
 	/*vtot = dsi->cur_mode.vtotal;*/
@@ -1298,29 +1302,39 @@ static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 		rect.w + dsi->ldi.h_front_porch) * dsi->phy.lane_byte_clk, pixel_clk);
 
 	DRM_INFO("hsa_time=%d, hbp_time=%d, hline_time=%d\n",
-	    hsa_time, hbp_time, hline_time);
+		 hsa_time, hbp_time, hline_time);
 	DRM_INFO("lane_byte_clk=%llu, pixel_clk=%llu\n",
-	    dsi->phy.lane_byte_clk, pixel_clk);
+		 dsi->phy.lane_byte_clk, pixel_clk);
 	set_reg(mipi_dsi_base + MIPIDSI_VID_HSA_TIME_OFFSET, hsa_time, 12, 0);
 	set_reg(mipi_dsi_base + MIPIDSI_VID_HBP_TIME_OFFSET, hbp_time, 12, 0);
 	set_reg(mipi_dsi_base + MIPIDSI_VID_HLINE_TIME_OFFSET, hline_time, 15, 0);
 
 	/* Define the Vertical line configuration*/
-	set_reg(mipi_dsi_base + MIPIDSI_VID_VSA_LINES_OFFSET, dsi->ldi.v_pulse_width, 10, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_VID_VBP_LINES_OFFSET, dsi->ldi.v_back_porch, 10, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_VID_VFP_LINES_OFFSET, dsi->ldi.v_front_porch, 10, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_VID_VACTIVE_LINES_OFFSET, rect.h, 14, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_TO_CNT_CFG_OFFSET, 0x7FF, 16, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_VID_VSA_LINES_OFFSET,
+		dsi->ldi.v_pulse_width, 10, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_VID_VBP_LINES_OFFSET,
+		dsi->ldi.v_back_porch, 10, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_VID_VFP_LINES_OFFSET,
+		dsi->ldi.v_front_porch, 10, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_VID_VACTIVE_LINES_OFFSET,
+		rect.h, 14, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_TO_CNT_CFG_OFFSET,
+		0x7FF, 16, 0);
 
 	/* Configure core's phy parameters*/
-	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET, dsi->phy.clk_lane_lp2hs_time, 10, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET, dsi->phy.clk_lane_hs2lp_time, 10, 16);
+	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET,
+		dsi->phy.clk_lane_lp2hs_time, 10, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET,
+		dsi->phy.clk_lane_hs2lp_time, 10, 16);
 
-	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_RD_CFG_OFFSET, 0x7FFF, 15, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET, dsi->phy.data_lane_lp2hs_time, 10, 0);
-	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET, dsi->phy.data_lane_hs2lp_time, 10, 16);
+	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_RD_CFG_OFFSET,
+		0x7FFF, 15, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET,
+		dsi->phy.data_lane_lp2hs_time, 10, 0);
+	set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET,
+		dsi->phy.data_lane_hs2lp_time, 10, 16);
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	//16~19bit:pclk_en, pclk_sel, dpipclk_en, dpipclk_sel
 	set_reg(mipi_dsi_base + MIPIDSI_CLKMGR_CFG_OFFSET, 0x5, 4, 16);
 	//0:dphy
@@ -1368,7 +1382,7 @@ static int mipi_dsi_on_sub1(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 	dsi_mipi_init(dsi, mipi_dsi_base, id);
 
 	/* dsi memory init */
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	outp32(mipi_dsi_base + DSI_MEM_CTRL, 0x02600008);
 #endif
 
@@ -1388,6 +1402,7 @@ static int mipi_dsi_on_sub1(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
 static int mipi_dsi_on_sub2(struct dw_dsi *dsi, char __iomem *mipi_dsi_base)
 {
 	u64 pctrl_dphytx_stopcnt = 0;
+
 	WARN_ON(!mipi_dsi_base);
 
 	/* switch to video mode */
@@ -1458,7 +1473,7 @@ static void dsi_encoder_enable(struct drm_encoder *encoder)
 }
 
 static enum drm_mode_status dsi_encoder_mode_valid(struct drm_encoder *encoder,
-					const struct drm_display_mode *mode)
+						   const struct drm_display_mode *mode)
 
 {
 	const struct drm_crtc_helper_funcs *crtc_funcs;
@@ -1675,7 +1690,7 @@ static int dsi_dcs_long_write(void __iomem *base,
 }
 
 static ssize_t dsi_host_transfer(struct mipi_dsi_host *host,
-				    const struct mipi_dsi_msg *msg)
+				 const struct mipi_dsi_msg *msg)
 {
 	struct dw_dsi *dsi = host_to_dsi(host);
 	struct dsi_hw_ctx *ctx = dsi->ctx;
@@ -1718,12 +1733,12 @@ static int dsi_host_init(struct device *dev, struct dw_dsi *dsi)
 	mipi->vc = 0;
 	mipi->color_mode = DSI_24BITS_1;
 	mipi->clk_post_adjust = 120;
-	mipi->clk_pre_adjust= 0;
-	mipi->clk_t_hs_prepare_adjust= 0;
-	mipi->clk_t_lpx_adjust= 0;
-	mipi->clk_t_hs_trial_adjust= 0;
-	mipi->clk_t_hs_exit_adjust= 0;
-	mipi->clk_t_hs_zero_adjust= 0;
+	mipi->clk_pre_adjust = 0;
+	mipi->clk_t_hs_prepare_adjust = 0;
+	mipi->clk_t_lpx_adjust = 0;
+	mipi->clk_t_hs_trial_adjust = 0;
+	mipi->clk_t_hs_exit_adjust = 0;
+	mipi->clk_t_hs_zero_adjust = 0;
 
 	dsi->ldi.data_en_plr = 0;
 	dsi->ldi.vsync_plr = 0;
@@ -1762,7 +1777,7 @@ dsi_connector_best_encoder(struct drm_connector *connector)
 	return &dsi->encoder;
 }
 
-static struct drm_connector_helper_funcs dsi_connector_helper_funcs = {
+static const struct drm_connector_helper_funcs dsi_connector_helper_funcs = {
 	.get_modes = dsi_connector_get_modes,
 	.mode_valid = dsi_connector_mode_valid,
 	.best_encoder = dsi_connector_best_encoder,
@@ -1825,6 +1840,7 @@ static int dsi_connector_init(struct drm_device *dev, struct dw_dsi *dsi)
 	DRM_INFO("connector init\n");
 	return 0;
 }
+
 static int dsi_bind(struct device *dev, struct device *master, void *data)
 {
 	struct dsi_data *ddata = dev_get_drvdata(dev);
@@ -1964,26 +1980,26 @@ static int dsi_parse_dt(struct platform_device *pdev, struct dw_dsi *dsi)
 	np = of_find_compatible_node(NULL, NULL, DTS_COMP_DSI_NAME);
 	if (!np) {
 		DRM_ERROR("NOT FOUND device node %s!\n",
-				   DTS_COMP_DSI_NAME);
+			  DTS_COMP_DSI_NAME);
 		return -ENXIO;
 	}
 
 	ctx->base = of_iomap(np, 0);
 	if (!(ctx->base)) {
-		DRM_ERROR ("failed to get dsi base resource.\n");
+		DRM_ERROR("failed to get dsi base resource.\n");
 		return -ENXIO;
 	}
 
 	ctx->peri_crg_base = of_iomap(np, 1);
 	if (!(ctx->peri_crg_base)) {
-		DRM_ERROR ("failed to get peri_crg_base resource.\n");
+		DRM_ERROR("failed to get peri_crg_base resource.\n");
 		return -ENXIO;
 	}
 
-#if defined (CONFIG_DRM_HISI_KIRIN970)
+#if defined(CONFIG_DRM_HISI_KIRIN970)
 	ctx->pctrl_base = of_iomap(np, 2);
 	if (!(ctx->pctrl_base)) {
-		DRM_ERROR ("failed to get dss pctrl_base resource.\n");
+		DRM_ERROR("failed to get dss pctrl_base resource.\n");
 		return -ENXIO;
 	}
 #endif
@@ -1996,7 +2012,7 @@ static int dsi_parse_dt(struct platform_device *pdev, struct dw_dsi *dsi)
 	dsi->cur_client = OUT_PANEL;
 	dsi->attached_client = dsi->cur_client;
 
-	DRM_INFO("dsi  cur_client is %d  <0->hdmi;1->panel> \n", dsi->cur_client);
+	DRM_INFO("dsi  cur_client is %d  <0->hdmi;1->panel>\n", dsi->cur_client);
 	/*dis-reset*/
 	/*ip_reset_dis_dsi0, ip_reset_dis_dsi1*/
 	outp32(ctx->peri_crg_base + PERRSTDIS3, 0x30000000);
@@ -2010,12 +2026,12 @@ static int dsi_parse_dt(struct platform_device *pdev, struct dw_dsi *dsi)
 	ret = clk_set_rate(ctx->dss_dphy0_ref_clk, DEFAULT_MIPI_CLK_RATE);
 	if (ret < 0) {
 		DRM_ERROR("dss_dphy0_ref_clk clk_set_rate(%lu) failed, error=%d!\n",
-			DEFAULT_MIPI_CLK_RATE, ret);
+			  DEFAULT_MIPI_CLK_RATE, ret);
 		return -EINVAL;
 	}
 
 	DRM_DEBUG("dss_dphy0_ref_clk:[%lu]->[%lu].\n",
-		DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_ref_clk));
+		  DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_ref_clk));
 
 	ctx->dss_dphy0_cfg_clk = devm_clk_get(&pdev->dev, "clk_txdphy0_cfg");
 	if (IS_ERR(ctx->dss_dphy0_cfg_clk)) {
@@ -2026,12 +2042,12 @@ static int dsi_parse_dt(struct platform_device *pdev, struct dw_dsi *dsi)
 	ret = clk_set_rate(ctx->dss_dphy0_cfg_clk, DEFAULT_MIPI_CLK_RATE);
 	if (ret < 0) {
 		DRM_ERROR("dss_dphy0_cfg_clk clk_set_rate(%lu) failed, error=%d!\n",
-			DEFAULT_MIPI_CLK_RATE, ret);
+			  DEFAULT_MIPI_CLK_RATE, ret);
 		return -EINVAL;
 	}
 
 	DRM_DEBUG("dss_dphy0_cfg_clk:[%lu]->[%lu].\n",
-		DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_cfg_clk));
+		  DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_cfg_clk));
 
 	ctx->dss_pclk_dsi0_clk = devm_clk_get(&pdev->dev, "pclk_dsi0");
 	if (IS_ERR(ctx->dss_pclk_dsi0_clk)) {
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_fb_panel.h b/drivers/staging/hikey9xx/gpu/kirin9xx_fb_panel.h
index 69c7f1fd7ccf..2ebf4dd9f09e 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_fb_panel.h
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_fb_panel.h
@@ -41,7 +41,6 @@
 /* vcc name */
 #define REGULATOR_PDP_NAME	"ldo3"
 
-
 /* irq name */
 #define IRQ_PDP_NAME	"irq_pdp"
 #define IRQ_SDP_NAME	"irq_sdp"
@@ -91,9 +90,9 @@ enum MIPI_LP11_MODE {
 
 /* resource desc */
 struct resource_desc {
-	uint32_t flag;
+	u32 flag;
 	char *name;
-	uint32_t *value;
+	u32 *value;
 };
 
 /* dtype for vcc */
@@ -136,6 +135,7 @@ struct pinctrl_data {
 	struct pinctrl_state *pinctrl_def;
 	struct pinctrl_state *pinctrl_idle;
 };
+
 struct pinctrl_cmd_desc {
 	int dtype;
 	struct pinctrl_data *pctrl_data;
@@ -156,7 +156,7 @@ struct gpio_desc {
 	int waittype;
 	int wait;
 	char *label;
-	uint32_t *gpio;
+	u32 *gpio;
 	int value;
 };
 
@@ -170,15 +170,15 @@ enum bl_control_mode {
 	COMMON_IC_MODE = 8,
 };
 
-/*******************************************************************************
-** FUNCTIONS PROTOTYPES
-*/
+/*
+ * FUNCTIONS PROTOTYPES
+ */
 #define MIPI_DPHY_NUM	(2)
 
-extern uint32_t g_dts_resouce_ready;
+extern u32 g_dts_resource_ready;
 
 int resource_cmds_tx(struct platform_device *pdev,
-	struct resource_desc *cmds, int cnt);
+		     struct resource_desc *cmds, int cnt);
 int vcc_cmds_tx(struct platform_device *pdev, struct vcc_desc *cmds, int cnt);
 int pinctrl_cmds_tx(struct platform_device *pdev, struct pinctrl_cmd_desc *cmds, int cnt);
 int gpio_cmds_tx(struct gpio_desc *cmds, int cnt);
diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_pwm.c b/drivers/staging/hikey9xx/gpu/kirin9xx_pwm.c
index f2b9cfe8fa54..519e8f0232de 100644
--- a/drivers/staging/hikey9xx/gpu/kirin9xx_pwm.c
+++ b/drivers/staging/hikey9xx/gpu/kirin9xx_pwm.c
@@ -66,7 +66,6 @@ static struct pinctrl_cmd_desc pwm_pinctrl_finit_cmds[] = {
 
 #define PWM_OUT_PRECISION	(800)
 
-
 int pinctrl_cmds_tx(struct platform_device *pdev, struct pinctrl_cmd_desc *cmds, int cnt)
 {
 	int ret = 0;
@@ -77,13 +76,13 @@ int pinctrl_cmds_tx(struct platform_device *pdev, struct pinctrl_cmd_desc *cmds,
 	cm = cmds;
 
 	for (i = 0; i < cnt; i++) {
-		if (cm == NULL) {
+		if (!cm) {
 			DRM_ERROR("cm is null! index=%d\n", i);
 			continue;
 		}
 
 		if (cm->dtype == DTYPE_PINCTRL_GET) {
-			if (NULL == pdev) {
+			if (!pdev) {
 				DRM_ERROR("pdev is NULL");
 				return -EINVAL;
 			}
@@ -158,7 +157,7 @@ int pinctrl_cmds_tx(struct platform_device *pdev, struct pinctrl_cmd_desc *cmds,
 int hisi_pwm_set_backlight(struct backlight_device *bl, uint32_t bl_level)
 {
 	char __iomem *pwm_base = NULL;
-	uint32_t bl_max = bl->props.max_brightness;
+	u32 bl_max = bl->props.max_brightness;
 
 	pwm_base = hisifd_pwm_base;
 	if (!pwm_base) {
@@ -173,9 +172,8 @@ int hisi_pwm_set_backlight(struct backlight_device *bl, uint32_t bl_level)
 		return -EINVAL;
 	}
 
-	if (bl_level > bl_max) {
+	if (bl_level > bl_max)
 		bl_level = bl_max;
-	}
 
 	bl_level = (bl_level * PWM_OUT_PRECISION) / bl_max;
 
@@ -232,11 +230,11 @@ int hisi_pwm_on(void)
 			return -EINVAL;
 		}
 
-		DRM_INFO("dss_pwm_clk clk_enable successed, ret=%d!\n", ret);
+		DRM_INFO("dss_pwm_clk clk_enable succeeded, ret=%d!\n", ret);
 	}
 
 	ret = pinctrl_cmds_tx(g_pwm_pdev, pwm_pinctrl_normal_cmds,
-		ARRAY_SIZE(pwm_pinctrl_normal_cmds));
+			      ARRAY_SIZE(pwm_pinctrl_normal_cmds));
 
 	//if enable PWM, please set IOMG_004 in IOC_AO module
 	//set IOMG_004: select PWM_OUT0
@@ -269,7 +267,7 @@ int hisi_pwm_off(void)
 		return 0;
 
 	ret = pinctrl_cmds_tx(g_pwm_pdev, pwm_pinctrl_lowpower_cmds,
-		ARRAY_SIZE(pwm_pinctrl_lowpower_cmds));
+			      ARRAY_SIZE(pwm_pinctrl_lowpower_cmds));
 
 	clk_tmp = g_pwm_clk;
 	if (clk_tmp) {
@@ -290,7 +288,7 @@ static int hisi_pwm_probe(struct platform_device *pdev)
 	struct device_node *np = NULL;
 	int ret = 0;
 
-	if (NULL == pdev) {
+	if (!pdev) {
 		DRM_ERROR("pdev is NULL");
 		return -EINVAL;
 	}
@@ -320,7 +318,7 @@ static int hisi_pwm_probe(struct platform_device *pdev)
 
 	/* pwm pinctrl init */
 	ret = pinctrl_cmds_tx(pdev, pwm_pinctrl_init_cmds,
-		ARRAY_SIZE(pwm_pinctrl_init_cmds));
+			      ARRAY_SIZE(pwm_pinctrl_init_cmds));
 	if (ret != 0) {
 		DRM_ERROR("Init pwm pinctrl failed! ret=%d.\n", ret);
 		goto err_return;
@@ -330,13 +328,13 @@ static int hisi_pwm_probe(struct platform_device *pdev)
 	g_pwm_clk = of_clk_get(np, 0);
 	if (IS_ERR(g_pwm_clk)) {
 		DRM_ERROR("%s clock not found: %d!\n",
-			np->name, (int)PTR_ERR(g_pwm_clk));
+			  np->name, (int)PTR_ERR(g_pwm_clk));
 		ret = -ENXIO;
 		goto err_return;
 	}
 
 	DRM_INFO("dss_pwm_clk:[%lu]->[%lu].\n",
-		DEFAULT_PWM_CLK_RATE, clk_get_rate(g_pwm_clk));
+		 DEFAULT_PWM_CLK_RATE, clk_get_rate(g_pwm_clk));
 
 	return 0;
 
@@ -350,7 +348,7 @@ static int hisi_pwm_remove(struct platform_device *pdev)
 	int ret = 0;
 
 	ret = pinctrl_cmds_tx(pdev, pwm_pinctrl_finit_cmds,
-		ARRAY_SIZE(pwm_pinctrl_finit_cmds));
+			      ARRAY_SIZE(pwm_pinctrl_finit_cmds));
 
 	clk_tmp = g_pwm_clk;
 	if (clk_tmp) {
-- 
2.26.2



More information about the devel mailing list