From mboxrd@z Thu Jan 1 00:00:00 1970 From: William Breathitt Gray Subject: Re: [PATCH v9 1/9] bitops: Introduce the for_each_set_clump8 macro Date: Tue, 12 Mar 2019 19:43:22 +0900 Message-ID: <20190312104322.GA3425@icarus> References: <12b3a364ac7fc32194fa6f55911dd800aca92a56.1551598603.git.vilhelm.gray@gmail.com> <20190311180113.c283fdbcba3df10db2c815aa@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20190311180113.c283fdbcba3df10db2c815aa@linux-foundation.org> Sender: linux-kernel-owner@vger.kernel.org To: linus.walleij@linaro.org Cc: akpm@linux-foundation.org, "open list:GPIO SUBSYSTEM" , linux-arch@vger.kernel.org, "linux-kernel@vger.kernel.org" , Andy Shevchenko , Rasmus Villemoes , Masahiro Yamada , Bartosz Golaszewski , Linux ARM , Andy Shevchenko , Arnd Bergmann List-Id: linux-gpio@vger.kernel.org On Mon, Mar 11, 2019 at 06:01:13PM -0700, Andrew Morton wrote: > On Fri, 8 Mar 2019 09:31:00 +0100 Linus Walleij wrote: > > > On Sun, Mar 3, 2019 at 8:47 AM William Breathitt Gray > > wrote: > > > > > This macro iterates for each 8-bit group of bits (clump) with set bits, > > > within a bitmap memory region. For each iteration, "start" is set to the > > > bit offset of the found clump, while the respective clump value is > > > stored to the location pointed by "clump". Additionally, the > > > bitmap_get_value8 and bitmap_set_value8 functions are introduced to > > > respectively get and set an 8-bit value in a bitmap memory region. > > > > > > Suggested-by: Andy Shevchenko > > > Suggested-by: Rasmus Villemoes > > > Cc: Arnd Bergmann > > > Cc: Andrew Morton > > > Reviewed-by: Andy Shevchenko > > > Reviewed-by: Linus Walleij > > > Signed-off-by: William Breathitt Gray > > > > Andrew: would you be OK with this being merged in v5.1? > > Yup. We have quite a few users there. I assume this will go via the > gpio tree? > > Feel free to add Acked-by: Andrew Morton , > although it probably isn't worth churning the git tree to do so at this > late stage - your cvall. Linus, I discovered a bug in this version of the patchset. I'll release a version 10 once I've resolved the issue. William Breathitt Gray