author | Dandawate Saket <dsaket@ti.com> | |
Thu, 13 Sep 2012 01:12:05 +0000 (18:12 -0700) | ||
committer | Praneeth Bajjuri <praneeth@ti.com> | |
Fri, 12 Jul 2013 22:28:58 +0000 (17:28 -0500) | ||
commit | cf34b7825b94cff8bf0ae5a2c07e54c02651cc59 | |
tree | 97bb355a9796d5f1b416f11f8c265e686258ecaa | tree | snapshot (tar.xz tar.gz zip) |
parent | b01c08d14f30cace79650e02cbb3319b6ed29158 | commit | diff |
OMAPDSS: OMAPFB: Do not suspend HDMI driver in early suspend.
Issue:
HDMI Audio can be active even if display activity
times out.
Analysis:
To save power approach to suspend panel on an
early suspend call was taken. This call of panel
suspend is done by OMAPLFB on early suspend
callback.
There is no issue in case of LCD since it doesnt
carry any audio. Also we only suspend the default
panel and not all the connected panels.
If hdmi is default display we cut HDMI clocks via
omapfb->blank call with NORMAL enum. HDMI audio
driver will close DMA only after the timeout
in userspace.
So we are voilating the expectation of HDMI audio
driver in case HDMI is default panel.
In case HDMI is default there is almost no use case
with battery power devices.
With this fix we can have an HDMI audio playback
happening when display inactivity times out
triggering early suspend.
Change-Id: I2180a20b1579bec8a3f910a5e349fb219437089a
Signed-off-by: Dandawate Saket <dsaket@ti.com>
Issue:
HDMI Audio can be active even if display activity
times out.
Analysis:
To save power approach to suspend panel on an
early suspend call was taken. This call of panel
suspend is done by OMAPLFB on early suspend
callback.
There is no issue in case of LCD since it doesnt
carry any audio. Also we only suspend the default
panel and not all the connected panels.
If hdmi is default display we cut HDMI clocks via
omapfb->blank call with NORMAL enum. HDMI audio
driver will close DMA only after the timeout
in userspace.
So we are voilating the expectation of HDMI audio
driver in case HDMI is default panel.
In case HDMI is default there is almost no use case
with battery power devices.
With this fix we can have an HDMI audio playback
happening when display inactivity times out
triggering early suspend.
Change-Id: I2180a20b1579bec8a3f910a5e349fb219437089a
Signed-off-by: Dandawate Saket <dsaket@ti.com>
drivers/video/omap2/omapfb/omapfb-main.c | diff | blob | history |