Re: [PATCH] drm/i915/dgfx: Disable d3cold Correctly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2022-06-06 at 17:56 +0530, Anshuman Gupta wrote:
> Currently i915 disables d3cold for i915 pci dev.
> This blocks D3 for i915 gfx pci upstream bridge (VSP).
> Let's disable d3cold at gfx root port to make sure that
> i915 gfx VSP can transition to D3 to save some power.

(nit) It's better to use imperative in the commit message, so "Disable
d3cold at gfx root port..." would be easier to read.

Also, from the code, it's clear that you're disabling d3cold in the
root now, instead of in the pdev itself.  But you don't explain why you
moved the enabling/disabling to hw_probe and remove_hw from the actual
PM calls...

--
Cheers,
Luca.




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux