09.04.2020 20:52, Thierry Reding пишет: ... > +static int tegra210_emc_remove(struct platform_device *pdev) > +{ > + struct tegra210_emc *emc = platform_get_drvdata(pdev); > + > + debugfs_remove_recursive(emc->debugfs.root); > + tegra210_clk_emc_detach(emc->clk); It's great that you want to make this driver modular, but I don't think that it's a good idea to suddenly rip off the EMC clock from the users. It should be better to simply disallow unloading of this driver once it's loaded.