Hi Krzysztof, Am Mittwoch, 5. März 2025, 08:17:23 CET schrieb Krzysztof Kozlowski: > On Tue, Mar 04, 2025 at 04:49:22PM +0100, Alexander Stein wrote: > > +#include <linux/of_platform.h> > > #include <linux/platform_device.h> > > #include <linux/pm_domain.h> > > #include <linux/pm_runtime.h> > > @@ -297,8 +298,14 @@ static int imx93_blk_ctrl_probe(struct platform_device *pdev) > > > > dev_set_drvdata(dev, bc); > > > > + ret = devm_of_platform_populate(dev); > > This means in remove() you will depopulate in different order than error > path (e.g. after genpd removal). This is rather unexpected - remove() > should be cleaning up in exactly reversed order of probe. Not sure if it > can lead to any issues, but usual recommendation is that you either use > devm() or not. Thanks for pointing out. I will add a devm_of_platform_depopulate() to remove(). Best regards Alexander -- TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018 Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider http://www.tq-group.com/
