Messages in this thread Patch in this message | | | Subject | [PATCH 2.6.2-rc1-mm3] drivers/usb/storage/dpcm.c | Date | Sat, 24 Jan 2004 21:03:42 -0800 (PST) | From | (Bryan Whitehead) |
| |
In function dpcm_transport the compiler complains about ret not being used: drivers/usb/storage/dpcm.c: In function `dpcm_transport': drivers/usb/storage/dpcm.c:46: warning: unused variable `ret'
ret is not used if CONFIG_USB_STORAGE_SDDR09 is not set. Instead of adding more ifdef's to the code this patch puts ret to use for the other 2 cases in the switch statement (case 0 and default).
--- drivers/usb/storage/dpcm.c.orig 2004-01-24 20:51:40.631038904 -0800 +++ drivers/usb/storage/dpcm.c 2004-01-24 20:50:05.155553384 -0800 @@ -56,7 +56,8 @@ int dpcm_transport(Scsi_Cmnd *srb, struc /* * LUN 0 corresponds to the CompactFlash card reader. */ - return usb_stor_CB_transport(srb, us); + ret = usb_stor_CB_transport(srb, us); + break; #ifdef CONFIG_USB_STORAGE_SDDR09 case 1: @@ -72,11 +73,12 @@ int dpcm_transport(Scsi_Cmnd *srb, struc ret = sddr09_transport(srb, us); srb->device->lun = 1; us->srb->device->lun = 1; - return ret; + break; #endif default: US_DEBUGP("dpcm_transport: Invalid LUN %d\n", srb->device->lun); - return USB_STOR_TRANSPORT_ERROR; + ret = USB_STOR_TRANSPORT_ERROR; } + return ret; } -- Bryan Whitehead driver@megahappy.net - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |