Thread (25 messages) 25 messages, 3 authors, 2011-11-17

Re: [PATCH 16/16] HID: wiimote: Enable NO_INIT_REPORTS quirk

flat view

From: David Herrmann <hidden>
Date: 2011-11-17 13:11:52

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help