[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMpxmJVB_L+otX2u80qwGjw4TXCJtwOXe=t11O4Daq3miMVk6Q@mail.gmail.com>
Date: Thu, 28 May 2020 16:49:06 +0200
From: Bartosz Golaszewski <bgolaszewski@...libre.com>
To: Mark Brown <broonie@...nel.org>
Cc: Bartosz Golaszewski <brgl@...ev.pl>,
John Crispin <john@...ozen.org>,
Sean Wang <sean.wang@...iatek.com>,
Mark Lee <Mark-MC.Lee@...iatek.com>,
"David S . Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>,
Matthias Brugger <matthias.bgg@...il.com>,
netdev <netdev@...r.kernel.org>,
arm-soc <linux-arm-kernel@...ts.infradead.org>,
linux-mediatek@...ts.infradead.org,
LKML <linux-kernel@...r.kernel.org>,
Fabien Parent <fparent@...libre.com>,
Stephane Le Provost <stephane.leprovost@...iatek.com>,
Pedro Tsai <pedro.tsai@...iatek.com>,
Andrew Perepech <andrew.perepech@...iatek.com>
Subject: Re: [PATCH v2 1/2] regmap: provide helpers for simple bit operations
czw., 28 maj 2020 o 16:45 Mark Brown <broonie@...nel.org> napisaĆ(a):
>
> On Thu, May 28, 2020 at 04:22:40PM +0200, Bartosz Golaszewski wrote:
>
> > + return (val & bits) == bits ? 1 : 0;
>
> The tenery here is redundant, it's converting a boolean value into a
> boolean value. Otherwise this looks good.
Do you mind if I respin it right away? I don't want to spam the list.
Bartosz
Powered by blists - more mailing lists