--- On Fri, 11/5/10, Greg KH <greg@xxxxxxxxx> wrote: > > This patch changes 86% of the driver. More is to come, > but this is > > a self-contained patch so it can be applied. > > No it can not at all, sorry. Greg, would you accept a brand new driver, say uastoo.c (just like 8139too.c came about, remember?), or would you say "This is 14% the same as uas.c, and thus cannot go in"? Is 14% difference too much or too little? Surely, the sum of small tiny individual patches, *would* replace 86% of the driver. BTW, there are more things to fix even on top of that. (I wonder how much similar 8139too.c was to rtl8139.c... when 8139too.c got in back about 10 years ago? I bet it was more than 14%. Hey, I'm building precedent here.) Or would you say "only individual patches even if it changes 100% of the code". > Remember, patches do one single thing at a time. I wholeheartedly agree. One thing at a time. Self-contained, well defined. I really did want to do that with uas.c but it was really hard. I was like "where to begin?" There was no one single change to say "here it is" without changing something else. > This patch is equivalant to deleting the file and replacing it with a > totally different driver, which is unacceptable. I did want to submit them one at a time. I really did. But one change necessitated another in order to keep things consistent, and it ended up like this, one single commit in my git tree. I really do have one single commit as this "patch". I really did want to submit it as individual one at a time. But there was no well defined "one at a time". > Please break all of these changes up into individual > patches and resubmit them. That's the only way we can take > them. That, or a new driver perhaps? The net effect of individual patches would be the same, leaving only 14% of the code the same and maybe less with the more work pending. Perhaps a new driver acknowledging it is based on this one FWIW? > I also suggest you go re-read Documentation/development_process/ as well. Did Willy reply to your query about the previous patches? I didn't see his reply. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
