drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx
authorLyude Paul <lyude@redhat.com>
Mon, 13 Jul 2020 17:07:45 +0000 (13:07 -0400)
committerLyude Paul <lyude@redhat.com>
Mon, 13 Jul 2020 17:29:20 +0000 (13:29 -0400)
commit1c26b8e09004477c070f0dccb2369438ef9e1fc9
treee56da702ee561b6d017470e59381590875fea24f
parent1d9221e9d395c8c80d99d002bea3c9b6dd192d6a
drm/probe_helper: Add drm_connector_helper_funcs.mode_valid_ctx

This is just an atomic version of mode_valid, which is intended to be
used for situations where a driver might need to check the atomic state
of objects other than the connector itself. One such example is with
MST, where the maximum possible bandwidth on a connector can change
dynamically irregardless of the display configuration.

Changes since v1:
* Use new drm logging functions
* Make some corrections in the mode_valid_ctx kdoc
* Return error codes or 0 from ->mode_valid_ctx() on fail, and store the
  drm_mode_status in an additional function parameter
Changes since v2:
* Don't accidentally assign ret to mode->status on success, or we'll
  squash legitimate mode validation results
* Don't forget to assign MODE_OK to status in drm_connector_mode_valid()
  if we have no callbacks
* Drop leftover hunk in drm_modes.h around enum drm_mode_status
Changes since v3:
* s/return ret/return 0/ in drm_mode_validate_pipeline()
* Minor cleanup in drm_connector_mode_valid()

Tested-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Cc: Lee Shawn C <shawn.c.lee@intel.com>
Signed-off-by: Lyude Paul <lyude@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200713170746.254388-2-lyude@redhat.com
drivers/gpu/drm/drm_crtc_helper_internal.h
drivers/gpu/drm/drm_probe_helper.c
include/drm/drm_modeset_helper_vtables.h