diff options
| author | Maarten Lankhorst <[email protected]> | 2018-09-20 12:27:06 +0200 |
|---|---|---|
| committer | Maarten Lankhorst <[email protected]> | 2018-09-21 19:15:10 +0200 |
| commit | 62ef0dd3cc0c217c3d58a1e92577c66c08598f7b (patch) | |
| tree | 1ce6775941825367b2f1353d21f6677c12d91aab /tools/perf/scripts/python | |
| parent | c249c5f64336ca7f8b314c16dbf440b091514f50 (diff) | |
drm/i915: Unconditionally clear plane visibility, v2.
We need to assume the plane has been visible before, even if no CRTC
is assigned to the plane. This is because when enabling a nv12 plane
on gen11, we will have to enable an extra plane and make it visible
by marking it in crtc_state->active_planes for
intel_update_planes_on_crtc().
Additionally, clear visible flag in intel_plane_atomic_check, in case
we ever hit a bug with visibility. Our code implicitly assumes that
plane_state->visible is only true when crtc and fb are set,
so we will either null deref in intel_fbc_choose_crtc() or
do something bad during the actual commit which cares even more.
Changes since v1:
- Unconditionally clear crtc_state->active_planes as well.
- Reword commit message, since this is now a preparation patch for
NV12 Y / UV plane linking.
Signed-off-by: Maarten Lankhorst <[email protected]>
[mlankhorst: Clear nv12_planes in the beginning as well, clarify commit message]
Reviewed-by: Matt Roper <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions