Hi Geert,
Fix panning on ARAnyM (Falcon emulation). Without this, `fbtest test011' fails
(for a 320x200x8 display with virtual 640x400).
Fails how, exactly?
As the original code in 2.4.x was the same, I'm wondering:
o Is this a bug in the ARAnyM emulation?
o xpanstep is 1, but judging from the visual output on ARAnyM, it looks like
it should be 16?
Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
---
drivers/video/atafb.c | 2 ++
1 file changed, 2 insertions(+)
--- a/drivers/video/atafb.c
+++ b/drivers/video/atafb.c
@@ -1697,8 +1697,10 @@ static int falcon_pan_display(struct fb_
}
par->hw.falcon.line_offset = bpp *
(info->var.xres_virtual - info->var.xres) / 16;
+#if 0
if (par->hw.falcon.xoffset)
par->hw.falcon.line_offset -= bpp;
+#endif;
The way I understand this code is:
If var->xoffset has the low order bits set (sub-word pixel offset)
par->hw.falcon.xoffset takes care of that, and xoffset takes care of the rest.
Since the partial word offset shifts the beginnig of the scan line into the
first word, the offset to the next scan line (par->hw.falcon.line_offset) has to
be shortened.
I am pretty sure the pan step can be one (see par->hw.falcon.xoffset).
I'll test your patch as soon as I get fbtest to work (missing libgcc_s.so.2).
Michael
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
