This quiets a huge explosion of warnings that appeared with the __must_check patch, by shifting the burden to the caller. Incidentally, there is only a single user of this function: mptsas Signed-off-by: Jeff Garzik <jeff@xxxxxxxxxx> diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h index 895d212..b401c82 100644 --- a/include/scsi/scsi_device.h +++ b/include/scsi/scsi_device.h @@ -298,9 +298,9 @@ extern int scsi_execute_async(struct scs void (*done)(void *, char *, int, int), gfp_t gfp); -static inline void scsi_device_reprobe(struct scsi_device *sdev) +static inline int __must_check scsi_device_reprobe(struct scsi_device *sdev) { - device_reprobe(&sdev->sdev_gendev); + return device_reprobe(&sdev->sdev_gendev); } static inline unsigned int sdev_channel(struct scsi_device *sdev) - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html