On Tue, 2007-09-25 at 17:38 -0500, Michael Reed wrote: > Jeff Garzik wrote: > > Moore, Eric wrote: > >> On Tuesday, September 25, 2007 11:32 AM, James Bottomley wrote: > >> > >>>> Youve rejected the error recovery patchs, which is fair enough. > >>> Just the separation ... the actual patch looks OK. > >>> > >> > >> I'll will continue working to separate the "error recovery > >> improvements:" into smaller feature add, but will take some time. I > >> want some constructive feedback, and too big of patch is deterring some > >> people from looking at it. > > > > I think it's fair to break it up, as long as its clearly noted that the > > patch is for review only. > > Any chance the changes, properly documented, could be accepted as > one big patch? Breaking this code into smaller patches, when it's > really intended as a driver update, could prove problematic. Yes, that's OK ... separate patches with signoffs are not absolute requirements as long as the description and the heritage is in the change log (although if you can do it, I prefer the separate signoffs and distinct patches). James - 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