staging/olpc: lock_fb_info can't fail
Simply because olpc never unregisters the damn thing. It also registers the framebuffer directly by poking around in fbdev core internals, so it's all around rather broken. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by: Sam Ravnborg <sam@ravnborg.org> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Jens Frederich <jfrederich@gmail.com> Cc: Daniel Drake <dsd@laptop.org> Cc: Jon Nettleton <jon.nettleton@gmail.com> Link: https://patchwork.freedesktop.org/patch/msgid/20190528090304.9388-15-daniel.vetter@ffwll.ch
This commit is contained in:
parent
34f31f14c4
commit
9ef3739237
|
@ -250,11 +250,7 @@ static bool dcon_blank_fb(struct dcon_priv *dcon, bool blank)
|
|||
int err;
|
||||
|
||||
console_lock();
|
||||
if (!lock_fb_info(dcon->fbinfo)) {
|
||||
console_unlock();
|
||||
dev_err(&dcon->client->dev, "unable to lock framebuffer\n");
|
||||
return false;
|
||||
}
|
||||
lock_fb_info(dcon->fbinfo);
|
||||
|
||||
dcon->ignore_fb_events = true;
|
||||
err = fb_blank(dcon->fbinfo,
|
||||
|
|
Loading…
Reference in New Issue