Re: [PATCH v2] Removes OF_UNITTEST dependency on OF_DYNAMIC config symbol
From: Grant Likely <hidden>
Date: 2015-01-22 16:59:33
On Thu, Jan 22, 2015 at 4:56 PM, Grant Likely [off-list ref] wrote:
On Sat, 10 Jan 2015 23:19:51 -0800 , Gaurav Minocha [off-list ref] wrote:quoted
This patch intends to remove the unittests dependency on the functions defined in dynamic.c. So, rather than calling of_attach_node defined in dynamic.c, minimal functionality required to attach a new node is re-defined in unittest.c. Also, now after executing the tests the test data is not removed from the device tree so there is no need to call of_detach_node. Tested with and without OF_DYNAMIC enabled on ppc, arm and x86 Signed-off-by: Gaurav Minocha <redacted>Looks good, but it leaves in some stale node removal code. I've fixed it up and applied for v3.19. Here is my fixup:
On reflection, picking this up for v3.19 is a bad decision because it is not a regression fix. I'll submit a separate fixup patch to remove the stale code. g.
quoted hunk
---diff --git a/drivers/of/unittest.c b/drivers/of/unittest.c index 139363af5c88..28beed1b3c2c 100644 --- a/drivers/of/unittest.c +++ b/drivers/of/unittest.c@@ -27,11 +27,6 @@ static struct selftest_results { int failed; } selftest_results; -#define NO_OF_NODES 3 -static struct device_node *nodes[NO_OF_NODES]; -static int last_node_index; -static bool selftest_live_tree; - #define selftest(result, fmt, ...) ({ \ bool failed = !(result); \ if (failed) { \@@ -830,13 +825,6 @@ static int attach_node_and_children(struct device_node *np) return 0; } - /* Children of the root need to be remembered for removal */ - if (np->parent == of_root) { - if (WARN_ON(last_node_index >= NO_OF_NODES)) - return -EINVAL; - nodes[last_node_index++] = np; - } - child = np->child; np->child = NULL;@@ -899,8 +887,6 @@ static int __init selftest_data_add(void) } if (!of_root) { - /* enabling flag for removing nodes */ - selftest_live_tree = true; of_root = selftest_data_node; for_each_of_allnodes(np)
-- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html