diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-03-14 16:08:28 -0400 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2018-03-19 23:52:40 -0400 |
commit | 827473268ee492b30b576ef156ab524a2b9f3520 (patch) | |
tree | 4798ef8eb4e41b684488333fb3f7dfc1df37f96f /drivers/of/unittest.c | |
parent | 202fbf4865e97b64ca2d94b7dc1a4efd6f7b0729 (diff) |
of: unittest: fix an error code in of_unittest_apply_overlay()
We accidentally return zero on failure instead of a negative error code.
Fixes: 39a751a4cb7e ("of: change overlay apply input data from unflattened to FDT")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drivers/of/unittest.c')
-rw-r--r-- | drivers/of/unittest.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/of/unittest.c b/drivers/of/unittest.c index a08c9e2d5c4c..02c5984ab09b 100644 --- a/drivers/of/unittest.c +++ b/drivers/of/unittest.c | |||
@@ -1423,15 +1423,13 @@ static int __init of_unittest_apply_overlay(int overlay_nr, int unittest_nr, | |||
1423 | int *overlay_id) | 1423 | int *overlay_id) |
1424 | { | 1424 | { |
1425 | const char *overlay_name; | 1425 | const char *overlay_name; |
1426 | int ret; | ||
1427 | 1426 | ||
1428 | overlay_name = overlay_name_from_nr(overlay_nr); | 1427 | overlay_name = overlay_name_from_nr(overlay_nr); |
1429 | 1428 | ||
1430 | ret = overlay_data_apply(overlay_name, overlay_id); | 1429 | if (!overlay_data_apply(overlay_name, overlay_id)) { |
1431 | if (!ret) { | ||
1432 | unittest(0, "could not apply overlay \"%s\"\n", | 1430 | unittest(0, "could not apply overlay \"%s\"\n", |
1433 | overlay_name); | 1431 | overlay_name); |
1434 | return ret; | 1432 | return -EFAULT; |
1435 | } | 1433 | } |
1436 | of_unittest_track_overlay(*overlay_id); | 1434 | of_unittest_track_overlay(*overlay_id); |
1437 | 1435 | ||