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

Reply via email to