On Fri, Mar 23, 2018 at 6:31 PM, Andy Shevchenko
[off-list ref] wrote:

On Fri, Mar 23, 2018 at 6:21 PM, Stephen Boyd [off-list ref] wrote:

quoted

       bitmap_zero(chip->valid_mask, max_gpios);
       for (i = 0; i < len; i++)
               set_bit(tmp[i], chip->valid_mask);

Looking to this code I just realized it would be nice to have

{of,device}_property_read_bitmask() where bitmask_parse() is called.

Since it related to change a binding, I would really take couple of
days to hear other opinions.

In the above case, you need to supply a string, like

"1-5,16,18,25"

s/bitmask/bitmap/g

-- 
With Best Regards,
Andy Shevchenko