> This looks like it is missing from the driver:
>
>    /* Workaround for some hardware bugs */
>    if (info->ChipFamily < CHIP_FAMILY_R200)
>       OUTREG(RADEON_AGP_CNTL, INREG(RADEON_AGP_CNTL) | 0x000e0000);

should be in drm IMHO...

>
> Second one:
>
>    /* Initialize Radeon's AGP registers */
>    /* Ring buffer is at AGP offset 0 */
>    OUTREG(RADEON_AGP_BASE, info->ringHandle);
>
> DRM is setting this already:
>     /* set RADEON_AGP_BASE here instead of relying on X from user space */
>    RADEON_WRITE(RADEON_AGP_BASE, (unsigned int)dev->agp->base);

> I seem to remember making this change, did I already fix this in the driver?
>

Yeah I think you fixed this one ..

Dave.

-- 
David Airlie, Software Engineer
http://www.skynet.ie/~airlied / airlied at skynet.ie
Linux kernel - DRI, VAX / pam_smb / ILUG



-------------------------------------------------------
This SF.Net email is sponsored by: NEC IT Guy Games.  How far can you shotput
a projector? How fast can you ride your desk chair down the office luge track?
If you want to score the big prize, get to know the little guy.  
Play to win an NEC 61" plasma display: http://www.necitguy.com/?r=20
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to