* Nikita Yushchenko <nikita.yoush@xxxxxxxxxxxxxxxxxx> [170512 01:22]: > >> + /* > >> + * If pctldev is not null, we are claiming hog for it, > >> + * that means, setting that is served by pctldev by itself. > >> + * > >> + * Thus we must skip map that is for this device but is served > >> + * by other device. > >> + */ > >> + if (pctldev && > >> + strcmp(dev_name(pctldev->dev), map->ctrl_dev_name)) > >> + continue; > >> > >> ret = add_setting(p, pctldev, map); > >> /* > >> -- > > > > Maybe add a comment saying pctldev is NULL in the regular case > > and only exists in the hog case? > > Isn't comment above saying exactly that? Yes it is, you're right :) Tony -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html