summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>2023-12-04 16:13:48 +0300
committerDmitry Baryshkov <dmitry.baryshkov@linaro.org>2023-12-04 22:33:05 +0300
commit1c0a80f160965c88f16e73ff69015db2f044c486 (patch)
tree535c48f7b9736f1e8d460078f4d53ece2e1cd24b /include
parentd0b3c318e04cc6c4e2a3c30ee0f6f619aa8d0db5 (diff)
downloadlinux-1c0a80f160965c88f16e73ff69015db2f044c486.tar.xz
Revert "drm/atomic: Loosen FB atomic checks"
This reverts commit f1e75da5364e780905d9cd6043f9c74cdcf84073. Although the Solid Fill planes patchset got all reviews and acknowledgements, it doesn't fulfill requirements for the new uABI. It has neither corresponding open-source userspace implementation nor the IGT tests coverage. Reverting this patchset until userspace obligations are fulfilled. Acked-by: Simon Ser <contact@emersion.fr> Acked-by: Maxime Ripard <mripard@kernel.org> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Link: https://patchwork.freedesktop.org/patch/msgid/20231204131455.19023-2-dmitry.baryshkov@linaro.org
Diffstat (limited to 'include')
-rw-r--r--include/drm/drm_atomic_helper.h4
-rw-r--r--include/drm/drm_plane.h29
2 files changed, 2 insertions, 31 deletions
diff --git a/include/drm/drm_atomic_helper.h b/include/drm/drm_atomic_helper.h
index 6d97f38ac1f6..536a0b0091c3 100644
--- a/include/drm/drm_atomic_helper.h
+++ b/include/drm/drm_atomic_helper.h
@@ -256,8 +256,8 @@ drm_atomic_plane_disabling(struct drm_plane_state *old_plane_state,
* Anything else should be considered a bug in the atomic core, so we
* gently warn about it.
*/
- WARN_ON((new_plane_state->crtc == NULL && drm_plane_has_visible_data(new_plane_state)) ||
- (new_plane_state->crtc != NULL && !drm_plane_has_visible_data(new_plane_state)));
+ WARN_ON((new_plane_state->crtc == NULL && new_plane_state->fb != NULL) ||
+ (new_plane_state->crtc != NULL && new_plane_state->fb == NULL));
return old_plane_state->crtc && !new_plane_state->crtc;
}
diff --git a/include/drm/drm_plane.h b/include/drm/drm_plane.h
index 3b187f3f5466..d14e2f1db234 100644
--- a/include/drm/drm_plane.h
+++ b/include/drm/drm_plane.h
@@ -1016,35 +1016,6 @@ static inline struct drm_plane *drm_plane_find(struct drm_device *dev,
#define drm_for_each_plane(plane, dev) \
list_for_each_entry(plane, &(dev)->mode_config.plane_list, head)
-/**
- * drm_plane_solid_fill_enabled - Check if solid fill is enabled on plane
- * @state: plane state
- *
- * Returns:
- * Whether the plane has been assigned a solid_fill_blob
- */
-static inline bool drm_plane_solid_fill_enabled(struct drm_plane_state *state)
-{
- if (!state)
- return false;
- return state->pixel_source == DRM_PLANE_PIXEL_SOURCE_SOLID_FILL && state->solid_fill_blob;
-}
-
-static inline bool drm_plane_has_visible_data(const struct drm_plane_state *state)
-{
- switch (state->pixel_source) {
- case DRM_PLANE_PIXEL_SOURCE_NONE:
- return false;
- case DRM_PLANE_PIXEL_SOURCE_SOLID_FILL:
- return state->solid_fill_blob != NULL;
- case DRM_PLANE_PIXEL_SOURCE_FB:
- default:
- WARN_ON(state->pixel_source != DRM_PLANE_PIXEL_SOURCE_FB);
- }
-
- return state->fb != NULL;
-}
-
bool drm_any_plane_has_format(struct drm_device *dev,
u32 format, u64 modifier);