On 29/01/2020 03:41, Dmitry Osipenko wrote: ... > The !RPM case isn't supported by Tegra anymore in upstream kernel. I'm > trying to help to make yours driver better and gave you reasons to > remove the unneeded code, while you're keep saying that "there is no > harm to retain it", which is not a reason to clutter up the code. I > don't feel that it's worthwhile to continue arguing here. On further review, it does look like a reasonable argument to get rid of this now completely. Like you see less clutter and the soc-core should ensure that the card is shutdown before unloading. OK, so fine with me. Jon -- nvpublic