Hi Krzysztof, On Thu, 20 Aug 2026 at 22:50, Krzysztof Kozlowski <[email protected]> wrote: > > On 08/08/2026 16:22, Guangshuo Li wrote: > > gr3d_probe() calls pm_runtime_use_autosuspend(), but gr3d_remove() > > does not call the matching pm_runtime_dont_use_autosuspend() during > > teardown. > > > > If the autosuspend delay is set to a negative value while autosuspend > > is enabled, the runtime PM core increments usage_count to prevent > > runtime suspend. Without calling pm_runtime_dont_use_autosuspend() > > during teardown, this reference is not dropped. > > > > The documentation for pm_runtime_use_autosuspend() also notes that it > > is important to undo it with pm_runtime_dont_use_autosuspend() at > > driver exit time, unless runtime PM was initially enabled with > > devm_pm_runtime_enable(). > > > > Add the missing pm_runtime_dont_use_autosuspend() call to the driver > > remove path. > > > > This issue was found by manual code inspection. > > > > Fixes: 62fa0a985e2c ("drm/tegra: Enable runtime PM during probe") > > Cc: [email protected] > > Signed-off-by: Guangshuo Li <[email protected]> > > You sent vast amount of patches, all separate, making it very difficult > to track and respond in efficient way. Do not do that. > > Group your work per subsystem. > > You were asked to clarify and respond to incorrect fixes statement. I do > not see how you clarified and responded at all. > > Best regards, > Krzysztof
Sorry about that. I should have replied to the earlier review comments explicitly, and I also should not have sent so many separate patches. I will group future patches by subsystem and make sure to respond clearly to review feedback before resending. Thanks for pointing this out. Best regards, Guangshuo
