On Sun, 1 Apr 2007, Marr wrote: > On Saturday 31 March 2007 2:14pm, Neil Whelchel wrote: > > > BTW I agree with Bill, any time ANYTHING is changed in the module, the > > version number should be bumped. If you change the spelling of colour to > > color, this should bump the revision. Changing anything that changes the > > flow of the module such as adding the cypress_set_dead() function should > > have bumped the minor. > > I'm kind of surprised that it seems the norm (per Greg KH's message) that > this > is done the way it is (i.e. no change to revision). Either way, I intend to > bump the revision number to "1.10" if my VID/PID-check patch (which includes > some other trivial indentation/whitespace fixes) makes it into mainline. > > Thanks again for your input, Neil!
Neil: IMHO It doesn't make a lot of sense to be tracking version info for every little thing in the Linux kernel. Version information is of course important, but that's where the kernel version comes in. One can also suggest that versioning of modules is important for traceability but really it's the signed-off changesets that provide the traceability. Also consider this: If every kernel janitor / submitter / random person were required to bump up that version number as part of his / her patch, now that 1 line change becomes a guaranteed merge conflict for EVERY single patch that touches that driver. The pvrusb2 driver I maintain has an out-of-tree variant, and that one _is_ versioned - it has to be otherwise there's no way to identify a random out-of-tree driver build out in the wild. But that reasoning falters when the driver is in the kernel. In those cases it's the kernel version I follow not the driver version. I've worked it this way for some time now and it simply hasn't been a problem. Nevertheless with all that said it's not my driver we're talking about here. If the official cypress_m8 maintainer wants to require that version number to be bumped up (Neil: Why aren't you listed as a maintainer?) then it's not my place to debate it. I may have my opinion but it's not my driver. Greg KH, as a shepherd for all this might feel differently however... -Mike -- | Mike Isely | PGP fingerprint Spammers Die!! | | 03 54 43 4D 75 E5 CC 92 | isely @ pobox (dot) com | 71 16 01 E2 B5 F5 C1 E8 | | ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel