From 7c346c2ce866959fc89a14d1379869084f32b3d8 Mon Sep 17 00:00:00 2001 From: H Hartley Sweeten Date: Tue, 14 May 2013 13:32:32 -0700 Subject: [PATCH] staging: comedi: usbduxfast: remove 'attached' from private data The subdevice function pointers are only setup if the device is attached. The 'attached' flag in the private data is unnecessary. Remove it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/usbduxfast.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/drivers/staging/comedi/drivers/usbduxfast.c b/drivers/staging/comedi/drivers/usbduxfast.c index 304302424981..59bbd4d94a2b 100644 --- a/drivers/staging/comedi/drivers/usbduxfast.c +++ b/drivers/staging/comedi/drivers/usbduxfast.c @@ -148,7 +148,6 @@ static const struct comedi_lrange range_usbduxfast_ai_range = { * one sub device just now: A/D */ struct usbduxfast_private { - int attached; /* is attached? */ struct usb_device *usb; /* pointer to the usb-device */ struct urb *urbIn; /* BULK-transfer handling: urb */ int8_t *transfer_buffer; @@ -276,11 +275,6 @@ static void usbduxfast_ai_interrupt(struct urb *urb) return; } - if (unlikely(!devpriv->attached)) { - /* no comedi device there */ - return; - } - /* first we test if something unusual has just happened */ switch (urb->status) { case 0: @@ -1246,8 +1240,6 @@ static int usbduxfast_attach_common(struct comedi_device *dev) s->maxdata = 0x1000; s->range_table = &range_usbduxfast_ai_range; - devpriv->attached = 1; - up(&devpriv->sem); return 0; @@ -1343,7 +1335,6 @@ static void usbduxfast_detach(struct comedi_device *dev) down(&devpriv->sem); - devpriv->attached = 0; devpriv->comedidev = NULL; if (devpriv->intf) -- 2.20.1