mbox series

[RFC,0/2] suppress the wrong long hotplug events

Message ID 20220314225837.42816-1-vinod.govindapillai@intel.com (mailing list archive)
Headers show
Series suppress the wrong long hotplug events | expand

Message

Vinod Govindapillai March 14, 2022, 10:58 p.m. UTC
Monitors like LG 27UL650-W, 27UK850 goes into power sleep state
and generates long duration hotplug events even when the monitor
is connected for display. Here is a proposal to detect and
suppress such hotplug events by "sleep" for 2 secs for power
state monitor become available before enable atomic commit.
A debugfs entry is created to enable the suppression of the
hotplug event in such scenarios.

Cc: Imre Deak <imre.deak@intel.com>

Mohammed Khajapasha (2):
  drm/i915/display: Add disable wait time for power state connector
  drm/i915/display: Add sleep for power state connector

 .../gpu/drm/i915/display/intel_connector.c    |  3 +
 drivers/gpu/drm/i915/display/intel_display.c  | 80 +++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_display.h  |  8 ++
 .../drm/i915/display/intel_display_debugfs.c  | 58 ++++++++++++++
 .../drm/i915/display/intel_display_debugfs.h  |  7 ++
 .../drm/i915/display/intel_display_types.h    |  2 +
 drivers/gpu/drm/i915/i915_drv.h               |  2 +
 7 files changed, 160 insertions(+)

Comments

Jani Nikula March 16, 2022, 8:12 a.m. UTC | #1
On Tue, 15 Mar 2022, Vinod Govindapillai <vinod.govindapillai@intel.com> wrote:
> Monitors like LG 27UL650-W, 27UK850 goes into power sleep state
> and generates long duration hotplug events even when the monitor
> is connected for display. Here is a proposal to detect and
> suppress such hotplug events by "sleep" for 2 secs for power
> state monitor become available before enable atomic commit.

I don't understand the failure mode. Please elaborate. Do we have a bug
report?

BR,
Jani.

> A debugfs entry is created to enable the suppression of the
> hotplug event in such scenarios.
>
> Cc: Imre Deak <imre.deak@intel.com>
>
> Mohammed Khajapasha (2):
>   drm/i915/display: Add disable wait time for power state connector
>   drm/i915/display: Add sleep for power state connector
>
>  .../gpu/drm/i915/display/intel_connector.c    |  3 +
>  drivers/gpu/drm/i915/display/intel_display.c  | 80 +++++++++++++++++++
>  drivers/gpu/drm/i915/display/intel_display.h  |  8 ++
>  .../drm/i915/display/intel_display_debugfs.c  | 58 ++++++++++++++
>  .../drm/i915/display/intel_display_debugfs.h  |  7 ++
>  .../drm/i915/display/intel_display_types.h    |  2 +
>  drivers/gpu/drm/i915/i915_drv.h               |  2 +
>  7 files changed, 160 insertions(+)