Hi Mario,
On 28 March 2018 at 20:37, Mario Six <mario....@gdsys.cc> wrote:
> Implement a set of functions to manipulate properties in a live device
> tree:
>
> * ofnode_set_property() to set generic properties of a node
> * ofnode_write_string() to set string properties of a node
> * ofnode_enable() to enable a node
> * ofnode_disable() to disable a node
>
Please add a test for these functions.
> Signed-off-by: Mario Six <mario....@gdsys.cc>
> ---
> drivers/core/ofnode.c | 58
> +++++++++++++++++++++++++++++++++++++++++++++++++++
> include/dm/ofnode.h | 49 +++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 107 insertions(+)
>
> diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c
> index ca002063b3..90d05aa559 100644
> --- a/drivers/core/ofnode.c
> +++ b/drivers/core/ofnode.c
> @@ -699,3 +699,61 @@ u64 ofnode_translate_address(ofnode node, const fdt32_t
> *in_addr)
> else
> return fdt_translate_address(gd->fdt_blob,
> ofnode_to_offset(node), in_addr);
> }
> +
> +#ifdef CONFIG_OF_LIVE
Is this needed?
> +int ofnode_set_property(ofnode node, const char *propname, int len,
> + const void *value)
> +{
> + const struct device_node *np = ofnode_to_np(node);
> + struct property *pp;
> + struct property *new;
> +
> + if (!np)
> + return -EINVAL;
> +
> + for (pp = np->properties; pp; pp = pp->next) {
> + if (strcmp(pp->name, propname) == 0) {
> + /* Property exists -> change value */
> + pp->value = (void *)value;
> + pp->length = len;
> + return 0;
> + }
> + }
> +
> + /* Property does not exist -> append new property */
> + new = malloc(sizeof(struct property));
> +
> + new->name = strdup(propname);
> + new->value = malloc(len);
> + memcpy(new->value, value, len);
> + new->length = len;
> + new->next = NULL;
> +
> + pp->next = new;
> +
> + return 0;
> +}
> +
> +int ofnode_write_string(ofnode node, const char *propname, const char *value)
> +{
> + assert(ofnode_valid(node));
What does this function do if livetree is not enabled?
> + debug("%s: %s = %s", __func__, propname, value);
> +
> + return ofnode_set_property(node, propname, strlen(value) + 1, value);
> +}
> +
> +int ofnode_enable(ofnode node)
> +{
> + assert(ofnode_valid(node));
> +
> + return ofnode_write_string(node, "status", "okay");
> +}
> +
> +int ofnode_disable(ofnode node)
> +{
> + assert(ofnode_valid(node));
> +
> + return ofnode_write_string(node, "status", "disable");
> +}
> +
> +#endif
> diff --git a/include/dm/ofnode.h b/include/dm/ofnode.h
> index aec205eb80..e82ebf19c5 100644
> --- a/include/dm/ofnode.h
> +++ b/include/dm/ofnode.h
> @@ -689,4 +689,53 @@ int ofnode_read_resource_byname(ofnode node, const char
> *name,
> * @return the translated address; OF_BAD_ADDR on error
> */
> u64 ofnode_translate_address(ofnode node, const fdt32_t *in_addr);
> +
> +#ifdef CONFIG_OF_LIVE
> +
> +/**
> + * ofnode_set_property() - Set a property of a ofnode
> + *
> + * @node: The node for whose property should be set
> + * @propname: The name of the property to set
> + * @len: The length of the new value of the property
> + * @value: The new value of the property
> + * @return 0 if successful, -ve on error
> + */
> +int ofnode_set_property(ofnode node, const char *propname, int len,
> + const void *value);
We should think about consistency here. Maybe
ofnode_write_prop()?
> +
> +/**
> + * ofnode_write_string() - Set a string property of a ofnode
> + *
> + * @node: The node for whose string property should be set
> + * @propname: The name of the string property to set
> + * @value: The new value of the string property
> + * @return 0 if successful, -ve on error
> + */
> +int ofnode_write_string(ofnode node, const char *propname, const char
> *value);
> +
> +/**
> + * ofnode_enable() - Enable a device tree node given by its ofnode
> + *
> + * This function effectively sets the node's "status" property to "okay",
> hence
> + * making it available for driver model initialization.
> + *
> + * @node: The node to enable
> + * @return 0 if successful, -ve on error
> + */
> +int ofnode_enable(ofnode node);
> +
> +/**
> + * ofnode_disable() - Disable a device tree node given by its ofnode
> + *
> + * This function effectively sets the node's "status" property to "disable",
> + * hence stopping it from being picked up by driver model initialization.
> + *
> + * @node: The node to disable
> + * @return 0 if successful, -ve on error
> + */
> +int ofnode_disable(ofnode node);
Would it be OK to have one function like ofnode_set_enabled(ofnode
node, bool enable) ?
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
