[PATCH] of: base: add parameter doc to of_parse_phandle_with_optional_args()

Michael Walle posted 1 patch 4 years, 5 months ago
include/linux/of.h | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
[PATCH] of: base: add parameter doc to of_parse_phandle_with_optional_args()
Posted by Michael Walle 4 years, 5 months ago
htmldocs produces warnings about the missing documentation. Add them.
While at it, fix the typo in the referenced function name.

Fixes: 952c4865d27a ("of: base: add of_parse_phandle_with_optional_args()")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Michael Walle <michael@walle.cc>
---
 include/linux/of.h | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/include/linux/of.h b/include/linux/of.h
index eeb2910b51da..062a0d899403 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -1021,9 +1021,14 @@ static inline int of_parse_phandle_with_fixed_args(const struct device_node *np,
 
 /**
  * of_parse_phandle_with_optional_args() - Find a node pointed by phandle in a list
+ * @np:		pointer to a device tree node containing a list
+ * @list_name:	property name that contains a list
+ * @cells_name:	property name that specifies phandles' arguments count
+ * @index:	index of a phandle to parse out
+ * @out_args:	optional pointer to output arguments structure (will be filled)
  *
- * Same as of_parse_phandle_args() except that if the cells_name property is
- * not found, cell_count of 0 is assumed.
+ * Same as of_parse_phandle_with_args() except that if the cells_name property
+ * is not found, cell_count of 0 is assumed.
  *
  * This is used to useful, if you have a phandle which didn't have arguments
  * before and thus doesn't have a '#*-cells' property but is now migrated to
-- 
2.30.2

Re: [PATCH] of: base: add parameter doc to of_parse_phandle_with_optional_args()
Posted by Rob Herring 4 years, 5 months ago
On Sat, 15 Jan 2022 12:31:56 +0100, Michael Walle wrote:
> htmldocs produces warnings about the missing documentation. Add them.
> While at it, fix the typo in the referenced function name.
> 
> Fixes: 952c4865d27a ("of: base: add of_parse_phandle_with_optional_args()")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Michael Walle <michael@walle.cc>
> ---
>  include/linux/of.h | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 

Applied, thanks!