[PATCH v5 5/5] staging: unisys: remove comparison

From: Sudip Mukherjee
Date: Tue Mar 24 2015 - 11:18:09 EST


the comparison is always true as the dev_t has been initialized in the
init function and we are sending that initialized dev_t to the
cleanup().

Signed-off-by: Sudip Mukherjee <sudip@xxxxxxxxxxxxxxx>
---

v5: reordered the patch series
v4: messed up the subject in v3
v3: broke the previous patch in series

drivers/staging/unisys/visorchipset/file.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/staging/unisys/visorchipset/file.c b/drivers/staging/unisys/visorchipset/file.c
index 890869a..39b19af 100644
--- a/drivers/staging/unisys/visorchipset/file.c
+++ b/drivers/staging/unisys/visorchipset/file.c
@@ -76,9 +76,7 @@ visorchipset_file_cleanup(dev_t major_dev)
if (file_cdev.ops != NULL)
cdev_del(&file_cdev);
file_cdev.ops = NULL;
- if (MAJOR(major_dev) >= 0) {
- unregister_chrdev_region(major_dev, 1);
- }
+ unregister_chrdev_region(major_dev, 1);
}

static int
--
1.8.1.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/