commit 5e2cd5e67a488c6a69d877f5e1294d2c5792161f
author Simon Glass <sjg@chromium.org> Sat Jul 30 15:52:10 2022 -0600
committer Tom Rini <trini@konsulko.com> Fri Aug 12 08:14:24 2022 -0400
tree 4ba504834c7df3cf5029337703da754a9f4287d0
parent 270a4437aaa44f78cebaae47cedfec47176c954d

dm: core: Swap parameters of ofnode_write_prop()

It is normal for the length to come after the value in libfdt. Follow this
same convention with ofnode.

Signed-off-by: Simon Glass <sjg@chromium.org>

drivers/core/ofnode.c

diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c
index bd41ef5..1c9542a 100644
--- a/drivers/core/ofnode.c
+++ b/drivers/core/ofnode.c
@@ -1105,8 +1105,8 @@
 	}
 }
 
-int ofnode_write_prop(ofnode node, const char *propname, int len,
-		      const void *value)
+int ofnode_write_prop(ofnode node, const char *propname, const void *value,
+		      int len)
 {
 	const struct device_node *np = ofnode_to_np(node);
 	struct property *pp;
@@ -1161,7 +1161,7 @@
 
 	debug("%s: %s = %s", __func__, propname, value);
 
-	return ofnode_write_prop(node, propname, strlen(value) + 1, value);
+	return ofnode_write_prop(node, propname, value, strlen(value) + 1);
 }
 
 int ofnode_set_enabled(ofnode node, bool value)

include/dm/ofnode.h

diff --git a/include/dm/ofnode.h b/include/dm/ofnode.h
index d7ad5dc..071a9d6 100644
--- a/include/dm/ofnode.h
+++ b/include/dm/ofnode.h
@@ -1114,13 +1114,13 @@
  *
  * @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 (must be valid prior to calling
  *		the function)
+ * @len:	The length of the new value of the property
  * Return: 0 if successful, -ve on error
  */
-int ofnode_write_prop(ofnode node, const char *propname, int len,
-		      const void *value);
+int ofnode_write_prop(ofnode node, const char *propname, const void *value,
+		      int len);
 
 /**
  * ofnode_write_string() - Set a string property of a ofnode

test/dm/ofnode.c

diff --git a/test/dm/ofnode.c b/test/dm/ofnode.c
index b8d8e44..0aeaaeb 100644
--- a/test/dm/ofnode.c
+++ b/test/dm/ofnode.c
@@ -576,8 +576,8 @@
 	/* Non-existent in DTB */
 	ut_asserteq_64(FDT_ADDR_T_NONE, dev_read_addr(dev));
 	/* reg = 0x42, size = 0x100 */
-	ut_assertok(ofnode_write_prop(node, "reg", 8,
-				      "\x00\x00\x00\x42\x00\x00\x01\x00"));
+	ut_assertok(ofnode_write_prop(node, "reg",
+				      "\x00\x00\x00\x42\x00\x00\x01\x00", 8));
 	ut_asserteq(0x42, dev_read_addr(dev));
 
 	/* Test disabling devices */