On Fri, Aug 20, 2004 at 11:32:06AM -0700, Erdi Chen wrote:
> >
> >3) What's the purpose of "#define __NO_VERSION__"
> Came from the i915_dma.c that I've used as a template. I have no idea
> what it is for. It seems to exist in many drm source files.
I was not sure but google confirmed.
__NO_VERSION__ was required in the linux kernel 2.3 days, which has
passed even for drm. So it should go in all drivers.
No drivers in linux kernel (2.6.8.1 has this definition.
> >
> >10) What doeis this do?
> > qwPadCount = (CMDBUF_ALIGNMENT_SIZE>>3) -
> > ((((uint32_t)vb) & CMDBUF_ALIGNMENT_MASK) >> 3);
> > There is _no_ guarantee a pointer will fir into uint_32
> > Above construction used in several places.
> qwPadCount is the number of quad-words (inherited from windows driver
> code) to pad the buffer to meet alignment requirement.
Maybe include/linux/kernel-h - ALIGN is usefull here?
Sam
-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
--
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel
