diff options
| author | David Kershner <david.kershner@unisys.com> | 2017-08-22 13:27:22 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-08-22 15:12:52 -0700 |
| commit | 727bb6485e1db639621eb3d96d7fb067b07810db (patch) | |
| tree | 01b0a6ebb6c7655a636a08b98f1e3f4a4487ac30 /drivers/staging/unisys | |
| parent | f51963f54d9f5f82f721c6bbf8785a58d29d3fdd (diff) | |
| download | linux-727bb6485e1db639621eb3d96d7fb067b07810db.tar.bz2 | |
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>
Diffstat (limited to 'drivers/staging/unisys')
| -rw-r--r-- | drivers/staging/unisys/visorbus/visorchipset.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c index 29475ccf81c1..98ce796e85c1 100644 --- 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; |