[PATCH] Signedness issue in drivers/net/phy/phy_device.c
authorEric Sesterhenn <snakebyte@gmx.de>
Mon, 25 Sep 2006 23:39:22 +0000 (16:39 -0700)
committerJeff Garzik <jeff@garzik.org>
Tue, 26 Sep 2006 00:01:19 +0000 (20:01 -0400)
commit84c22d7901f793bd267b5f79270080964b252826
tree253633abda9f66686e8d5bea5a6f00a07911d663
parent0fe2480aded9504434e24815c410cbebb4f4efad
[PATCH] Signedness issue in drivers/net/phy/phy_device.c

While checking gcc 4.1 -Wextra warnings, I stumbled across the following
two warnings:

drivers/net/phy/phy_device.c:528: warning: comparison of unsigned expression < 0 is always false
drivers/net/phy/phy_device.c:546: warning: comparison of unsigned expression < 0 is always false

Since phy_read() returns an integer and can return negative values, it seems
to me the best way to get proper error handling working again is to make val
an int.  Currently it is an u32, so the < 0 check always fails.

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/net/phy/phy_device.c