linux-kernel - Re: [PATCH v2] USB: add support for Dream Cheeky DL100B Webmail Notifier (1d34:0004)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 21 Dec 2010 16:19:07 -0800
From:	Greg KH <greg@...ah.com>
To:	Melchior FRANZ <melchior.franz@...il.com>
Cc:	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] USB: add support for Dream Cheeky DL100B Webmail
 Notifier (1d34:0004)

On Wed, Dec 22, 2010 at 12:58:42AM +0100, Melchior FRANZ wrote:
> Greg: I hope I understood your instructions correctly.  :-)
> 
> Some notes:
> - 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 :)

> - There were two differently capitalized ENOMEM messages with the same
>   contents before, and as I had to add a third one, I de-capitalized one
>   for consistency reasons.

That's fine.

> - The patch removes the three color #define-s, as they are no longer
>   suitable on the top level, and puts the values directly in the code.
>   This should be clear enough, as the respective color is mentioned
>   in the line before. (I considered to use "const" values or an enum,
>   but both generated different, more verbose code.)

That's fine too.

> Please review again and consider for inclusion. (The patch is diffed
> against 2.6.36.2.)

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:

> +		retval = usb_control_msg(led->udev,
> +					usb_sndctrlpipe(led->udev, 0),
> +					0x09,
> +					0x21,
> +					0x200, The price difference to Delcom's thingy lets you a lot of room. :-)
> +					0,
> +					buffer,
> +					8,
> +					2000);

I think you might want to fix that up :)

Care to resend?

Third times a charm?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/