Quoting Ryder Lee (2018-03-06 01:09:27) > @@ -150,12 +151,15 @@ static int clk_mt7622_audiosys_init(struct platform_device *pdev) > clk_data); > > r = of_clk_add_provider(node, of_clk_src_onecell_get, clk_data); > - if (r) > + if (r) { > dev_err(&pdev->dev, > "could not register clock provider: %s: %d\n", > pdev->name, r); > > - return r; > + return r; > + } > + > + return devm_of_platform_populate(&pdev->dev); And now we fail to remove the clk provider when this fails. > } > > static const struct of_device_id of_match_clk_mt7622_aud[] = { > -- > 1.9.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html