Thread (5 messages) 5 messages, 3 authors, 2012-02-28

Re: [PATCH 1/1] HID: kye: Add support for 3 tablets

From: Nikolai Kondrashov <hidden>
Date: 2012-02-27 15:10:07

Possibly related (same subject, not in this thread)

Hi Jiri,

On Mon, Feb 27, 2012 at 4:54 PM, Jiri Kosina [off-list ref] wrote:
Would you strongly object to dropping all those you have added into
various hid drivers already? I think the new rdesc + a few words of
comment why we had to change the descriptor would be enough.
No, I wouldn't object. I think it will make the code clearer. There is some
value in having the original descriptors there, but not that much.

I think I'll post the original descriptors on the emerging DIGImend project
wiki instead.
So I am not applying this patch now, and I'd like to ask you to respin it
with the original rdesc comment removed and send a followup patch that
does the substitution/removal in the other drivers.

Sounds ok?
Sure.

I'll be sending a new version of this patch in about a day and later the
removal patch.

Thanks :)

Sincerely,
Nick
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help