diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2024-05-23 21:28:17 +0300 |
|---|---|---|
| committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2024-05-27 14:30:58 +0300 |
| commit | 325257f2e9cb753a1581f8ca50e84012010d2718 (patch) | |
| tree | bc05f4e8a9d781db2b4bb027e1a0e9ee742e0171 /drivers/gpu/drm/i915/display/intel_display.c | |
| parent | 5f128aa1052e7d28074ac9b5279d0a09232b699a (diff) | |
| download | linux-325257f2e9cb753a1581f8ca50e84012010d2718.tar.gz linux-325257f2e9cb753a1581f8ca50e84012010d2718.tar.bz2 linux-325257f2e9cb753a1581f8ca50e84012010d2718.zip | |
drm/i915: Hide the intel_crtc_needs_color_update() inside intel_color_check()
Move the intel_crtc_needs_color_update() into intel_color_check()
so that the caller doesn't have to care about this. This will
also enable us to hide the c8_planes_changed() thing better.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240523182818.15382-3-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/display/intel_display.c')
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_display.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 61c073fc0a46..d035a12cbeb0 100644 --- a/drivers/gpu/drm/i915/display/intel_display.c +++ b/drivers/gpu/drm/i915/display/intel_display.c @@ -4254,11 +4254,9 @@ static int intel_crtc_atomic_check(struct intel_atomic_state *state, if (c8_planes_changed(crtc_state)) crtc_state->uapi.color_mgmt_changed = true; - if (intel_crtc_needs_color_update(crtc_state)) { - ret = intel_color_check(state, crtc); - if (ret) - return ret; - } + ret = intel_color_check(state, crtc); + if (ret) + return ret; ret = intel_compute_pipe_wm(state, crtc); if (ret) { |