Commit c8320789 authored by Tomi Valkeinen's avatar Tomi Valkeinen

drm/omap: dsi: remove ulps support

ULPS is a niche power-saving feature which only really affects command
mode panels showing a static picture. I know the ULPS code used to work
very long time ago, but I could not get it working with the current
driver. As the ULPS code is not trivial and includes delayed work (so
lots of chances for race issues), and just keeping DSI video and command
mode panels working has been challenging enough even without ULPS, lets
remove ULPS support.

When the DSI driver works reliably for command and video mode displays,
someone interested can work on ULPS and add it back if the power saving
is substantial enough.
Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201215104657.802264-83-tomi.valkeinen@ti.com
parent 9a521118
This diff is collapsed.
...@@ -394,12 +394,8 @@ struct dsi_data { ...@@ -394,12 +394,8 @@ struct dsi_data {
atomic_t do_ext_te_update; atomic_t do_ext_te_update;
bool te_enabled; bool te_enabled;
bool ulps_enabled;
bool ulps_auto_idle;
bool video_enabled; bool video_enabled;
struct delayed_work ulps_work;
struct delayed_work framedone_timeout_work; struct delayed_work framedone_timeout_work;
#ifdef DSI_CATCH_MISSING_TE #ifdef DSI_CATCH_MISSING_TE
......
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