struct fb_info::screen_size is in units of bytes, but we allocated 8
times as much so far, because of bits vs bytes confusion.

Signed-off-by: Ahmad Fatoum <[email protected]>
---
 drivers/video/ramfb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/ramfb.c b/drivers/video/ramfb.c
index 45b75e890c59..5b03d8a9c821 100644
--- a/drivers/video/ramfb.c
+++ b/drivers/video/ramfb.c
@@ -86,7 +86,7 @@ static int ramfb_activate_var(struct fb_info *fbi)
                dma_free_coherent(DMA_DEVICE_BROKEN,
                                  fbi->screen_base, ramfb->screen_dma, 
fbi->screen_size);
 
-       fbi->screen_size = fbi->xres * fbi->yres * fbi->bits_per_pixel;
+       fbi->screen_size = fbi->xres * fbi->yres * fbi->bits_per_pixel / 
BITS_PER_BYTE;
        fbi->screen_base = dma_alloc_coherent(DMA_DEVICE_BROKEN,
                                              fbi->screen_size, 
&ramfb->screen_dma);
 
-- 
2.39.5


Reply via email to