On 03/12/11 3:26 AM, "Mike Christie" <michaelc@xxxxxxxxxxx> wrote: >On 12/02/2011 12:42 AM, vikas.chaudhary@xxxxxxxxxx wrote: >> >> /** >> + * qla4xxx_is_eh_active: > >You need a description of the function above after the colon. And >actually qla4xxx is doing a dash in other places. So you should do: > >/** > * qla4xxx_is_eh_active - check if eh is rinning > >And just to make sure, if the host state is in SHOST_CANCEL_RECOVERY the >drivers shutdown code will have cleaned up the command right? It looks >like it, but I was not 100% sure. I updated patch to take care of command cleanup on driver shutdown. I am sending updated patch in next email. Thanks, Vikas. This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message. -- 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
