On Tue, Feb 20, 2018 at 5:10 PM, Laurent Pinchart <[email protected]> wrote: > From: Pantelis Antoniou <[email protected]> > > Changesets are very powerful, but the lack of a helper API > makes using them cumbersome. Introduce a simple copy based > API that makes things considerably easier. > > To wit, adding a property using the raw API. > > struct property *prop; > prop = kzalloc(sizeof(*prop)), GFP_KERNEL); > prop->name = kstrdup("compatible"); > prop->value = kstrdup("foo,bar"); > prop->length = strlen(prop->value) + 1; > of_changeset_add_property(ocs, np, prop); > > while using the helper API > > of_changeset_add_property_string(ocs, np, "compatible", > "foo,bar"); > > Signed-off-by: Pantelis Antoniou <[email protected]> > [Fixed memory leak in __of_changeset_add_update_property_copy()] > Signed-off-by: Laurent Pinchart <[email protected]> > --- > drivers/of/dynamic.c | 222 ++++++++++++++++++++++++++++++++++ > include/linux/of.h | 328 > +++++++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 550 insertions(+)
Other than what Geert pointed out, Reviewed-by: Rob Herring <[email protected]> _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
