[Bug 1881] New: stretched panel size derivation incorrect?

[bugzilla-daemon at freedesktop.org]

Please do not reply to this email: if you want to comment on the bug, go to          
the URL shown below and enter yourcomments there.   
 
https://bugs.freedesktop.org/show_bug.cgi?id=1881        
   
           Summary: stretched panel size derivation incorrect?
           Product: xorg
           Version: CVS_head
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Driver/Radeon
        AssignedTo: xorg-bugzilla-noise at freedesktop.org
        ReportedBy: daniel at freedesktop.org


Looking at radeon_driver.c, the panel size derivation for stretched panels looks
incorrect to me.  It may be right; without specs, I don't know.

    CARD32 fp_vert_stretch = INREG(RADEON_FP_VERT_STRETCH);
    CARD32 fp_horz_stretch = INREG(RADEON_FP_HORZ_STRETCH);

    info->PanelPwrDly = 200;
    if (fp_vert_stretch & RADEON_VERT_STRETCH_ENABLE) {
        info->PanelYRes = (fp_vert_stretch>>12) + 1;
[...]
    }
    if (fp_horz_stretch & RADEON_HORZ_STRETCH_ENABLE) {
        info->PanelXRes = ((fp_vert_stretch>>16) + 1) * 8;
[...]
    }

A simple but obvious patch to change the second usage of fp_vert_stretch to
fp_horz_stretch is attached.        
   
   
--         
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email       
   
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.