On Tue, Jan 10, 2012 at 01:51:07PM -0600, James Bottomley wrote: > > The changes since the Nov 4 RFC are listed in the patch commit log: > > > > ib_srpt: Make compilation with BUG=n proceed` > > ib_srpt: Use new target_core_fabric.h include > > ib_srpt: Check hex2bin() return code to silence build warning > > > > These are all very minor and did not warrant another full RFC posting. > > They might not warrant a full RFC reposting, but individually they > should have been posted to the list, so Bart is right. At least 2 and 3 list, just not as global patches against the target tree, not ib_srpt specificly: 2 is from: [PATCH] target: header reshuffle, part2 3 is from: [PATCH 5/5] target: check hex2bin result Not sure about 1. I'm really getting tired of this bickering. It's a brand new driver and doesn't have to be perfect. While Bart generally has technically good points there's really no reason to put his remaining fixups in after the driver has been merged and available to a wieder audience. -- 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