On Wed, Oct 26, 2011 at 10:17:19AM -0700, Tirumala Marri wrote: > Greg, > > < > <No, just start over from scratch. Just leave the crap driver behind, > <use it for reference but write the new one. What email client are you using, I have never seen this type of formatting, and it makes it impossible to read your replies. Please resolve this. > <It's obvious given that you are already at iteration v15 and it's > <still looking this bad that this is not realistic to get reviewed and > <accepted as-is. I don't think staging is a good target either -- what > <the driver really needs is _functional_ cut-down to only cover the use > <cases that your product uses, and staging cleanups are mostly around > <style and refactoring, not changing, fixing or removing functionality. > < > <> <Compare this to the dwc3 driver, which is much much cleaner. > <> < > Could you please comment on this. This has been reviewed so many times > And we did the major changes. I think it is un-acceptable to say to > re-write > The driver now after 15 reviews? This should have happened in first 5 > reviews. Sometimes people miss things on previous reviews, this is not a perfict system, only the best that we currently know how to do, don't you agree? If this driver contains lots of things that are not necessary for it to work at this point in time, why wouldn't that be removed now? Anyway, I think it would probably be best to get this into staging for now, which will help you get a way to clean it up in an easier manner, and still have users have access to it. Is that acceptable? thanks, greg k-h -- 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
