From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.5 required=3.0 tests=BAYES_00,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UPPERCASE_50_75, URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6F1E6C433F5 for ; Sun, 5 Sep 2021 13:06:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5041C60F5E for ; Sun, 5 Sep 2021 13:06:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231304AbhIENH0 (ORCPT ); Sun, 5 Sep 2021 09:07:26 -0400 Received: from mail.kernel.org ([198.145.29.99]:35048 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229759AbhIENHZ (ORCPT ); Sun, 5 Sep 2021 09:07:25 -0400 Received: from jic23-huawei (cpc108967-cmbg20-2-0-cust86.5-4.cable.virginm.net [81.101.6.87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 2AA996054F; Sun, 5 Sep 2021 13:06:15 +0000 (UTC) Date: Sun, 5 Sep 2021 14:09:38 +0100 From: Jonathan Cameron To: Miquel Raynal Cc: Lars-Peter Clausen , Peter Meerwald-Stadler , Rob Herring , Dmitry Torokhov , Lee Jones , bcousson@baylibre.com, Tony Lindgren , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-input@vger.kernel.org, linux-omap@vger.kernel.org, Thomas Petazzoni , Vignesh Raghavendra , Lokesh Vutla , Tero Kristo , Ryan Barnett , Grygorii Strashko , Jason Reeder , Jonathan Cameron Subject: Re: [PATCH v2 25/46] mfd: ti_am335x_tscadc: Use BIT() and GENMASK() when relevant in the header Message-ID: <20210905140938.76385326@jic23-huawei> In-Reply-To: <20210902215144.507243-26-miquel.raynal@bootlin.com> References: <20210902215144.507243-1-miquel.raynal@bootlin.com> <20210902215144.507243-26-miquel.raynal@bootlin.com> X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.30; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Thu, 2 Sep 2021 23:51:23 +0200 Miquel Raynal wrote: > Convert masks to GENMASK() and regular shifts to BIT() in > ti_am335x_tscadc.h header. > > Suggested-by: Jonathan Cameron > Signed-off-by: Miquel Raynal Might be better two squash this and next patch together. I wrote a bunch of 'use FIELD_PREP() here' comments before realising you've done that separately. Still, for what is here looks good. Reviewed-by: Jonathan Cameron > --- > include/linux/mfd/ti_am335x_tscadc.h | 40 ++++++++++++++-------------- > 1 file changed, 20 insertions(+), 20 deletions(-) > > diff --git a/include/linux/mfd/ti_am335x_tscadc.h b/include/linux/mfd/ti_am335x_tscadc.h > index efafecfc87a7..4de16fc3d74f 100644 > --- a/include/linux/mfd/ti_am335x_tscadc.h > +++ b/include/linux/mfd/ti_am335x_tscadc.h > @@ -39,11 +39,11 @@ > #define IRQWKUP_ENB BIT(0) > > /* Step Enable */ > -#define STEPENB_MASK (0x1FFFF << 0) > +#define STEPENB_MASK GENMASK(16, 0) > #define STEPENB(val) ((val) << 0) > -#define ENB(val) (1 << (val)) > -#define STPENB_STEPENB STEPENB(0x1FFFF) > -#define STPENB_STEPENB_TC STEPENB(0x1FFF) > +#define ENB(val) BIT(val) > +#define STPENB_STEPENB STEPENB(GENMASK(16, 0)) > +#define STPENB_STEPENB_TC STEPENB(GENMASK(12, 0)) > > /* IRQ enable */ > #define IRQENB_HW_PEN BIT(0) > @@ -57,11 +57,11 @@ > #define IRQENB_PENUP BIT(9) > > /* Step Configuration */ > -#define STEPCONFIG_MODE_MASK (3 << 0) > +#define STEPCONFIG_MODE_MASK GENMASK(1, 0) > #define STEPCONFIG_MODE(val) ((val) << 0) > #define STEPCONFIG_MODE_SWCNT STEPCONFIG_MODE(1) > #define STEPCONFIG_MODE_HWSYNC STEPCONFIG_MODE(2) > -#define STEPCONFIG_AVG_MASK (7 << 2) > +#define STEPCONFIG_AVG_MASK GENMASK(4, 2) > #define STEPCONFIG_AVG(val) ((val) << 2) > #define STEPCONFIG_AVG_16 STEPCONFIG_AVG(4) > #define STEPCONFIG_XPP BIT(5) > @@ -71,41 +71,41 @@ > #define STEPCONFIG_XNP BIT(9) > #define STEPCONFIG_YPN BIT(10) > #define STEPCONFIG_RFP(val) ((val) << 12) > -#define STEPCONFIG_RFP_VREFP (0x3 << 12) > -#define STEPCONFIG_INM_MASK (0xF << 15) > +#define STEPCONFIG_RFP_VREFP GENMASK(13, 12) > +#define STEPCONFIG_INM_MASK GENMASK(18, 15) > #define STEPCONFIG_INM(val) ((val) << 15) > #define STEPCONFIG_INM_ADCREFM STEPCONFIG_INM(8) > -#define STEPCONFIG_INP_MASK (0xF << 19) > +#define STEPCONFIG_INP_MASK GENMASK(22, 19) > #define STEPCONFIG_INP(val) ((val) << 19) > #define STEPCONFIG_INP_AN4 STEPCONFIG_INP(4) > #define STEPCONFIG_INP_ADCREFM STEPCONFIG_INP(8) > #define STEPCONFIG_FIFO1 BIT(26) > #define STEPCONFIG_RFM(val) ((val) << 23) > -#define STEPCONFIG_RFM_VREFN (0x3 << 23) > +#define STEPCONFIG_RFM_VREFN GENMASK(24, 23) > > /* Delay register */ > -#define STEPDELAY_OPEN_MASK (0x3FFFF << 0) > +#define STEPDELAY_OPEN_MASK GENMASK(17, 0) > #define STEPDELAY_OPEN(val) ((val) << 0) > #define STEPCONFIG_OPENDLY STEPDELAY_OPEN(0x098) > -#define STEPDELAY_SAMPLE_MASK (0xFF << 24) > +#define STEPDELAY_SAMPLE_MASK GENMASK(31, 24) > #define STEPDELAY_SAMPLE(val) ((val) << 24) > #define STEPCONFIG_SAMPLEDLY STEPDELAY_SAMPLE(0) > > /* Charge Config */ > -#define STEPCHARGE_RFP_MASK (7 << 12) > +#define STEPCHARGE_RFP_MASK GENMASK(14, 12) > #define STEPCHARGE_RFP(val) ((val) << 12) > #define STEPCHARGE_RFP_XPUL STEPCHARGE_RFP(1) > -#define STEPCHARGE_INM_MASK (0xF << 15) > +#define STEPCHARGE_INM_MASK GENMASK(18, 15) > #define STEPCHARGE_INM(val) ((val) << 15) > #define STEPCHARGE_INM_AN1 STEPCHARGE_INM(1) > -#define STEPCHARGE_INP_MASK (0xF << 19) > +#define STEPCHARGE_INP_MASK GENMASK(22, 19) > #define STEPCHARGE_INP(val) ((val) << 19) > -#define STEPCHARGE_RFM_MASK (3 << 23) > +#define STEPCHARGE_RFM_MASK GENMASK(24, 23) > #define STEPCHARGE_RFM(val) ((val) << 23) > #define STEPCHARGE_RFM_XNUR STEPCHARGE_RFM(1) > > /* Charge delay */ > -#define CHARGEDLY_OPEN_MASK (0x3FFFF << 0) > +#define CHARGEDLY_OPEN_MASK GENMASK(17, 0) > #define CHARGEDLY_OPEN(val) ((val) << 0) > #define CHARGEDLY_OPENDLY CHARGEDLY_OPEN(0x400) > > @@ -114,7 +114,7 @@ > #define CNTRLREG_STEPID BIT(1) > #define CNTRLREG_STEPCONFIGWRT BIT(2) > #define CNTRLREG_POWERDOWN BIT(4) > -#define CNTRLREG_AFE_CTRL_MASK (3 << 5) > +#define CNTRLREG_AFE_CTRL_MASK GENMASK(6, 5) > #define CNTRLREG_AFE_CTRL(val) ((val) << 5) > #define CNTRLREG_4WIRE CNTRLREG_AFE_CTRL(1) > #define CNTRLREG_5WIRE CNTRLREG_AFE_CTRL(2) > @@ -122,8 +122,8 @@ > #define CNTRLREG_TSCENB BIT(7) > > /* FIFO READ Register */ > -#define FIFOREAD_DATA_MASK (0xfff << 0) > -#define FIFOREAD_CHNLID_MASK (0xf << 16) > +#define FIFOREAD_DATA_MASK GENMASK(11, 0) > +#define FIFOREAD_CHNLID_MASK GENMASK(19, 16) > > /* DMA ENABLE/CLEAR Register */ > #define DMA_FIFO0 BIT(0)