Re: [PATCH] of: platform: rename parameter 'bus' to 'node' in of_platform_bus_create()

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

On 7/17/25 15:14, Bo Sun wrote:
> The parameter represents a device tree node, not specifically a bus.
> Use more accurate naming following device tree conventions.
> 
> Signed-off-by: Bo Sun <bo@mboxify.com>

Acked-by: Ahmad Fatoum <a.fatoum@pengutronix.de>

> ---
>  drivers/of/platform.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/of/platform.c b/drivers/of/platform.c
> index 3600b5f9c1..092ec39b05 100644
> --- a/drivers/of/platform.c
> +++ b/drivers/of/platform.c
> @@ -399,14 +399,14 @@ static inline struct device *of_amba_device_create(struct device_node *np)
>  
>  /**
>   * of_platform_bus_create() - Create a device for a node and its children.
> - * @bus: device node of the bus to instantiate
> + * @node: device node of the bus to instantiate
>   * @matches: match table for bus nodes
>   * @parent: parent for new device, or NULL for top level.
>   *
>   * Creates a platform_device for the provided device_node, and optionally
>   * recursively create devices for all the child nodes.
>   */
> -static int of_platform_bus_create(struct device_node *bus,
> +static int of_platform_bus_create(struct device_node *node,
>  				const struct of_device_id *matches,
>  				struct device *parent)
>  {
> @@ -415,22 +415,22 @@ static int of_platform_bus_create(struct device_node *bus,
>  	int rc = 0;
>  
>  	/* Make sure it has a compatible property */
> -	if (!of_get_property(bus, "compatible", NULL)) {
> +	if (!of_get_property(node, "compatible", NULL)) {
>  		pr_debug("%s() - skipping %pOF, no compatible prop\n",
> -			__func__, bus);
> +			__func__, node);
>  		return 0;
>  	}
>  
> -	if (of_device_is_compatible(bus, "arm,primecell")) {
> -		if (of_amba_device_create(bus))
> +	if (of_device_is_compatible(node, "arm,primecell")) {
> +		if (of_amba_device_create(node))
>  			return 0;
>  	}
>  
> -	dev = of_platform_device_create(bus, parent);
> -	if (!dev || !of_match_node(matches, bus))
> +	dev = of_platform_device_create(node, parent);
> +	if (!dev || !of_match_node(matches, node))
>  		return 0;
>  
> -	for_each_child_of_node(bus, child) {
> +	for_each_child_of_node(node, child) {
>  		pr_debug("   create child: %pOF\n", child);
>  		rc = of_platform_bus_create(child, matches, dev);
>  		if (rc)
> 
> 

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |