quires lot of code refactoring.
> > > > Or Is there any way we follow 1) with less code refactoring.?
> > > > Or Do you think there is any other proper approach ?
> > > >
> > > > (Please note rev3 is not clean, d3cold off support need to be
> &
+Rafael
On Thu, Jul 21, 2022 at 03:29:48PM +0530, tilak.tang...@intel.com wrote:
> From: Tilak Tangudu
>
> Added is_intel_rpm_allowed function to query the runtime_pm
> status and disllow during suspending and resuming.
>
> v2: Return -2 if runtime pm is not allowed in runtime_pm_get
> and sk
>
> > > (Please note rev3 is not clean, d3cold off support need to be
> > > restricted to Headless clients for now , we see some Display related
> > > warnings in headed case ).
>
> I believe this warnings shows that the solution 2 has some flaws or corner
> c
Headless clients for now , we see some Display related warnings in
> > headed
> > case ).
I believe this warnings shows that the solution 2 has some flaws or
corner cases that we don't fully understand.
I honestly believe we need to go with option 1, moving the runtime_pm_
ires lot of code refactoring.
> Or Is there any way we follow 1) with less code refactoring.?
> Or Do you think there is any other proper approach ?
>
> (Please note rev3 is not clean, d3cold off support need to be restricted to
> Headless clients for now , we see some Dis
v3 is not clean, d3cold off support need to be restricted to
Headless clients for now , we see some Display related warnings in headed case
).
Thanks
Tilak
> -Original Message-----
> From: Intel-gfx On Behalf Of
> Tangudu, Tilak
> Sent: Thursday, August 4, 2022 11:03 AM
> To: Vivi,
On Thu, 2022-08-04 at 05:32 +, Tangudu, Tilak wrote:
>
>
> > -Original Message-
> > From: Vivi, Rodrigo
> > Sent: Thursday, August 4, 2022 2:01 AM
> > To: Tangudu, Tilak
> > Cc: Ewins, Jon ; Belgaumkar, Vinay
> > ; Roper, Matthew D
> > ; Wilson, Chris P
> > ;
> > Nikula, Jani ; Gupt
> -Original Message-
> From: Tangudu, Tilak
> Sent: Thursday, July 21, 2022 3:30 PM
> To: Ewins, Jon ; Belgaumkar, Vinay
> ; Roper, Matthew D
> ; Wilson, Chris P ;
> Nikula, Jani ; Gupta, saurabhg
> ; Vivi, Rodrigo ; Gupta,
> Anshuman ; Nilawar, Badal
> ; Tangudu, Tilak ; Deak,
> Imre ;
> -Original Message-
> From: Vivi, Rodrigo
> Sent: Thursday, August 4, 2022 2:01 AM
> To: Tangudu, Tilak
> Cc: Ewins, Jon ; Belgaumkar, Vinay
> ; Roper, Matthew D
> ; Wilson, Chris P ;
> Nikula, Jani ; Gupta, saurabhg
> ; Gupta, Anshuman
> ; Nilawar, Badal ;
> Deak, Imre ; Iddamsetty,
On Thu, Jul 21, 2022 at 03:29:48PM +0530, tilak.tang...@intel.com wrote:
> From: Tilak Tangudu
>
> Added is_intel_rpm_allowed function to query the runtime_pm
> status and disllow during suspending and resuming.
>
> v2: Return -2 if runtime pm is not allowed in runtime_pm_get
> and skip wakeref
Hi,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip]
url:
https://github.com/intel-lab-lkp/linux/commits/tilak-tangudu-intel-com/drm-i915-Add-D3Cold-Off-support-for-runtime-pm/20220721-174913
base: git://anongit.freedesktop.org/drm/drm-tip drm-t
From: Tilak Tangudu
Added is_intel_rpm_allowed function to query the runtime_pm
status and disllow during suspending and resuming.
v2: Return -2 if runtime pm is not allowed in runtime_pm_get
and skip wakeref release in runtime_pm_put if wakeref value
is -2. - Jani N
Signed-off-by: Tilak Tangudu
12 matches
Mail list logo
