On Fri, Oct 17, 2014 at 06:34:48PM +0000, Arvind Kumar wrote: > Hi Christoph, > > Thanks for the change. Sorry for the delay. The change looks fine to me. I just have a question. > > The comment in include/scsi/scsi_cmnd.h says: > > struct scsi_cmnd { > ... > unsigned char tag; /* SCSI-II queued command tag */ > } > > Is that comment not right? Should we update that too? The comment looks very confusing and I'm fine with a patch that just removes the comment. That being said Hannes has patches to remove the whole field which I'd like to merge once he fixes up the remaining issues, so it might not worth the 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