While I was working on mergedfb support for the radeon video overlay, I
happened upon and fixed a bug in the setting of ECP.  in
RADEONAllocAdaptor() and RADEONDisplayVideo(), the current code looks
something like this:

    if(info->ModeReg.dot_clock_freq < 17500) 
        ecp_div = 0;
    else
        ecp_div = 1;

If the overlay is on the second crtc, I would think it should look
something like this:

    if ((info->MergedFB && info->OverlayOnCRTC2) || info->IsSecondary)
        dot_clock = info->ModeReg.dot_clock_freq_2;
    else
        dot_clock = info->ModeReg.dot_clock_freq;

    if(dot_clock < 17500)
        ecp_div = 0;
    else
        ecp_div = 1;

This fix in already in my mergedfb code
(http://bugs.xfree86.org/show_bug.cgi?id=276). 


Alex

__________________________________
Do you Yahoo!?
Yahoo! SiteBuilder - Free, easy-to-use web site design software
http://sitebuilder.yahoo.com
_______________________________________________
Devel mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/devel

Reply via email to