Commit 0a7f54ea authored by Tomi Valkeinen's avatar Tomi Valkeinen

drm/omap: fix bus_flags for panel-dpi

panel-dpi used to convey the bus-flags via the videomode, but recent
changes changed the use of videomode to DRM's drm_display_mode which
does not contain bus-flags. This broke panel-dpi, which didn't
explicitly store the bus-flags into dssdev->bus_flags.

Fix this by setting dssdev->bus_flags. Also change the bus_flags type to
u32, as that is the type used in the DRM framework, and we would get a
warning with drm_bus_flags_from_videomode() otherwise.

Fixes: 3fbda31e ("drm/omap: Split mode fixup and mode set from encoder enable")
Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ti.com>
Reported-by: default avatarH. Nikolaus Schaller <hns@goldelico.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181126092447.11864-1-tomi.valkeinen@ti.comReviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
parent 4265b0fe
...@@ -177,6 +177,7 @@ static int panel_dpi_probe(struct platform_device *pdev) ...@@ -177,6 +177,7 @@ static int panel_dpi_probe(struct platform_device *pdev)
dssdev->type = OMAP_DISPLAY_TYPE_DPI; dssdev->type = OMAP_DISPLAY_TYPE_DPI;
dssdev->owner = THIS_MODULE; dssdev->owner = THIS_MODULE;
dssdev->of_ports = BIT(0); dssdev->of_ports = BIT(0);
drm_bus_flags_from_videomode(&ddata->vm, &dssdev->bus_flags);
omapdss_display_init(dssdev); omapdss_display_init(dssdev);
omapdss_device_register(dssdev); omapdss_device_register(dssdev);
......
...@@ -432,7 +432,7 @@ struct omap_dss_device { ...@@ -432,7 +432,7 @@ struct omap_dss_device {
const struct omap_dss_driver *driver; const struct omap_dss_driver *driver;
const struct omap_dss_device_ops *ops; const struct omap_dss_device_ops *ops;
unsigned long ops_flags; unsigned long ops_flags;
unsigned long bus_flags; u32 bus_flags;
/* helper variable for driver suspend/resume */ /* helper variable for driver suspend/resume */
bool activate_after_resume; bool activate_after_resume;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment