Date:   Sat, 11 Feb 2023 21:50:59 +0530
From:   Deepak R Varma <drv@mailo.com>
To:     Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Cc:     Antoniu Miclaus <antoniu.miclaus@analog.com>,
        Lars-Peter Clausen <lars@metafoo.de>,
        Michael Hennerich <Michael.Hennerich@analog.com>,
        Jonathan Cameron <jic23@kernel.org>, linux-iio@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Saurabh Singh Sengar <ssengar@microsoft.com>,
        Praveen Kumar <kumarpraveen@linux.microsoft.com>,
        Deepak R Varma <drv@mailo.com>
Subject: Re: [PATCH] iio: frequency: adf4377: remove duplicate/repeating
 constant
Message-ID: <Y+fAa5N3YbMW6yJt@ubun2204.myguest.virtualbox.org>
References: <Y+c+mwNnlPpSH49/@ubun2204.myguest.virtualbox.org>
 <1f9d31ad-db91-a6b9-3a70-cfc983cd99b7@wanadoo.fr>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <1f9d31ad-db91-a6b9-3a70-cfc983cd99b7@wanadoo.fr>
Precedence: bulk

On Sat, Feb 11, 2023 at 03:57:51PM +0100, Christophe JAILLET wrote:
> Le 11/02/2023 ? 08:07, Deepak R Varma a ?crit?:
> > Constant ADF4377_0000_SOFT_RESET_R_MSK is unnecessarily or'ed with
> > itself. Remove the redundant constant from the expression.
> > Issue identified using doublebitand.cocci Coccinelle semantic patch.
> > 
> > Signed-off-by: Deepak R Varma <drv@mailo.com>
> > ---
> >   drivers/iio/frequency/adf4377.c | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/iio/frequency/adf4377.c b/drivers/iio/frequency/adf4377.c
> > index 26abecbd51e0..caefd7ea6b14 100644
> > --- a/drivers/iio/frequency/adf4377.c
> > +++ b/drivers/iio/frequency/adf4377.c
> > @@ -495,8 +495,8 @@ static int adf4377_soft_reset(struct adf4377_state *st)
> >   		return ret;
> >   	return regmap_read_poll_timeout(st->regmap, 0x0, read_val,
> > -					!(read_val & (ADF4377_0000_SOFT_RESET_R_MSK |
> > -					ADF4377_0000_SOFT_RESET_R_MSK)), 200, 200 * 100);
> > +					!(read_val & ADF4377_0000_SOFT_RESET_R_MSK),
> > +					200, 200 * 100);
> 
> Based on the code just above, it is likely that one is expected to be
> ADF4377_0000_SOFT_RESET_MSK.

Hello CJ,
I agree, that appears to be a close possibility. I also tried looking to the
data sheet, but could not conclude.
How can I make sure if the other one should be ADF4377_0000_SOFT_RESET_MSK?

Thank you,
./drv

> 
> CJ
> 
> >   }
> >   static int adf4377_get_freq(struct adf4377_state *st, u64 *freq)
>