On Mon, Aug 1, 2011 at 10:35 PM, Geert Uytterhoeven <ge...@linux-m68k.org> wrote: > On Mon, Aug 1, 2011 at 22:03, Andre Heider <a.hei...@gmail.com> wrote: >> priv->gendisk = gendisk; >> + >> + /* find first accessible region */ >> + for (region_idx = 0; region_idx < dev->num_regions; region_idx++) >> + if (test_bit(region_idx, &dev->accessible_regions)) { >> + priv->region_idx = region_idx; >> + break; >> + } >> + > > Why not > > priv->region_idx = __ffs(dev->accessible_regions);
Right, thanks. Will change that for v2 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev