On Thu, Dec 16, 2010 at 04:48:15PM +0100, Kevin Wolf wrote: > > + scsi_req_set_status(r, GOOD, NO_SENSE); > > + scsi_req_complete(&r->req); > > + scsi_remove_request(r); > > Isn't this the same as scsi_command_complete()?
Yes. > > > + return 0; > > And if you break; here instead of returning (like all other commands) > and remove the three lines above completely, I think it would just do > the right thing. Yes, that looks doable.
