Hi Geert,

> Make sure fix->line_length is always set, as some applications need it because
> they don't have fallback code if line_length is zero.
> 
> Works on ARAnyM (Falcon emulation), but par->next_line is not set on any other
> Atari variant?

You're right - while testing on the Falcon framebuffer I noticed 
fix->line_length needs to be set, and kludged around to get it set to something 
more or less sane, but I could not test the other code paths. 

It looks like par->next_line should be set to linelen in all other decode_var 
variants. The quilt tree here does have fix->line_length = par->next_line 
already, what is your patch based on? 

I had 

fix->line_length = (par->hw.falcon.line_width + par->hw.falcon.line_offset) *
                par->hw.falcon.bpp / 8;

originally - I suppose this had to change with your cfb fixes? 

        Michael

--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to