14.02.2020 17:05, Jon Hunter пишет: > > 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. And the RPM core ensures that state is synced on disable, sorry if I didn't make it clear before. +1 for less clutter :)