diff options
author | Tim Sell <Timothy.Sell@unisys.com> | 2016-09-27 14:42:21 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-09-28 11:30:52 +0200 |
commit | f7a34ff7c04527776a93683f4ac4bf92e9e778a5 (patch) | |
tree | b865cb02f6d9eb555a25517034cefd58c0cd901d | |
parent | 8681a1d47b3332b4dbeb0321b347af1bb7cb0cd6 (diff) | |
download | lwn-f7a34ff7c04527776a93683f4ac4bf92e9e778a5.tar.gz lwn-f7a34ff7c04527776a93683f4ac4bf92e9e778a5.zip |
staging: unisys: visorbus: fix sloppy unsigned/signed int conversions
Several implicit unsigned --> signed int conversions were corrected.
No runtime effects were observed here, nevertheless implicit conversions
between integral types can lead to unexpected behavior.
Signed-off-by: Tim Sell <Timothy.Sell@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/unisys/visorbus/visorbus_main.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbus/visorbus_main.c index 1f3d2e0bf966..ccdb5a1f45d9 100644 --- a/drivers/staging/unisys/visorbus/visorbus_main.c +++ b/drivers/staging/unisys/visorbus/visorbus_main.c @@ -762,7 +762,7 @@ write_vbus_bus_info(struct visorchannel *chan, static void write_vbus_dev_info(struct visorchannel *chan, struct spar_vbus_headerinfo *hdr_info, - struct ultra_vbus_deviceinfo *info, int devix) + struct ultra_vbus_deviceinfo *info, unsigned int devix) { int off = (sizeof(struct channel_header) + hdr_info->dev_info_offset) + @@ -787,8 +787,8 @@ fix_vbus_dev_info(struct visor_device *visordev) int i; struct visor_device *bdev; struct visor_driver *visordrv; - int bus_no = visordev->chipset_bus_no; - int dev_no = visordev->chipset_dev_no; + u32 bus_no = visordev->chipset_bus_no; + u32 dev_no = visordev->chipset_dev_no; struct ultra_vbus_deviceinfo dev_info; const char *chan_type_name = NULL; struct spar_vbus_headerinfo *hdr_info; |