Skip to content

Commit 1dbc790

Browse files
Liu Yingsuperna9999
authored andcommitted
drm/bridge: fsl-ldb: Drop DE signal polarity inversion
It's unnecessary to invert input data enable signal polarity according to the output one. Let's drop the inversion. Since ->atomic_check() does nothing more than the inversion, it can be dropped entirely as well. Without this patch, 'koe,tx26d202vm0bwa' LVDS panel connected with i.MX8MP EVK board does not show any data on screen. Fixes: 463db5c ("drm: bridge: ldb: Implement simple Freescale i.MX8MP LDB bridge") Cc: Andrzej Hajda <[email protected]> Cc: Neil Armstrong <[email protected]> Cc: Robert Foss <[email protected]> Cc: Laurent Pinchart <[email protected]> Cc: Jonas Karlman <[email protected]> Cc: Jernej Skrabec <[email protected]> Cc: David Airlie <[email protected]> Cc: Daniel Vetter <[email protected]> Cc: Sam Ravnborg <[email protected]> Cc: Marek Vasut <[email protected]> Cc: NXP Linux Team <[email protected]> Signed-off-by: Liu Ying <[email protected]> Reviewed-by: Marek Vasut <[email protected]> Signed-off-by: Neil Armstrong <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent 57ef278 commit 1dbc790

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

drivers/gpu/drm/bridge/fsl-ldb.c

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -74,22 +74,6 @@ static int fsl_ldb_attach(struct drm_bridge *bridge,
7474
bridge, flags);
7575
}
7676

77-
static int fsl_ldb_atomic_check(struct drm_bridge *bridge,
78-
struct drm_bridge_state *bridge_state,
79-
struct drm_crtc_state *crtc_state,
80-
struct drm_connector_state *conn_state)
81-
{
82-
/* Invert DE signal polarity. */
83-
bridge_state->input_bus_cfg.flags &= ~(DRM_BUS_FLAG_DE_LOW |
84-
DRM_BUS_FLAG_DE_HIGH);
85-
if (bridge_state->output_bus_cfg.flags & DRM_BUS_FLAG_DE_LOW)
86-
bridge_state->input_bus_cfg.flags |= DRM_BUS_FLAG_DE_HIGH;
87-
else if (bridge_state->output_bus_cfg.flags & DRM_BUS_FLAG_DE_HIGH)
88-
bridge_state->input_bus_cfg.flags |= DRM_BUS_FLAG_DE_LOW;
89-
90-
return 0;
91-
}
92-
9377
static void fsl_ldb_atomic_enable(struct drm_bridge *bridge,
9478
struct drm_bridge_state *old_bridge_state)
9579
{
@@ -241,7 +225,6 @@ fsl_ldb_mode_valid(struct drm_bridge *bridge,
241225

242226
static const struct drm_bridge_funcs funcs = {
243227
.attach = fsl_ldb_attach,
244-
.atomic_check = fsl_ldb_atomic_check,
245228
.atomic_enable = fsl_ldb_atomic_enable,
246229
.atomic_disable = fsl_ldb_atomic_disable,
247230
.atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state,

0 commit comments

Comments
 (0)