staging: unisys: don't copy to local variable

Do the check with the values in the structure instead of copying them to
local variables.

Reviewed-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: Tim Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c
index 29475cc..98ce796 100644 (file)
--- a/drivers/staging/unisys/visorbus/visorchipset.c
+++ b/drivers/staging/unisys/visorbus/visorchipset.c
@@ -373,11 +373,9 @@ static int match_visorbus_dev_by_id(struct device *dev, void *data)
 {
        struct visor_device *vdev = to_visor_device(dev);
        struct visor_busdev *id = data;
-       u32 bus_no = id->bus_no;
-       u32 dev_no = id->dev_no;
 
-       if ((vdev->chipset_bus_no == bus_no) &&
-           (vdev->chipset_dev_no == dev_no))
+       if ((vdev->chipset_bus_no == id->bus_no) &&
+           (vdev->chipset_dev_no == id->dev_no))
                return 1;
 
        return 0;