diff options
author | Eric Sesterhenn <[email protected]> | 2006-09-25 16:39:22 -0700 |
---|---|---|
committer | Jeff Garzik <[email protected]> | 2006-09-25 20:01:19 -0400 |
commit | 84c22d7901f793bd267b5f79270080964b252826 (patch) | |
tree | 253633abda9f66686e8d5bea5a6f00a07911d663 /net/unix/af_unix.c | |
parent | 0fe2480aded9504434e24815c410cbebb4f4efad (diff) |
[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 <[email protected]>
Cc: Jeff Garzik <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Jeff Garzik <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions