[Bug 19098] New: [cosmetic] various fixes

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=19098

           Summary: [cosmetic] various fixes
           Product: xorg
           Version: git
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Driver/Radeon
        AssignedTo: xorg-driver-ati at lists.x.org
        ReportedBy: fabio.ped at libero.it
         QAContact: xorg-team at lists.x.org


Created an attachment (id=21175)
 --> (http://bugs.freedesktop.org/attachment.cgi?id=21175)
radeon.man.patch

1) radeon.man.patch
- clarify supported features (hoping that users stop asking things like "why I
can't use Xv on my RV670?" in irc/forum/ml);
- improve readability avoiding adding an empty line between every card in the
SUPPORTED HARDWARE;

2) ati_pciids.csv.patch
- remove some duplicated "ATI ATI " in cards name

Also some things to ask:

a) ati_pciids.csv says that the "ATI Radeon HD 4870 x2" is a RV770. Shouldn't
says R700?

b) MergedFB support was removed some time ago. However a grep -ri mergedfb
revealed there are some traces of it (mostly in comments), e.g.:

src/radeon_driver.c:

   if (xf86IsEntityShared(info->pEnt->index)) {
        xf86DrvMsg(pScrn->scrnIndex, X_WARNING,
                   "Direct Rendering Disabled -- "
                   "Dual-head configuration is not working with "
                   "DRI at present.\n"
                   "Please use the radeon MergedFB option if you "
                   "want Dual-head with DRI.\n");
        return FALSE;
    }

src/legacy_crtc.c:

/* note cannot use pScrn->pScreen since this is unitialized when called from
RADEONScreenInit, and we need to call from there to get mergedfb + pageflip
working */


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.