Many Linux distributions would enable vesafb in order to display
early stage boot splash. In this case, we will get garbled X
Window screen if running X fbdev on psbfb.
This is because fb0 is occupied by vesafb while psbfb is on fb1.
They tried to drive the same pieces of hardware at the same
time. With unmodified X start-up, it would try to use default
fb0 framebuffer device and unfortunately it is now broken
becaues fb1 supersedes it.
We should let psbfb takeover framebuffer control from vesafb
to get around this problem.
See also commit : 4410f3910947dcea8672280b3adecd
53cec4e85e
Signed-off-by: Michael Chang <[email protected]>
---
?drivers/staging/gma500/psb_fb.c | ? ?8 ++++++++
?1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/gma500/psb_fb.c b/drivers/staging/gma500/psb_fb.c
index 99c03a2..259596c 100644
--- a/drivers/staging/gma500/psb_fb.c
+++ b/drivers/staging/gma500/psb_fb.c
@@ -441,6 +441,14 @@ static int psbfb_create(struct psb_fbdev *fbdev,
? ? ? ?info->screen_size = size;
? ? ? ?memset(info->screen_base, 0, size);
+ ? ? ? info->apertures = alloc_apertures(1);
+ ? ? ? if (!info->apertures) {
+ ? ? ? ? ? ? ? ret = -ENOMEM;
+ ? ? ? ? ? ? ? goto out_err0;
+ ? ? ? }
+ ? ? ? info->apertures->ranges[0].base = dev->mode_config.fb_base;
+ ? ? ? info->apertures->ranges[0].size = dev_priv->pg->stolen_size;
+
? ? ? ?drm_fb_helper_fill_fix(info, fb->pitch, fb->depth);
? ? ? ?drm_fb_helper_fill_var(info, &fbdev->psb_fb_helper,
? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?sizes->fb_width, sizes->fb_height);
--
1.7.3.4
> We should let psbfb takeover framebuffer control from vesafb
> to get around this problem.
Looks good to me assuming we have stolen memory. This maybe wants to
test
if (pg->stolen_size)
although I'm not sure that adding a zero size aperture would do harm
any way ? The devices without stolen ram also won't have vesa.
Alan