diff options
author | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-10-03 15:14:09 +0000 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2015-02-26 12:15:13 +0200 |
commit | 4e1d3ca0836b6b44c358c5ace79aa15a091a4142 (patch) | |
tree | a71c7575f7227d9c9cf01db2cd3f3e54f3fa032b /drivers/video/fbdev | |
parent | 386f167c69ca1c6920581f2cb78be8a1f6691539 (diff) | |
download | op-kernel-dev-4e1d3ca0836b6b44c358c5ace79aa15a091a4142.zip op-kernel-dev-4e1d3ca0836b6b44c358c5ace79aa15a091a4142.tar.gz |
OMAPDSS: DISPC: fix div by zero issue in overlay scaling
omapdrm doesn't always configure the overlays correctly, causing the
overlay setup functions to be called with zero timings. This leads to
division by zero error.
This happens, for example, when a HDMI cable is not connected, but a
user tries to setup a plane with scaling.
Fixing omapdrm is a big job, so for now let's check for the bad timings
in DISPC and return an error.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video/fbdev')
-rw-r--r-- | drivers/video/fbdev/omap2/dss/dispc.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/video/fbdev/omap2/dss/dispc.c b/drivers/video/fbdev/omap2/dss/dispc.c index 8805266..1123111d 100644 --- a/drivers/video/fbdev/omap2/dss/dispc.c +++ b/drivers/video/fbdev/omap2/dss/dispc.c @@ -2322,6 +2322,11 @@ static int dispc_ovl_calc_scaling(unsigned long pclk, unsigned long lclk, if (width == out_width && height == out_height) return 0; + if (pclk == 0 || mgr_timings->pixelclock == 0) { + DSSERR("cannot calculate scaling settings: pclk is zero\n"); + return -EINVAL; + } + if ((caps & OMAP_DSS_OVL_CAP_SCALE) == 0) return -EINVAL; |