On Fri, Mar 23, 2018 at 6:21 PM, Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote: > 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" -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html