Re: [PATCH 10/15] ps3stor_lib: Add support for multiple regions

On Mon, Aug 1, 2011 at 10:35 PM, Geert Uytterhoeven
[off-list ref] wrote:

On Mon, Aug 1, 2011 at 22:03, Andre Heider [off-list ref] wrote:

quoted

=A0 =A0 =A0 =A0priv->gendisk =3D gendisk;
+
+ =A0 =A0 =A0 /* find first accessible region */
+ =A0 =A0 =A0 for (region_idx =3D 0; region_idx < dev->num_regions; regi=

on_idx++)

quoted

+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 if (test_bit(region_idx, &dev->accessible_=

regions)) {

quoted

+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 priv->region_idx =3D regio=

n_idx;

quoted

+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 break;
+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 }
+

Why not

priv->region_idx =3D __ffs(dev->accessible_regions);

Right, thanks. Will change that for v2