From: William Dean <williamsukat...@163.com>

It could directly return 'devm_add_action_or_reset' to simplify code.

Signed-off-by: William Dean <williamsukat...@163.com>
---
 drivers/dax/bus.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c
index 1dad813ee4a6..251c53061764 100644
--- a/drivers/dax/bus.c
+++ b/drivers/dax/bus.c
@@ -772,11 +772,8 @@ static int devm_register_dax_mapping(struct dev_dax 
*dev_dax, int range_id)
                return rc;
        }
 
-       rc = devm_add_action_or_reset(dax_region->dev, unregister_dax_mapping,
+       return devm_add_action_or_reset(dax_region->dev, unregister_dax_mapping,
                        dev);
-       if (rc)
-               return rc;
-       return 0;
 }
 
 static int alloc_dev_dax_range(struct dev_dax *dev_dax, u64 start,
-- 
2.25.1


Reply via email to