Re: usbutils 0.82 release

From: Xiaofan Chen
Date: Fri May 08 2009 - 02:41:01 EST


On Fri, May 8, 2009 at 12:01 PM, Greg KH <greg@xxxxxxxxx> wrote:
> On Thu, May 07, 2009 at 07:51:23PM +0800, Xiaofan Chen wrote:
>> Not sure if this is a real problem or not. There is a compiler warning
>> for gcc 4.4.0.
>>
>> gcc -DHAVE_CONFIG_H -I.   -DDATADIR=\"/usr/share\"   -g -O2 -MT
>> lsusb-names.o -MD -MP -MF .deps/lsusb-names.Tpo -c -o lsusb-names.o
>> `test -f 'names.c' || echo './'`names.c
>> names.c: In function ‘names_init’:
>> names.c:803: warning: assignment makes pointer from integer without a cast
>
> Care to provide a patch for this?

To be honest, I do not know the reason of the warning. It looks fine to me
but I am not a real programmer.

In fact, I tried version 0.73 and there is no warning there and the file name.c
is the same between 0.73 and 0.81/0.82.

For 0.73:
[mcuee@myhost usbutils-0.73]$ make
make all-am
make[1]: Entering directory `/home/mcuee/Desktop/build/usbutils/usbutils-0.73'
if gcc -DHAVE_CONFIG_H -I. -I. -I. -DUSBIDS_DIR='"/usr/local/share"'
-Wall -W -Wunused -g -O2 -MT lsusb.o -MD -MP -MF ".deps/lsusb.Tpo" -c
-o lsusb.o lsusb.c; \
then mv -f ".deps/lsusb.Tpo" ".deps/lsusb.Po"; else rm -f
".deps/lsusb.Tpo"; exit 1; fi
lsusb.c: In function ‘do_wireless’:
lsusb.c:2648: warning: unused parameter ‘fd’
if gcc -DHAVE_CONFIG_H -I. -I. -I. -DUSBIDS_DIR='"/usr/local/share"'
-Wall -W -Wunused -g -O2 -MT names.o -MD -MP -MF ".deps/names.Tpo" -c
-o names.o names.c; \
then mv -f ".deps/names.Tpo" ".deps/names.Po"; else rm -f
".deps/names.Tpo"; exit 1; fi
if gcc -DHAVE_CONFIG_H -I. -I. -I. -DUSBIDS_DIR='"/usr/local/share"'
-Wall -W -Wunused -g -O2 -MT devtree.o -MD -MP -MF ".deps/devtree.Tpo"
-c -o devtree.o devtree.c; \
then mv -f ".deps/devtree.Tpo" ".deps/devtree.Po"; else rm -f
".deps/devtree.Tpo"; exit 1; fi
if gcc -DHAVE_CONFIG_H -I. -I. -I. -DUSBIDS_DIR='"/usr/local/share"'
-Wall -W -Wunused -g -O2 -MT usbmisc.o -MD -MP -MF ".deps/usbmisc.Tpo"
-c -o usbmisc.o usbmisc.c; \
then mv -f ".deps/usbmisc.Tpo" ".deps/usbmisc.Po"; else rm -f
".deps/usbmisc.Tpo"; exit 1; fi
gcc -Wall -W -Wunused -g -O2 -o lsusb lsusb.o names.o devtree.o
usbmisc.o -lz -lusb
make[1]: Leaving directory `/home/mcuee/Desktop/build/usbutils/usbutils-0.73'


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