Hi Frank, > On Oct 25, 2016, at 23:59 , frowand.list@xxxxxxxxx wrote: > > From: Frank Rowand <frank.rowand@xxxxxxxxxxx> > > This unused variable is a reminder that symbols in overlays are > not available to subsequent overlays. If such a feature is > desired then there are several ways it could be implemented. > Please don’t apply that. There’s a patch that actually imports the symbol table from overlays that subsequent operations work. Please see: https://patchwork.kernel.org/patch/9104701/ > Signed-off-by: Frank Rowand <frank.rowand@xxxxxxxxxxx> > --- > drivers/of/resolver.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c > index 3f7cf569c7ea..b48d16200ccd 100644 > --- a/drivers/of/resolver.c > +++ b/drivers/of/resolver.c > @@ -272,7 +272,7 @@ static int adjust_local_phandle_references(struct device_node *local_fixups, > int of_resolve_phandles(struct device_node *overlay) > { > struct device_node *child, *local_fixups, *refnode; > - struct device_node *tree_symbols, *overlay_symbols, *overlay_fixups; > + struct device_node *tree_symbols, *overlay_fixups; > struct property *prop; > const char *refpath; > phandle phandle, phandle_delta; > @@ -302,12 +302,9 @@ int of_resolve_phandles(struct device_node *overlay) > if (err) > goto err_out; > > - overlay_symbols = NULL; > overlay_fixups = NULL; > > for_each_child_of_node(overlay, child) { > - if (!of_node_cmp(child->name, "__symbols__")) > - overlay_symbols = child; > if (!of_node_cmp(child->name, "__fixups__")) > overlay_fixups = child; > } > -- > 1.9.1 > > -- > 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 Regards — Pantelis -- 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
