Messages in this thread | | | Subject | Re: [PATCH v5 2/3] of/selftest: Add self tests for manipulation of properties | From | Pantelis Antoniou <> | Date | Sat, 16 Nov 2013 19:50:37 +0200 |
| |
Hi Grant,
On Nov 15, 2013, at 7:46 PM, Grant Likely wrote:
> Adds a few simple test cases to ensure that addition, update and removal > of device tree node properties works correctly. > > Signed-off-by: Grant Likely <grant.likely@linaro.org> > Cc: Rob Herring <rob.herring@calxeda.com> > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: David S. Miller <davem@davemloft.net> > Cc: Nathan Fontenot <nfont@linux.vnet.ibm.com> > Cc: Pantelis Antoniou <panto@antoniou-consulting.com> > --- > drivers/of/selftest.c | 62 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 62 insertions(+) > > diff --git a/drivers/of/selftest.c b/drivers/of/selftest.c > index e21012bde639..cb8d3e722f76 100644 > --- a/drivers/of/selftest.c > +++ b/drivers/of/selftest.c > @@ -30,6 +30,67 @@ static struct selftest_results { > } \ > } > > +static void __init of_selftest_dynamic(void) > +{ > + struct device_node *np; > + struct property *prop; > + > + np = of_find_node_by_path("/testcase-data"); > + if (!np) { > + pr_err("missing testcase data\n"); > + return; > + } > + > + /* Array of 4 properties for the purpose of testing */ > + prop = kzalloc(sizeof(*prop) * 4, GFP_KERNEL); > + if (!prop) { > + selftest(0, "kzalloc() failed\n"); > + return; > + } > + > + /* Add a new property - should pass*/ > + prop->name = "new-property"; > + prop->value = "new-property-data"; > + prop->length = strlen(prop->value); > + selftest(of_add_property(np, prop) == 0, "Adding a new property failed\n"); > + > + /* Try to add an existing property - should fail */ > + prop++; > + prop->name = "new-property"; > + prop->value = "new-property-data-should-fail"; > + prop->length = strlen(prop->value); > + selftest(of_add_property(np, prop) != 0, > + "Adding an existing property should have failed\n"); > + > + /* Try to modify an existing property - should pass */ > + prop->value = "modify-property-data-should-pass"; > + prop->length = strlen(prop->value); > + selftest(of_update_property(np, prop) == 0, > + "Updating an existing property should have passed\n"); > + > + /* Try to modify non-existent property - should pass*/ > + prop++; > + prop->name = "modify-property"; > + prop->value = "modify-missing-property-data-should-pass"; > + prop->length = strlen(prop->value); > + selftest(of_update_property(np, prop) == 0, > + "Updating a missing property should have passed\n"); > + > + /* Remove property - should pass */ > + selftest(of_remove_property(np, prop) == 0, > + "Removing a property should have passed\n"); > + > + /* Adding very large property - should pass */ > + prop++; > + prop->name = "large-property-PAGE_SIZEx8"; > + prop->length = PAGE_SIZE * 8; > + prop->value = kzalloc(prop->length, GFP_KERNEL); > + selftest(prop->value != NULL, "Unable to allocate large buffer\n"); > + if (prop->value) > + selftest(of_add_property(np, prop) == 0, > + "Adding a large property should have passed\n"); > +} > + > static void __init of_selftest_parse_phandle_with_args(void) > { > struct device_node *np; > @@ -312,6 +373,7 @@ static int __init of_selftest(void) > of_node_put(np); > > pr_info("start of selftest - you will see error messages\n"); > + of_selftest_dynamic(); > of_selftest_parse_phandle_with_args(); > of_selftest_property_match_string(); > of_selftest_parse_interrupts(); > -- > 1.8.3.2 >
I like this. I'll add my overlay test cases somewhere around here.
I might need to introduce some new helper functions in order to generate the overlay completely programmatically.
Regards
-- Pantelis
| |