On Wed, Apr 20, 2016 at 3:10 AM, Alexandre Courbot <gnurou@xxxxxxxxx> wrote: >> + if (!ret) >> + gdev->descs[i].name = name; > > Shouldn't we check for name collision (by calling gpio_name_to_desc() > as gpiochip_set_desc_names() does) here? That check is there to avoid getting the same name twice in sysfs (which would fail), and this is not for sysfs, it is for the chardev. And I don't think so: atleast we should not look globally like that thing does. It need not be a unique name for the system, but I don't know if we should even enforce it to be a unique name for the chip. What do people think? Yours, Linus Walleij -- 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