diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2017-03-16 10:23:53 -0400 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2017-03-22 10:42:17 -0400 |
commit | 3f6db6591a2decad5f223a7dcfc01d2a3c15e187 (patch) | |
tree | b609766a89ae1bb29661d27c9f32781b1f85daf9 | |
parent | 4a8ed2b819402ae450e3c53a1fe5eec59e3f423e (diff) |
iommu/dmar: Remove redundant assignment of ret
There is no need to assign ret to 0 in some cases. Moreover it might
shadow some errors in the future.
Remove such assignments.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r-- | drivers/iommu/dmar.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c index 71d774f1d406..9a44e20d7d88 100644 --- a/drivers/iommu/dmar.c +++ b/drivers/iommu/dmar.c | |||
@@ -391,7 +391,7 @@ static int dmar_parse_one_drhd(struct acpi_dmar_header *header, void *arg) | |||
391 | { | 391 | { |
392 | struct acpi_dmar_hardware_unit *drhd; | 392 | struct acpi_dmar_hardware_unit *drhd; |
393 | struct dmar_drhd_unit *dmaru; | 393 | struct dmar_drhd_unit *dmaru; |
394 | int ret = 0; | 394 | int ret; |
395 | 395 | ||
396 | drhd = (struct acpi_dmar_hardware_unit *)header; | 396 | drhd = (struct acpi_dmar_hardware_unit *)header; |
397 | dmaru = dmar_find_dmaru(drhd); | 397 | dmaru = dmar_find_dmaru(drhd); |
@@ -609,8 +609,8 @@ static int __init | |||
609 | parse_dmar_table(void) | 609 | parse_dmar_table(void) |
610 | { | 610 | { |
611 | struct acpi_table_dmar *dmar; | 611 | struct acpi_table_dmar *dmar; |
612 | int ret = 0; | ||
613 | int drhd_count = 0; | 612 | int drhd_count = 0; |
613 | int ret; | ||
614 | struct dmar_res_callback cb = { | 614 | struct dmar_res_callback cb = { |
615 | .print_entry = true, | 615 | .print_entry = true, |
616 | .ignore_unhandled = true, | 616 | .ignore_unhandled = true, |