On Thu, Mar 28, 2019 at 01:40:04PM +0900, William Breathitt Gray wrote:
> On Wed, Mar 27, 2019 at 02:33:14PM +0200, Andy Shevchenko wrote:
> > On Wed, Mar 27, 2019 at 02:02:39PM +0900, William Breathitt Gray wrote:
> > > Replace verbose implementation in set_multiple callback with
> > > for_each_set_clump8 macro to simplify code and improve clarity.
> >
> > > + for_each_set_clump8(offset, bankmask, mask, chip->registers * 8) {
> >
> > > + bank = (chip->registers - 1) - (offset / 8);
> >
> > Excessive parens, but it's minor.
>
> Fair point, this could do without parens around (chip->registers - 1).
And around division too.
> Since this and renaming 'idx' to 'index' in find_bit.c are such a minor
> changes, I'll wait first to see if something else comes up to be fixed
> in this review. If so, I'll add these changes to the next version of
> this patchset.
It seems inline proposal would going to be implemented.
--
With Best Regards,
Andy Shevchenko
