Returning !bbu_find_handler() from barebox_update_handler_exists() would return the opposite result from what the name of that funciton implies. Drop the "!" to make it behave as expected. Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> --- common/bbu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/bbu.c b/common/bbu.c index 11e44f4a7..9e206227a 100644 --- a/common/bbu.c +++ b/common/bbu.c @@ -151,7 +151,7 @@ bool barebox_update_handler_exists(struct bbu_data *data) if (!data->handler_name) return false; - return !bbu_find_handler(data->handler_name); + return bbu_find_handler(data->handler_name) != NULL; } static int bbu_check_of_compat(struct bbu_data *data) -- 2.17.1 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox