Re: [PATCH 07/29] ncr5380: Cleanup TAG_NEXT and TAG_NONE macros

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux