On Wed, Nov 16, 2011 at 7:39 PM, Jiri Kosina [off-list ref] wrote:
On Wed, 16 Nov 2011, David Herrmann wrote:
quoted
quoted
quoted
-#define WIIMOTE_VERSION "0.2"
+#define WIIMOTE_VERSION "0.3"
Do we really need this?
I generally prefer not having any file versioning for the files that are
implicitly tracked by kernel git.
No, not at all. But I thought this was common practice. I have removed
it all together. The next series will be updated.
It definitely might make sense for your internal tracking during driver
development. I just would like to have it dropped for final merge.
Otherwise, I have just finished first round of review, and overall the
driver looks fine, thanks!
Great. I've resent the revised patches. Let me know if there are any
other issues.
--
Jiri Kosina
SUSE Labs
Cheers
David