On 03/06/18 22:18, Dan Carpenter wrote: > We changed this from of_overlay_apply() to overlay_data_apply(). The > overlay_data_apply() function returns 1 on success and 0 on error so > the check for less than zero needs to be updated. > > Fixes: 39a751a4cb7e ("of: change overlay apply input data from unflattened to FDT") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > diff --git a/drivers/of/unittest.c b/drivers/of/unittest.c > index ca5e86dadae2..b233f170b2ec 100644 > --- a/drivers/of/unittest.c > +++ b/drivers/of/unittest.c > @@ -1666,8 +1666,7 @@ static void __init of_unittest_overlay_8(void) > > overlay_name = overlay_name_from_nr(overlay_nr + i); > > - ret = overlay_data_apply(overlay_name, &ovcs_id); > - if (ret < 0) { > + if (!overlay_data_apply(overlay_name, &ovcs_id)) { > unittest(0, "could not apply overlay \"%s\"\n", > overlay_name); > return; > Thank you. Reviewed-by: Frank Rowand <frank.rowand@xxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html