mirror of
https://github.com/torvalds/linux.git
synced 2026-07-27 17:47:41 +02:00
Merge tag 'drm-intel-fixes-2026-06-05' of https://gitlab.freedesktop.org/drm/i915/kernel into drm-fixes
- Fix color blob reference handling in intel_plane_state (Chaitanya Kumar Borah) - Revert "drm/i915/backlight: Remove try_vesa_interface" [backlight] (Suraj Kandpal) Signed-off-by: Dave Airlie <airlied@redhat.com> From: Tvrtko Ursulin <tursulin@igalia.com> Link: https://patch.msgid.link/aiKgmwz7VGOaFXIv@linux
This commit is contained in:
commit
8ff3adc8c3
|
|
@ -691,10 +691,9 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *connector)
|
|||
struct intel_dp *intel_dp = intel_attached_dp(connector);
|
||||
struct drm_device *dev = connector->base.dev;
|
||||
struct intel_panel *panel = &connector->panel;
|
||||
bool try_intel_interface = false;
|
||||
bool try_intel_interface = false, try_vesa_interface = false;
|
||||
|
||||
/*
|
||||
* Check the VBT and user's module parameters to figure out which
|
||||
/* Check the VBT and user's module parameters to figure out which
|
||||
* interfaces to probe
|
||||
*/
|
||||
switch (display->params.enable_dpcd_backlight) {
|
||||
|
|
@ -703,6 +702,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *connector)
|
|||
case INTEL_DP_AUX_BACKLIGHT_AUTO:
|
||||
switch (panel->vbt.backlight.type) {
|
||||
case INTEL_BACKLIGHT_VESA_EDP_AUX_INTERFACE:
|
||||
try_vesa_interface = true;
|
||||
break;
|
||||
case INTEL_BACKLIGHT_DISPLAY_DDI:
|
||||
try_intel_interface = true;
|
||||
|
|
@ -715,12 +715,20 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *connector)
|
|||
if (panel->vbt.backlight.type != INTEL_BACKLIGHT_VESA_EDP_AUX_INTERFACE)
|
||||
try_intel_interface = true;
|
||||
|
||||
try_vesa_interface = true;
|
||||
break;
|
||||
case INTEL_DP_AUX_BACKLIGHT_FORCE_VESA:
|
||||
try_vesa_interface = true;
|
||||
break;
|
||||
case INTEL_DP_AUX_BACKLIGHT_FORCE_INTEL:
|
||||
try_intel_interface = true;
|
||||
break;
|
||||
}
|
||||
|
||||
/* For eDP 1.5 and above we are supposed to use VESA interface for brightness control */
|
||||
if (intel_dp->edp_dpcd[0] >= DP_EDP_15)
|
||||
try_vesa_interface = true;
|
||||
|
||||
/*
|
||||
* Since Intel has their own backlight control interface, the majority of machines out there
|
||||
* using DPCD backlight controls with Intel GPUs will be using this interface as opposed to
|
||||
|
|
@ -733,9 +741,6 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *connector)
|
|||
* panel with Intel's OUI - which is also required for us to be able to detect Intel's
|
||||
* backlight interface at all. This means that the only sensible way for us to detect both
|
||||
* interfaces is to probe for Intel's first, and VESA's second.
|
||||
*
|
||||
* Also there is a chance some VBTs may advertise false Intel backlight support even if the
|
||||
* TCON DPCD says otherwise. This means we keep VESA interface as fallback in that case.
|
||||
*/
|
||||
if (try_intel_interface && intel_dp->edp_dpcd[0] <= DP_EDP_14b &&
|
||||
intel_dp_aux_supports_hdr_backlight(connector)) {
|
||||
|
|
@ -745,7 +750,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *connector)
|
|||
return 0;
|
||||
}
|
||||
|
||||
if (intel_dp_aux_supports_vesa_backlight(connector)) {
|
||||
if (try_vesa_interface && intel_dp_aux_supports_vesa_backlight(connector)) {
|
||||
drm_dbg_kms(dev, "[CONNECTOR:%d:%s] Using VESA eDP backlight controls\n",
|
||||
connector->base.base.id, connector->base.name);
|
||||
panel->backlight.funcs = &intel_dp_vesa_bl_funcs;
|
||||
|
|
|
|||
|
|
@ -144,6 +144,15 @@ intel_plane_duplicate_state(struct drm_plane *plane)
|
|||
if (intel_state->hw.fb)
|
||||
drm_framebuffer_get(intel_state->hw.fb);
|
||||
|
||||
if (intel_state->hw.degamma_lut)
|
||||
drm_property_blob_get(intel_state->hw.degamma_lut);
|
||||
if (intel_state->hw.gamma_lut)
|
||||
drm_property_blob_get(intel_state->hw.gamma_lut);
|
||||
if (intel_state->hw.ctm)
|
||||
drm_property_blob_get(intel_state->hw.ctm);
|
||||
if (intel_state->hw.lut_3d)
|
||||
drm_property_blob_get(intel_state->hw.lut_3d);
|
||||
|
||||
return &intel_state->uapi;
|
||||
}
|
||||
|
||||
|
|
@ -167,6 +176,16 @@ intel_plane_destroy_state(struct drm_plane *plane,
|
|||
__drm_atomic_helper_plane_destroy_state(&plane_state->uapi);
|
||||
if (plane_state->hw.fb)
|
||||
drm_framebuffer_put(plane_state->hw.fb);
|
||||
|
||||
if (plane_state->hw.degamma_lut)
|
||||
drm_property_blob_put(plane_state->hw.degamma_lut);
|
||||
if (plane_state->hw.gamma_lut)
|
||||
drm_property_blob_put(plane_state->hw.gamma_lut);
|
||||
if (plane_state->hw.ctm)
|
||||
drm_property_blob_put(plane_state->hw.ctm);
|
||||
if (plane_state->hw.lut_3d)
|
||||
drm_property_blob_put(plane_state->hw.lut_3d);
|
||||
|
||||
kfree(plane_state);
|
||||
}
|
||||
|
||||
|
|
@ -317,6 +336,14 @@ static void intel_plane_clear_hw_state(struct intel_plane_state *plane_state)
|
|||
{
|
||||
if (plane_state->hw.fb)
|
||||
drm_framebuffer_put(plane_state->hw.fb);
|
||||
if (plane_state->hw.degamma_lut)
|
||||
drm_property_blob_put(plane_state->hw.degamma_lut);
|
||||
if (plane_state->hw.gamma_lut)
|
||||
drm_property_blob_put(plane_state->hw.gamma_lut);
|
||||
if (plane_state->hw.ctm)
|
||||
drm_property_blob_put(plane_state->hw.ctm);
|
||||
if (plane_state->hw.lut_3d)
|
||||
drm_property_blob_put(plane_state->hw.lut_3d);
|
||||
|
||||
memset(&plane_state->hw, 0, sizeof(plane_state->hw));
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user