Re: [PATCH v5 5/5] Add ioctls to enable and disable local controls on an instrument

From: Andy Shevchenko
Date: Wed Nov 25 2015 - 15:46:41 EST


On Wed, Nov 25, 2015 at 11:12 AM, Dave Penkler <dpenkler@xxxxxxxxx> wrote:
> On Sun, Nov 22, 2015 at 12:36:53PM +0200, Andy Shevchenko wrote:


> OK so now I have the same as in v4 again:
>
> if (rv < 0) {
> dev_err(dev, "simple usb_control_msg failed %d\n", rv);
> goto exit;
> } else if (rv != 1) {
> dev_warn(dev, "simple usb_control_msg returned %d\n", rv);
> rv = -EIO;
> goto exit;
> }

This looks correct. Sorry if I misled you, but I really asked about
error code in the first place.

--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/