Hi Pantelis,
On Mon, Sep 21, 2015 at 2:36 PM, Pantelis Antoniou
<[email protected]> wrote:
>> On Sep 21, 2015, at 15:35 , Geert Uytterhoeven <[email protected]> wrote:
>> On Wed, Sep 16, 2015 at 6:11 PM, Pantelis Antoniou
>> <[email protected]> wrote:
>>> 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");
>>
>> What about removing properties?
>
> Once upon a time there was that capability. It was removed after we didn’t
> have
> a good use for them yet. Do you have any? I’d be happy to re-add it :)
Aliases?
If an overlay removes e.g. a serial port, it should remove its alias, too.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html