Re: [PATCH 7/8] all: replace find_next{,_zero}_bit with find_first{,_zero}_bit where appropriate

From: Andy Shevchenko <andy.shevchenko@gmail.com>
To: Yury Norov <yury.norov@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux-Arch <linux-arch@vger.kernel.org>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Brian Cain <bcain@codeaurora.org>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	Jonas Bonn <jonas@southpole.se>,
	Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>,
	"James E.J. Bottomley" <James.Bottomley@hansenpartnership.com>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Heiko Carstens <hca@linux.ibm.com>, Rich Felker <dalias@libc.org>,
	David Hildenbrand <david@redhat.com>,
	Jaegeuk Kim <jaegeuk@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Rasmus Villemoes <linux@rasmusvillemoes.dk>,
	Andrew Morton <akpm@linux-foundation.org>,
	Alexander Lobakin <alobakin@pm.me>,
	Samuel Mendoza-Jonas <sam@mendozajonas.com>,
	Jakub Kicinski <kuba@kernel.org>,
	Daniel Bristot de Oliveira <bristot@redhat.com>,
	Alexey Klimov <aklimov@redhat.com>,
	Ingo Molnar <mingo@redhat.com>
Subject: Re: [PATCH 7/8] all: replace find_next{,_zero}_bit with find_first{,_zero}_bit where appropriate
Date: Sun, 13 Jun 2021 00:47:31 +0300	[thread overview]
Message-ID: <CAHp75VerU1NJMweWCR7MsE9hiMFZyJP8m751OFKmGrJ1gVhMWw@mail.gmail.com> (raw)
In-Reply-To: <20210612123639.329047-8-yury.norov@gmail.com>

On Sat, Jun 12, 2021 at 3:39 PM Yury Norov <yury.norov@gmail.com> wrote:
>
> find_first{,_zero}_bit is a more effective analogue of 'next' version if
> start == 0. This patch replaces 'next' with 'first' where things look
> trivial.

Depending on the maintainers (but I think there will be at least few
in this case) they would like to have this be split on a per-driver
basis.
I counted 17 patches. I would split.

Since many of them are independent you may send without Cc'ing all
non-relevant people in each case.

>  arch/powerpc/platforms/pasemi/dma_lib.c |  4 ++--
>  arch/s390/kvm/kvm-s390.c                |  2 +-
>  drivers/block/rnbd/rnbd-clt.c           |  2 +-
>  drivers/dma/ti/edma.c                   |  2 +-
>  drivers/iio/adc/ad7124.c                |  2 +-
>  drivers/infiniband/hw/irdma/hw.c        | 16 ++++++++--------
>  drivers/media/cec/core/cec-core.c       |  2 +-
>  drivers/media/mc/mc-devnode.c           |  2 +-
>  drivers/pci/controller/dwc/pci-dra7xx.c |  2 +-
>  drivers/scsi/lpfc/lpfc_sli.c            | 10 +++++-----
>  drivers/soc/ti/k3-ringacc.c             |  4 ++--
>  drivers/tty/n_tty.c                     |  2 +-
>  drivers/virt/acrn/ioreq.c               |  3 +--
>  fs/f2fs/segment.c                       |  8 ++++----
>  fs/ocfs2/cluster/heartbeat.c            |  2 +-
>  fs/ocfs2/dlm/dlmdomain.c                |  4 ++--
>  fs/ocfs2/dlm/dlmmaster.c                | 18 +++++++++---------
>  fs/ocfs2/dlm/dlmrecovery.c              |  2 +-
>  fs/ocfs2/dlm/dlmthread.c                |  2 +-
>  lib/genalloc.c                          |  2 +-
>  net/ncsi/ncsi-manage.c                  |  4 ++--
>  21 files changed, 47 insertions(+), 48 deletions(-)

-- 
With Best Regards,
Andy Shevchenko