Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S941911AbcJYVAs (ORCPT ); Tue, 25 Oct 2016 17:00:48 -0400 Received: from mail-pf0-f196.google.com ([209.85.192.196]:36190 "EHLO mail-pf0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964946AbcJYVAM (ORCPT ); Tue, 25 Oct 2016 17:00:12 -0400 From: frowand.list@gmail.com To: Rob Herring , pantelis.antoniou@konsulko.com, Pantelis Antoniou Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [RFC PATCH 10/13] of: Update comments to reflect changes and increase clarity Date: Tue, 25 Oct 2016 13:59:03 -0700 Message-Id: <1477429146-27039-11-git-send-email-frowand.list@gmail.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1477429146-27039-1-git-send-email-frowand.list@gmail.com> References: <1477429146-27039-1-git-send-email-frowand.list@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4195 Lines: 108 From: Frank Rowand Signed-off-by: Frank Rowand --- drivers/of/resolver.c | 51 ++++++++++++++++++++++++++++++++------------------- 1 file changed, 32 insertions(+), 19 deletions(-) diff --git a/drivers/of/resolver.c b/drivers/of/resolver.c index 76c09cb57eae..4e6df385118b 100644 --- a/drivers/of/resolver.c +++ b/drivers/of/resolver.c @@ -50,9 +50,6 @@ static struct device_node *find_node_by_full_name(struct device_node *node, return NULL; } -/* - * Find live tree's maximum phandle value. - */ static phandle live_tree_max_phandle(void) { struct device_node *node; @@ -71,9 +68,6 @@ static phandle live_tree_max_phandle(void) return phandle; } -/* - * Adjust a subtree's phandle values by a given delta. - */ static void adjust_overlay_phandles(struct device_node *overlay, int phandle_delta) { @@ -118,6 +112,7 @@ static int update_usages_of_a_phandle_reference(struct device_node *overlay, return -ENOMEM; memcpy(value, prop_fixup->value, prop_fixup->length); + /* prop_fixup contains a list of tuples of path:property_name:offset */ end = value + prop_fixup->length; for (cur = value; cur < end; cur += len + 1) { len = strlen(cur); @@ -177,10 +172,14 @@ static int node_name_cmp(const struct device_node *dn1, /* * Adjust the local phandle references by the given phandle delta. - * Assumes the existances of a __local_fixups__ node at the root. - * Assumes that __of_verify_tree_phandle_references has been called. - * Does not take any devtree locks so make sure you call this on a tree - * which is at the detached state. + * + * Subtree @local_fixups, which is overlay node __local_fixups__, + * mirrors the fragment node structure at the root of the overlay. + * + * For each property in the fragments that contains a phandle reference, + * @local_fixups has a property of the same name that contains a list + * of offsets of the phandle reference(s) within the respective property + * value(s). The values at these offsets will be fixed up. */ static int adjust_local_phandle_references(struct device_node *local_fixups, struct device_node *overlay, int phandle_delta) @@ -225,6 +224,13 @@ static int adjust_local_phandle_references(struct device_node *local_fixups, } } + /* + * These nested loops recurse down two subtrees in parallel, where the + * node names in the two subtrees match. + * + * The roots of the subtrees are the overlay's __local_fixups__ node + * and the overlay's root node. + */ for_each_child_of_node(local_fixups, child) { for_each_child_of_node(overlay, overlay_child) @@ -244,17 +250,24 @@ static int adjust_local_phandle_references(struct device_node *local_fixups, } /** - * of_resolve - Resolve the given node against the live tree. + * of_resolve_phandles - Relocate and resolve overlay against live tree + * + * @overlay: Pointer to devicetree overlay to relocate and resolve + * + * Modify (relocate) values of local phandles in @overlay to a range that + * does not conflict with the live expanded devicetree. Update references + * to the local phandles in @overlay. Update (resolve) phandle references + * in @overlay that refer to the live expanded devicetree. + * + * @overlay must be detached. * - * @resolve: Node to resolve + * Resolving and applying @overlay to the live expanded devicetree must be + * protected by a mechanism to ensure that multiple overlays are processed + * in a single threaded manner so that multiple overlays will not relocate + * phandles to overlapping ranges. The mechanism to enforce this is not + * yet implemented. * - * Perform dynamic Device Tree resolution against the live tree - * to the given node to resolve. This depends on the live tree - * having a __symbols__ node, and the resolve node the __fixups__ & - * __local_fixups__ nodes (if needed). - * The result of the operation is a resolve node that it's contents - * are fit to be inserted or operate upon the live tree. - * Returns 0 on success or a negative error value on error. + * Return: %0 on success or a negative error value on error. */ int of_resolve_phandles(struct device_node *overlay) { -- 1.9.1