On Tue, Mar 07, 2017 at 08:05:10PM +0300, Dan Carpenter wrote:
> On Tue, Mar 07, 2017 at 05:33:07PM +1100, Tobin C. Harding wrote:
> > + /* Only allow 1 open at a time on mgmt device */
> > + if (dgnc_mgmt_in_use[minor]) {
> > + rc = -EBUSY;
> > + goto out;
> > + }
> > + dgnc_mgmt_in_use[minor]++;
> >
> > +out:
>
> Better to use "unlock" instead of "out". "out" is very ambigous.
Noted. Will include in v3.
thanks,
Tobin.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
