diff options
author | Sergio Aguirre <saaguirre@ti.com> | 2009-10-05 20:31:46 (GMT) |
---|---|---|
committer | Tony Lindgren <tony@atomide.com> | 2009-10-06 15:31:50 (GMT) |
commit | 03bb2b493cf58edf11e5966f8469534259d2fffe (patch) | |
tree | 6141e8f6e990e9cf917ceffab2948a1fa4985078 /drivers/firmware/edd.c | |
parent | 265489003c463f3d78d622fba60d56d16b3009dd (diff) | |
download | linux-fsl-qoriq-03bb2b493cf58edf11e5966f8469534259d2fffe.tar.xz |
omapfb: Condition mutex acquisition
This fixes a bug introduced by this commit ID:
commit 537a1bf059fa312355696fa6db80726e655e7f17
Author: Krzysztof Helt <krzysztof.h1@wp.pl>
Date: Tue Jun 30 11:41:29 2009 -0700
fbdev: add mutex for fb_mmap locking
In which a mutex was added when changing smem_start and smem_len fields,
so the mutex inside the fb_mmap() call is actually used.
The problem was that set_fb_fix, which modifies the above 2 fields,
was called before and after registering the framebuffer,
which when used before registration, lead to a failed attempt to
use an uninitialized mutex.
Solution: Don't use mutex before framebuffer registration.
Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
Acked-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
Acked-by: Imre Deak <imre.deak@nokia.com>
Tested-by: Cory Maccarrone <darkstar6262@gmail.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Diffstat (limited to 'drivers/firmware/edd.c')
0 files changed, 0 insertions, 0 deletions