[PATCH 4/4] mmc: sdhci: aspeed: Simplify with scoped for each OF child loop
Ulf Hansson
ulf.hansson at linaro.org
Wed Dec 31 02:33:39 AEDT 2025
On Wed, 24 Dec 2025 at 13:44, Krzysztof Kozlowski
<krzysztof.kozlowski at oss.qualcomm.com> wrote:
>
> Use scoped for-each loop when iterating over device nodes to make code a
> bit simpler.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski at oss.qualcomm.com>
All patches in the series applied for next, thanks!
Kind regards
Uffe
> ---
> drivers/mmc/host/sdhci-of-aspeed.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-of-aspeed.c b/drivers/mmc/host/sdhci-of-aspeed.c
> index ca97b01996b1..4296def69436 100644
> --- a/drivers/mmc/host/sdhci-of-aspeed.c
> +++ b/drivers/mmc/host/sdhci-of-aspeed.c
> @@ -519,7 +519,7 @@ static struct platform_driver aspeed_sdhci_driver = {
> static int aspeed_sdc_probe(struct platform_device *pdev)
>
> {
> - struct device_node *parent, *child;
> + struct device_node *parent;
> struct aspeed_sdc *sdc;
> int ret;
>
> @@ -548,12 +548,11 @@ static int aspeed_sdc_probe(struct platform_device *pdev)
> dev_set_drvdata(&pdev->dev, sdc);
>
> parent = pdev->dev.of_node;
> - for_each_available_child_of_node(parent, child) {
> + for_each_available_child_of_node_scoped(parent, child) {
> struct platform_device *cpdev;
>
> cpdev = of_platform_device_create(child, NULL, &pdev->dev);
> if (!cpdev) {
> - of_node_put(child);
> ret = -ENODEV;
> goto err_clk;
> }
> --
> 2.51.0
>
More information about the Linux-aspeed
mailing list