dri-devel Archive on lore.kernel.org
 help / color / Atom feed
From: "Chrisanthus, Anitha" <anitha.chrisanthus@intel.com>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: "Vetter, Daniel" <daniel.vetter@intel.com>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"Dea, Edmund J" <edmund.j.dea@intel.com>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>
Subject: RE: [PATCH v4] drm/kmb: Add support for KeemBay Display
Date: Mon, 3 Aug 2020 21:02:24 +0000
Message-ID: <BY5PR11MB4182B032DC268C4156B4EEE58C4D0@BY5PR11MB4182.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20200802152342.GA1764350@ravnborg.org>

Hi Sam,
I installed codespell, but the dictionary.txt in usr/share/codespell/dictionary.txt seems to be different from yours. Mine is version 1.8. Where can I get the dictionary.txt that you are using?
I have corrected the relevant spelling warnings from your email and have sent v5.

Thanks,
Anitha
> -----Original Message-----
> From: Sam Ravnborg <sam@ravnborg.org>
> Sent: Sunday, August 2, 2020 8:24 AM
> To: Chrisanthus, Anitha <anitha.chrisanthus@intel.com>
> Cc: dri-devel@lists.freedesktop.org; Vetter, Daniel <daniel.vetter@intel.com>;
> intel-gfx@lists.freedesktop.org; Dea, Edmund J <edmund.j.dea@intel.com>
> Subject: Re: [PATCH v4] drm/kmb: Add support for KeemBay Display
> 
> Hi Anitha.
> 
> On Thu, Jul 30, 2020 at 01:44:44PM -0700, Anitha Chrisanthus wrote:
> > This is a basic KMS atomic modesetting display driver for KeemBay family of
> > SOCs. Driver has no 2D or 3D graphics.It calls into the ADV bridge
> > driver at the connector level.
> >
> > Single CRTC with LCD controller->mipi DSI-> ADV bridge
> >
> > Only 1080p resolution and single plane is supported at this time.
> >
> > v2: moved extern to .h, removed license text
> >     use drm_dev_init, upclassed dev_private, removed HAVE_IRQ.
> >
> > v3: Squashed all 59 commits to one
> >
> > v4: review changes from Sam Ravnborg
> > 	renamed dev_p to kmb
> > 	moved clocks under kmb_clock, consolidated clk initializations
> > 	use drmm functions
> > 	use DRM_GEM_CMA_DRIVER_OPS_VMAP
> >
> 
> I have not found time neither energy to take a look at v4.
> But I applied locally and ran it through checkpatch
> with my options:
> checkpatch -q --emacs --strict --show-types --codespell --codespellfile
> /usr/lib/python3/dist-packages/codespell_lib/data/dictionary.txt
> 
> The options are from the dim script used when maintaining drm-misc-next with
> codespell options added.
> 
> Please try to reproduce locally and fix relevant warnings.
> 
> 	Sam
> 
> -:146: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #146: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:58:
> +	kmb_clr_bitmask_lcd(kmb, LCD_INT_ENABLE,
> +			LCD_INT_VERT_COMP);
> 
> -:173: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #173: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:85:
> +	drm_info(dev,
> +		"vfp= %d vbp= %d vsyc_len=%d hfp=%d hbp=%d
> hsync_len=%d\n",
> 
> -:194: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #194: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:106:
> +	drm_dbg(dev, "%s : %dactive height= %d vbp=%d vfp=%d vsync-w=%d h-
> active=%d h-bp=%d h-fp=%d hysnc-l=%d",
> +			__func__, __LINE__,
> 
> -:199: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #199: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:111:
> +	kmb_write_lcd(kmb, LCD_V_ACTIVEHEIGHT,
> +			m->crtc_vdisplay - 1);
> 
> -:204: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #204: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:116:
> +	kmb_write_lcd(kmb, LCD_H_ACTIVEWIDTH,
> +			m->crtc_hdisplay - 1);
> 
> -:217: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #217: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:129:
> +		kmb_write_lcd(kmb,
> +				LCD_V_BACKPORCH_EVEN, vm.vback_porch);
> 
> -:219: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #219: FILE: drivers/gpu/drm/kmb/kmb_crtc.c:131:
> +		kmb_write_lcd(kmb,
> +				LCD_V_FRONTPORCH_EVEN,
> vm.vfront_porch);
> 
> -:413: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #413: FILE: drivers/gpu/drm/kmb/kmb_drv.c:60:
> +		drm_err(&kmb->drm,
> +				"Failed to enable MIPI_ECFG clock: %d\n", ret);
> 
> -:420: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #420: FILE: drivers/gpu/drm/kmb/kmb_drv.c:67:
> +		drm_err(&kmb->drm,
> +				"Failed to enable MIPI_CFG clock: %d\n", ret);
> 
> -:427: CHECK:LINE_SPACING: Please don't use multiple blank lines
> #427: FILE: drivers/gpu/drm/kmb/kmb_drv.c:74:
> +
> +
> 
> -:463: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #463: FILE: drivers/gpu/drm/kmb/kmb_drv.c:110:
> +	kmb->sys_clk_mhz = clk_get_rate(kmb_clk.clk_pll0)/1000000;
>  	                                                 ^
> 
> -:470: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #470: FILE: drivers/gpu/drm/kmb/kmb_drv.c:117:
> +		drm_err(&kmb->drm, "failed to set to clk_lcd to %d\n",
> +			  KMB_LCD_DEFAULT_CLK);
> 
> -:479: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #479: FILE: drivers/gpu/drm/kmb/kmb_drv.c:126:
> +		drm_err(&kmb->drm, "failed to set to clk_mipi to %d\n",
> +			  KMB_MIPI_DEFAULT_CLK);
> 
> -:506: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #506: FILE: drivers/gpu/drm/kmb/kmb_drv.c:153:
> +			drm_err(&kmb->drm,
> +					"failed to set clk_mipi_cfg to %d\n",
> 
> -:511: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #511: FILE: drivers/gpu/drm/kmb/kmb_drv.c:158:
> +		drm_info(&kmb->drm,
> +				"Get clk_mipi_cfg after set = %ld\n", clk);
> 
> -:561: CHECK:LINE_SPACING: Please don't use multiple blank lines
> #561: FILE: drivers/gpu/drm/kmb/kmb_drv.c:208:
> +
> +
> 
> -:688: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
> #688: FILE: drivers/gpu/drm/kmb/kmb_drv.c:335:
> +	if (status & LCD_INT_EOF) {
> +
> 
> -:701: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_DMA_CFG>
> #701: FILE: drivers/gpu/drm/kmb/kmb_drv.c:348:
> +						    LCD_LAYERn_DMA_CFG
> 
> -:706: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #706: FILE: drivers/gpu/drm/kmb/kmb_drv.c:353:
> +				kmb_clr_bitmask_lcd(kmb, LCD_CONTROL,
> +					    plane_status[plane_id].ctrl);
> 
> -:733: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #733: FILE: drivers/gpu/drm/kmb/kmb_drv.c:380:
> +
> +	}
> 
> -:774: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #774: FILE: drivers/gpu/drm/kmb/kmb_drv.c:421:
> +			drm_info(&kmb->drm,
> +			    "!LAYER0:VL0 DMA UNDERFLOW val =
> 0x%lx,under_flow=%d",
> 
> -:776: WARNING:TYPO_SPELLING: 'inerrupt' may be misspelled - perhaps
> 'interrupt'?
> #776: FILE: drivers/gpu/drm/kmb/kmb_drv.c:423:
> +			/* disable underflow inerrupt */
> 
> -:787: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #787: FILE: drivers/gpu/drm/kmb/kmb_drv.c:434:
> +			kmb_clr_bitmask_lcd(kmb, LCD_LAYERn_DMA_CFG(0),
> +
> LCD_DMA_LAYER_CONT_PING_PONG_UPDATE);
> 
> -:793: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #793: FILE: drivers/gpu/drm/kmb/kmb_drv.c:440:
> +			drm_info(&kmb->drm,
> +				"LAYER0:VL0 DMA OVERFLOW val = 0x%lx",
> val);
> 
> -:796: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #796: FILE: drivers/gpu/drm/kmb/kmb_drv.c:443:
> +			drm_info(&kmb->drm,
> +				"LAYER0:VL0 DMA CB OVERFLOW val = 0x%lx",
> val);
> 
> -:799: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #799: FILE: drivers/gpu/drm/kmb/kmb_drv.c:446:
> +			drm_info(&kmb->drm,
> +				"LAYER0:VL0 DMA CR OVERFLOW val = 0x%lx",
> val);
> 
> -:807: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #807: FILE: drivers/gpu/drm/kmb/kmb_drv.c:454:
> +			drm_info(&kmb->drm,
> +			    "!LAYER1:VL1 DMA UNDERFLOW val = 0x%lx,
> under_flow=%d",
> 
> -:809: WARNING:TYPO_SPELLING: 'inerrupt' may be misspelled - perhaps
> 'interrupt'?
> #809: FILE: drivers/gpu/drm/kmb/kmb_drv.c:456:
> +			/* disable underflow inerrupt */
> 
> -:820: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #820: FILE: drivers/gpu/drm/kmb/kmb_drv.c:467:
> +			kmb_clr_bitmask_lcd(kmb, LCD_LAYERn_DMA_CFG(1),
> +
> LCD_DMA_LAYER_CONT_PING_PONG_UPDATE);
> 
> -:827: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #827: FILE: drivers/gpu/drm/kmb/kmb_drv.c:474:
> +			drm_info(&kmb->drm,
> +				"LAYER1:VL1 DMA OVERFLOW val = 0x%lx",
> val);
> 
> -:830: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #830: FILE: drivers/gpu/drm/kmb/kmb_drv.c:477:
> +			drm_info(&kmb->drm,
> +				"LAYER1:VL1 DMA CB OVERFLOW val = 0x%lx",
> val);
> 
> -:833: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #833: FILE: drivers/gpu/drm/kmb/kmb_drv.c:480:
> +			drm_info(&kmb->drm,
> +				"LAYER1:VL1 DMA CR OVERFLOW val = 0x%lx",
> val);
> 
> -:838: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #838: FILE: drivers/gpu/drm/kmb/kmb_drv.c:485:
> +			drm_info(&kmb->drm,
> +				"LAYER2:GL0 DMA UNDERFLOW val = 0x%lx",
> val);
> 
> -:841: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #841: FILE: drivers/gpu/drm/kmb/kmb_drv.c:488:
> +			drm_info(&kmb->drm,
> +				"LAYER2:GL0 DMA OVERFLOW val = 0x%lx",
> val);
> 
> -:846: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #846: FILE: drivers/gpu/drm/kmb/kmb_drv.c:493:
> +			drm_info(&kmb->drm,
> +				"LAYER3:GL1 DMA UNDERFLOW val = 0x%lx",
> val);
> 
> -:849: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #849: FILE: drivers/gpu/drm/kmb/kmb_drv.c:496:
> +			drm_info(&kmb->drm,
> +				"LAYER3:GL1 DMA OVERFLOW val = 0x%lx",
> val);
> 
> -:952: WARNING:TYPO_SPELLING: 'procees' may be misspelled - perhaps
> 'process'?
> #952: FILE: drivers/gpu/drm/kmb/kmb_drv.c:599:
> +	 *  and then the rest of the driver initialization can procees
> 
> -:957: CHECK:BRACES: braces {} should be used on all arms of this statement
> #957: FILE: drivers/gpu/drm/kmb/kmb_drv.c:604:
> +	if (adv_bridge == ERR_PTR(-EPROBE_DEFER))
> [...]
> +	else if (IS_ERR(adv_bridge)) {
> [...]
> 
> -:966: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #966: FILE: drivers/gpu/drm/kmb/kmb_drv.c:613:
> +	kmb = devm_drm_dev_alloc(dev, &kmb_driver,
> +			struct kmb_drm_private, drm);
> 
> -:980: CHECK:BRACES: braces {} should be used on all arms of this statement
> #980: FILE: drivers/gpu/drm/kmb/kmb_drv.c:627:
> +	if (ret == -EPROBE_DEFER) {
> [...]
> +	} else if (ret)
> [...]
> 
> -:990: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #990: FILE: drivers/gpu/drm/kmb/kmb_drv.c:637:
> +	drm_info(&kmb->drm, "mode_config.num_crtc=%d\n",
> +			kmb->drm.mode_config.num_crtc);
> 
> -:1026: WARNING:UNDOCUMENTED_DT_STRING: DT compatible string
> "intel,kmb_display" appears un-documented -- check
> ./Documentation/devicetree/bindings/
> #1026: FILE: drivers/gpu/drm/kmb/kmb_drv.c:673:
> +	{.compatible = "intel,kmb_display"},
> 
> -:1122: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without
> comment
> #1122: FILE: drivers/gpu/drm/kmb/kmb_drv.h:35:
> +	spinlock_t			irq_lock;
> 
> -:1360: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #1360: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:95:
> +	uint16_t default_bit_rate_mbps;
> 
> -:1361: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #1361: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:96:
> +	uint8_t hsfreqrange_code;
> 
> -:1462: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1462: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:197:
> +	if (mode->hdisplay < mode_config->min_width ||
> +			mode->hdisplay > mode_config->max_width)
> 
> -:1466: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1466: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:201:
> +	if (mode->vdisplay < mode_config->min_height ||
> +			mode->vdisplay > mode_config->max_height)
> 
> -:1477: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1477: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:212:
> +	num_modes = drm_add_modes_noedid(connector,
> +			 connector->dev->mode_config.max_width,
> 
> -:1772: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1772: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:507:
> +	drm_dbg(&kmb->drm,
> +		 "ctrl=%d frame_id=%d section=%d cfg=%x packed=%d\n",
> 
> -:1775: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TXm_HS_FGn_SECTo_PH>
> #1775: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:510:
> +		       (MIPI_TXm_HS_FGn_SECTo_PH(ctrl_no, frame_id, section)),
> 
> -:1787: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_FGn_SECT_UNPACKED_BYTES0>
> #1787: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:522:
> +	    MIPI_TXm_HS_FGn_SECT_UNPACKED_BYTES0(ctrl_no,
> 
> -:1792: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1792: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:527:
> +	drm_dbg(&kmb->drm,
> +		 "unpacked_bytes = %d, wordcount = %d\n", unpacked_bytes,
> 
> -:1796: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_FGn_SECTo_LINE_CFG>
> #1796: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:531:
> +	reg_adr = MIPI_TXm_HS_FGn_SECTo_LINE_CFG(ctrl_no, frame_id,
> section);
> 
> -:1828: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1828: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:563:
> +	unpacked_bytes = compute_unpacked_bytes(*wc,
> +					data_type_parameters.bits_per_pclk);
> 
> -:1868: WARNING:TYPO_SPELLING: 'activ' may be misspelled - perhaps 'active'?
> #1868: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:603:
> +	drm_dbg(&kmb->drm, "bpp=%d sysclk=%d lane-rate=%d activ-
> lanes=%d\n",
> 
> -:1869: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #1869: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:604:
> +	drm_dbg(&kmb->drm, "bpp=%d sysclk=%d lane-rate=%d activ-
> lanes=%d\n",
> +		 fg_cfg->bpp, sysclk, fg_cfg->lane_rate_mbps,
> 
> -:1873: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_FGn_NUM_LINES>
> #1873: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:608:
> +	reg_adr = MIPI_TXm_HS_FGn_NUM_LINES(ctrl_no, frame_gen);
> 
> -:1883: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #1883: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:618:
> +	reg_adr = MIPI_TXm_HS_VSYNC_WIDTHn(ctrl_no, frame_gen / 2);
> 
> -:1883: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_VSYNC_WIDTHn>
> #1883: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:618:
> +	reg_adr = MIPI_TXm_HS_VSYNC_WIDTHn(ctrl_no, frame_gen / 2);
> 
> -:1887: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_V_BACKPORCHESn>
> #1887: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:622:
> +	reg_adr = MIPI_TXm_HS_V_BACKPORCHESn(ctrl_no, frame_gen / 2);
> 
> -:1891: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_V_FRONTPORCHESn>
> #1891: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:626:
> +	reg_adr = MIPI_TXm_HS_V_FRONTPORCHESn(ctrl_no, frame_gen / 2);
> 
> -:1895: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TXm_HS_V_ACTIVEn>
> #1895: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:630:
> +	reg_adr = MIPI_TXm_HS_V_ACTIVEn(ctrl_no, frame_gen / 2);
> 
> -:1899: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #1899: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:634:
> +	reg_adr = MIPI_TXm_HS_HSYNC_WIDTHn(ctrl_no, frame_gen);
> 
> -:1899: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_HSYNC_WIDTHn>
> #1899: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:634:
> +	reg_adr = MIPI_TXm_HS_HSYNC_WIDTHn(ctrl_no, frame_gen);
> 
> -:1904: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_H_BACKPORCHn>
> #1904: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:639:
> +	reg_adr = MIPI_TXm_HS_H_BACKPORCHn(ctrl_no, frame_gen);
> 
> -:1909: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_H_FRONTPORCHn>
> #1909: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:644:
> +	reg_adr = MIPI_TXm_HS_H_FRONTPORCHn(ctrl_no, frame_gen);
> 
> -:1914: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TXm_HS_H_ACTIVEn>
> #1914: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:649:
> +	reg_adr = MIPI_TXm_HS_H_ACTIVEn(ctrl_no, frame_gen);
> 
> -:1923: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #1923: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:658:
> +	reg_adr = MIPI_TXm_HS_LLP_HSYNC_WIDTHn(ctrl_no, frame_gen);
> 
> -:1923: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_LLP_HSYNC_WIDTHn>
> #1923: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:658:
> +	reg_adr = MIPI_TXm_HS_LLP_HSYNC_WIDTHn(ctrl_no, frame_gen);
> 
> -:1927: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_LLP_H_BACKPORCHn>
> #1927: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:662:
> +	reg_adr = MIPI_TXm_HS_LLP_H_BACKPORCHn(ctrl_no, frame_gen);
> 
> -:1931: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_LLP_H_FRONTPORCHn>
> #1931: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:666:
> +	reg_adr = MIPI_TXm_HS_LLP_H_FRONTPORCHn(ctrl_no, frame_gen);
> 
> -:1947: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written
> "fg_cfg->sections[i]"
> #1947: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:682:
> +		if (fg_cfg->sections[i] != NULL)
> 
> -:1995: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_MC_FIFO_CTRL_EN>
> #1995: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:730:
> +	kmb_set_bit_mipi(kmb, MIPI_TXm_HS_MC_FIFO_CTRL_EN(ctrl_no),
> 
> -:2052: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TXm_HS_SYNC_CFG>
> #2052: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:787:
> +	kmb_write_mipi(kmb, MIPI_TXm_HS_SYNC_CFG(ctrl_no), sync_cfg);
> 
> -:2053: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TXm_HS_CTRL>
> #2053: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:788:
> +	kmb_write_mipi(kmb, MIPI_TXm_HS_CTRL(ctrl_no), ctrl);
> 
> -:2064: CHECK:CAMELCASE: Avoid CamelCase: <TP_SEL_VCm>
> #2064: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:799:
> +	val = TP_SEL_VCm(vc, tp_sel);
> 
> -:2067: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_TEST_PAT_COLOR0>
> #2067: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:802:
> +	kmb_write_mipi(kmb, MIPI_TXm_HS_TEST_PAT_COLOR0(ctrl_no),
> color0);
> 
> -:2068: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_TEST_PAT_COLOR1>
> #2068: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:803:
> +	kmb_write_mipi(kmb, MIPI_TXm_HS_TEST_PAT_COLOR1(ctrl_no),
> color1);
> 
> -:2071: CHECK:CAMELCASE: Avoid CamelCase: <TP_EN_VCm>
> #2071: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:806:
> +	val |= TP_EN_VCm(vc);
> 
> -:2072: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_TEST_PAT_CTRL>
> #2072: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:807:
> +	kmb_write_mipi(kmb, MIPI_TXm_HS_TEST_PAT_CTRL(ctrl_no), val);
> 
> -:2074: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #2074: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:809:
> +
> +}
> 
> -:2099: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written
> "!frame"
> #2099: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:834:
> +		if (frame == NULL)
> 
> -:2107: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written
> "!frame->sections[sect]"
> #2107: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:842:
> +			if (frame->sections[sect] == NULL)
> 
> -:2117: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #2117: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:852:
> +
> +		}
> 
> -:2146: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2146: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:881:
> +	drm_dbg(&kmb->drm, "IRQ_STATUS = 0x%x\n",
> +		  GET_MIPI_TX_HS_IRQ_STATUS(kmb, MIPI_CTRL6));
> 
> -:2259: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #2259: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:994:
> +		reg_rd_data = kmb_read_mipi(kmb, DPHY_TEST_DOUT0_3);
> 
> -:2261: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #2261: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:996:
> +		reg_rd_data = kmb_read_mipi(kmb, DPHY_TEST_DOUT4_7);
> 
> -:2263: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #2263: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:998:
> +		reg_rd_data = kmb_read_mipi(kmb, DPHY_TEST_DOUT8_9);
> 
> -:2338: CHECK:SPACING: spaces preferred around that '*' (ctx:ExV)
> #2338: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1073:
> +							      *kmb,
>  							      ^
> 
> -:2351: CHECK:SPACING: spaces preferred around that '*' (ctx:ExV)
> #2351: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1086:
> +							     *kmb,
>  							     ^
> 
> -:2518: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #2518: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1253:
> +		 kmb_read_mipi(kmb, DPHY_TEST_DOUT4_7));
> 
> -:2518: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2518: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1253:
> +	drm_dbg(&kmb->drm, "dphy %d fsm_state = 0%x\n", dphy_no,
> +		 kmb_read_mipi(kmb, DPHY_TEST_DOUT4_7));
> 
> -:2530: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2530: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1265:
> +	drm_info(&kmb->drm,
> +		"dphy=%d mode=%d active_lanes=%d\n", dphy_no, mode,
> 
> -:2533: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2533: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1268:
> +	drm_dbg(&kmb->drm, "MIPI_DPHY_STAT0_4_7 = 0x%x)\n",
> +		  kmb_read_mipi(kmb, MIPI_DPHY_STAT4_7));
> 
> -:2557: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2557: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1292:
> +	drm_dbg(&kmb->drm, "DPHY_TEST_CTRL0=0x%x\n",
> +		  kmb_read_mipi(kmb, DPHY_TEST_CTRL0));
> 
> -:2658: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #2658: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1393:
> +
> +	}
> 
> -:2665: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2665: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1400:
> +	drm_dbg(&kmb->drm, "DPHY_FREQ = 0x%x\n",
> +		 kmb_read_mipi(kmb, DPHY_FREQ_CTRL0_3 + 4));
> 
> -:2667: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2667: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1402:
> +	drm_dbg(&kmb->drm, "MIPI_DPHY_STAT0_4_7 = 0x%x)\n",
> +		  kmb_read_mipi(kmb, MIPI_DPHY_STAT4_7));
> 
> -:2673: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2673: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1408:
> +	drm_dbg(&kmb->drm, "DPHY_CFG_CLK_EN = 0x%x\n",
> +		 kmb_read_mipi(kmb, DPHY_CFG_CLK_EN));
> 
> -:2694: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2694: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1429:
> +		drm_dbg(&kmb->drm, "DPHY_INIT_CTRL1 = 0x%x\n",
> +			 kmb_read_mipi(kmb, DPHY_INIT_CTRL1));
> 
> -:2698: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2698: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1433:
> +	drm_dbg(&kmb->drm, "MIPI_DPHY_STAT0_4_7 = 0x%x)\n",
> +		  kmb_read_mipi(kmb, MIPI_DPHY_STAT4_7));
> 
> -:2739: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2739: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1474:
> +	drm_dbg(&kmb->drm, "MIPI_DPHY_STAT0_4_7 = 0x%x)\n",
> +		  kmb_read_mipi(kmb, MIPI_DPHY_STAT4_7));
> 
> -:2751: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2751: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1486:
> +	drm_dbg(&kmb->drm, "DPHY_INIT_CTRL2 = 0x%x\n",
> +		 kmb_read_mipi(kmb, DPHY_INIT_CTRL2));
> 
> -:2764: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2764: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1499:
> +	drm_dbg(&kmb->drm, "MIPI_DPHY_STAT0_4_7 = 0x%x)\n",
> +		 kmb_read_mipi(kmb, MIPI_DPHY_STAT4_7));
> 
> -:2771: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2771: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1506:
> +	drm_dbg(&kmb->drm, "DPHY_INIT_CTRL0 = 0x%x\n",
> +		 kmb_read_mipi(kmb, DPHY_INIT_CTRL0));
> 
> -:2784: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #2784: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1519:
> +		val = GET_TEST_DOUT4_7(kmb, dphy_no);
> 
> -:2806: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2806: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1541:
> +	drm_dbg(&kmb->drm, "dphy=%d active_lanes=%d data_lanes=%d\n",
> dphy_no,
> +		 active_lanes, data_lanes);
> 
> -:2819: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2819: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1554:
> +			drm_info(&kmb->drm,
> +				"! WAIT_INIT_DONE: TIMING
> OUT!(err_stat=%d)",
> 
> -:2878: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2878: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1613:
> +	drm_info(&kmb->drm,
> +		"active_lanes=%d lane_rate=%d\n", cfg->active_lanes,
> 
> -:2945: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written
> "mode"
> #2945: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1680:
> +	if (mode != NULL) {
> 
> -:2964: CHECK:SPACING: No space is necessary after a cast
> #2964: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1699:
> +		data_rate = ((((u32) mode->crtc_vtotal *
> 
> -:2965: CHECK:SPACING: No space is necessary after a cast
> #2965: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1700:
> +				(u32) mode->crtc_htotal) *
> 
> -:2974: CHECK:SPACING: No space is necessary after a cast
> #2974: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1709:
> +			 (u32) data_rate, mipi_tx_init_cfg.active_lanes);
> 
> -:2986: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2986: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1721:
> +		drm_info(&kmb->drm,
> +			"lane rate=%d\n", mipi_tx_init_cfg.lane_rate_mbps);
> 
> -:2988: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #2988: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1723:
> +		drm_dbg(&kmb->drm,
> +		     "vfp= %d vbp= %d vsyc_len=%d hfp=%d hbp=%d
> hsync_len=%d lane-rate=%d",
> 
> -:2997: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #2997: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1732:
> +
> +	}
> 
> -:3013: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3013: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1748:
> +	drm_dbg(&kmb->drm, "IRQ_STATUS = 0x%x\n",
> +		 GET_MIPI_TX_HS_IRQ_STATUS(kmb, MIPI_CTRL6));
> 
> -:3022: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3022: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1757:
> +	drm_dbg(&kmb->drm, "IRQ_STATUS = 0x%x\n",
> +		 GET_MIPI_TX_HS_IRQ_STATUS(kmb, MIPI_CTRL6));
> 
> -:3028: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3028: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1763:
> +	drm_dbg(&kmb->drm, "MIPI_TXm_HS_CTRL = 0x%x\n",
> +		 kmb_read_mipi(kmb, MIPI_TXm_HS_CTRL(6)));
> 
> -:3030: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3030: FILE: drivers/gpu/drm/kmb/kmb_dsi.c:1765:
> +	drm_dbg(&kmb->drm, "MIPI LOOP BACK = %x\n",
> +		 kmb_read_mipi(kmb, MIPI_CTRL_DIG_LOOPBACK));
> 
> -:3137: CHECK:BIT_MACRO: Prefer using the BIT macro
> #3137: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:32:
> +#define   PLL_FEEDBACK_DIVIDER_HIGH			(1 << 7)
> 
> -:3139: CHECK:BIT_MACRO: Prefer using the BIT macro
> #3139: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:34:
> +#define   PLL_N_OVR_EN					(1 << 4)
> 
> -:3140: CHECK:BIT_MACRO: Prefer using the BIT macro
> #3140: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:35:
> +#define   PLL_M_OVR_EN					(1 << 5)
> 
> -:3362: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3362: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:257:
> +	uint8_t size_constraint_pixels;
> 
> -:3363: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3363: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:258:
> +	uint8_t size_constraint_bytes;
> 
> -:3364: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3364: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:259:
> +	uint8_t pixels_per_pclk;
> 
> -:3365: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3365: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:260:
> +	uint8_t bits_per_pclk;
> 
> -:3369: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3369: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:264:
> +	uint8_t hfp_blank_en;	/*horizontal front porch blanking enable */
> 
> -:3370: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3370: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:265:
> +	uint8_t eotp_en;	/*End of transmission packet enable */
> 
> -:3372: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3372: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:267:
> +	uint8_t lpm_last_vfp_line;
> 
> -:3374: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3374: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:269:
> +	uint8_t lpm_first_vsa_line;
> 
> -:3375: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3375: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:270:
> +	uint8_t sync_pulse_eventn;	/*sync type */
> 
> -:3376: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3376: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:271:
> +	uint8_t hfp_blanking;	/*horizontal front porch blanking mode */
> 
> -:3377: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3377: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:272:
> +	uint8_t hbp_blanking;	/*horizontal back porch blanking mode */
> 
> -:3378: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3378: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:273:
> +	uint8_t hsa_blanking;	/*horizontal sync active blanking mode */
> 
> -:3379: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3379: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:274:
> +	uint8_t v_blanking;	/*vertical timing blanking mode */
> 
> -:3383: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3383: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:278:
> +	uint32_t dma_v_stride;
> 
> -:3384: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3384: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:279:
> +	uint16_t dma_v_scale_cfg;
> 
> -:3385: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3385: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:280:
> +	uint16_t width_pixels;
> 
> -:3386: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3386: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:281:
> +	uint16_t height_lines;
> 
> -:3387: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3387: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:282:
> +	uint8_t dma_packed;
> 
> -:3388: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3388: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:283:
> +	uint8_t bpp;
> 
> -:3389: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3389: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:284:
> +	uint8_t bpp_unpacked;
> 
> -:3390: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3390: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:285:
> +	uint8_t dma_h_stride;
> 
> -:3391: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3391: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:286:
> +	uint8_t data_type;
> 
> -:3392: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3392: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:287:
> +	uint8_t data_mode;
> 
> -:3393: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3393: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:288:
> +	uint8_t dma_flip_rotate_sel;
> 
> -:3397: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3397: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:292:
> +	uint32_t bpp;
> 
> -:3398: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3398: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:293:
> +	uint32_t lane_rate_mbps;
> 
> -:3399: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3399: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:294:
> +	uint32_t hsync_width;
> 
> -:3400: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3400: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:295:
> +	uint32_t h_backporch;
> 
> -:3401: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3401: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:296:
> +	uint32_t h_frontporch;
> 
> -:3402: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3402: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:297:
> +	uint32_t h_active;
> 
> -:3403: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3403: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:298:
> +	uint16_t vsync_width;
> 
> -:3404: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3404: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:299:
> +	uint16_t v_backporch;
> 
> -:3405: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3405: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:300:
> +	uint16_t v_frontporch;
> 
> -:3406: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3406: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:301:
> +	uint16_t v_active;
> 
> -:3407: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3407: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:302:
> +	uint8_t active_lanes;
> 
> -:3411: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3411: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:306:
> +	uint32_t wc;
> 
> -:3414: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3414: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:309:
> +	uint8_t vchannel;
> 
> -:3415: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3415: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:310:
> +	uint8_t dma_packed;
> 
> -:3420: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3420: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:315:
> +	uint32_t hsync_width;	/*in pixels */
> 
> -:3421: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3421: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:316:
> +	uint32_t h_backporch;	/*in pixels */
> 
> -:3422: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3422: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:317:
> +	uint32_t h_frontporch;	/*in pixels */
> 
> -:3423: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3423: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:318:
> +	uint16_t vsync_width;	/*in lines */
> 
> -:3424: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3424: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:319:
> +	uint16_t v_backporch;	/*in lines */
> 
> -:3425: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t'
> #3425: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:320:
> +	uint16_t v_frontporch;	/*in lines */
> 
> -:3431: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3431: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:326:
> +	uint8_t line_sync_pkt_en;
> 
> -:3432: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3432: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:327:
> +	uint8_t line_counter_active;
> 
> -:3433: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3433: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:328:
> +	uint8_t frame_counter_active;
> 
> -:3434: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3434: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:329:
> +	uint8_t tx_hsclkkidle_cnt;
> 
> -:3435: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3435: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:330:
> +	uint8_t tx_hsexit_cnt;
> 
> -:3436: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3436: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:331:
> +	uint8_t tx_crc_en;
> 
> -:3437: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3437: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:332:
> +	uint8_t tx_hact_wait_stop;
> 
> -:3438: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3438: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:333:
> +	uint8_t tx_always_use_hact;
> 
> -:3439: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3439: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:334:
> +	uint8_t tx_wait_trig;
> 
> -:3440: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3440: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:335:
> +	uint8_t tx_wait_all_sect;
> 
> -:3446: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3446: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:341:
> +	uint8_t index;
> 
> -:3447: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3447: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:342:
> +	uint8_t type;		/* controller type : MIPI_DSI */
> 
> -:3448: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3448: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:343:
> +	uint8_t dir;		/* controller direction : MIPI_TX */
> 
> -:3449: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3449: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:344:
> +	uint8_t active_lanes;	/* # active lanes per controller 2/4 */
> 
> -:3450: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3450: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:345:
> +	uint32_t lane_rate_mbps;	/*MBPS */
> 
> -:3451: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3451: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:346:
> +	uint32_t ref_clk_khz;
> 
> -:3452: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3452: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:347:
> +	uint32_t cfg_clk_khz;
> 
> -:3453: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3453: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:348:
> +	uint32_t data_if;	/*MIPI_IF_DMA or MIPI_IF_PARALLEL */
> 
> -:3459: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u8' over 'uint8_t'
> #3459: FILE: drivers/gpu/drm/kmb/kmb_dsi.h:354:
> +	uint8_t value;
> 
> -:3509: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3509: FILE: drivers/gpu/drm/kmb/kmb_plane.c:28:
> +const uint32_t layer_irqs[] = {
> 
> -:3658: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3658: FILE: drivers/gpu/drm/kmb/kmb_plane.c:177:
> +	DRM_INFO_ONCE("%s : %d format=0x%x val=0x%x\n",
> +			 __func__, __LINE__, format, val);
> 
> -:3696: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF11>
> #3696: FILE: drivers/gpu/drm/kmb/kmb_plane.c:215:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF11(plane_id),
> csc_coef_lcd[0]);
> 
> -:3697: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF12>
> #3697: FILE: drivers/gpu/drm/kmb/kmb_plane.c:216:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF12(plane_id),
> csc_coef_lcd[1]);
> 
> -:3698: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF13>
> #3698: FILE: drivers/gpu/drm/kmb/kmb_plane.c:217:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF13(plane_id),
> csc_coef_lcd[2]);
> 
> -:3699: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF21>
> #3699: FILE: drivers/gpu/drm/kmb/kmb_plane.c:218:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF21(plane_id),
> csc_coef_lcd[3]);
> 
> -:3700: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF22>
> #3700: FILE: drivers/gpu/drm/kmb/kmb_plane.c:219:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF22(plane_id),
> csc_coef_lcd[4]);
> 
> -:3701: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF23>
> #3701: FILE: drivers/gpu/drm/kmb/kmb_plane.c:220:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF23(plane_id),
> csc_coef_lcd[5]);
> 
> -:3702: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF31>
> #3702: FILE: drivers/gpu/drm/kmb/kmb_plane.c:221:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF31(plane_id),
> csc_coef_lcd[6]);
> 
> -:3703: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF32>
> #3703: FILE: drivers/gpu/drm/kmb/kmb_plane.c:222:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF32(plane_id),
> csc_coef_lcd[7]);
> 
> -:3704: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_COEFF33>
> #3704: FILE: drivers/gpu/drm/kmb/kmb_plane.c:223:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_COEFF33(plane_id),
> csc_coef_lcd[8]);
> 
> -:3705: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_OFF1>
> #3705: FILE: drivers/gpu/drm/kmb/kmb_plane.c:224:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_OFF1(plane_id),
> csc_coef_lcd[9]);
> 
> -:3706: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_OFF2>
> #3706: FILE: drivers/gpu/drm/kmb/kmb_plane.c:225:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_OFF2(plane_id),
> csc_coef_lcd[10]);
> 
> -:3707: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CSC_OFF3>
> #3707: FILE: drivers/gpu/drm/kmb/kmb_plane.c:226:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CSC_OFF3(plane_id),
> csc_coef_lcd[11]);
> 
> -:3756: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_DMA_LEN>
> #3756: FILE: drivers/gpu/drm/kmb/kmb_plane.c:275:
> +	kmb_write_lcd(kmb, LCD_LAYERn_DMA_LEN(plane_id), dma_len);
> 
> -:3757: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_LEN_SHADOW>
> #3757: FILE: drivers/gpu/drm/kmb/kmb_plane.c:276:
> +	kmb_write_lcd(kmb, LCD_LAYERn_DMA_LEN_SHADOW(plane_id),
> dma_len);
> 
> -:3758: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_LINE_VSTRIDE>
> #3758: FILE: drivers/gpu/drm/kmb/kmb_plane.c:277:
> +	kmb_write_lcd(kmb, LCD_LAYERn_DMA_LINE_VSTRIDE(plane_id),
> 
> -:3760: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_LINE_WIDTH>
> #3760: FILE: drivers/gpu/drm/kmb/kmb_plane.c:279:
> +	kmb_write_lcd(kmb, LCD_LAYERn_DMA_LINE_WIDTH(plane_id),
> 
> -:3765: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_ADDR>
> #3765: FILE: drivers/gpu/drm/kmb/kmb_plane.c:284:
> +	kmb_write_lcd(kmb, LCD_LAYERn_DMA_START_ADDR(plane_id),
> 
> -:3771: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_CB_LINE_VSTRIDE>
> #3771: FILE: drivers/gpu/drm/kmb/kmb_plane.c:290:
> +		kmb_write_lcd(kmb,
> LCD_LAYERn_DMA_CB_LINE_VSTRIDE(plane_id),
> 
> -:3772: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3772: FILE: drivers/gpu/drm/kmb/kmb_plane.c:291:
> +		kmb_write_lcd(kmb,
> LCD_LAYERn_DMA_CB_LINE_VSTRIDE(plane_id),
> +				width*fb->format->cpp[0]);
> 
> -:3772: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #3772: FILE: drivers/gpu/drm/kmb/kmb_plane.c:291:
> +				width*fb->format->cpp[0]);
>  				     ^
> 
> -:3773: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_CB_LINE_WIDTH>
> #3773: FILE: drivers/gpu/drm/kmb/kmb_plane.c:292:
> +		kmb_write_lcd(kmb,
> LCD_LAYERn_DMA_CB_LINE_WIDTH(plane_id),
> 
> -:3777: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3777: FILE: drivers/gpu/drm/kmb/kmb_plane.c:296:
> +		addr[U_PLANE] = drm_fb_cma_get_gem_addr(fb, plane->state,
> +				U_PLANE);
> 
> -:3779: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around
> 'num_planes == 3'
> #3779: FILE: drivers/gpu/drm/kmb/kmb_plane.c:298:
> +		if ((num_planes == 3) && (val & LCD_LAYER_CRCB_ORDER))
> 
> -:3781: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3781: FILE: drivers/gpu/drm/kmb/kmb_plane.c:300:
> +			kmb_write_lcd(kmb,
> +
> 	LCD_LAYERn_DMA_START_CR_ADR(plane_id),
> 
> -:3781: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_CR_ADR>
> #3781: FILE: drivers/gpu/drm/kmb/kmb_plane.c:300:
> +
> 	LCD_LAYERn_DMA_START_CR_ADR(plane_id),
> 
> -:3785: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3785: FILE: drivers/gpu/drm/kmb/kmb_plane.c:304:
> +			kmb_write_lcd(kmb,
> +
> 	LCD_LAYERn_DMA_START_CB_ADR(plane_id),
> 
> -:3785: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_CB_ADR>
> #3785: FILE: drivers/gpu/drm/kmb/kmb_plane.c:304:
> +
> 	LCD_LAYERn_DMA_START_CB_ADR(plane_id),
> 
> -:3790: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3790: FILE: drivers/gpu/drm/kmb/kmb_plane.c:309:
> +			kmb_write_lcd(kmb,
> +
> 	LCD_LAYERn_DMA_CR_LINE_VSTRIDE(plane_id),
> 
> -:3790: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_CR_LINE_VSTRIDE>
> #3790: FILE: drivers/gpu/drm/kmb/kmb_plane.c:309:
> +
> 	LCD_LAYERn_DMA_CR_LINE_VSTRIDE(plane_id),
> 
> -:3791: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #3791: FILE: drivers/gpu/drm/kmb/kmb_plane.c:310:
> +				((width)*fb->format->cpp[0]));
>  				        ^
> 
> -:3794: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3794: FILE: drivers/gpu/drm/kmb/kmb_plane.c:313:
> +			kmb_write_lcd(kmb,
> +				LCD_LAYERn_DMA_CR_LINE_WIDTH(plane_id),
> 
> -:3794: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_CR_LINE_WIDTH>
> #3794: FILE: drivers/gpu/drm/kmb/kmb_plane.c:313:
> +				LCD_LAYERn_DMA_CR_LINE_WIDTH(plane_id),
> 
> -:3795: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #3795: FILE: drivers/gpu/drm/kmb/kmb_plane.c:314:
> +				((width)*fb->format->cpp[0]));
>  				        ^
> 
> -:3798: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3798: FILE: drivers/gpu/drm/kmb/kmb_plane.c:317:
> +			addr[V_PLANE] = drm_fb_cma_get_gem_addr(fb,
> +					plane->state, V_PLANE);
> 
> -:3803: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3803: FILE: drivers/gpu/drm/kmb/kmb_plane.c:322:
> +				kmb_write_lcd(kmb,
> +
> 	LCD_LAYERn_DMA_START_CB_ADR(plane_id),
> 
> -:3807: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3807: FILE: drivers/gpu/drm/kmb/kmb_plane.c:326:
> +				kmb_write_lcd(kmb,
> +
> 	LCD_LAYERn_DMA_START_CR_ADR(plane_id),
> 
> -:3812: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_WIDTH>
> #3812: FILE: drivers/gpu/drm/kmb/kmb_plane.c:331:
> +	kmb_write_lcd(kmb, LCD_LAYERn_WIDTH(plane_id), src_w - 1);
> 
> -:3813: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_HEIGHT>
> #3813: FILE: drivers/gpu/drm/kmb/kmb_plane.c:332:
> +	kmb_write_lcd(kmb, LCD_LAYERn_HEIGHT(plane_id), src_h - 1);
> 
> -:3814: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_COL_START>
> #3814: FILE: drivers/gpu/drm/kmb/kmb_plane.c:333:
> +	kmb_write_lcd(kmb, LCD_LAYERn_COL_START(plane_id), crtc_x);
> 
> -:3815: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_ROW_START>
> #3815: FILE: drivers/gpu/drm/kmb/kmb_plane.c:334:
> +	kmb_write_lcd(kmb, LCD_LAYERn_ROW_START(plane_id), crtc_y);
> 
> -:3826: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CFG>
> #3826: FILE: drivers/gpu/drm/kmb/kmb_plane.c:345:
> +	kmb_write_lcd(kmb, LCD_LAYERn_CFG(plane_id), val);
> 
> -:3869: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3869: FILE: drivers/gpu/drm/kmb/kmb_plane.c:388:
> +	drm_dbg(&kmb->drm, "dma_cfg=0x%x LCD_DMA_CFG=0x%x\n",
> dma_cfg,
> +		  kmb_read_lcd(kmb, LCD_LAYERn_DMA_CFG(plane_id)));
> 
> -:3951: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
> #3951: FILE: drivers/gpu/drm/kmb/kmb_plane.c:470:
> +	const uint32_t *plane_formats;
> 
> -:3978: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3978: FILE: drivers/gpu/drm/kmb/kmb_plane.c:497:
> +			drm_err(drm, "drm_universal_plane_init failed
> (ret=%d)",
> +				  ret);
> 
> -:3982: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3982: FILE: drivers/gpu/drm/kmb/kmb_plane.c:501:
> +		drm_dbg(drm, "%s : %d i=%d type=%d",
> +			  __func__, __LINE__,
> 
> -:3991: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open
> parenthesis
> #3991: FILE: drivers/gpu/drm/kmb/kmb_plane.c:510:
> +		drm_dbg(drm, "%s : %d primary=%p\n", __func__, __LINE__,
> +			  &primary->base_plane);
> 
> -:4160: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4160: FILE: drivers/gpu/drm/kmb/kmb_regs.h:25:
> +#define LCD_CTRL_PROGRESSIVE			  (0<<0)
>                              			    ^
> 
> -:4161: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4161: FILE: drivers/gpu/drm/kmb/kmb_regs.h:26:
> +#define LCD_CTRL_INTERLACED			  (1<<0)
>                             			    ^
> 
> -:4161: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4161: FILE: drivers/gpu/drm/kmb/kmb_regs.h:26:
> +#define LCD_CTRL_INTERLACED			  (1<<0)
> 
> -:4162: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4162: FILE: drivers/gpu/drm/kmb/kmb_regs.h:27:
> +#define LCD_CTRL_ENABLE				  (1<<1)
>                         				    ^
> 
> -:4162: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4162: FILE: drivers/gpu/drm/kmb/kmb_regs.h:27:
> +#define LCD_CTRL_ENABLE				  (1<<1)
> 
> -:4163: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4163: FILE: drivers/gpu/drm/kmb/kmb_regs.h:28:
> +#define LCD_CTRL_VL1_ENABLE			  (1<<2)
>                             			    ^
> 
> -:4163: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4163: FILE: drivers/gpu/drm/kmb/kmb_regs.h:28:
> +#define LCD_CTRL_VL1_ENABLE			  (1<<2)
> 
> -:4164: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4164: FILE: drivers/gpu/drm/kmb/kmb_regs.h:29:
> +#define LCD_CTRL_VL2_ENABLE			  (1<<3)
>                             			    ^
> 
> -:4164: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4164: FILE: drivers/gpu/drm/kmb/kmb_regs.h:29:
> +#define LCD_CTRL_VL2_ENABLE			  (1<<3)
> 
> -:4165: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4165: FILE: drivers/gpu/drm/kmb/kmb_regs.h:30:
> +#define LCD_CTRL_GL1_ENABLE			  (1<<4)
>                             			    ^
> 
> -:4165: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4165: FILE: drivers/gpu/drm/kmb/kmb_regs.h:30:
> +#define LCD_CTRL_GL1_ENABLE			  (1<<4)
> 
> -:4166: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4166: FILE: drivers/gpu/drm/kmb/kmb_regs.h:31:
> +#define LCD_CTRL_GL2_ENABLE			  (1<<5)
>                             			    ^
> 
> -:4166: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4166: FILE: drivers/gpu/drm/kmb/kmb_regs.h:31:
> +#define LCD_CTRL_GL2_ENABLE			  (1<<5)
> 
> -:4167: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4167: FILE: drivers/gpu/drm/kmb/kmb_regs.h:32:
> +#define LCD_CTRL_ALPHA_BLEND_VL1		  (0<<6)
>                                  		    ^
> 
> -:4168: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4168: FILE: drivers/gpu/drm/kmb/kmb_regs.h:33:
> +#define LCD_CTRL_ALPHA_BLEND_VL2		  (1<<6)
>                                  		    ^
> 
> -:4168: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4168: FILE: drivers/gpu/drm/kmb/kmb_regs.h:33:
> +#define LCD_CTRL_ALPHA_BLEND_VL2		  (1<<6)
> 
> -:4169: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4169: FILE: drivers/gpu/drm/kmb/kmb_regs.h:34:
> +#define LCD_CTRL_ALPHA_BLEND_GL1		  (2<<6)
>                                  		    ^
> 
> -:4170: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4170: FILE: drivers/gpu/drm/kmb/kmb_regs.h:35:
> +#define LCD_CTRL_ALPHA_BLEND_GL2		  (3<<6)
>                                  		    ^
> 
> -:4171: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4171: FILE: drivers/gpu/drm/kmb/kmb_regs.h:36:
> +#define LCD_CTRL_ALPHA_TOP_VL1			  (0<<8)
>                                			    ^
> 
> -:4172: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4172: FILE: drivers/gpu/drm/kmb/kmb_regs.h:37:
> +#define LCD_CTRL_ALPHA_TOP_VL2			  (1<<8)
>                                			    ^
> 
> -:4172: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4172: FILE: drivers/gpu/drm/kmb/kmb_regs.h:37:
> +#define LCD_CTRL_ALPHA_TOP_VL2			  (1<<8)
> 
> -:4173: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4173: FILE: drivers/gpu/drm/kmb/kmb_regs.h:38:
> +#define LCD_CTRL_ALPHA_TOP_GL1			  (2<<8)
>                                			    ^
> 
> -:4174: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4174: FILE: drivers/gpu/drm/kmb/kmb_regs.h:39:
> +#define LCD_CTRL_ALPHA_TOP_GL2			  (3<<8)
>                                			    ^
> 
> -:4175: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4175: FILE: drivers/gpu/drm/kmb/kmb_regs.h:40:
> +#define LCD_CTRL_ALPHA_MIDDLE_VL1		  (0<<10)
>                                   		    ^
> 
> -:4176: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4176: FILE: drivers/gpu/drm/kmb/kmb_regs.h:41:
> +#define LCD_CTRL_ALPHA_MIDDLE_VL2		  (1<<10)
>                                   		    ^
> 
> -:4176: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4176: FILE: drivers/gpu/drm/kmb/kmb_regs.h:41:
> +#define LCD_CTRL_ALPHA_MIDDLE_VL2		  (1<<10)
> 
> -:4177: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4177: FILE: drivers/gpu/drm/kmb/kmb_regs.h:42:
> +#define LCD_CTRL_ALPHA_MIDDLE_GL1		  (2<<10)
>                                   		    ^
> 
> -:4178: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4178: FILE: drivers/gpu/drm/kmb/kmb_regs.h:43:
> +#define LCD_CTRL_ALPHA_MIDDLE_GL2		  (3<<10)
>                                   		    ^
> 
> -:4179: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4179: FILE: drivers/gpu/drm/kmb/kmb_regs.h:44:
> +#define LCD_CTRL_ALPHA_BOTTOM_VL1		  (0<<12)
>                                   		    ^
> 
> -:4180: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4180: FILE: drivers/gpu/drm/kmb/kmb_regs.h:45:
> +#define LCD_CTRL_ALPHA_BOTTOM_VL2		  (1<<12)
>                                   		    ^
> 
> -:4180: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4180: FILE: drivers/gpu/drm/kmb/kmb_regs.h:45:
> +#define LCD_CTRL_ALPHA_BOTTOM_VL2		  (1<<12)
> 
> -:4181: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4181: FILE: drivers/gpu/drm/kmb/kmb_regs.h:46:
> +#define LCD_CTRL_ALPHA_BOTTOM_GL1		  (2<<12)
>                                   		    ^
> 
> -:4182: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4182: FILE: drivers/gpu/drm/kmb/kmb_regs.h:47:
> +#define LCD_CTRL_ALPHA_BOTTOM_GL2		  (3<<12)
>                                   		    ^
> 
> -:4183: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4183: FILE: drivers/gpu/drm/kmb/kmb_regs.h:48:
> +#define LCD_CTRL_TIM_GEN_ENABLE			  (1<<14)
>                                 			    ^
> 
> -:4183: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4183: FILE: drivers/gpu/drm/kmb/kmb_regs.h:48:
> +#define LCD_CTRL_TIM_GEN_ENABLE			  (1<<14)
> 
> -:4184: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4184: FILE: drivers/gpu/drm/kmb/kmb_regs.h:49:
> +#define LCD_CTRL_CONTINUOUS			  (0<<15)
>                             			    ^
> 
> -:4185: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4185: FILE: drivers/gpu/drm/kmb/kmb_regs.h:50:
> +#define LCD_CTRL_ONE_SHOT			  (1<<15)
>                           			    ^
> 
> -:4185: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4185: FILE: drivers/gpu/drm/kmb/kmb_regs.h:50:
> +#define LCD_CTRL_ONE_SHOT			  (1<<15)
> 
> -:4186: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4186: FILE: drivers/gpu/drm/kmb/kmb_regs.h:51:
> +#define LCD_CTRL_PWM0_EN			  (1<<16)
>                          			    ^
> 
> -:4186: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4186: FILE: drivers/gpu/drm/kmb/kmb_regs.h:51:
> +#define LCD_CTRL_PWM0_EN			  (1<<16)
> 
> -:4187: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4187: FILE: drivers/gpu/drm/kmb/kmb_regs.h:52:
> +#define LCD_CTRL_PWM1_EN			  (1<<17)
>                          			    ^
> 
> -:4187: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4187: FILE: drivers/gpu/drm/kmb/kmb_regs.h:52:
> +#define LCD_CTRL_PWM1_EN			  (1<<17)
> 
> -:4188: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4188: FILE: drivers/gpu/drm/kmb/kmb_regs.h:53:
> +#define LCD_CTRL_PWM2_EN			  (1<<18)
>                          			    ^
> 
> -:4188: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4188: FILE: drivers/gpu/drm/kmb/kmb_regs.h:53:
> +#define LCD_CTRL_PWM2_EN			  (1<<18)
> 
> -:4189: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4189: FILE: drivers/gpu/drm/kmb/kmb_regs.h:54:
> +#define LCD_CTRL_OUTPUT_DISABLED		  (0<<19)
>                                  		    ^
> 
> -:4190: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4190: FILE: drivers/gpu/drm/kmb/kmb_regs.h:55:
> +#define LCD_CTRL_OUTPUT_ENABLED			  (1<<19)
>                                 			    ^
> 
> -:4190: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4190: FILE: drivers/gpu/drm/kmb/kmb_regs.h:55:
> +#define LCD_CTRL_OUTPUT_ENABLED			  (1<<19)
> 
> -:4191: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4191: FILE: drivers/gpu/drm/kmb/kmb_regs.h:56:
> +#define LCD_CTRL_BPORCH_ENABLE			  (1<<21)
>                                			    ^
> 
> -:4191: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4191: FILE: drivers/gpu/drm/kmb/kmb_regs.h:56:
> +#define LCD_CTRL_BPORCH_ENABLE			  (1<<21)
> 
> -:4192: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4192: FILE: drivers/gpu/drm/kmb/kmb_regs.h:57:
> +#define LCD_CTRL_FPORCH_ENABLE			  (1<<22)
>                                			    ^
> 
> -:4192: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4192: FILE: drivers/gpu/drm/kmb/kmb_regs.h:57:
> +#define LCD_CTRL_FPORCH_ENABLE			  (1<<22)
> 
> -:4193: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4193: FILE: drivers/gpu/drm/kmb/kmb_regs.h:58:
> +#define LCD_CTRL_PIPELINE_DMA			  (1<<28)
>                               			    ^
> 
> -:4193: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4193: FILE: drivers/gpu/drm/kmb/kmb_regs.h:58:
> +#define LCD_CTRL_PIPELINE_DMA			  (1<<28)
> 
> -:4194: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4194: FILE: drivers/gpu/drm/kmb/kmb_regs.h:59:
> +#define LCD_CTRL_VHSYNC_IDLE_LVL		  (1<<31)
>                                  		    ^
> 
> -:4194: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4194: FILE: drivers/gpu/drm/kmb/kmb_regs.h:59:
> +#define LCD_CTRL_VHSYNC_IDLE_LVL		  (1<<31)
> 
> -:4198: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4198: FILE: drivers/gpu/drm/kmb/kmb_regs.h:63:
> +#define LCD_INT_EOF				  (1<<0)
>                     				    ^
> 
> -:4198: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4198: FILE: drivers/gpu/drm/kmb/kmb_regs.h:63:
> +#define LCD_INT_EOF				  (1<<0)
> 
> -:4199: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4199: FILE: drivers/gpu/drm/kmb/kmb_regs.h:64:
> +#define LCD_INT_LINE_CMP			  (1<<1)
>                          			    ^
> 
> -:4199: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4199: FILE: drivers/gpu/drm/kmb/kmb_regs.h:64:
> +#define LCD_INT_LINE_CMP			  (1<<1)
> 
> -:4200: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4200: FILE: drivers/gpu/drm/kmb/kmb_regs.h:65:
> +#define LCD_INT_VERT_COMP			  (1<<2)
>                           			    ^
> 
> -:4200: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4200: FILE: drivers/gpu/drm/kmb/kmb_regs.h:65:
> +#define LCD_INT_VERT_COMP			  (1<<2)
> 
> -:4201: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4201: FILE: drivers/gpu/drm/kmb/kmb_regs.h:66:
> +#define LAYER0_DMA_DONE				  (1<<3)
>                         				    ^
> 
> -:4201: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4201: FILE: drivers/gpu/drm/kmb/kmb_regs.h:66:
> +#define LAYER0_DMA_DONE				  (1<<3)
> 
> -:4202: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4202: FILE: drivers/gpu/drm/kmb/kmb_regs.h:67:
> +#define LAYER0_DMA_IDLE				  (1<<4)
>                         				    ^
> 
> -:4202: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4202: FILE: drivers/gpu/drm/kmb/kmb_regs.h:67:
> +#define LAYER0_DMA_IDLE				  (1<<4)
> 
> -:4203: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4203: FILE: drivers/gpu/drm/kmb/kmb_regs.h:68:
> +#define LAYER0_DMA_FIFO_OVERFLOW		  (1<<5)
>                                  		    ^
> 
> -:4203: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4203: FILE: drivers/gpu/drm/kmb/kmb_regs.h:68:
> +#define LAYER0_DMA_FIFO_OVERFLOW		  (1<<5)
> 
> -:4204: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4204: FILE: drivers/gpu/drm/kmb/kmb_regs.h:69:
> +#define LAYER0_DMA_FIFO_UNDERFLOW		  (1<<6)
>                                   		    ^
> 
> -:4204: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4204: FILE: drivers/gpu/drm/kmb/kmb_regs.h:69:
> +#define LAYER0_DMA_FIFO_UNDERFLOW		  (1<<6)
> 
> -:4205: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4205: FILE: drivers/gpu/drm/kmb/kmb_regs.h:70:
> +#define LAYER0_DMA_CB_FIFO_OVERFLOW		  (1<<7)
>                                     		    ^
> 
> -:4205: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4205: FILE: drivers/gpu/drm/kmb/kmb_regs.h:70:
> +#define LAYER0_DMA_CB_FIFO_OVERFLOW		  (1<<7)
> 
> -:4206: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4206: FILE: drivers/gpu/drm/kmb/kmb_regs.h:71:
> +#define LAYER0_DMA_CB_FIFO_UNDERFLOW		  (1<<8)
>                                      		    ^
> 
> -:4206: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4206: FILE: drivers/gpu/drm/kmb/kmb_regs.h:71:
> +#define LAYER0_DMA_CB_FIFO_UNDERFLOW		  (1<<8)
> 
> -:4207: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4207: FILE: drivers/gpu/drm/kmb/kmb_regs.h:72:
> +#define LAYER0_DMA_CR_FIFO_OVERFLOW		  (1<<9)
>                                     		    ^
> 
> -:4207: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4207: FILE: drivers/gpu/drm/kmb/kmb_regs.h:72:
> +#define LAYER0_DMA_CR_FIFO_OVERFLOW		  (1<<9)
> 
> -:4208: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4208: FILE: drivers/gpu/drm/kmb/kmb_regs.h:73:
> +#define LAYER0_DMA_CR_FIFO_UNDERFLOW		  (1<<10)
>                                      		    ^
> 
> -:4208: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4208: FILE: drivers/gpu/drm/kmb/kmb_regs.h:73:
> +#define LAYER0_DMA_CR_FIFO_UNDERFLOW		  (1<<10)
> 
> -:4209: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4209: FILE: drivers/gpu/drm/kmb/kmb_regs.h:74:
> +#define LAYER1_DMA_DONE				  (1<<11)
>                         				    ^
> 
> -:4209: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4209: FILE: drivers/gpu/drm/kmb/kmb_regs.h:74:
> +#define LAYER1_DMA_DONE				  (1<<11)
> 
> -:4210: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4210: FILE: drivers/gpu/drm/kmb/kmb_regs.h:75:
> +#define LAYER1_DMA_IDLE				  (1<<12)
>                         				    ^
> 
> -:4210: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4210: FILE: drivers/gpu/drm/kmb/kmb_regs.h:75:
> +#define LAYER1_DMA_IDLE				  (1<<12)
> 
> -:4211: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4211: FILE: drivers/gpu/drm/kmb/kmb_regs.h:76:
> +#define LAYER1_DMA_FIFO_OVERFLOW		  (1<<13)
>                                  		    ^
> 
> -:4211: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4211: FILE: drivers/gpu/drm/kmb/kmb_regs.h:76:
> +#define LAYER1_DMA_FIFO_OVERFLOW		  (1<<13)
> 
> -:4212: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4212: FILE: drivers/gpu/drm/kmb/kmb_regs.h:77:
> +#define LAYER1_DMA_FIFO_UNDERFLOW		  (1<<14)
>                                   		    ^
> 
> -:4212: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4212: FILE: drivers/gpu/drm/kmb/kmb_regs.h:77:
> +#define LAYER1_DMA_FIFO_UNDERFLOW		  (1<<14)
> 
> -:4213: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4213: FILE: drivers/gpu/drm/kmb/kmb_regs.h:78:
> +#define LAYER1_DMA_CB_FIFO_OVERFLOW		  (1<<15)
>                                     		    ^
> 
> -:4213: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4213: FILE: drivers/gpu/drm/kmb/kmb_regs.h:78:
> +#define LAYER1_DMA_CB_FIFO_OVERFLOW		  (1<<15)
> 
> -:4214: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4214: FILE: drivers/gpu/drm/kmb/kmb_regs.h:79:
> +#define LAYER1_DMA_CB_FIFO_UNDERFLOW		  (1<<16)
>                                      		    ^
> 
> -:4214: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4214: FILE: drivers/gpu/drm/kmb/kmb_regs.h:79:
> +#define LAYER1_DMA_CB_FIFO_UNDERFLOW		  (1<<16)
> 
> -:4215: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4215: FILE: drivers/gpu/drm/kmb/kmb_regs.h:80:
> +#define LAYER1_DMA_CR_FIFO_OVERFLOW		  (1<<17)
>                                     		    ^
> 
> -:4215: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4215: FILE: drivers/gpu/drm/kmb/kmb_regs.h:80:
> +#define LAYER1_DMA_CR_FIFO_OVERFLOW		  (1<<17)
> 
> -:4216: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4216: FILE: drivers/gpu/drm/kmb/kmb_regs.h:81:
> +#define LAYER1_DMA_CR_FIFO_UNDERFLOW		  (1<<18)
>                                      		    ^
> 
> -:4216: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4216: FILE: drivers/gpu/drm/kmb/kmb_regs.h:81:
> +#define LAYER1_DMA_CR_FIFO_UNDERFLOW		  (1<<18)
> 
> -:4217: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4217: FILE: drivers/gpu/drm/kmb/kmb_regs.h:82:
> +#define LAYER2_DMA_DONE				  (1<<19)
>                         				    ^
> 
> -:4217: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4217: FILE: drivers/gpu/drm/kmb/kmb_regs.h:82:
> +#define LAYER2_DMA_DONE				  (1<<19)
> 
> -:4218: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4218: FILE: drivers/gpu/drm/kmb/kmb_regs.h:83:
> +#define LAYER2_DMA_IDLE				  (1<<20)
>                         				    ^
> 
> -:4218: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4218: FILE: drivers/gpu/drm/kmb/kmb_regs.h:83:
> +#define LAYER2_DMA_IDLE				  (1<<20)
> 
> -:4219: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4219: FILE: drivers/gpu/drm/kmb/kmb_regs.h:84:
> +#define LAYER2_DMA_FIFO_OVERFLOW		  (1<<21)
>                                  		    ^
> 
> -:4219: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4219: FILE: drivers/gpu/drm/kmb/kmb_regs.h:84:
> +#define LAYER2_DMA_FIFO_OVERFLOW		  (1<<21)
> 
> -:4220: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4220: FILE: drivers/gpu/drm/kmb/kmb_regs.h:85:
> +#define LAYER2_DMA_FIFO_UNDERFLOW		  (1<<22)
>                                   		    ^
> 
> -:4220: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4220: FILE: drivers/gpu/drm/kmb/kmb_regs.h:85:
> +#define LAYER2_DMA_FIFO_UNDERFLOW		  (1<<22)
> 
> -:4221: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4221: FILE: drivers/gpu/drm/kmb/kmb_regs.h:86:
> +#define LAYER3_DMA_DONE				  (1<<23)
>                         				    ^
> 
> -:4221: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4221: FILE: drivers/gpu/drm/kmb/kmb_regs.h:86:
> +#define LAYER3_DMA_DONE				  (1<<23)
> 
> -:4222: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4222: FILE: drivers/gpu/drm/kmb/kmb_regs.h:87:
> +#define LAYER3_DMA_IDLE				  (1<<24)
>                         				    ^
> 
> -:4222: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4222: FILE: drivers/gpu/drm/kmb/kmb_regs.h:87:
> +#define LAYER3_DMA_IDLE				  (1<<24)
> 
> -:4223: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4223: FILE: drivers/gpu/drm/kmb/kmb_regs.h:88:
> +#define LAYER3_DMA_FIFO_OVERFLOW		  (1<<25)
>                                  		    ^
> 
> -:4223: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4223: FILE: drivers/gpu/drm/kmb/kmb_regs.h:88:
> +#define LAYER3_DMA_FIFO_OVERFLOW		  (1<<25)
> 
> -:4224: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4224: FILE: drivers/gpu/drm/kmb/kmb_regs.h:89:
> +#define LAYER3_DMA_FIFO_UNDERFLOW		  (1<<26)
>                                   		    ^
> 
> -:4224: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4224: FILE: drivers/gpu/drm/kmb/kmb_regs.h:89:
> +#define LAYER3_DMA_FIFO_UNDERFLOW		  (1<<26)
> 
> -:4237: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4237: FILE: drivers/gpu/drm/kmb/kmb_regs.h:102:
> +#define LCD_VSTATUS_VERTICAL_STATUS_MASK	  (3<<13)
>                                          	    ^
> 
> -:4238: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4238: FILE: drivers/gpu/drm/kmb/kmb_regs.h:103:
> +#define LCD_VSTATUS_COMPARE_VSYNC		  (0<<13)
>                                   		    ^
> 
> -:4239: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4239: FILE: drivers/gpu/drm/kmb/kmb_regs.h:104:
> +#define LCD_VSTATUS_COMPARE_BACKPORCH		  (1<<13)
>                                       		    ^
> 
> -:4239: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4239: FILE: drivers/gpu/drm/kmb/kmb_regs.h:104:
> +#define LCD_VSTATUS_COMPARE_BACKPORCH		  (1<<13)
> 
> -:4240: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4240: FILE: drivers/gpu/drm/kmb/kmb_regs.h:105:
> +#define LCD_VSTATUS_COMPARE_ACTIVE		  (2<<13)
>                                    		    ^
> 
> -:4241: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4241: FILE: drivers/gpu/drm/kmb/kmb_regs.h:106:
> +#define LCD_VSTATUS_COMPARE_FRONT_PORCH		  (3<<13)
>                                         		    ^
> 
> -:4256: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4256: FILE: drivers/gpu/drm/kmb/kmb_regs.h:121:
> +#define LCD_LAYERn_CFG(N)			(LCD_LAYER0_CFG +
> (0x400*N))
>                           			                        ^
> 
> -:4256: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4256: FILE: drivers/gpu/drm/kmb/kmb_regs.h:121:
> +#define LCD_LAYERn_CFG(N)			(LCD_LAYER0_CFG +
> (0x400*N))
> 
> -:4257: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4257: FILE: drivers/gpu/drm/kmb/kmb_regs.h:122:
> +#define LCD_LAYER_SCALE_H			  (1<<1)
>                           			    ^
> 
> -:4257: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4257: FILE: drivers/gpu/drm/kmb/kmb_regs.h:122:
> +#define LCD_LAYER_SCALE_H			  (1<<1)
> 
> -:4258: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4258: FILE: drivers/gpu/drm/kmb/kmb_regs.h:123:
> +#define LCD_LAYER_SCALE_V			  (1<<2)
>                           			    ^
> 
> -:4258: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4258: FILE: drivers/gpu/drm/kmb/kmb_regs.h:123:
> +#define LCD_LAYER_SCALE_V			  (1<<2)
> 
> -:4261: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4261: FILE: drivers/gpu/drm/kmb/kmb_regs.h:126:
> +#define LCD_LAYER_CSC_EN			  (1<<3)
>                          			    ^
> 
> -:4261: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4261: FILE: drivers/gpu/drm/kmb/kmb_regs.h:126:
> +#define LCD_LAYER_CSC_EN			  (1<<3)
> 
> -:4262: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4262: FILE: drivers/gpu/drm/kmb/kmb_regs.h:127:
> +#define LCD_LAYER_ALPHA_STATIC			  (1<<4)
>                                			    ^
> 
> -:4262: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4262: FILE: drivers/gpu/drm/kmb/kmb_regs.h:127:
> +#define LCD_LAYER_ALPHA_STATIC			  (1<<4)
> 
> -:4263: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4263: FILE: drivers/gpu/drm/kmb/kmb_regs.h:128:
> +#define LCD_LAYER_ALPHA_EMBED			  (1<<5)
>                               			    ^
> 
> -:4263: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4263: FILE: drivers/gpu/drm/kmb/kmb_regs.h:128:
> +#define LCD_LAYER_ALPHA_EMBED			  (1<<5)
> 
> -:4267: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4267: FILE: drivers/gpu/drm/kmb/kmb_regs.h:132:
> +#define LCD_LAYER_ALPHA_PREMULT			  (1<<6)
>                                 			    ^
> 
> -:4267: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4267: FILE: drivers/gpu/drm/kmb/kmb_regs.h:132:
> +#define LCD_LAYER_ALPHA_PREMULT			  (1<<6)
> 
> -:4268: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4268: FILE: drivers/gpu/drm/kmb/kmb_regs.h:133:
> +#define LCD_LAYER_INVERT_COL			  (1<<7)
>                              			    ^
> 
> -:4268: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4268: FILE: drivers/gpu/drm/kmb/kmb_regs.h:133:
> +#define LCD_LAYER_INVERT_COL			  (1<<7)
> 
> -:4269: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4269: FILE: drivers/gpu/drm/kmb/kmb_regs.h:134:
> +#define LCD_LAYER_TRANSPARENT_EN		  (1<<8)
>                                  		    ^
> 
> -:4269: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4269: FILE: drivers/gpu/drm/kmb/kmb_regs.h:134:
> +#define LCD_LAYER_TRANSPARENT_EN		  (1<<8)
> 
> -:4270: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4270: FILE: drivers/gpu/drm/kmb/kmb_regs.h:135:
> +#define LCD_LAYER_FORMAT_YCBCR444PLAN		  (0<<9)
>                                       		    ^
> 
> -:4271: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4271: FILE: drivers/gpu/drm/kmb/kmb_regs.h:136:
> +#define LCD_LAYER_FORMAT_YCBCR422PLAN		  (1<<9)
>                                       		    ^
> 
> -:4271: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4271: FILE: drivers/gpu/drm/kmb/kmb_regs.h:136:
> +#define LCD_LAYER_FORMAT_YCBCR422PLAN		  (1<<9)
> 
> -:4272: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4272: FILE: drivers/gpu/drm/kmb/kmb_regs.h:137:
> +#define LCD_LAYER_FORMAT_YCBCR420PLAN		  (2<<9)
>                                       		    ^
> 
> -:4273: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4273: FILE: drivers/gpu/drm/kmb/kmb_regs.h:138:
> +#define LCD_LAYER_FORMAT_RGB888PLAN		  (3<<9)
>                                     		    ^
> 
> -:4274: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4274: FILE: drivers/gpu/drm/kmb/kmb_regs.h:139:
> +#define LCD_LAYER_FORMAT_YCBCR444LIN		  (4<<9)
>                                      		    ^
> 
> -:4275: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4275: FILE: drivers/gpu/drm/kmb/kmb_regs.h:140:
> +#define LCD_LAYER_FORMAT_YCBCR422LIN		  (5<<9)
>                                      		    ^
> 
> -:4276: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4276: FILE: drivers/gpu/drm/kmb/kmb_regs.h:141:
> +#define LCD_LAYER_FORMAT_RGB888			  (6<<9)
>                                 			    ^
> 
> -:4277: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4277: FILE: drivers/gpu/drm/kmb/kmb_regs.h:142:
> +#define LCD_LAYER_FORMAT_RGBA8888		  (7<<9)
>                                   		    ^
> 
> -:4278: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4278: FILE: drivers/gpu/drm/kmb/kmb_regs.h:143:
> +#define LCD_LAYER_FORMAT_RGBX8888		  (8<<9)
>                                   		    ^
> 
> -:4279: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4279: FILE: drivers/gpu/drm/kmb/kmb_regs.h:144:
> +#define LCD_LAYER_FORMAT_RGB565			  (9<<9)
>                                 			    ^
> 
> -:4280: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4280: FILE: drivers/gpu/drm/kmb/kmb_regs.h:145:
> +#define LCD_LAYER_FORMAT_RGBA1555		  (0xa<<9)
>                                   		      ^
> 
> -:4281: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4281: FILE: drivers/gpu/drm/kmb/kmb_regs.h:146:
> +#define LCD_LAYER_FORMAT_XRGB1555		  (0xb<<9)
>                                   		      ^
> 
> -:4282: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4282: FILE: drivers/gpu/drm/kmb/kmb_regs.h:147:
> +#define LCD_LAYER_FORMAT_RGB444			  (0xc<<9)
>                                 			      ^
> 
> -:4283: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4283: FILE: drivers/gpu/drm/kmb/kmb_regs.h:148:
> +#define LCD_LAYER_FORMAT_RGBA4444		  (0xd<<9)
>                                   		      ^
> 
> -:4284: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4284: FILE: drivers/gpu/drm/kmb/kmb_regs.h:149:
> +#define LCD_LAYER_FORMAT_RGBX4444		  (0xe<<9)
>                                   		      ^
> 
> -:4285: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4285: FILE: drivers/gpu/drm/kmb/kmb_regs.h:150:
> +#define LCD_LAYER_FORMAT_RGB332			  (0xf<<9)
>                                 			      ^
> 
> -:4286: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4286: FILE: drivers/gpu/drm/kmb/kmb_regs.h:151:
> +#define LCD_LAYER_FORMAT_RGBA3328		  (0x10<<9)
>                                   		       ^
> 
> -:4287: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4287: FILE: drivers/gpu/drm/kmb/kmb_regs.h:152:
> +#define LCD_LAYER_FORMAT_RGBX3328		  (0x11<<9)
>                                   		       ^
> 
> -:4288: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4288: FILE: drivers/gpu/drm/kmb/kmb_regs.h:153:
> +#define LCD_LAYER_FORMAT_CLUT			  (0x12<<9)
>                               			       ^
> 
> -:4289: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4289: FILE: drivers/gpu/drm/kmb/kmb_regs.h:154:
> +#define LCD_LAYER_FORMAT_NV12			  (0x1c<<9)
>                               			       ^
> 
> -:4290: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4290: FILE: drivers/gpu/drm/kmb/kmb_regs.h:155:
> +#define LCD_LAYER_PLANAR_STORAGE		  (1<<14)
>                                  		    ^
> 
> -:4290: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4290: FILE: drivers/gpu/drm/kmb/kmb_regs.h:155:
> +#define LCD_LAYER_PLANAR_STORAGE		  (1<<14)
> 
> -:4291: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4291: FILE: drivers/gpu/drm/kmb/kmb_regs.h:156:
> +#define LCD_LAYER_8BPP				  (0<<15)
>                        				    ^
> 
> -:4292: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4292: FILE: drivers/gpu/drm/kmb/kmb_regs.h:157:
> +#define LCD_LAYER_16BPP				  (1<<15)
>                         				    ^
> 
> -:4292: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4292: FILE: drivers/gpu/drm/kmb/kmb_regs.h:157:
> +#define LCD_LAYER_16BPP				  (1<<15)
> 
> -:4293: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4293: FILE: drivers/gpu/drm/kmb/kmb_regs.h:158:
> +#define LCD_LAYER_24BPP				  (2<<15)
>                         				    ^
> 
> -:4294: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4294: FILE: drivers/gpu/drm/kmb/kmb_regs.h:159:
> +#define LCD_LAYER_32BPP				  (3<<15)
>                         				    ^
> 
> -:4295: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4295: FILE: drivers/gpu/drm/kmb/kmb_regs.h:160:
> +#define LCD_LAYER_Y_ORDER			  (1<<17)
>                           			    ^
> 
> -:4295: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4295: FILE: drivers/gpu/drm/kmb/kmb_regs.h:160:
> +#define LCD_LAYER_Y_ORDER			  (1<<17)
> 
> -:4296: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4296: FILE: drivers/gpu/drm/kmb/kmb_regs.h:161:
> +#define LCD_LAYER_CRCB_ORDER			  (1<<18)
>                              			    ^
> 
> -:4296: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4296: FILE: drivers/gpu/drm/kmb/kmb_regs.h:161:
> +#define LCD_LAYER_CRCB_ORDER			  (1<<18)
> 
> -:4297: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4297: FILE: drivers/gpu/drm/kmb/kmb_regs.h:162:
> +#define LCD_LAYER_BGR_ORDER			  (1<<19)
>                             			    ^
> 
> -:4297: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4297: FILE: drivers/gpu/drm/kmb/kmb_regs.h:162:
> +#define LCD_LAYER_BGR_ORDER			  (1<<19)
> 
> -:4298: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4298: FILE: drivers/gpu/drm/kmb/kmb_regs.h:163:
> +#define LCD_LAYER_LUT_2ENT			  (0<<20)
>                            			    ^
> 
> -:4299: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4299: FILE: drivers/gpu/drm/kmb/kmb_regs.h:164:
> +#define LCD_LAYER_LUT_4ENT			  (1<<20)
>                            			    ^
> 
> -:4299: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4299: FILE: drivers/gpu/drm/kmb/kmb_regs.h:164:
> +#define LCD_LAYER_LUT_4ENT			  (1<<20)
> 
> -:4300: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4300: FILE: drivers/gpu/drm/kmb/kmb_regs.h:165:
> +#define LCD_LAYER_LUT_16ENT			  (2<<20)
>                             			    ^
> 
> -:4301: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4301: FILE: drivers/gpu/drm/kmb/kmb_regs.h:166:
> +#define LCD_LAYER_NO_FLIP			  (0<<22)
>                           			    ^
> 
> -:4302: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4302: FILE: drivers/gpu/drm/kmb/kmb_regs.h:167:
> +#define LCD_LAYER_FLIP_V			  (1<<22)
>                          			    ^
> 
> -:4302: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4302: FILE: drivers/gpu/drm/kmb/kmb_regs.h:167:
> +#define LCD_LAYER_FLIP_V			  (1<<22)
> 
> -:4303: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4303: FILE: drivers/gpu/drm/kmb/kmb_regs.h:168:
> +#define LCD_LAYER_FLIP_H			  (2<<22)
>                          			    ^
> 
> -:4304: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4304: FILE: drivers/gpu/drm/kmb/kmb_regs.h:169:
> +#define LCD_LAYER_ROT_R90			  (3<<22)
>                           			    ^
> 
> -:4305: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4305: FILE: drivers/gpu/drm/kmb/kmb_regs.h:170:
> +#define LCD_LAYER_ROT_L90			  (4<<22)
>                           			    ^
> 
> -:4306: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4306: FILE: drivers/gpu/drm/kmb/kmb_regs.h:171:
> +#define LCD_LAYER_ROT_180			  (5<<22)
>                           			    ^
> 
> -:4307: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4307: FILE: drivers/gpu/drm/kmb/kmb_regs.h:172:
> +#define LCD_LAYER_FIFO_00			  (0<<25)
>                           			    ^
> 
> -:4308: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4308: FILE: drivers/gpu/drm/kmb/kmb_regs.h:173:
> +#define LCD_LAYER_FIFO_25			  (1<<25)
>                           			    ^
> 
> -:4308: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4308: FILE: drivers/gpu/drm/kmb/kmb_regs.h:173:
> +#define LCD_LAYER_FIFO_25			  (1<<25)
> 
> -:4309: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4309: FILE: drivers/gpu/drm/kmb/kmb_regs.h:174:
> +#define LCD_LAYER_FIFO_50			  (2<<25)
>                           			    ^
> 
> -:4310: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4310: FILE: drivers/gpu/drm/kmb/kmb_regs.h:175:
> +#define LCD_LAYER_FIFO_100			  (3<<25)
>                            			    ^
> 
> -:4311: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4311: FILE: drivers/gpu/drm/kmb/kmb_regs.h:176:
> +#define LCD_LAYER_INTERLEAVE_DIS		  (0<<27)
>                                  		    ^
> 
> -:4312: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4312: FILE: drivers/gpu/drm/kmb/kmb_regs.h:177:
> +#define LCD_LAYER_INTERLEAVE_V			  (1<<27)
>                                			    ^
> 
> -:4312: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4312: FILE: drivers/gpu/drm/kmb/kmb_regs.h:177:
> +#define LCD_LAYER_INTERLEAVE_V			  (1<<27)
> 
> -:4313: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4313: FILE: drivers/gpu/drm/kmb/kmb_regs.h:178:
> +#define LCD_LAYER_INTERLEAVE_H			  (2<<27)
>                                			    ^
> 
> -:4314: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4314: FILE: drivers/gpu/drm/kmb/kmb_regs.h:179:
> +#define LCD_LAYER_INTERLEAVE_CH			  (3<<27)
>                                 			    ^
> 
> -:4315: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4315: FILE: drivers/gpu/drm/kmb/kmb_regs.h:180:
> +#define LCD_LAYER_INTERLEAVE_V_SUB		  (4<<27)
>                                    		    ^
> 
> -:4316: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4316: FILE: drivers/gpu/drm/kmb/kmb_regs.h:181:
> +#define LCD_LAYER_INTERLEAVE_H_SUB		  (5<<27)
>                                    		    ^
> 
> -:4317: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4317: FILE: drivers/gpu/drm/kmb/kmb_regs.h:182:
> +#define LCD_LAYER_INTERLEAVE_CH_SUB		  (6<<27)
>                                     		    ^
> 
> -:4318: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4318: FILE: drivers/gpu/drm/kmb/kmb_regs.h:183:
> +#define LCD_LAYER_INTER_POS_EVEN		  (0<<30)
>                                  		    ^
> 
> -:4319: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4319: FILE: drivers/gpu/drm/kmb/kmb_regs.h:184:
> +#define LCD_LAYER_INTER_POS_ODD			  (1<<30)
>                                 			    ^
> 
> -:4319: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4319: FILE: drivers/gpu/drm/kmb/kmb_regs.h:184:
> +#define LCD_LAYER_INTER_POS_ODD			  (1<<30)
> 
> -:4322: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4322: FILE: drivers/gpu/drm/kmb/kmb_regs.h:187:
> +#define LCD_LAYERn_COL_START(N)		(LCD_LAYER0_COL_START +
> (0x400*N))
>                                 		                              ^
> 
> -:4322: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4322: FILE: drivers/gpu/drm/kmb/kmb_regs.h:187:
> +#define LCD_LAYERn_COL_START(N)		(LCD_LAYER0_COL_START +
> (0x400*N))
> 
> -:4324: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4324: FILE: drivers/gpu/drm/kmb/kmb_regs.h:189:
> +#define LCD_LAYERn_ROW_START(N)		(LCD_LAYER0_ROW_START +
> (0x400*N))
>                                 		                              ^
> 
> -:4324: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4324: FILE: drivers/gpu/drm/kmb/kmb_regs.h:189:
> +#define LCD_LAYERn_ROW_START(N)		(LCD_LAYER0_ROW_START +
> (0x400*N))
> 
> -:4326: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4326: FILE: drivers/gpu/drm/kmb/kmb_regs.h:191:
> +#define LCD_LAYERn_WIDTH(N)		(LCD_LAYER0_WIDTH +
> (0x400*N))
>                             		                          ^
> 
> -:4326: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4326: FILE: drivers/gpu/drm/kmb/kmb_regs.h:191:
> +#define LCD_LAYERn_WIDTH(N)		(LCD_LAYER0_WIDTH +
> (0x400*N))
> 
> -:4328: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4328: FILE: drivers/gpu/drm/kmb/kmb_regs.h:193:
> +#define LCD_LAYERn_HEIGHT(N)		(LCD_LAYER0_HEIGHT +
> (0x400*N))
>                              		                           ^
> 
> -:4328: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4328: FILE: drivers/gpu/drm/kmb/kmb_regs.h:193:
> +#define LCD_LAYERn_HEIGHT(N)		(LCD_LAYER0_HEIGHT +
> (0x400*N))
> 
> -:4330: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4330: FILE: drivers/gpu/drm/kmb/kmb_regs.h:195:
> +#define LCD_LAYERn_SCALE_CFG(N)		(LCD_LAYER0_SCALE_CFG +
> (0x400*N))
>                                 		                              ^
> 
> -:4330: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_SCALE_CFG>
> #4330: FILE: drivers/gpu/drm/kmb/kmb_regs.h:195:
> +#define LCD_LAYERn_SCALE_CFG(N)		(LCD_LAYER0_SCALE_CFG +
> (0x400*N))
> 
> -:4330: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4330: FILE: drivers/gpu/drm/kmb/kmb_regs.h:195:
> +#define LCD_LAYERn_SCALE_CFG(N)		(LCD_LAYER0_SCALE_CFG +
> (0x400*N))
> 
> -:4332: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4332: FILE: drivers/gpu/drm/kmb/kmb_regs.h:197:
> +#define LCD_LAYERn_ALPHA(N)		(LCD_LAYER0_ALPHA +
> (0x400*N))
>                             		                          ^
> 
> -:4332: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_ALPHA>
> #4332: FILE: drivers/gpu/drm/kmb/kmb_regs.h:197:
> +#define LCD_LAYERn_ALPHA(N)		(LCD_LAYER0_ALPHA +
> (0x400*N))
> 
> -:4332: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4332: FILE: drivers/gpu/drm/kmb/kmb_regs.h:197:
> +#define LCD_LAYERn_ALPHA(N)		(LCD_LAYER0_ALPHA +
> (0x400*N))
> 
> -:4334: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4334: FILE: drivers/gpu/drm/kmb/kmb_regs.h:199:
> +#define LCD_LAYERn_INV_COLOUR_LS(N)
> 	(LCD_LAYER0_INV_COLOUR_LS + (0x400*N))
>                                     	                                  ^
> 
> -:4334: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_INV_COLOUR_LS>
> #4334: FILE: drivers/gpu/drm/kmb/kmb_regs.h:199:
> +#define LCD_LAYERn_INV_COLOUR_LS(N)
> 	(LCD_LAYER0_INV_COLOUR_LS + (0x400*N))
> 
> -:4334: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4334: FILE: drivers/gpu/drm/kmb/kmb_regs.h:199:
> +#define LCD_LAYERn_INV_COLOUR_LS(N)
> 	(LCD_LAYER0_INV_COLOUR_LS + (0x400*N))
> 
> -:4336: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4336: FILE: drivers/gpu/drm/kmb/kmb_regs.h:201:
> +#define LCD_LAYERn_INV_COLOUR_MS(N)
> 	(LCD_LAYER0_INV_COLOUR_MS + (0x400*N))
>                                     	                                  ^
> 
> -:4336: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_INV_COLOUR_MS>
> #4336: FILE: drivers/gpu/drm/kmb/kmb_regs.h:201:
> +#define LCD_LAYERn_INV_COLOUR_MS(N)
> 	(LCD_LAYER0_INV_COLOUR_MS + (0x400*N))
> 
> -:4336: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4336: FILE: drivers/gpu/drm/kmb/kmb_regs.h:201:
> +#define LCD_LAYERn_INV_COLOUR_MS(N)
> 	(LCD_LAYER0_INV_COLOUR_MS + (0x400*N))
> 
> -:4338: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4338: FILE: drivers/gpu/drm/kmb/kmb_regs.h:203:
> +#define LCD_LAYERn_TRANS_COLOUR_LS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_LS + (0x400*N))
>                                       	                                    ^
> 
> -:4338: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_TRANS_COLOUR_LS>
> #4338: FILE: drivers/gpu/drm/kmb/kmb_regs.h:203:
> +#define LCD_LAYERn_TRANS_COLOUR_LS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_LS + (0x400*N))
> 
> -:4338: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4338: FILE: drivers/gpu/drm/kmb/kmb_regs.h:203:
> +#define LCD_LAYERn_TRANS_COLOUR_LS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_LS + (0x400*N))
> 
> -:4340: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4340: FILE: drivers/gpu/drm/kmb/kmb_regs.h:205:
> +#define LCD_LAYERn_TRANS_COLOUR_MS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_MS + (0x400*N))
>                                       	                                    ^
> 
> -:4340: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_TRANS_COLOUR_MS>
> #4340: FILE: drivers/gpu/drm/kmb/kmb_regs.h:205:
> +#define LCD_LAYERn_TRANS_COLOUR_MS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_MS + (0x400*N))
> 
> -:4340: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4340: FILE: drivers/gpu/drm/kmb/kmb_regs.h:205:
> +#define LCD_LAYERn_TRANS_COLOUR_MS(N)
> 	(LCD_LAYER0_TRANS_COLOUR_MS + (0x400*N))
> 
> -:4342: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4342: FILE: drivers/gpu/drm/kmb/kmb_regs.h:207:
> +#define LCD_LAYERn_CSC_COEFF11(N)	(LCD_LAYER0_CSC_COEFF11 +
> (0x400*N))
>                                   	                                ^
> 
> -:4342: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4342: FILE: drivers/gpu/drm/kmb/kmb_regs.h:207:
> +#define LCD_LAYERn_CSC_COEFF11(N)	(LCD_LAYER0_CSC_COEFF11 +
> (0x400*N))
> 
> -:4344: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4344: FILE: drivers/gpu/drm/kmb/kmb_regs.h:209:
> +#define LCD_LAYERn_CSC_COEFF12(N)	(LCD_LAYER0_CSC_COEFF12 +
> (0x400*N))
>                                   	                                ^
> 
> -:4344: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4344: FILE: drivers/gpu/drm/kmb/kmb_regs.h:209:
> +#define LCD_LAYERn_CSC_COEFF12(N)	(LCD_LAYER0_CSC_COEFF12 +
> (0x400*N))
> 
> -:4346: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4346: FILE: drivers/gpu/drm/kmb/kmb_regs.h:211:
> +#define LCD_LAYERn_CSC_COEFF13(N)	(LCD_LAYER0_CSC_COEFF13 +
> (0x400*N))
>                                   	                                ^
> 
> -:4346: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4346: FILE: drivers/gpu/drm/kmb/kmb_regs.h:211:
> +#define LCD_LAYERn_CSC_COEFF13(N)	(LCD_LAYER0_CSC_COEFF13 +
> (0x400*N))
> 
> -:4348: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4348: FILE: drivers/gpu/drm/kmb/kmb_regs.h:213:
> +#define LCD_LAYERn_CSC_COEFF21(N)	(LCD_LAYER0_CSC_COEFF21 +
> (0x400*N))
>                                   	                                ^
> 
> -:4348: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4348: FILE: drivers/gpu/drm/kmb/kmb_regs.h:213:
> +#define LCD_LAYERn_CSC_COEFF21(N)	(LCD_LAYER0_CSC_COEFF21 +
> (0x400*N))
> 
> -:4350: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4350: FILE: drivers/gpu/drm/kmb/kmb_regs.h:215:
> +#define LCD_LAYERn_CSC_COEFF22(N)	(LCD_LAYER0_CSC_COEFF22 +
> (0x400*N))
>                                   	                                ^
> 
> -:4350: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4350: FILE: drivers/gpu/drm/kmb/kmb_regs.h:215:
> +#define LCD_LAYERn_CSC_COEFF22(N)	(LCD_LAYER0_CSC_COEFF22 +
> (0x400*N))
> 
> -:4352: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4352: FILE: drivers/gpu/drm/kmb/kmb_regs.h:217:
> +#define LCD_LAYERn_CSC_COEFF23(N)	(LCD_LAYER0_CSC_COEFF23 +
> (0x400*N))
>                                   	                                ^
> 
> -:4352: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4352: FILE: drivers/gpu/drm/kmb/kmb_regs.h:217:
> +#define LCD_LAYERn_CSC_COEFF23(N)	(LCD_LAYER0_CSC_COEFF23 +
> (0x400*N))
> 
> -:4354: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4354: FILE: drivers/gpu/drm/kmb/kmb_regs.h:219:
> +#define LCD_LAYERn_CSC_COEFF31(N)	(LCD_LAYER0_CSC_COEFF31 +
> (0x400*N))
>                                   	                                ^
> 
> -:4354: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4354: FILE: drivers/gpu/drm/kmb/kmb_regs.h:219:
> +#define LCD_LAYERn_CSC_COEFF31(N)	(LCD_LAYER0_CSC_COEFF31 +
> (0x400*N))
> 
> -:4356: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4356: FILE: drivers/gpu/drm/kmb/kmb_regs.h:221:
> +#define LCD_LAYERn_CSC_COEFF32(N)	  (LCD_LAYER0_CSC_COEFF32 +
> (0x400*N))
>                                   	                                  ^
> 
> -:4356: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4356: FILE: drivers/gpu/drm/kmb/kmb_regs.h:221:
> +#define LCD_LAYERn_CSC_COEFF32(N)	  (LCD_LAYER0_CSC_COEFF32 +
> (0x400*N))
> 
> -:4358: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4358: FILE: drivers/gpu/drm/kmb/kmb_regs.h:223:
> +#define LCD_LAYERn_CSC_COEFF33(N)	(LCD_LAYER0_CSC_COEFF33 +
> (0x400*N))
>                                   	                                ^
> 
> -:4358: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4358: FILE: drivers/gpu/drm/kmb/kmb_regs.h:223:
> +#define LCD_LAYERn_CSC_COEFF33(N)	(LCD_LAYER0_CSC_COEFF33 +
> (0x400*N))
> 
> -:4360: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4360: FILE: drivers/gpu/drm/kmb/kmb_regs.h:225:
> +#define LCD_LAYERn_CSC_OFF1(N)		(LCD_LAYER0_CSC_OFF1 +
> (0x400*N))
>                                		                             ^
> 
> -:4360: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4360: FILE: drivers/gpu/drm/kmb/kmb_regs.h:225:
> +#define LCD_LAYERn_CSC_OFF1(N)		(LCD_LAYER0_CSC_OFF1 +
> (0x400*N))
> 
> -:4362: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4362: FILE: drivers/gpu/drm/kmb/kmb_regs.h:227:
> +#define LCD_LAYERn_CSC_OFF2(N)		(LCD_LAYER0_CSC_OFF2 +
> (0x400*N))
>                                		                             ^
> 
> -:4362: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4362: FILE: drivers/gpu/drm/kmb/kmb_regs.h:227:
> +#define LCD_LAYERn_CSC_OFF2(N)		(LCD_LAYER0_CSC_OFF2 +
> (0x400*N))
> 
> -:4364: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4364: FILE: drivers/gpu/drm/kmb/kmb_regs.h:229:
> +#define LCD_LAYERn_CSC_OFF3(N)		(LCD_LAYER0_CSC_OFF3 +
> (0x400*N))
>                                		                             ^
> 
> -:4364: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4364: FILE: drivers/gpu/drm/kmb/kmb_regs.h:229:
> +#define LCD_LAYERn_CSC_OFF3(N)		(LCD_LAYER0_CSC_OFF3 +
> (0x400*N))
> 
> -:4368: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4368: FILE: drivers/gpu/drm/kmb/kmb_regs.h:233:
> +#define LCD_LAYERn_DMA_CFG(N)
> 	(LCD_LAYER0_DMA_CFG + (0x400*N))
>                               			                            ^
> 
> -:4368: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4368: FILE: drivers/gpu/drm/kmb/kmb_regs.h:233:
> +#define LCD_LAYERn_DMA_CFG(N)
> 	(LCD_LAYER0_DMA_CFG + (0x400*N))
> 
> -:4369: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4369: FILE: drivers/gpu/drm/kmb/kmb_regs.h:234:
> +#define LCD_DMA_LAYER_ENABLE			  (1<<0)
>                              			    ^
> 
> -:4369: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4369: FILE: drivers/gpu/drm/kmb/kmb_regs.h:234:
> +#define LCD_DMA_LAYER_ENABLE			  (1<<0)
> 
> -:4370: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4370: FILE: drivers/gpu/drm/kmb/kmb_regs.h:235:
> +#define LCD_DMA_LAYER_STATUS			  (1<<1)
>                              			    ^
> 
> -:4370: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4370: FILE: drivers/gpu/drm/kmb/kmb_regs.h:235:
> +#define LCD_DMA_LAYER_STATUS			  (1<<1)
> 
> -:4371: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4371: FILE: drivers/gpu/drm/kmb/kmb_regs.h:236:
> +#define LCD_DMA_LAYER_AUTO_UPDATE		  (1<<2)
>                                   		    ^
> 
> -:4371: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4371: FILE: drivers/gpu/drm/kmb/kmb_regs.h:236:
> +#define LCD_DMA_LAYER_AUTO_UPDATE		  (1<<2)
> 
> -:4372: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4372: FILE: drivers/gpu/drm/kmb/kmb_regs.h:237:
> +#define LCD_DMA_LAYER_CONT_UPDATE		  (1<<3)
>                                   		    ^
> 
> -:4372: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4372: FILE: drivers/gpu/drm/kmb/kmb_regs.h:237:
> +#define LCD_DMA_LAYER_CONT_UPDATE		  (1<<3)
> 
> -:4375: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4375: FILE: drivers/gpu/drm/kmb/kmb_regs.h:240:
> +#define LCD_DMA_LAYER_FIFO_ADR_MODE		  (1<<4)
>                                     		    ^
> 
> -:4375: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4375: FILE: drivers/gpu/drm/kmb/kmb_regs.h:240:
> +#define LCD_DMA_LAYER_FIFO_ADR_MODE		  (1<<4)
> 
> -:4376: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4376: FILE: drivers/gpu/drm/kmb/kmb_regs.h:241:
> +#define LCD_DMA_LAYER_AXI_BURST_1		  (1<<5)
>                                   		    ^
> 
> -:4376: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4376: FILE: drivers/gpu/drm/kmb/kmb_regs.h:241:
> +#define LCD_DMA_LAYER_AXI_BURST_1		  (1<<5)
> 
> -:4377: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4377: FILE: drivers/gpu/drm/kmb/kmb_regs.h:242:
> +#define LCD_DMA_LAYER_AXI_BURST_2		  (2<<5)
>                                   		    ^
> 
> -:4378: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4378: FILE: drivers/gpu/drm/kmb/kmb_regs.h:243:
> +#define LCD_DMA_LAYER_AXI_BURST_3		  (3<<5)
>                                   		    ^
> 
> -:4379: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4379: FILE: drivers/gpu/drm/kmb/kmb_regs.h:244:
> +#define LCD_DMA_LAYER_AXI_BURST_4		  (4<<5)
>                                   		    ^
> 
> -:4380: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4380: FILE: drivers/gpu/drm/kmb/kmb_regs.h:245:
> +#define LCD_DMA_LAYER_AXI_BURST_5		  (5<<5)
>                                   		    ^
> 
> -:4381: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4381: FILE: drivers/gpu/drm/kmb/kmb_regs.h:246:
> +#define LCD_DMA_LAYER_AXI_BURST_6		  (6<<5)
>                                   		    ^
> 
> -:4382: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4382: FILE: drivers/gpu/drm/kmb/kmb_regs.h:247:
> +#define LCD_DMA_LAYER_AXI_BURST_7		  (7<<5)
>                                   		    ^
> 
> -:4383: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4383: FILE: drivers/gpu/drm/kmb/kmb_regs.h:248:
> +#define LCD_DMA_LAYER_AXI_BURST_8		  (8<<5)
>                                   		    ^
> 
> -:4384: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4384: FILE: drivers/gpu/drm/kmb/kmb_regs.h:249:
> +#define LCD_DMA_LAYER_AXI_BURST_9		  (9<<5)
>                                   		    ^
> 
> -:4385: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4385: FILE: drivers/gpu/drm/kmb/kmb_regs.h:250:
> +#define LCD_DMA_LAYER_AXI_BURST_10		  (0xa<<5)
>                                    		      ^
> 
> -:4386: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4386: FILE: drivers/gpu/drm/kmb/kmb_regs.h:251:
> +#define LCD_DMA_LAYER_AXI_BURST_11		  (0xb<<5)
>                                    		      ^
> 
> -:4387: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4387: FILE: drivers/gpu/drm/kmb/kmb_regs.h:252:
> +#define LCD_DMA_LAYER_AXI_BURST_12		  (0xc<<5)
>                                    		      ^
> 
> -:4388: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4388: FILE: drivers/gpu/drm/kmb/kmb_regs.h:253:
> +#define LCD_DMA_LAYER_AXI_BURST_13		  (0xd<<5)
>                                    		      ^
> 
> -:4389: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4389: FILE: drivers/gpu/drm/kmb/kmb_regs.h:254:
> +#define LCD_DMA_LAYER_AXI_BURST_14		  (0xe<<5)
>                                    		      ^
> 
> -:4390: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4390: FILE: drivers/gpu/drm/kmb/kmb_regs.h:255:
> +#define LCD_DMA_LAYER_AXI_BURST_15		  (0xf<<5)
>                                    		      ^
> 
> -:4391: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4391: FILE: drivers/gpu/drm/kmb/kmb_regs.h:256:
> +#define LCD_DMA_LAYER_AXI_BURST_16		  (0x10<<5)
>                                    		       ^
> 
> -:4392: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4392: FILE: drivers/gpu/drm/kmb/kmb_regs.h:257:
> +#define LCD_DMA_LAYER_VSTRIDE_EN		  (1<<10)
>                                  		    ^
> 
> -:4392: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4392: FILE: drivers/gpu/drm/kmb/kmb_regs.h:257:
> +#define LCD_DMA_LAYER_VSTRIDE_EN		  (1<<10)
> 
> -:4395: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4395: FILE: drivers/gpu/drm/kmb/kmb_regs.h:260:
> +#define LCD_LAYERn_DMA_START_ADDR(N)
> 	(LCD_LAYER0_DMA_START_ADR \
> +						+ (0x400*N))
> 
> -:4396: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4396: FILE: drivers/gpu/drm/kmb/kmb_regs.h:261:
> +						+ (0x400*N))
>  						        ^
> 
> -:4398: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_SHADOW>
> #4398: FILE: drivers/gpu/drm/kmb/kmb_regs.h:263:
> +#define LCD_LAYERn_DMA_START_SHADOW(N)
> 	(LCD_LAYER0_DMA_START_SHADOW \
> 
> -:4398: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4398: FILE: drivers/gpu/drm/kmb/kmb_regs.h:263:
> +#define LCD_LAYERn_DMA_START_SHADOW(N)
> 	(LCD_LAYER0_DMA_START_SHADOW \
> +						+ (0x400*N))
> 
> -:4399: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4399: FILE: drivers/gpu/drm/kmb/kmb_regs.h:264:
> +						+ (0x400*N))
>  						        ^
> 
> -:4401: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4401: FILE: drivers/gpu/drm/kmb/kmb_regs.h:266:
> +#define LCD_LAYERn_DMA_LEN(N)
> 	(LCD_LAYER0_DMA_LEN + \
> +						(0x400*N))
> 
> -:4402: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4402: FILE: drivers/gpu/drm/kmb/kmb_regs.h:267:
> +						(0x400*N))
>  						      ^
> 
> -:4404: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4404: FILE: drivers/gpu/drm/kmb/kmb_regs.h:269:
> +#define LCD_LAYERn_DMA_LEN_SHADOW(N)
> 	(LCD_LAYER0_DMA_LEN_SHADOW + \
> +						(0x400*N))
> 
> -:4405: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4405: FILE: drivers/gpu/drm/kmb/kmb_regs.h:270:
> +						(0x400*N))
>  						      ^
> 
> -:4407: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_DMA_STATUS>
> #4407: FILE: drivers/gpu/drm/kmb/kmb_regs.h:272:
> +#define LCD_LAYERn_DMA_STATUS(N)		(LCD_LAYER0_DMA_STATUS +
> \
> 
> -:4407: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4407: FILE: drivers/gpu/drm/kmb/kmb_regs.h:272:
> +#define LCD_LAYERn_DMA_STATUS(N)		(LCD_LAYER0_DMA_STATUS +
> \
> +						(0x400*N))
> 
> -:4408: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4408: FILE: drivers/gpu/drm/kmb/kmb_regs.h:273:
> +						(0x400*N))
>  						      ^
> 
> -:4410: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4410: FILE: drivers/gpu/drm/kmb/kmb_regs.h:275:
> +#define LCD_LAYERn_DMA_LINE_WIDTH(N)
> 	(LCD_LAYER0_DMA_LINE_WIDTH + \
> +						(0x400*N))
> 
> -:4411: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4411: FILE: drivers/gpu/drm/kmb/kmb_regs.h:276:
> +						(0x400*N))
>  						      ^
> 
> -:4413: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4413: FILE: drivers/gpu/drm/kmb/kmb_regs.h:278:
> +#define LCD_LAYERn_DMA_LINE_VSTRIDE(N)
> 	(LCD_LAYER0_DMA_LINE_VSTRIDE +\
> +						(0x400*N))
> 
> -:4414: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4414: FILE: drivers/gpu/drm/kmb/kmb_regs.h:279:
> +						(0x400*N))
>  						      ^
> 
> -:4416: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_FIFO_STATUS>
> #4416: FILE: drivers/gpu/drm/kmb/kmb_regs.h:281:
> +#define LCD_LAYERn_DMA_FIFO_STATUS(N)
> 	(LCD_LAYER0_DMA_FIFO_STATUS + \
> 
> -:4416: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4416: FILE: drivers/gpu/drm/kmb/kmb_regs.h:281:
> +#define LCD_LAYERn_DMA_FIFO_STATUS(N)
> 	(LCD_LAYER0_DMA_FIFO_STATUS + \
> +						(0x400*N))
> 
> -:4417: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4417: FILE: drivers/gpu/drm/kmb/kmb_regs.h:282:
> +						(0x400*N))
>  						      ^
> 
> -:4419: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4419: FILE: drivers/gpu/drm/kmb/kmb_regs.h:284:
> +#define LCD_LAYERn_CFG2(N)			(LCD_LAYER0_CFG2 +
> (0x400*N))
>                            			                         ^
> 
> -:4419: CHECK:CAMELCASE: Avoid CamelCase: <LCD_LAYERn_CFG2>
> #4419: FILE: drivers/gpu/drm/kmb/kmb_regs.h:284:
> +#define LCD_LAYERn_CFG2(N)			(LCD_LAYER0_CFG2 +
> (0x400*N))
> 
> -:4419: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4419: FILE: drivers/gpu/drm/kmb/kmb_regs.h:284:
> +#define LCD_LAYERn_CFG2(N)			(LCD_LAYER0_CFG2 +
> (0x400*N))
> 
> -:4421: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4421: FILE: drivers/gpu/drm/kmb/kmb_regs.h:286:
> +#define LCD_LAYERn_DMA_START_CB_ADR(N)
> 	(LCD_LAYER0_DMA_START_CB_ADR + \
> +						(0x20*N))
> 
> -:4422: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4422: FILE: drivers/gpu/drm/kmb/kmb_regs.h:287:
> +						(0x20*N))
>  						     ^
> 
> -:4424: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_CB_SHADOW>
> #4424: FILE: drivers/gpu/drm/kmb/kmb_regs.h:289:
> +#define LCD_LAYERn_DMA_START_CB_SHADOW(N)
> 	(LCD_LAYER0_DMA_START_CB_SHADOW\
> 
> -:4424: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4424: FILE: drivers/gpu/drm/kmb/kmb_regs.h:289:
> +#define LCD_LAYERn_DMA_START_CB_SHADOW(N)
> 	(LCD_LAYER0_DMA_START_CB_SHADOW\
> +						+ (0x20*N))
> 
> -:4425: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4425: FILE: drivers/gpu/drm/kmb/kmb_regs.h:290:
> +						+ (0x20*N))
>  						       ^
> 
> -:4427: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4427: FILE: drivers/gpu/drm/kmb/kmb_regs.h:292:
> +#define LCD_LAYERn_DMA_CB_LINE_WIDTH(N)
> 	(LCD_LAYER0_DMA_CB_LINE_WIDTH +\
> +						(0x20*N))
> 
> -:4428: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4428: FILE: drivers/gpu/drm/kmb/kmb_regs.h:293:
> +						(0x20*N))
>  						     ^
> 
> -:4430: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4430: FILE: drivers/gpu/drm/kmb/kmb_regs.h:295:
> +#define LCD_LAYERn_DMA_CB_LINE_VSTRIDE(N)
> 	(LCD_LAYER0_DMA_CB_LINE_VSTRIDE\
> +						+ (0x20*N))
> 
> -:4431: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4431: FILE: drivers/gpu/drm/kmb/kmb_regs.h:296:
> +						+ (0x20*N))
>  						       ^
> 
> -:4433: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4433: FILE: drivers/gpu/drm/kmb/kmb_regs.h:298:
> +#define LCD_LAYERn_DMA_START_CR_ADR(N)
> 	(LCD_LAYER0_DMA_START_CR_ADR + \
> +						(0x20*N))
> 
> -:4434: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4434: FILE: drivers/gpu/drm/kmb/kmb_regs.h:299:
> +						(0x20*N))
>  						     ^
> 
> -:4436: CHECK:CAMELCASE: Avoid CamelCase:
> <LCD_LAYERn_DMA_START_CR_SHADOW>
> #4436: FILE: drivers/gpu/drm/kmb/kmb_regs.h:301:
> +#define LCD_LAYERn_DMA_START_CR_SHADOW(N)	\
> 
> -:4436: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4436: FILE: drivers/gpu/drm/kmb/kmb_regs.h:301:
> +#define LCD_LAYERn_DMA_START_CR_SHADOW(N)	\
> +
> 	(LCD_LAYER0_DMA_START_CR_SHADOW\
> +						 + (0x20*N))
> 
> -:4438: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4438: FILE: drivers/gpu/drm/kmb/kmb_regs.h:303:
> +						 + (0x20*N))
>  						        ^
> 
> -:4440: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4440: FILE: drivers/gpu/drm/kmb/kmb_regs.h:305:
> +#define LCD_LAYERn_DMA_CR_LINE_WIDTH(N)
> 	(LCD_LAYER0_DMA_CR_LINE_WIDTH +\
> +						(0x20*N))
> 
> -:4441: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4441: FILE: drivers/gpu/drm/kmb/kmb_regs.h:306:
> +						(0x20*N))
>  						     ^
> 
> -:4443: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4443: FILE: drivers/gpu/drm/kmb/kmb_regs.h:308:
> +#define LCD_LAYERn_DMA_CR_LINE_VSTRIDE(N)
> 	(LCD_LAYER0_DMA_CR_LINE_VSTRIDE\
> +						+ (0x20*N))
> 
> -:4444: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4444: FILE: drivers/gpu/drm/kmb/kmb_regs.h:309:
> +						+ (0x20*N))
>  						       ^
> 
> -:4485: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4485: FILE: drivers/gpu/drm/kmb/kmb_regs.h:350:
> +#define LCD_OUTF_BGR_ORDER			  (1 << 5)
> 
> -:4486: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4486: FILE: drivers/gpu/drm/kmb/kmb_regs.h:351:
> +#define LCD_OUTF_Y_ORDER			  (1 << 6)
> 
> -:4487: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4487: FILE: drivers/gpu/drm/kmb/kmb_regs.h:352:
> +#define LCD_OUTF_CRCB_ORDER			  (1 << 7)
> 
> -:4488: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4488: FILE: drivers/gpu/drm/kmb/kmb_regs.h:353:
> +#define LCD_OUTF_SYNC_MODE			  (1 << 11)
> 
> -:4489: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4489: FILE: drivers/gpu/drm/kmb/kmb_regs.h:354:
> +#define LCD_OUTF_RGB_CONV_MODE			  (1 << 14)
> 
> -:4490: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4490: FILE: drivers/gpu/drm/kmb/kmb_regs.h:355:
> +#define LCD_OUTF_MIPI_RGB_MODE			  (1 << 18)
> 
> -:4520: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4520: FILE: drivers/gpu/drm/kmb/kmb_regs.h:385:
> +#define LCD_DMA_STATE_ACTIVE			  (1 << 3)
> 
> -:4540: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'M' may be better
> as '(M)' to avoid precedence issues
> #4540: FILE: drivers/gpu/drm/kmb/kmb_regs.h:405:
> +#define HS_OFFSET(M)				((M + 1) * 0x400)
> 
> -:4544: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4544: FILE: drivers/gpu/drm/kmb/kmb_regs.h:409:
> +#define   HS_CTRL_EN				(1 << 0)
> 
> -:4546: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4546: FILE: drivers/gpu/drm/kmb/kmb_regs.h:411:
> +#define   HS_CTRL_CSIDSIN			(1 << 2)
> 
> -:4548: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4548: FILE: drivers/gpu/drm/kmb/kmb_regs.h:413:
> +#define   TX_SOURCE				(1 << 3)
> 
> -:4551: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4551: FILE: drivers/gpu/drm/kmb/kmb_regs.h:416:
> +#define   DSI_EOTP_EN				(1 << 11)
> 
> -:4552: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4552: FILE: drivers/gpu/drm/kmb/kmb_regs.h:417:
> +#define   DSI_CMD_HFP_EN			(1 << 12)
> 
> -:4553: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4553: FILE: drivers/gpu/drm/kmb/kmb_regs.h:418:
> +#define   CRC_EN				(1 << 14)
> 
> -:4555: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4555: FILE: drivers/gpu/drm/kmb/kmb_regs.h:420:
> +#define   HSCLKIDLE_CNT				(1 << 24)
> 
> -:4559: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4559: FILE: drivers/gpu/drm/kmb/kmb_regs.h:424:
> +#define   LINE_SYNC_PKT_ENABLE			(1 << 0)
> 
> -:4560: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4560: FILE: drivers/gpu/drm/kmb/kmb_regs.h:425:
> +#define   FRAME_COUNTER_ACTIVE			(1 << 1)
> 
> -:4561: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4561: FILE: drivers/gpu/drm/kmb/kmb_regs.h:426:
> +#define   LINE_COUNTER_ACTIVE			(1 << 2)
> 
> -:4562: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4562: FILE: drivers/gpu/drm/kmb/kmb_regs.h:427:
> +#define   DSI_V_BLANKING			(1 << 4)
> 
> -:4563: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4563: FILE: drivers/gpu/drm/kmb/kmb_regs.h:428:
> +#define   DSI_HSA_BLANKING			(1 << 5)
> 
> -:4564: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4564: FILE: drivers/gpu/drm/kmb/kmb_regs.h:429:
> +#define   DSI_HBP_BLANKING			(1 << 6)
> 
> -:4565: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4565: FILE: drivers/gpu/drm/kmb/kmb_regs.h:430:
> +#define   DSI_HFP_BLANKING			(1 << 7)
> 
> -:4566: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4566: FILE: drivers/gpu/drm/kmb/kmb_regs.h:431:
> +#define   DSI_SYNC_PULSE_EVENTN			(1 << 8)
> 
> -:4567: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4567: FILE: drivers/gpu/drm/kmb/kmb_regs.h:432:
> +#define   DSI_LPM_FIRST_VSA_LINE		(1 << 9)
> 
> -:4568: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4568: FILE: drivers/gpu/drm/kmb/kmb_regs.h:433:
> +#define   DSI_LPM_LAST_VFP_LINE			(1 << 10)
> 
> -:4569: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4569: FILE: drivers/gpu/drm/kmb/kmb_regs.h:434:
> +#define   WAIT_ALL_SECT				(1 << 11)
> 
> -:4570: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4570: FILE: drivers/gpu/drm/kmb/kmb_regs.h:435:
> +#define   WAIT_TRIG_POS				(1 << 15)
> 
> -:4575: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4575: FILE: drivers/gpu/drm/kmb/kmb_regs.h:440:
> +#define   MIPI_TXm_HS_FGn_SECTo_PH(M, N, O)
> 	(MIPI_TX0_HS_FG0_SECT0_PH + \
> +						HS_OFFSET(M) + (0x2C*N) +
> (8*O))
> 
> -:4575: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'O' may be better
> as '(O)' to avoid precedence issues
> #4575: FILE: drivers/gpu/drm/kmb/kmb_regs.h:440:
> +#define   MIPI_TXm_HS_FGn_SECTo_PH(M, N, O)
> 	(MIPI_TX0_HS_FG0_SECT0_PH + \
> +						HS_OFFSET(M) + (0x2C*N) +
> (8*O))
> 
> -:4576: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4576: FILE: drivers/gpu/drm/kmb/kmb_regs.h:441:
> +						HS_OFFSET(M) + (0x2C*N) +
> (8*O))
>  						                    ^
> 
> -:4576: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4576: FILE: drivers/gpu/drm/kmb/kmb_regs.h:441:
> +						HS_OFFSET(M) + (0x2C*N) +
> (8*O))
>  						                            ^
> 
> -:4584: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4584: FILE: drivers/gpu/drm/kmb/kmb_regs.h:449:
> +#define   MIPI_TX_SECT_DMA_PACKED		(1<<26)
>                                   		  ^
> 
> -:4584: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4584: FILE: drivers/gpu/drm/kmb/kmb_regs.h:449:
> +#define   MIPI_TX_SECT_DMA_PACKED		(1<<26)
> 
> -:4587: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4587: FILE: drivers/gpu/drm/kmb/kmb_regs.h:452:
> +#define   MIPI_TXm_HS_FGn_SECT_UNPACKED_BYTES0(M, N)	\
> +
> 	(MIPI_TX_HS_FG0_SECT_UNPACKED_BYTES0 \
> +					+ HS_OFFSET(M) + (0x2C*N))
> 
> -:4589: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4589: FILE: drivers/gpu/drm/kmb/kmb_regs.h:454:
> +					+ HS_OFFSET(M) + (0x2C*N))
>  					                      ^
> 
> -:4591: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4591: FILE: drivers/gpu/drm/kmb/kmb_regs.h:456:
> +#define   MIPI_TXm_HS_FGn_SECTo_LINE_CFG(M, N, O)	\
> +				(MIPI_TX_HS_FG0_SECT0_LINE_CFG +
> HS_OFFSET(M) \
> +				+ (0x2C*N) + (8*O))
> 
> -:4591: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'O' may be better
> as '(O)' to avoid precedence issues
> #4591: FILE: drivers/gpu/drm/kmb/kmb_regs.h:456:
> +#define   MIPI_TXm_HS_FGn_SECTo_LINE_CFG(M, N, O)	\
> +				(MIPI_TX_HS_FG0_SECT0_LINE_CFG +
> HS_OFFSET(M) \
> +				+ (0x2C*N) + (8*O))
> 
> -:4593: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4593: FILE: drivers/gpu/drm/kmb/kmb_regs.h:458:
> +				+ (0x2C*N) + (8*O))
>  				       ^
> 
> -:4593: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4593: FILE: drivers/gpu/drm/kmb/kmb_regs.h:458:
> +				+ (0x2C*N) + (8*O))
>  				               ^
> 
> -:4596: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4596: FILE: drivers/gpu/drm/kmb/kmb_regs.h:461:
> +#define   MIPI_TXm_HS_FGn_NUM_LINES(M, N)	\
> +				(MIPI_TX_HS_FG0_NUM_LINES +
> HS_OFFSET(M) \
> +				+ (0x2C*N))
> 
> -:4598: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4598: FILE: drivers/gpu/drm/kmb/kmb_regs.h:463:
> +				+ (0x2C*N))
>  				       ^
> 
> -:4600: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #4600: FILE: drivers/gpu/drm/kmb/kmb_regs.h:465:
> +#define   MIPI_TXm_HS_VSYNC_WIDTHn(M, N)		\
> 
> -:4600: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4600: FILE: drivers/gpu/drm/kmb/kmb_regs.h:465:
> +#define   MIPI_TXm_HS_VSYNC_WIDTHn(M, N)		\
> +				(MIPI_TX_HS_VSYNC_WIDTHS0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4602: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4602: FILE: drivers/gpu/drm/kmb/kmb_regs.h:467:
> +				+ (0x4*N))
>  				      ^
> 
> -:4604: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4604: FILE: drivers/gpu/drm/kmb/kmb_regs.h:469:
> +#define   MIPI_TXm_HS_V_BACKPORCHESn(M, N)	\
> +				(MIPI_TX_HS_V_BACKPORCHES0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4606: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4606: FILE: drivers/gpu/drm/kmb/kmb_regs.h:471:
> +				+ (0x4*N))
>  				      ^
> 
> -:4608: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4608: FILE: drivers/gpu/drm/kmb/kmb_regs.h:473:
> +#define   MIPI_TXm_HS_V_FRONTPORCHESn(M, N)	\
> +				(MIPI_TX_HS_V_FRONTPORCHES0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4610: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4610: FILE: drivers/gpu/drm/kmb/kmb_regs.h:475:
> +				+ (0x4*N))
>  				      ^
> 
> -:4612: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4612: FILE: drivers/gpu/drm/kmb/kmb_regs.h:477:
> +#define   MIPI_TXm_HS_V_ACTIVEn(M, N)		\
> +				(MIPI_TX_HS_V_ACTIVE0 + HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4614: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4614: FILE: drivers/gpu/drm/kmb/kmb_regs.h:479:
> +				+ (0x4*N))
>  				      ^
> 
> -:4616: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #4616: FILE: drivers/gpu/drm/kmb/kmb_regs.h:481:
> +#define   MIPI_TXm_HS_HSYNC_WIDTHn(M, N)		\
> 
> -:4616: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4616: FILE: drivers/gpu/drm/kmb/kmb_regs.h:481:
> +#define   MIPI_TXm_HS_HSYNC_WIDTHn(M, N)		\
> +				(MIPI_TX_HS_HSYNC_WIDTH0 + HS_OFFSET(M)
> \
> +				+ (0x4*N))
> 
> -:4618: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4618: FILE: drivers/gpu/drm/kmb/kmb_regs.h:483:
> +				+ (0x4*N))
>  				      ^
> 
> -:4620: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4620: FILE: drivers/gpu/drm/kmb/kmb_regs.h:485:
> +#define   MIPI_TXm_HS_H_BACKPORCHn(M, N)		\
> +				(MIPI_TX_HS_H_BACKPORCH0 + HS_OFFSET(M)
> \
> +				+ (0x4*N))
> 
> -:4622: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4622: FILE: drivers/gpu/drm/kmb/kmb_regs.h:487:
> +				+ (0x4*N))
>  				      ^
> 
> -:4624: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4624: FILE: drivers/gpu/drm/kmb/kmb_regs.h:489:
> +#define   MIPI_TXm_HS_H_FRONTPORCHn(M, N)	\
> +				(MIPI_TX_HS_H_FRONTPORCH0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4626: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4626: FILE: drivers/gpu/drm/kmb/kmb_regs.h:491:
> +				+ (0x4*N))
>  				      ^
> 
> -:4628: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4628: FILE: drivers/gpu/drm/kmb/kmb_regs.h:493:
> +#define   MIPI_TXm_HS_H_ACTIVEn(M, N)		\
> +				(MIPI_TX_HS_H_ACTIVE0 + HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4630: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4630: FILE: drivers/gpu/drm/kmb/kmb_regs.h:495:
> +				+ (0x4*N))
>  				      ^
> 
> -:4632: WARNING:TYPO_SPELLING: 'WIDTHn' may be misspelled - perhaps
> 'Width'?
> #4632: FILE: drivers/gpu/drm/kmb/kmb_regs.h:497:
> +#define   MIPI_TXm_HS_LLP_HSYNC_WIDTHn(M, N)	\
> 
> -:4632: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4632: FILE: drivers/gpu/drm/kmb/kmb_regs.h:497:
> +#define   MIPI_TXm_HS_LLP_HSYNC_WIDTHn(M, N)	\
> +				(MIPI_TX_HS_LLP_HSYNC_WIDTH0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4634: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4634: FILE: drivers/gpu/drm/kmb/kmb_regs.h:499:
> +				+ (0x4*N))
>  				      ^
> 
> -:4636: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4636: FILE: drivers/gpu/drm/kmb/kmb_regs.h:501:
> +#define   MIPI_TXm_HS_LLP_H_BACKPORCHn(M, N)	\
> +				(MIPI_TX_HS_LLP_H_BACKPORCH0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4638: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4638: FILE: drivers/gpu/drm/kmb/kmb_regs.h:503:
> +				+ (0x4*N))
>  				      ^
> 
> -:4640: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4640: FILE: drivers/gpu/drm/kmb/kmb_regs.h:505:
> +#define   MIPI_TXm_HS_LLP_H_FRONTPORCHn(M, N)	\
> +				(MIPI_TX_HS_LLP_H_FRONTPORCH0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4642: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4642: FILE: drivers/gpu/drm/kmb/kmb_regs.h:507:
> +				+ (0x4*N))
>  				      ^
> 
> -:4650: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_MC_FIFO_CHAN_ALLOCn>
> #4650: FILE: drivers/gpu/drm/kmb/kmb_regs.h:515:
> +#define   MIPI_TXm_HS_MC_FIFO_CHAN_ALLOCn(M, N)	\
> 
> -:4650: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4650: FILE: drivers/gpu/drm/kmb/kmb_regs.h:515:
> +#define   MIPI_TXm_HS_MC_FIFO_CHAN_ALLOCn(M, N)	\
> +			(MIPI_TX_HS_MC_FIFO_CHAN_ALLOC0 +
> HS_OFFSET(M) \
> +			+ (0x4*N))
> 
> -:4652: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4652: FILE: drivers/gpu/drm/kmb/kmb_regs.h:517:
> +			+ (0x4*N))
>  			      ^
> 
> -:4653: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'vc' - possible side-
> effects?
> #4653: FILE: drivers/gpu/drm/kmb/kmb_regs.h:518:
> +#define   SET_MC_FIFO_CHAN_ALLOC(dev, ctrl, vc, sz)	\
> +		kmb_write_bits_mipi(dev, \
> +				MIPI_TXm_HS_MC_FIFO_CHAN_ALLOCn(ctrl, \
> +				vc/2), (vc % 2)*16, 16, sz)
> 
> -:4653: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'vc' may be better
> as '(vc)' to avoid precedence issues
> #4653: FILE: drivers/gpu/drm/kmb/kmb_regs.h:518:
> +#define   SET_MC_FIFO_CHAN_ALLOC(dev, ctrl, vc, sz)	\
> +		kmb_write_bits_mipi(dev, \
> +				MIPI_TXm_HS_MC_FIFO_CHAN_ALLOCn(ctrl, \
> +				vc/2), (vc % 2)*16, 16, sz)
> 
> -:4656: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #4656: FILE: drivers/gpu/drm/kmb/kmb_regs.h:521:
> +				vc/2), (vc % 2)*16, 16, sz)
>  				  ^
> 
> -:4656: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4656: FILE: drivers/gpu/drm/kmb/kmb_regs.h:521:
> +				vc/2), (vc % 2)*16, 16, sz)
>  				               ^
> 
> -:4659: CHECK:CAMELCASE: Avoid CamelCase:
> <MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn>
> #4659: FILE: drivers/gpu/drm/kmb/kmb_regs.h:524:
> +#define   MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn(M, N)	\
> 
> -:4659: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4659: FILE: drivers/gpu/drm/kmb/kmb_regs.h:524:
> +#define   MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn(M, N)	\
> +				(MIPI_TX_HS_MC_FIFO_RTHRESHOLD0 +
> HS_OFFSET(M) \
> +				+ (0x4*N))
> 
> -:4661: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4661: FILE: drivers/gpu/drm/kmb/kmb_regs.h:526:
> +				+ (0x4*N))
>  				      ^
> 
> -:4662: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'vc' - possible side-
> effects?
> #4662: FILE: drivers/gpu/drm/kmb/kmb_regs.h:527:
> +#define   SET_MC_FIFO_RTHRESHOLD(dev, ctrl, vc, th)	\
> +	kmb_write_bits_mipi(dev, MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn(ctrl,
> vc/2), \
> +			(vc % 2)*16, 16, th)
> 
> -:4662: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'vc' may be better
> as '(vc)' to avoid precedence issues
> #4662: FILE: drivers/gpu/drm/kmb/kmb_regs.h:527:
> +#define   SET_MC_FIFO_RTHRESHOLD(dev, ctrl, vc, th)	\
> +	kmb_write_bits_mipi(dev, MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn(ctrl,
> vc/2), \
> +			(vc % 2)*16, 16, th)
> 
> -:4663: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #4663: FILE: drivers/gpu/drm/kmb/kmb_regs.h:528:
> +	kmb_write_bits_mipi(dev, MIPI_TXm_HS_MC_FIFO_RTHRESHOLDn(ctrl,
> vc/2), \
>  	                                                                 ^
> 
> -:4664: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4664: FILE: drivers/gpu/drm/kmb/kmb_regs.h:529:
> +			(vc % 2)*16, 16, th)
>  			        ^
> 
> -:4684: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'M' may be better
> as '(M)' to avoid precedence issues
> #4684: FILE: drivers/gpu/drm/kmb/kmb_regs.h:549:
> +#define   SET_MIPI_CTRL_IRQ_ENABLE0(dev, M, N)	kmb_set_bit_mipi(dev,
> \
> +						MIPI_CTRL_IRQ_ENABLE0,
> M+N)
> 
> -:4684: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4684: FILE: drivers/gpu/drm/kmb/kmb_regs.h:549:
> +#define   SET_MIPI_CTRL_IRQ_ENABLE0(dev, M, N)	kmb_set_bit_mipi(dev,
> \
> +						MIPI_CTRL_IRQ_ENABLE0,
> M+N)
> 
> -:4685: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4685: FILE: drivers/gpu/drm/kmb/kmb_regs.h:550:
> +						MIPI_CTRL_IRQ_ENABLE0,
> M+N)
>  						                        ^
> 
> -:4692: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'M' may be better
> as '(M)' to avoid precedence issues
> #4692: FILE: drivers/gpu/drm/kmb/kmb_regs.h:557:
> +#define   SET_MIPI_CTRL_IRQ_CLEAR0(dev, M, N)		\
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR0, M+N)
> 
> -:4692: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4692: FILE: drivers/gpu/drm/kmb/kmb_regs.h:557:
> +#define   SET_MIPI_CTRL_IRQ_CLEAR0(dev, M, N)		\
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR0, M+N)
> 
> -:4693: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4693: FILE: drivers/gpu/drm/kmb/kmb_regs.h:558:
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR0, M+N)
>  		                                             ^
> 
> -:4695: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'M' may be better
> as '(M)' to avoid precedence issues
> #4695: FILE: drivers/gpu/drm/kmb/kmb_regs.h:560:
> +#define   SET_MIPI_CTRL_IRQ_CLEAR1(dev, M, N)		\
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR1, M+N)
> 
> -:4695: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4695: FILE: drivers/gpu/drm/kmb/kmb_regs.h:560:
> +#define   SET_MIPI_CTRL_IRQ_CLEAR1(dev, M, N)		\
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR1, M+N)
> 
> -:4696: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4696: FILE: drivers/gpu/drm/kmb/kmb_regs.h:561:
> +		kmb_set_bit_mipi(dev, MIPI_CTRL_IRQ_CLEAR1, M+N)
>  		                                             ^
> 
> -:4699: CHECK:CAMELCASE: Avoid CamelCase: <MIPI_TX_HS_IRQ_STATUSm>
> #4699: FILE: drivers/gpu/drm/kmb/kmb_regs.h:564:
> +#define   MIPI_TX_HS_IRQ_STATUSm(M)
> 	(MIPI_TX_HS_IRQ_STATUS + \
> 
> -:4703: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4703: FILE: drivers/gpu/drm/kmb/kmb_regs.h:568:
> +#define   MIPI_TX_HS_IRQ_LINE_COMPARE			(1<<1)
>                                       			  ^
> 
> -:4703: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4703: FILE: drivers/gpu/drm/kmb/kmb_regs.h:568:
> +#define   MIPI_TX_HS_IRQ_LINE_COMPARE			(1<<1)
> 
> -:4704: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4704: FILE: drivers/gpu/drm/kmb/kmb_regs.h:569:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_0			(1<<2)
>                                       			  ^
> 
> -:4704: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4704: FILE: drivers/gpu/drm/kmb/kmb_regs.h:569:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_0			(1<<2)
> 
> -:4705: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4705: FILE: drivers/gpu/drm/kmb/kmb_regs.h:570:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_1			(1<<3)
>                                       			  ^
> 
> -:4705: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4705: FILE: drivers/gpu/drm/kmb/kmb_regs.h:570:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_1			(1<<3)
> 
> -:4706: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4706: FILE: drivers/gpu/drm/kmb/kmb_regs.h:571:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_2			(1<<4)
>                                       			  ^
> 
> -:4706: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4706: FILE: drivers/gpu/drm/kmb/kmb_regs.h:571:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_2			(1<<4)
> 
> -:4707: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4707: FILE: drivers/gpu/drm/kmb/kmb_regs.h:572:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_3			(1<<5)
>                                       			  ^
> 
> -:4707: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4707: FILE: drivers/gpu/drm/kmb/kmb_regs.h:572:
> +#define   MIPI_TX_HS_IRQ_FRAME_DONE_3			(1<<5)
> 
> -:4708: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4708: FILE: drivers/gpu/drm/kmb/kmb_regs.h:573:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_0			(1<<6)
>                                     			  ^
> 
> -:4708: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4708: FILE: drivers/gpu/drm/kmb/kmb_regs.h:573:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_0			(1<<6)
> 
> -:4709: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4709: FILE: drivers/gpu/drm/kmb/kmb_regs.h:574:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_0			(1<<7)
>                                     			  ^
> 
> -:4709: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4709: FILE: drivers/gpu/drm/kmb/kmb_regs.h:574:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_0			(1<<7)
> 
> -:4710: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4710: FILE: drivers/gpu/drm/kmb/kmb_regs.h:575:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_1			(1<<8)
>                                     			  ^
> 
> -:4710: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4710: FILE: drivers/gpu/drm/kmb/kmb_regs.h:575:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_1			(1<<8)
> 
> -:4711: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4711: FILE: drivers/gpu/drm/kmb/kmb_regs.h:576:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_1			(1<<9)
>                                     			  ^
> 
> -:4711: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4711: FILE: drivers/gpu/drm/kmb/kmb_regs.h:576:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_1			(1<<9)
> 
> -:4712: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4712: FILE: drivers/gpu/drm/kmb/kmb_regs.h:577:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_2			(1<<10)
>                                     			  ^
> 
> -:4712: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4712: FILE: drivers/gpu/drm/kmb/kmb_regs.h:577:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_2			(1<<10)
> 
> -:4713: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4713: FILE: drivers/gpu/drm/kmb/kmb_regs.h:578:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_2			(1<<11)
>                                     			  ^
> 
> -:4713: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4713: FILE: drivers/gpu/drm/kmb/kmb_regs.h:578:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_2			(1<<11)
> 
> -:4714: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4714: FILE: drivers/gpu/drm/kmb/kmb_regs.h:579:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_3			(1<<12)
>                                     			  ^
> 
> -:4714: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4714: FILE: drivers/gpu/drm/kmb/kmb_regs.h:579:
> +#define   MIPI_TX_HS_IRQ_DMA_DONE_3			(1<<12)
> 
> -:4715: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4715: FILE: drivers/gpu/drm/kmb/kmb_regs.h:580:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_3			(1<<13)
>                                     			  ^
> 
> -:4715: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4715: FILE: drivers/gpu/drm/kmb/kmb_regs.h:580:
> +#define   MIPI_TX_HS_IRQ_DMA_IDLE_3			(1<<13)
> 
> -:4716: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4716: FILE: drivers/gpu/drm/kmb/kmb_regs.h:581:
> +#define   MIPI_TX_HS_IRQ_MC_FIFO_UNDERFLOW		(1<<14)
>                                            		  ^
> 
> -:4716: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4716: FILE: drivers/gpu/drm/kmb/kmb_regs.h:581:
> +#define   MIPI_TX_HS_IRQ_MC_FIFO_UNDERFLOW		(1<<14)
> 
> -:4717: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4717: FILE: drivers/gpu/drm/kmb/kmb_regs.h:582:
> +#define   MIPI_TX_HS_IRQ_MC_FIFO_OVERFLOW		(1<<15)
>                                           		  ^
> 
> -:4717: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4717: FILE: drivers/gpu/drm/kmb/kmb_regs.h:582:
> +#define   MIPI_TX_HS_IRQ_MC_FIFO_OVERFLOW		(1<<15)
> 
> -:4718: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4718: FILE: drivers/gpu/drm/kmb/kmb_regs.h:583:
> +#define   MIPI_TX_HS_IRQ_LLP_FIFO_EMPTY			(1<<16)
>                                         			  ^
> 
> -:4718: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4718: FILE: drivers/gpu/drm/kmb/kmb_regs.h:583:
> +#define   MIPI_TX_HS_IRQ_LLP_FIFO_EMPTY			(1<<16)
> 
> -:4719: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4719: FILE: drivers/gpu/drm/kmb/kmb_regs.h:584:
> +#define   MIPI_TX_HS_IRQ_LLP_REQUEST_QUEUE_FULL
> 	(1<<17)
>                                                 		  ^
> 
> -:4719: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4719: FILE: drivers/gpu/drm/kmb/kmb_regs.h:584:
> +#define   MIPI_TX_HS_IRQ_LLP_REQUEST_QUEUE_FULL
> 	(1<<17)
> 
> -:4720: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4720: FILE: drivers/gpu/drm/kmb/kmb_regs.h:585:
> +#define   MIPI_TX_HS_IRQ_LLP_REQUEST_QUEUE_ERROR	(1<<18)
>                                                  	  ^
> 
> -:4720: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4720: FILE: drivers/gpu/drm/kmb/kmb_regs.h:585:
> +#define   MIPI_TX_HS_IRQ_LLP_REQUEST_QUEUE_ERROR	(1<<18)
> 
> -:4721: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4721: FILE: drivers/gpu/drm/kmb/kmb_regs.h:586:
> +#define   MIPI_TX_HS_IRQ_LLP_WORD_COUNT_ERROR		(1<<20)
>                                               		  ^
> 
> -:4721: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4721: FILE: drivers/gpu/drm/kmb/kmb_regs.h:586:
> +#define   MIPI_TX_HS_IRQ_LLP_WORD_COUNT_ERROR		(1<<20)
> 
> -:4776: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'N' may be better
> as '(N)' to avoid precedence issues
> #4776: FILE: drivers/gpu/drm/kmb/kmb_regs.h:641:
> +#define   TP_SEL_VCm(M, N)				\
> +				(N << (((M) * 0x04) + 1))
> 
> -:4795: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4795: FILE: drivers/gpu/drm/kmb/kmb_regs.h:660:
> +#define   SET_DPHY_INIT_CTRL0(dev, dphy, offset)	\
> +			kmb_set_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
> 
> -:4795: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'offset' may be
> better as '(offset)' to avoid precedence issues
> #4795: FILE: drivers/gpu/drm/kmb/kmb_regs.h:660:
> +#define   SET_DPHY_INIT_CTRL0(dev, dphy, offset)	\
> +			kmb_set_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
> 
> -:4796: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4796: FILE: drivers/gpu/drm/kmb/kmb_regs.h:661:
> +			kmb_set_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
>  			                                            ^
> 
> -:4797: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4797: FILE: drivers/gpu/drm/kmb/kmb_regs.h:662:
> +#define   CLR_DPHY_INIT_CTRL0(dev, dphy, offset)	\
> +			kmb_clr_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
> 
> -:4797: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'offset' may be
> better as '(offset)' to avoid precedence issues
> #4797: FILE: drivers/gpu/drm/kmb/kmb_regs.h:662:
> +#define   CLR_DPHY_INIT_CTRL0(dev, dphy, offset)	\
> +			kmb_clr_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
> 
> -:4798: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4798: FILE: drivers/gpu/drm/kmb/kmb_regs.h:663:
> +			kmb_clr_bit_mipi(dev, DPHY_INIT_CTRL0,
> (dphy+offset))
>  			                                            ^
> 
> -:4805: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dphy' - possible
> side-effects?
> #4805: FILE: drivers/gpu/drm/kmb/kmb_regs.h:670:
> +#define   SET_DPHY_FREQ_CTRL0_3(dev, dphy, val)	\
> +			kmb_write_bits_mipi(dev, DPHY_FREQ_CTRL0_3 \
> +			+ ((dphy/4)*4), (dphy % 4) * 8, 6, val)
> 
> -:4805: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4805: FILE: drivers/gpu/drm/kmb/kmb_regs.h:670:
> +#define   SET_DPHY_FREQ_CTRL0_3(dev, dphy, val)	\
> +			kmb_write_bits_mipi(dev, DPHY_FREQ_CTRL0_3 \
> +			+ ((dphy/4)*4), (dphy % 4) * 8, 6, val)
> 
> -:4807: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #4807: FILE: drivers/gpu/drm/kmb/kmb_regs.h:672:
> +			+ ((dphy/4)*4), (dphy % 4) * 8, 6, val)
>  			        ^
> 
> -:4807: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4807: FILE: drivers/gpu/drm/kmb/kmb_regs.h:672:
> +			+ ((dphy/4)*4), (dphy % 4) * 8, 6, val)
>  			           ^
> 
> -:4813: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dphy' - possible
> side-effects?
> #4813: FILE: drivers/gpu/drm/kmb/kmb_regs.h:678:
> +#define	  GET_STOPSTATE_DATA(dev, dphy)		\
> +			(((kmb_read_mipi(dev, MIPI_DPHY_STAT0_3 +
> (dphy/4)*4)) \
> +					>> (((dphy % 4)*8)+4)) & 0x03)
> 
> -:4813: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4813: FILE: drivers/gpu/drm/kmb/kmb_regs.h:678:
> +#define	  GET_STOPSTATE_DATA(dev, dphy)		\
> +			(((kmb_read_mipi(dev, MIPI_DPHY_STAT0_3 +
> (dphy/4)*4)) \
> +					>> (((dphy % 4)*8)+4)) & 0x03)
> 
> -:4814: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #4814: FILE: drivers/gpu/drm/kmb/kmb_regs.h:679:
> +			(((kmb_read_mipi(dev, MIPI_DPHY_STAT0_3 +
> (dphy/4)*4)) \
>  			                                               ^
> 
> -:4814: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4814: FILE: drivers/gpu/drm/kmb/kmb_regs.h:679:
> +			(((kmb_read_mipi(dev, MIPI_DPHY_STAT0_3 +
> (dphy/4)*4)) \
>  			                                                  ^
> 
> -:4815: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4815: FILE: drivers/gpu/drm/kmb/kmb_regs.h:680:
> +					>> (((dphy % 4)*8)+4)) & 0x03)
>  					               ^
> 
> -:4815: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4815: FILE: drivers/gpu/drm/kmb/kmb_regs.h:680:
> +					>> (((dphy % 4)*8)+4)) & 0x03)
>  					                  ^
> 
> -:4830: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4830: FILE: drivers/gpu/drm/kmb/kmb_regs.h:695:
> +#define   SET_DPHY_TEST_CTRL1_EN(dev, dphy)	\
> +			kmb_set_bit_mipi(dev, DPHY_TEST_CTRL1, (dphy+12))
> 
> -:4831: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4831: FILE: drivers/gpu/drm/kmb/kmb_regs.h:696:
> +			kmb_set_bit_mipi(dev, DPHY_TEST_CTRL1, (dphy+12))
>  			                                            ^
> 
> -:4832: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4832: FILE: drivers/gpu/drm/kmb/kmb_regs.h:697:
> +#define   CLR_DPHY_TEST_CTRL1_EN(dev, dphy)	\
> +			kmb_clr_bit_mipi(dev, DPHY_TEST_CTRL1, (dphy+12))
> 
> -:4833: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV)
> #4833: FILE: drivers/gpu/drm/kmb/kmb_regs.h:698:
> +			kmb_clr_bit_mipi(dev, DPHY_TEST_CTRL1, (dphy+12))
>  			                                            ^
> 
> -:4837: CHECK:SPACING: spaces preferred around that '%' (ctx:VxV)
> #4837: FILE: drivers/gpu/drm/kmb/kmb_regs.h:702:
> +			4, ((val) << (((dphy)%4)*8)))
>  			                     ^
> 
> -:4837: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4837: FILE: drivers/gpu/drm/kmb/kmb_regs.h:702:
> +			4, ((val) << (((dphy)%4)*8)))
>  			                        ^
> 
> -:4838: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4838: FILE: drivers/gpu/drm/kmb/kmb_regs.h:703:
> +#define DPHY_TEST_DOUT0_3			(0x168)
> 
> -:4839: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4839: FILE: drivers/gpu/drm/kmb/kmb_regs.h:704:
> +#define   GET_TEST_DOUT0_3(dev, dphy)		\
> 
> -:4840: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4840: FILE: drivers/gpu/drm/kmb/kmb_regs.h:705:
> +			(kmb_read_mipi(dev, DPHY_TEST_DOUT0_3) \
> 
> -:4841: CHECK:SPACING: spaces preferred around that '%' (ctx:VxV)
> #4841: FILE: drivers/gpu/drm/kmb/kmb_regs.h:706:
> +			>> (((dphy)%4)*8) & 0xff)
>  			           ^
> 
> -:4841: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4841: FILE: drivers/gpu/drm/kmb/kmb_regs.h:706:
> +			>> (((dphy)%4)*8) & 0xff)
>  			              ^
> 
> -:4842: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4842: FILE: drivers/gpu/drm/kmb/kmb_regs.h:707:
> +#define DPHY_TEST_DOUT4_7			(0x16C)
> 
> -:4843: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4843: FILE: drivers/gpu/drm/kmb/kmb_regs.h:708:
> +#define   GET_TEST_DOUT4_7(dev, dphy)		\
> 
> -:4844: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4844: FILE: drivers/gpu/drm/kmb/kmb_regs.h:709:
> +			(kmb_read_mipi(dev, DPHY_TEST_DOUT4_7) \
> 
> -:4845: CHECK:SPACING: spaces preferred around that '%' (ctx:VxV)
> #4845: FILE: drivers/gpu/drm/kmb/kmb_regs.h:710:
> +			>> (((dphy)%4)*8) & 0xff)
>  			           ^
> 
> -:4845: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #4845: FILE: drivers/gpu/drm/kmb/kmb_regs.h:710:
> +			>> (((dphy)%4)*8) & 0xff)
>  			              ^
> 
> -:4846: WARNING:TYPO_SPELLING: 'DOUT' may be misspelled - perhaps
> 'DOUBT'?
> #4846: FILE: drivers/gpu/drm/kmb/kmb_regs.h:711:
> +#define DPHY_TEST_DOUT8_9			(0x170)
> 
> -:4850: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'dphy' may be
> better as '(dphy)' to avoid precedence issues
> #4850: FILE: drivers/gpu/drm/kmb/kmb_regs.h:715:
> +#define   GET_PLL_LOCK(dev, dphy)		\
> +			(kmb_read_mipi(dev, DPHY_PLL_LOCK) \
> +			& (1 << (dphy - MIPI_DPHY6)))
> 
> -:4859: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4859: FILE: drivers/gpu/drm/kmb/kmb_regs.h:724:
> +#define   LCD					(1<<1)
>               					  ^
> 
> -:4859: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4859: FILE: drivers/gpu/drm/kmb/kmb_regs.h:724:
> +#define   LCD					(1<<1)
> 
> -:4860: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4860: FILE: drivers/gpu/drm/kmb/kmb_regs.h:725:
> +#define   MIPI_COMMON				(1<<2)
>                       				  ^
> 
> -:4860: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4860: FILE: drivers/gpu/drm/kmb/kmb_regs.h:725:
> +#define   MIPI_COMMON				(1<<2)
> 
> -:4861: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
> #4861: FILE: drivers/gpu/drm/kmb/kmb_regs.h:726:
> +#define   MIPI_TX0				(1<<9)
>                    				  ^
> 
> -:4861: CHECK:BIT_MACRO: Prefer using the BIT macro
> #4861: FILE: drivers/gpu/drm/kmb/kmb_regs.h:726:
> +#define   MIPI_TX0				(1<<9)
> 
> -:4870: WARNING:TYPO_SPELLING: 'PRORITY' may be misspelled - perhaps
> 'PRIORITY'?
> #4870: FILE: drivers/gpu/drm/kmb/kmb_regs.h:735:
> +#define LCD_QOS_PRORITY				(0x8)
> 
> total: 0 errors, 25 warnings, 695 checks, 4767 lines checked
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply index

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-30 20:44 [PATCH v4] Add support for KeemBay DRM driver Anitha Chrisanthus
2020-07-30 20:44 ` [PATCH v4] drm/kmb: Add support for KeemBay Display Anitha Chrisanthus
2020-08-02 15:23   ` Sam Ravnborg
2020-08-03 21:02     ` Chrisanthus, Anitha [this message]
2020-08-05 18:04       ` Sam Ravnborg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BY5PR11MB4182B032DC268C4156B4EEE58C4D0@BY5PR11MB4182.namprd11.prod.outlook.com \
    --to=anitha.chrisanthus@intel.com \
    --cc=daniel.vetter@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=edmund.j.dea@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=sam@ravnborg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

dri-devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dri-devel/0 dri-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dri-devel dri-devel/ https://lore.kernel.org/dri-devel \
		dri-devel@lists.freedesktop.org
	public-inbox-index dri-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.freedesktop.lists.dri-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git