Force a modeset if the new FB has a different format than the
currently active one. While it might be possible to change between
compatible formats without a full modeset as the format control is
also supposed to be double buffered, the colorspace conversion is
not, so when the CSC changes we need a modeset.
For now just always force a modeset when the FB format changes to
properly reconfigure all parts of the device for the new format.
Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Marek Vasut <ma...@denx.de>
---
v3: no changes
v2: fix indentation
---
drivers/gpu/drm/mxsfb/lcdif_drv.c | 18 +++++++++++++++++-
drivers/gpu/drm/mxsfb/lcdif_kms.c | 26 ++++++++++++++++++++------
2 files changed, 37 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/mxsfb/lcdif_drv.c
b/drivers/gpu/drm/mxsfb/lcdif_drv.c
index 9df278adf3e8..667d3a4716b0 100644
--- a/drivers/gpu/drm/mxsfb/lcdif_drv.c
+++ b/drivers/gpu/drm/mxsfb/lcdif_drv.c
@@ -30,9 +30,25 @@
#include "lcdif_drv.h"
#include "lcdif_regs.h"
+static int lcdif_atomic_check(struct drm_device *dev,
+ struct drm_atomic_state *state)
+{
+ int ret;
+
+ ret = drm_atomic_helper_check(dev, state);
+ if (ret)
+ return ret;
+
+ /*
+ * Check modeset again in case crtc_state->mode_changed is
+ * updated in plane's ->atomic_check callback.
+ */
+ return drm_atomic_helper_check_modeset(dev, state);
+}
+
static const struct drm_mode_config_funcs lcdif_mode_config_funcs = {
.fb_create = drm_gem_fb_create,
- .atomic_check = drm_atomic_helper_check,
+ .atomic_check = lcdif_atomic_check,
.atomic_commit = drm_atomic_helper_commit,
};
diff --git a/drivers/gpu/drm/mxsfb/lcdif_kms.c
b/drivers/gpu/drm/mxsfb/lcdif_kms.c
index c3e31fa0335e..f98b9d8e3aa4 100644
--- a/drivers/gpu/drm/mxsfb/lcdif_kms.c
+++ b/drivers/gpu/drm/mxsfb/lcdif_kms.c
@@ -649,18 +649,32 @@ static const struct drm_crtc_funcs lcdif_crtc_funcs = {
static int lcdif_plane_atomic_check(struct drm_plane *plane,
struct drm_atomic_state *state)
{
- struct drm_plane_state *plane_state =
drm_atomic_get_new_plane_state(state,
-
plane);
+ struct drm_plane_state *new_state =
drm_atomic_get_new_plane_state(state,
+
plane);
+ struct drm_plane_state *old_state =
drm_atomic_get_old_plane_state(state,
+
plane);
struct lcdif_drm_private *lcdif = to_lcdif_drm_private(plane->dev);
struct drm_crtc_state *crtc_state;
+ int ret;
+
+ /* always okay to disable the plane */
+ if (!new_state->fb)
+ return 0;
crtc_state = drm_atomic_get_new_crtc_state(state,
&lcdif->crtc);
- return drm_atomic_helper_check_plane_state(plane_state, crtc_state,
- DRM_PLANE_NO_SCALING,
- DRM_PLANE_NO_SCALING,
- false, true);
+ ret = drm_atomic_helper_check_plane_state(new_state, crtc_state,
+ DRM_PLANE_NO_SCALING,
+ DRM_PLANE_NO_SCALING,
+ false, true);
+ if (ret)
+ return ret;
+
+ if (old_state->fb && new_state->fb->format != old_state->fb->format)
+ crtc_state->mode_changed = true;
+
+ return 0;
}
static void lcdif_plane_primary_atomic_update(struct drm_plane *plane,
--
2.39.2
