On Thu, May 8, 2014 at 3:51 AM, Frank Rowand <frowand.list@xxxxxxxxx> wrote: > On 5/7/2014 2:48 PM, Rob Herring wrote: >> From: Rob Herring <robh@xxxxxxxxxx> >> >> Add a test case for nodes which have the same name and same >> non-translatable unit address. > > If I apply patch 1 and 2 without applying 3 and 4 then console > warnings are printed, but from a different area of code than > the original problem reported. This probably is not a big deal, > but I'm trying to figure out if I can modify the test to also > show the original problem. > > The test case also properly reports the failure. > > Once all 4 patches are applied, then the test case passes. When adding testcases I do prefer the addition of the test to appear before the fix while the patch is under review. Makes it easy to see that, yes the bug exists, and yes the later patch resolves it. When the patches are actually committed for merging then it is probably a good idea to reverse them. Thanks for the testing. g. > > Thus: > > Tested-by: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx> > >> >> Signed-off-by: Rob Herring <robh@xxxxxxxxxx> >> --- >> drivers/of/selftest.c | 23 ++++++++++++++++++ >> drivers/of/testcase-data/testcases.dtsi | 1 + >> drivers/of/testcase-data/tests-platform.dtsi | 35 ++++++++++++++++++++++++++++ >> 3 files changed, 59 insertions(+) >> create mode 100644 drivers/of/testcase-data/tests-platform.dtsi >> > > < snip > > -- 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