On Fri, 3 Oct 2014, Geert Uytterhoeven wrote: > On Thu, Oct 2, 2014 at 8:56 AM, Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> wrote: > > --- linux.orig/drivers/scsi/NCR5380.h 2014-10-02 16:55:55.000000000 +1000 > > +++ linux/drivers/scsi/NCR5380.h 2014-10-02 16:55:57.000000000 +1000 > > @@ -224,14 +224,11 @@ > > #define DISCONNECT_LONG 2 > > > > /* > > - * These are "special" values for the tag parameter passed to NCR5380_select. > > + * "Special" value for the (unsigned char) command tag, to indicate > > + * I_T_L nexus instead of I_T_L_Q. > > */ > > I guess the above comment can just be removed instead of changed, as the > I_T_L nexus definitions below are removed? When I wrote this patch I didn't realize that the tag member in struct scsi_cmnd was going to be removed in favour of cmd->request.tag. I will rewrite this patch to remove all of the cmd->tag stuff. When I sent this patch I knew I'd have to revise it, but I wanted to get the entire series out there anyway to avoid further duplication of effort. -- -- 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