>> + /* >> + * 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?
