USB devfs patch

Andrew Thompson thompsa at FreeBSD.org
Thu Feb 26 08:54:54 PST 2009


On Thu, Feb 26, 2009 at 09:48:31AM +0100, Hans Petter Selasky wrote:
> On Thursday 26 February 2009, Andrew Thompson wrote:
> > http://people.freebsd.org/~thompsa/usb-cdevs.diff
> 
> Hi,
> 
> Here is a list of comments and bugs.
> The comments follow the diff from top to bottom.

Thanks Hans, I will go through them all.

> 6) Some non-gcc compilers will complain unless you do 0-1 when the
> destination variable is unsigned.
> 
> -       usb2_free_pipe_data(udev, iface_index, 0 - 1);
> +       usb2_free_pipe_data(udev, iface_index, -1);

This one in particular, if the variable is unsigned then you shouldnt
be passing a negative value. Either make signed or use ~0 for the max
value.


cheers,
Andrew


More information about the freebsd-current mailing list