2023-11-17 17:21:37

by Johan Hovold

Subject: [PATCH] of: fix recursion typo in kernel doc

Fix a typo in the kernel doc for the of_platform_depopulate() functions,
which remove children "recursively".

Signed-off-by: Johan Hovold <[email protected]>
---
drivers/of/platform.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index 126d265aa7d8..2293059758d1 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -668,7 +668,7 @@ EXPORT_SYMBOL_GPL(of_platform_device_destroy);
* @parent: device which children will be removed
*
* Complementary to of_platform_populate(), this function removes children
- * of the given device (and, recurrently, their children) that have been
+ * of the given device (and, recursively, their children) that have been
* created from their respective device tree nodes (and only those,
* leaving others - eg. manually created - unharmed).
*/
@@ -737,7 +737,7 @@ static int devm_of_platform_match(struct device *dev, void *res, void *data)
* @dev: device that requested to depopulate from device tree data
*
* Complementary to devm_of_platform_populate(), this function removes children
- * of the given device (and, recurrently, their children) that have been
+ * of the given device (and, recursively, their children) that have been
* created from their respective device tree nodes (and only those,
* leaving others - eg. manually created - unharmed).
*/
--
2.41.0

2023-11-17 18:08:27

by Randy Dunlap

Subject: Re: [PATCH] of: fix recursion typo in kernel doc

On 11/17/23 09:16, Johan Hovold wrote:
> Fix a typo in the kernel doc for the of_platform_depopulate() functions,
> which remove children "recursively".
>
> Signed-off-by: Johan Hovold <[email protected]>

Looks good. Thanks.

Reviewed-by: Randy Dunlap <[email protected]>

> ---
> drivers/of/platform.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/of/platform.c b/drivers/of/platform.c
> index 126d265aa7d8..2293059758d1 100644
> --- a/drivers/of/platform.c
> +++ b/drivers/of/platform.c
> @@ -668,7 +668,7 @@ EXPORT_SYMBOL_GPL(of_platform_device_destroy);
> * @parent: device which children will be removed
> *
> * Complementary to of_platform_populate(), this function removes children
> - * of the given device (and, recurrently, their children) that have been
> + * of the given device (and, recursively, their children) that have been
> * created from their respective device tree nodes (and only those,
> * leaving others - eg. manually created - unharmed).
> */
> @@ -737,7 +737,7 @@ static int devm_of_platform_match(struct device *dev, void *res, void *data)
> * @dev: device that requested to depopulate from device tree data
> *
> * Complementary to devm_of_platform_populate(), this function removes children
> - * of the given device (and, recurrently, their children) that have been
> + * of the given device (and, recursively, their children) that have been
> * created from their respective device tree nodes (and only those,
> * leaving others - eg. manually created - unharmed).
> */

--
~Randy

2023-11-19 16:11:04

by Rob Herring (Arm)

Subject: Re: [PATCH] of: fix recursion typo in kernel doc

On Fri, 17 Nov 2023 18:16:28 +0100, Johan Hovold wrote:
> Fix a typo in the kernel doc for the of_platform_depopulate() functions,
> which remove children "recursively".
>
> Signed-off-by: Johan Hovold <[email protected]>
> ---
> drivers/of/platform.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>

Applied, thanks!