On Thursday 02 August 2012 11:43:44 James Bottomley wrote:
 
> this should be inside the switch as
> 
> default:
>       return -EIO;
> 
> and no return statment outside of the switch.

OK

> Don't you still want to handle BUSY and QUEUE_FULL as -EBUSY?  Right at
> the moment they'll fall through the default case and return -EIO.

True. They should be handled.

        Regards
                Oliver

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to