From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AF6CF214210 for ; Tue, 23 Sep 2025 14:17:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758637028; cv=none; b=CVGA2Zagg38f0r5mQBEftj1khfyVHxgmkihKqrIhzIZN6aR5Ydpz18MvFrl/JsW/IwQHbCSGEKSL24ncxX/NEnmeThjxZHgTxMEL1xEiMW4Ea/3SN+snRehknAyPT2rA0R5Mo+6Bsd6xihORuSb/1/5T77Vtkxpz2UDbkn2tliU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758637028; c=relaxed/simple; bh=dk+w7GRKuyHJZDHPQMYhc/Zn0Rcn3budkb2GGiwLAcM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=lA2U7h+LrnIzJ/VhWY8coFPk9TX/0K2Wl2TJra/5TZYXY3w045XH+C7Qew3l1D+uTWKhZuxiroU9XRQEpXZOzTUxXno3X5tBfTG8Pv13OgJy0dBt0kOEcYkYwi1p466CWAHoh3gpKp4VmP7SCGY2mpIBhlCQgrFedAzN9zM9gsE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=EIr2t3Jw; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="EIr2t3Jw" Received: from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi [81.175.209.231]) by perceval.ideasonboard.com (Postfix) with UTF8SMTPSA id 567D4606; Tue, 23 Sep 2025 16:15:40 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1758636940; bh=dk+w7GRKuyHJZDHPQMYhc/Zn0Rcn3budkb2GGiwLAcM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EIr2t3JwzYIUjncx/qAyz3kCFAL7RGpOwUjxhjOmkn+wUn0ioOSE0lMNQA8roNFbI apmXXsSOzNmQiF1HH6PAWz8yfUN+CDKBJbXDlfbZh5EnOfNX5lNGh4xty4xrhRIPBv vFxp/JjqY0PGyh0w1B836TNucLyEVxnKRyr47aTI= Date: Tue, 23 Sep 2025 17:16:31 +0300 From: Laurent Pinchart To: Marek Vasut Cc: dri-devel@lists.freedesktop.org, David Airlie , Geert Uytterhoeven , Kieran Bingham , Maarten Lankhorst , Magnus Damm , Maxime Ripard , Simona Vetter , Thomas Zimmermann , Tomi Valkeinen , linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH 8/9] drm/rcar-du: dsi: Convert register bits to BIT() macro Message-ID: <20250923141631.GJ20765@pendragon.ideasonboard.com> References: <20250922185740.153759-1-marek.vasut+renesas@mailbox.org> <20250922185740.153759-9-marek.vasut+renesas@mailbox.org> Precedence: bulk X-Mailing-List: linux-renesas-soc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20250922185740.153759-9-marek.vasut+renesas@mailbox.org> On Mon, Sep 22, 2025 at 08:55:04PM +0200, Marek Vasut wrote: > Convert register bits to BIT() macro where applicable. This is done > automatically using regex 's@(1 << \([0-9]\+\))@BIT(\1)', except for > bitfields which are manually updated to use GENMASK(). > > Signed-off-by: Marek Vasut > --- > Cc: David Airlie > Cc: Geert Uytterhoeven > Cc: Kieran Bingham > Cc: Laurent Pinchart > Cc: Maarten Lankhorst > Cc: Magnus Damm > Cc: Maxime Ripard > Cc: Simona Vetter > Cc: Thomas Zimmermann > Cc: Tomi Valkeinen > Cc: dri-devel@lists.freedesktop.org > Cc: linux-renesas-soc@vger.kernel.org > --- > .../drm/renesas/rcar-du/rcar_mipi_dsi_regs.h | 248 +++++++++--------- > 1 file changed, 124 insertions(+), 124 deletions(-) > > diff --git a/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h b/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h > index 48c3b679b2663..29c806cae3557 100644 > --- a/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h > +++ b/drivers/gpu/drm/renesas/rcar-du/rcar_mipi_dsi_regs.h > @@ -9,37 +9,37 @@ > #define __RCAR_MIPI_DSI_REGS_H__ > > #define LINKSR 0x010 > -#define LINKSR_LPBUSY (1 << 1) > -#define LINKSR_HSBUSY (1 << 0) > +#define LINKSR_LPBUSY BIT(1) > +#define LINKSR_HSBUSY BIT(0) > > #define TXSETR 0x100 > -#define TXSETR_LANECNT_MASK (0x3 << 0) > +#define TXSETR_LANECNT_MASK GENMASK(1, 0) > > /* > * DSI Command Transfer Registers > */ > #define TXCMSETR 0x110 > -#define TXCMSETR_SPDTYP (1 << 8) /* 0:HS 1:LP */ > -#define TXCMSETR_LPPDACC (1 << 0) > +#define TXCMSETR_SPDTYP BIT(8) /* 0:HS 1:LP */ > +#define TXCMSETR_LPPDACC BIT(0) > #define TXCMCR 0x120 > -#define TXCMCR_BTATYP (1 << 2) > -#define TXCMCR_BTAREQ (1 << 1) > -#define TXCMCR_TXREQ (1 << 0) > +#define TXCMCR_BTATYP BIT(2) > +#define TXCMCR_BTAREQ BIT(1) > +#define TXCMCR_TXREQ BIT(0) > #define TXCMSR 0x130 > -#define TXCMSR_CLSNERR (1 << 18) > -#define TXCMSR_AXIERR (1 << 16) > -#define TXCMSR_TXREQEND (1 << 0) > +#define TXCMSR_CLSNERR BIT(18) > +#define TXCMSR_AXIERR BIT(16) > +#define TXCMSR_TXREQEND BIT(0) > #define TXCMSCR 0x134 > -#define TXCMSCR_CLSNERR (1 << 18) > -#define TXCMSCR_AXIERR (1 << 16) > -#define TXCMSCR_TXREQEND (1 << 0) > +#define TXCMSCR_CLSNERR BIT(18) > +#define TXCMSCR_AXIERR BIT(16) > +#define TXCMSCR_TXREQEND BIT(0) > #define TXCMIER 0x138 > -#define TXCMIER_CLSNERR (1 << 18) > -#define TXCMIER_AXIERR (1 << 16) > -#define TXCMIER_TXREQEND (1 << 0) > +#define TXCMIER_CLSNERR BIT(18) > +#define TXCMIER_AXIERR BIT(16) > +#define TXCMIER_TXREQEND BIT(0) > #define TXCMADDRSET0R 0x140 > #define TXCMPHDR 0x150 > -#define TXCMPHDR_FMT (1 << 24) /* 0:SP 1:LP */ > +#define TXCMPHDR_FMT BIT(24) /* 0:SP 1:LP */ > #define TXCMPHDR_VC(n) (((n) & 0x3) << 22) > #define TXCMPHDR_DT(n) (((n) & 0x3f) << 16) > #define TXCMPHDR_DATA1(n) (((n) & 0xff) << 8) > @@ -53,63 +53,63 @@ > #define RXSETR_CRCEN(n) (((n) & 0xf) << 24) > #define RXSETR_ECCEN(n) (((n) & 0xf) << 16) > #define RXPSETR 0x210 > -#define RXPSETR_LPPDACC (1 << 0) > +#define RXPSETR_LPPDACC BIT(0) > #define RXPSR 0x220 > -#define RXPSR_ECCERR1B (1 << 28) > -#define RXPSR_UEXTRGERR (1 << 25) > -#define RXPSR_RESPTOERR (1 << 24) > -#define RXPSR_OVRERR (1 << 23) > -#define RXPSR_AXIERR (1 << 22) > -#define RXPSR_CRCERR (1 << 21) > -#define RXPSR_WCERR (1 << 20) > -#define RXPSR_UEXDTERR (1 << 19) > -#define RXPSR_UEXPKTERR (1 << 18) > -#define RXPSR_ECCERR (1 << 17) > -#define RXPSR_MLFERR (1 << 16) > -#define RXPSR_RCVACK (1 << 14) > -#define RXPSR_RCVEOT (1 << 10) > -#define RXPSR_RCVAKE (1 << 9) > -#define RXPSR_RCVRESP (1 << 8) > -#define RXPSR_BTAREQEND (1 << 0) > +#define RXPSR_ECCERR1B BIT(28) > +#define RXPSR_UEXTRGERR BIT(25) > +#define RXPSR_RESPTOERR BIT(24) > +#define RXPSR_OVRERR BIT(23) > +#define RXPSR_AXIERR BIT(22) > +#define RXPSR_CRCERR BIT(21) > +#define RXPSR_WCERR BIT(20) > +#define RXPSR_UEXDTERR BIT(19) > +#define RXPSR_UEXPKTERR BIT(18) > +#define RXPSR_ECCERR BIT(17) > +#define RXPSR_MLFERR BIT(16) > +#define RXPSR_RCVACK BIT(14) > +#define RXPSR_RCVEOT BIT(10) > +#define RXPSR_RCVAKE BIT(9) > +#define RXPSR_RCVRESP BIT(8) > +#define RXPSR_BTAREQEND BIT(0) > #define RXPSCR 0x224 > -#define RXPSCR_ECCERR1B (1 << 28) > -#define RXPSCR_UEXTRGERR (1 << 25) > -#define RXPSCR_RESPTOERR (1 << 24) > -#define RXPSCR_OVRERR (1 << 23) > -#define RXPSCR_AXIERR (1 << 22) > -#define RXPSCR_CRCERR (1 << 21) > -#define RXPSCR_WCERR (1 << 20) > -#define RXPSCR_UEXDTERR (1 << 19) > -#define RXPSCR_UEXPKTERR (1 << 18) > -#define RXPSCR_ECCERR (1 << 17) > -#define RXPSCR_MLFERR (1 << 16) > -#define RXPSCR_RCVACK (1 << 14) > -#define RXPSCR_RCVEOT (1 << 10) > -#define RXPSCR_RCVAKE (1 << 9) > -#define RXPSCR_RCVRESP (1 << 8) > -#define RXPSCR_BTAREQEND (1 << 0) > +#define RXPSCR_ECCERR1B BIT(28) > +#define RXPSCR_UEXTRGERR BIT(25) > +#define RXPSCR_RESPTOERR BIT(24) > +#define RXPSCR_OVRERR BIT(23) > +#define RXPSCR_AXIERR BIT(22) > +#define RXPSCR_CRCERR BIT(21) > +#define RXPSCR_WCERR BIT(20) > +#define RXPSCR_UEXDTERR BIT(19) > +#define RXPSCR_UEXPKTERR BIT(18) > +#define RXPSCR_ECCERR BIT(17) > +#define RXPSCR_MLFERR BIT(16) > +#define RXPSCR_RCVACK BIT(14) > +#define RXPSCR_RCVEOT BIT(10) > +#define RXPSCR_RCVAKE BIT(9) > +#define RXPSCR_RCVRESP BIT(8) > +#define RXPSCR_BTAREQEND BIT(0) > #define RXPIER 0x228 > -#define RXPIER_ECCERR1B (1 << 28) > -#define RXPIER_UEXTRGERR (1 << 25) > -#define RXPIER_RESPTOERR (1 << 24) > -#define RXPIER_OVRERR (1 << 23) > -#define RXPIER_AXIERR (1 << 22) > -#define RXPIER_CRCERR (1 << 21) > -#define RXPIER_WCERR (1 << 20) > -#define RXPIER_UEXDTERR (1 << 19) > -#define RXPIER_UEXPKTERR (1 << 18) > -#define RXPIER_ECCERR (1 << 17) > -#define RXPIER_MLFERR (1 << 16) > -#define RXPIER_RCVACK (1 << 14) > -#define RXPIER_RCVEOT (1 << 10) > -#define RXPIER_RCVAKE (1 << 9) > -#define RXPIER_RCVRESP (1 << 8) > -#define RXPIER_BTAREQEND (1 << 0) > +#define RXPIER_ECCERR1B BIT(28) > +#define RXPIER_UEXTRGERR BIT(25) > +#define RXPIER_RESPTOERR BIT(24) > +#define RXPIER_OVRERR BIT(23) > +#define RXPIER_AXIERR BIT(22) > +#define RXPIER_CRCERR BIT(21) > +#define RXPIER_WCERR BIT(20) > +#define RXPIER_UEXDTERR BIT(19) > +#define RXPIER_UEXPKTERR BIT(18) > +#define RXPIER_ECCERR BIT(17) > +#define RXPIER_MLFERR BIT(16) > +#define RXPIER_RCVACK BIT(14) > +#define RXPIER_RCVEOT BIT(10) > +#define RXPIER_RCVAKE BIT(9) > +#define RXPIER_RCVRESP BIT(8) > +#define RXPIER_BTAREQEND BIT(0) > #define RXPADDRSET0R 0x230 > #define RXPSIZESETR 0x238 > #define RXPSIZESETR_SIZE(n) (((n) & 0xf) << 3) > #define RXPHDR 0x240 > -#define RXPHDR_FMT (1 << 24) /* 0:SP 1:LP */ > +#define RXPHDR_FMT BIT(24) /* 0:SP 1:LP */ > #define RXPHDR_VC(n) (((n) & 0x3) << 22) > #define RXPHDR_DT(n) (((n) & 0x3f) << 16) > #define RXPHDR_DATA1(n) (((n) & 0xff) << 8) > @@ -128,38 +128,38 @@ > #define TASCR 0x514 > #define TAIER 0x518 > #define TOSR 0x610 > -#define TOSR_TATO (1 << 2) > -#define TOSR_LRXHTO (1 << 1) > -#define TOSR_HRXTO (1 << 0) > +#define TOSR_TATO BIT(2) > +#define TOSR_LRXHTO BIT(1) > +#define TOSR_HRXTO BIT(0) > #define TOSCR 0x614 > -#define TOSCR_TATO (1 << 2) > -#define TOSCR_LRXHTO (1 << 1) > -#define TOSCR_HRXTO (1 << 0) > +#define TOSCR_TATO BIT(2) > +#define TOSCR_LRXHTO BIT(1) > +#define TOSCR_HRXTO BIT(0) > > /* > * Video Mode Register > */ > #define TXVMSETR 0x180 > -#define TXVMSETR_SYNSEQ_EVENTS (1 << 16) /* 0:Pulses 1:Events */ > -#define TXVMSETR_VSTPM (1 << 15) > -#define TXVMSETR_PIXWDTH (1 << 8) > -#define TXVMSETR_VSEN (1 << 4) > -#define TXVMSETR_HFPBPEN (1 << 2) > -#define TXVMSETR_HBPBPEN (1 << 1) > -#define TXVMSETR_HSABPEN (1 << 0) > +#define TXVMSETR_SYNSEQ_EVENTS BIT(16) /* 0:Pulses 1:Events */ > +#define TXVMSETR_VSTPM BIT(15) > +#define TXVMSETR_PIXWDTH BIT(8) This is a 3 bits field according to the documentation. > +#define TXVMSETR_VSEN BIT(4) > +#define TXVMSETR_HFPBPEN BIT(2) > +#define TXVMSETR_HBPBPEN BIT(1) > +#define TXVMSETR_HSABPEN BIT(0) > > #define TXVMCR 0x190 > -#define TXVMCR_VFCLR (1 << 12) > -#define TXVMCR_EN_VIDEO (1 << 0) > +#define TXVMCR_VFCLR BIT(12) > +#define TXVMCR_EN_VIDEO BIT(0) > > #define TXVMSR 0x1a0 > -#define TXVMSR_STR (1 << 16) > -#define TXVMSR_VFRDY (1 << 12) > -#define TXVMSR_ACT (1 << 8) > -#define TXVMSR_RDY (1 << 0) > +#define TXVMSR_STR BIT(16) > +#define TXVMSR_VFRDY BIT(12) > +#define TXVMSR_ACT BIT(8) > +#define TXVMSR_RDY BIT(0) > > #define TXVMSCR 0x1a4 > -#define TXVMSCR_STR (1 << 16) > +#define TXVMSCR_STR BIT(16) > > #define TXVMPSPHSETR 0x1c0 > #define TXVMPSPHSETR_DT_MASK (0x3f << 16) > @@ -170,10 +170,10 @@ > #define TXVMPSPHSETR_DT_YCBCR16 0x2c > > #define TXVMVPRMSET0R 0x1d0 > -#define TXVMVPRMSET0R_HSPOL_LOW (1 << 17) /* 0:High 1:Low */ > -#define TXVMVPRMSET0R_VSPOL_LOW (1 << 16) /* 0:High 1:Low */ > -#define TXVMVPRMSET0R_CSPC_YCbCr (1 << 4) /* 0:RGB 1:YCbCr */ > -#define TXVMVPRMSET0R_BPP_MASK (7 << 0) > +#define TXVMVPRMSET0R_HSPOL_LOW BIT(17) /* 0:High 1:Low */ > +#define TXVMVPRMSET0R_VSPOL_LOW BIT(16) /* 0:High 1:Low */ > +#define TXVMVPRMSET0R_CSPC_YCbCr BIT(4) /* 0:RGB 1:YCbCr */ This is a 2 bits field. > +#define TXVMVPRMSET0R_BPP_MASK GENMASK(2, 0) > #define TXVMVPRMSET0R_BPP_16 0 > #define TXVMVPRMSET0R_BPP_18 1 > #define TXVMVPRMSET0R_BPP_24 2 > @@ -198,51 +198,51 @@ > * PHY-Protocol Interface (PPI) Registers > */ > #define PPISETR 0x700 > -#define PPISETR_DLEN_MASK (0xf << 0) > -#define PPISETR_CLEN (1 << 8) > +#define PPISETR_DLEN_MASK GENMASK(3, 0) > +#define PPISETR_CLEN BIT(8) While at it I'd swap those two lines, to sort fields by decreasing bit position as in the rest of the file. > > #define PPICLCR 0x710 > -#define PPICLCR_TXREQHS (1 << 8) > -#define PPICLCR_TXULPSEXT (1 << 1) > -#define PPICLCR_TXULPSCLK (1 << 0) > +#define PPICLCR_TXREQHS BIT(8) > +#define PPICLCR_TXULPSEXT BIT(1) > +#define PPICLCR_TXULPSCLK BIT(0) > > #define PPICLSR 0x720 > -#define PPICLSR_HSTOLP (1 << 27) > -#define PPICLSR_TOHS (1 << 26) > -#define PPICLSR_STPST (1 << 0) > +#define PPICLSR_HSTOLP BIT(27) > +#define PPICLSR_TOHS BIT(26) > +#define PPICLSR_STPST BIT(0) > > #define PPICLSCR 0x724 > -#define PPICLSCR_HSTOLP (1 << 27) > -#define PPICLSCR_TOHS (1 << 26) > +#define PPICLSCR_HSTOLP BIT(27) > +#define PPICLSCR_TOHS BIT(26) > > #define PPIDL0SR 0x740 > -#define PPIDL0SR_DIR (1 << 10) > -#define PPIDL0SR_STPST (1 << 6) > +#define PPIDL0SR_DIR BIT(10) > +#define PPIDL0SR_STPST BIT(6) > > #define PPIDLSR 0x760 > -#define PPIDLSR_STPST (0xf << 0) > +#define PPIDLSR_STPST GENMASK(3, 0) > > /* > * Clocks registers > */ > #define LPCLKSET 0x1000 > -#define LPCLKSET_CKEN (1 << 8) > +#define LPCLKSET_CKEN BIT(8) > #define LPCLKSET_LPCLKDIV(x) (((x) & 0x3f) << 0) > > #define CFGCLKSET 0x1004 > -#define CFGCLKSET_CKEN (1 << 8) > +#define CFGCLKSET_CKEN BIT(8) > #define CFGCLKSET_CFGCLKDIV(x) (((x) & 0x3f) << 0) > > #define DOTCLKDIV 0x1008 > -#define DOTCLKDIV_CKEN (1 << 8) > +#define DOTCLKDIV_CKEN BIT(8) > #define DOTCLKDIV_DOTCLKDIV(x) (((x) & 0x3f) << 0) > > #define VCLKSET 0x100c > -#define VCLKSET_CKEN (1 << 16) > -#define VCLKSET_COLOR_YCC (1 << 8) /* 0:RGB 1:YCbCr */ > +#define VCLKSET_CKEN BIT(16) > +#define VCLKSET_COLOR_YCC BIT(8) /* 0:RGB 1:YCbCr */ > #define VCLKSET_DIV_V3U(x) (((x) & 0x3) << 4) > #define VCLKSET_DIV_V4H(x) (((x) & 0x7) << 4) > -#define VCLKSET_BPP_MASK (3 << 2) > +#define VCLKSET_BPP_MASK GENMASK(3, 2) > #define VCLKSET_BPP_16 0 > #define VCLKSET_BPP_18 1 > #define VCLKSET_BPP_18L 2 > @@ -250,24 +250,24 @@ > #define VCLKSET_LANE(x) (((x) & 0x3) << 0) > > #define VCLKEN 0x1010 > -#define VCLKEN_CKEN (1 << 0) > +#define VCLKEN_CKEN BIT(0) > > #define PHYSETUP 0x1014 > #define PHYSETUP_HSFREQRANGE(x) (((x) & 0x7f) << 16) > -#define PHYSETUP_HSFREQRANGE_MASK (0x7f << 16) > +#define PHYSETUP_HSFREQRANGE_MASK GENMASK(22, 16) > #define PHYSETUP_CFGCLKFREQRANGE(x) (((x) & 0x3f) << 8) > -#define PHYSETUP_SHUTDOWNZ (1 << 1) > -#define PHYSETUP_RSTZ (1 << 0) > +#define PHYSETUP_SHUTDOWNZ BIT(1) > +#define PHYSETUP_RSTZ BIT(0) > > #define CLOCKSET1 0x101c > -#define CLOCKSET1_LOCK_PHY (1 << 17) > -#define CLOCKSET1_CLKSEL (1 << 8) > -#define CLOCKSET1_CLKINSEL_MASK (3 << 2) > +#define CLOCKSET1_LOCK_PHY BIT(17) > +#define CLOCKSET1_CLKSEL BIT(8) This is a two bits field. > +#define CLOCKSET1_CLKINSEL_MASK GENMASK(3, 2) > #define CLOCKSET1_CLKINSEL_EXTAL 0 > #define CLOCKSET1_CLKINSEL_DIG 1 > #define CLOCKSET1_CLKINSEL_DU 2 > -#define CLOCKSET1_SHADOW_CLEAR (1 << 1) > -#define CLOCKSET1_UPDATEPLL (1 << 0) > +#define CLOCKSET1_SHADOW_CLEAR BIT(1) > +#define CLOCKSET1_UPDATEPLL BIT(0) > > #define CLOCKSET2 0x1020 > #define CLOCKSET2_M(x) (((x) & 0xfff) << 16) > @@ -281,15 +281,15 @@ > #define CLOCKSET3_GMP_CNTRL(x) (((x) & 0x3) << 0) > > #define PHTW 0x1034 > -#define PHTW_DWEN (1 << 24) > +#define PHTW_DWEN BIT(24) > #define PHTW_TESTDIN_DATA(x) (((x) & 0xff) << 16) > -#define PHTW_CWEN (1 << 8) > +#define PHTW_CWEN BIT(8) > #define PHTW_TESTDIN_CODE(x) (((x) & 0xff) << 0) > > #define PHTR 0x1038 > -#define PHTR_TEST (1 << 16) > +#define PHTR_TEST BIT(16) And this is a 8 bits field. > > #define PHTC 0x103c > -#define PHTC_TESTCLR (1 << 0) > +#define PHTC_TESTCLR BIT(0) > > #endif /* __RCAR_MIPI_DSI_REGS_H__ */ -- Regards, Laurent Pinchart