dm: core: Rename ofnode_get_property_by_prop()

The current name is quite unwieldy. Change it to use an ofprop_ prefix
and shorten it. Fix the return-value comment while we are here.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-09-06 20:27:14 -06:00 committed by Tom Rini
parent 4b1f571465
commit 9243224687
6 changed files with 15 additions and 16 deletions

View file

@ -816,7 +816,7 @@ int ofnode_next_property(struct ofprop *prop)
return 0; return 0;
} }
const void *ofnode_get_property_by_prop(const struct ofprop *prop, const void *ofprop_get_property(const struct ofprop *prop,
const char **propname, int *lenp) const char **propname, int *lenp)
{ {
if (ofnode_is_np(prop->node)) if (ofnode_is_np(prop->node))

View file

@ -298,7 +298,7 @@ int dev_read_next_prop(struct ofprop *prop)
const void *dev_read_prop_by_prop(struct ofprop *prop, const void *dev_read_prop_by_prop(struct ofprop *prop,
const char **propname, int *lenp) const char **propname, int *lenp)
{ {
return ofnode_get_property_by_prop(prop, propname, lenp); return ofprop_get_property(prop, propname, lenp);
} }
int dev_read_alias_seq(const struct udevice *dev, int *devnump) int dev_read_alias_seq(const struct udevice *dev, int *devnump)

2
env/common.c vendored
View file

@ -544,7 +544,7 @@ void env_import_fdt(void)
res = ofnode_next_property(&prop)) { res = ofnode_next_property(&prop)) {
const char *name, *val; const char *name, *val;
val = ofnode_get_property_by_prop(&prop, &name, NULL); val = ofprop_get_property(&prop, &name, NULL);
env_set(name, val); env_set(name, val);
} }
} }

View file

@ -802,7 +802,7 @@ const void *ofnode_get_property(ofnode node, const char *propname, int *lenp);
* ofnode_first_property()- get the reference of the first property * ofnode_first_property()- get the reference of the first property
* *
* Get reference to the first property of the node, it is used to iterate * Get reference to the first property of the node, it is used to iterate
* and read all the property with ofnode_get_property_by_prop(). * and read all the property with ofprop_get_property().
* *
* @node: node to read * @node: node to read
* @prop: place to put argument reference * @prop: place to put argument reference
@ -814,7 +814,7 @@ int ofnode_first_property(ofnode node, struct ofprop *prop);
* ofnode_next_property() - get the reference of the next property * ofnode_next_property() - get the reference of the next property
* *
* Get reference to the next property of the node, it is used to iterate * Get reference to the next property of the node, it is used to iterate
* and read all the property with ofnode_get_property_by_prop(). * and read all the property with ofprop_get_property().
* *
* @prop: reference of current argument and place to put reference of next one * @prop: reference of current argument and place to put reference of next one
* Return: 0 if OK, -ve on error. -FDT_ERR_NOTFOUND if not found * Return: 0 if OK, -ve on error. -FDT_ERR_NOTFOUND if not found
@ -822,16 +822,16 @@ int ofnode_first_property(ofnode node, struct ofprop *prop);
int ofnode_next_property(struct ofprop *prop); int ofnode_next_property(struct ofprop *prop);
/** /**
* ofnode_get_property_by_prop() - get a pointer to the value of a property * ofprop_get_property() - get a pointer to the value of a property
* *
* Get value for the property identified by the provided reference. * Get value for the property identified by the provided reference.
* *
* @prop: reference on property * @prop: reference on property
* @propname: If non-NULL, place to property name on success, * @propname: If non-NULL, place to property name on success,
* @lenp: If non-NULL, place to put length on success * @lenp: If non-NULL, place to put length on success, or error code on failure
* Return: 0 if OK, -ve on error. -FDT_ERR_NOTFOUND if not found * Return: pointer to property, or NULL if not found
*/ */
const void *ofnode_get_property_by_prop(const struct ofprop *prop, const void *ofprop_get_property(const struct ofprop *prop,
const char **propname, int *lenp); const char **propname, int *lenp);
/** /**

View file

@ -1091,7 +1091,7 @@ static inline const void *dev_read_prop_by_prop(struct ofprop *prop,
const char **propname, const char **propname,
int *lenp) int *lenp)
{ {
return ofnode_get_property_by_prop(prop, propname, lenp); return ofprop_get_property(prop, propname, lenp);
} }
static inline int dev_read_alias_seq(const struct udevice *dev, int *devnump) static inline int dev_read_alias_seq(const struct udevice *dev, int *devnump)

View file

@ -5,7 +5,7 @@
#include <dm/test.h> #include <dm/test.h>
#include <test/ut.h> #include <test/ut.h>
static int dm_test_ofnode_get_property_by_prop(struct unit_test_state *uts) static int dm_test_ofprop_get_property(struct unit_test_state *uts)
{ {
ofnode node; ofnode node;
struct ofprop prop; struct ofprop prop;
@ -17,7 +17,7 @@ static int dm_test_ofnode_get_property_by_prop(struct unit_test_state *uts)
for (res = ofnode_first_property(node, &prop); for (res = ofnode_first_property(node, &prop);
!res; !res;
res = ofnode_next_property(&prop)) { res = ofnode_next_property(&prop)) {
value = ofnode_get_property_by_prop(&prop, &propname, &len); value = ofprop_get_property(&prop, &propname, &len);
ut_assertnonnull(value); ut_assertnonnull(value);
switch (count) { switch (count) {
case 0: case 0:
@ -46,5 +46,4 @@ static int dm_test_ofnode_get_property_by_prop(struct unit_test_state *uts)
return 0; return 0;
} }
DM_TEST(dm_test_ofnode_get_property_by_prop, DM_TEST(dm_test_ofprop_get_property, UT_TESTF_SCAN_PDATA | UT_TESTF_SCAN_FDT);
UT_TESTF_SCAN_PDATA | UT_TESTF_SCAN_FDT);