Commit d1f5a653d2a53d4bf5285ff77284cd8cbf538af9
1 parent
bc089c2394
Exists in
smarc_8mq-imx_v2020.04_5.4.24_2.1.0
and in
1 other branch
MLK-24062-1 misc: scu: Remove child dev bind
Remove the child device binding codes for clk and iomux nodes, because the misc uclass has added such operation in post-bind. If we keep the codes in scu bind, child devices will be created twices. Find the problem by "dm uclass" command. uclass 24: clk 0 * clk @ fce22380, seq 0, (req -1) 1 clk @ fce23110 uclass 67: pinctrl 0 * iomuxc @ fce22470, seq 0, (req -1) 1 iomuxc @ fce231c0 After the fix: uclass 24: clk 0 * clk @ fce22380, seq 0, (req -1) uclass 67: pinctrl 0 * iomuxc @ fce22470, seq 0, (req -1) Signed-off-by: Ye Li <ye.li@nxp.com> Reviewed-by: Peng Fan <peng.fan@nxp.com> (cherry picked from commit 927674dd70cda8512f6bbc47a79babd9a3ec4081)
Showing 1 changed file with 0 additions and 12 deletions Side-by-side Diff
drivers/misc/imx8/scu.c
| ... | ... | @@ -210,18 +210,6 @@ |
| 210 | 210 | |
| 211 | 211 | static int imx8_scu_bind(struct udevice *dev) |
| 212 | 212 | { |
| 213 | - int ret; | |
| 214 | - struct udevice *child; | |
| 215 | - ofnode node; | |
| 216 | - | |
| 217 | - debug("%s(dev=%p)\n", __func__, dev); | |
| 218 | - ofnode_for_each_subnode(node, dev_ofnode(dev)) { | |
| 219 | - ret = lists_bind_fdt(dev, node, &child, true); | |
| 220 | - if (ret) | |
| 221 | - return ret; | |
| 222 | - debug("bind child dev %s\n", child->name); | |
| 223 | - } | |
| 224 | - | |
| 225 | 213 | return 0; |
| 226 | 214 | } |
| 227 | 215 |