Re: [PATCH] pinctrl: when claiming hog, skip maps not served by same device

From: Tony Lindgren
Date: Fri May 12 2017 - 10:03:32 EST


* 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