Messages in this thread | | | From | Melchior FRANZ <> | Subject | Re: [PATCH v2] USB: add support for Dream Cheeky DL100B Webmail Notifier (1d34:0004) | Date | Wed, 22 Dec 2010 01:37:51 +0100 |
| |
* Greg KH -- Wednesday 22 December 2010: > On Wed, Dec 22, 2010 at 12:58:42AM +0100, Melchior FRANZ wrote: > > - I increased the file's version number (1.1 -> 1.2). Hope that's OK. > > Actually, you could just remove it, it's pretty pointless :)
Yeah, that's what I thought. But leaving it at 1.1 was even more pointless, so I aimed for the most conservative solution. :-)
> I would, but it looks like you didn't test this patch out by building it > as the following line really would not compile properly at all, right:
> > + 0x200, The price difference to Delcom's thingy lets you a lot of room. :-)
Of course, I tested it. That garbage is not in the code, or in the generated diff. This happened after importing the correct diff into kmail. Probably and accidantal MMB click. Sigh. OK, it's my fault, no matter what ...
> Third times a charm?
Heh. Coming along ... Thanks for reviewing.
m.
| |