Use for_each_child_of_node() macro instead of open coding it.

Signed-off-by: Qinglang Miao <miaoqingl...@huawei.com>
---
 drivers/mtd/devices/spear_smi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/devices/spear_smi.c b/drivers/mtd/devices/spear_smi.c
index 79dcca164..2e0086238 100644
--- a/drivers/mtd/devices/spear_smi.c
+++ b/drivers/mtd/devices/spear_smi.c
@@ -793,7 +793,7 @@ static int spear_smi_probe_config_dt(struct platform_device 
*pdev,
                                     struct device_node *np)
 {
        struct spear_smi_plat_data *pdata = dev_get_platdata(&pdev->dev);
-       struct device_node *pp = NULL;
+       struct device_node *pp;
        const __be32 *addr;
        u32 val;
        int len;
@@ -812,7 +812,7 @@ static int spear_smi_probe_config_dt(struct platform_device 
*pdev,
                return -ENOMEM;
 
        /* Fill structs for each subnode (flash device) */
-       while ((pp = of_get_next_child(np, pp))) {
+       for_each_child_of_node(np, pp) {
                pdata->np[i] = pp;
 
                /* Read base-addr and size from DT */
-- 
2.23.0

Reply via email to