diff options
author | Peter Meerwald <pmeerw@pmeerw.net> | 2015-01-30 02:59:46 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2015-02-04 05:41:53 -0500 |
commit | 811fbb1f580ca024a0af603dfaef08a4d1dcb5ef (patch) | |
tree | 0a24b979abbe6070764198a9380d533af9f6fbb2 /drivers/video | |
parent | a2408154a14b5633b1f233e4b5fea85c09917eef (diff) |
omapfb: Return error code when applying overlay settings fails
the check of the return code is missing, user space does not get notified
about the error condition:
omapdss OVERLAY error: overlay 2 horizontally not inside the display area (403 + 800 >= 800)
omapdss APPLY error: failed to apply settings: illegal configuration.
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c b/drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c index 146b6f5428db..9ddfdd63b84c 100644 --- a/drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c +++ b/drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c | |||
@@ -137,8 +137,11 @@ static int omapfb_setup_plane(struct fb_info *fbi, struct omapfb_plane_info *pi) | |||
137 | goto undo; | 137 | goto undo; |
138 | } | 138 | } |
139 | 139 | ||
140 | if (ovl->manager) | 140 | if (ovl->manager) { |
141 | ovl->manager->apply(ovl->manager); | 141 | r = ovl->manager->apply(ovl->manager); |
142 | if (r) | ||
143 | goto undo; | ||
144 | } | ||
142 | 145 | ||
143 | if (pi->enabled) { | 146 | if (pi->enabled) { |
144 | r = ovl->enable(ovl); | 147 | r = ovl->enable(ovl); |