[PATCH 1/2] xfree86: Fix type of IOPortBase to be a pointer

Jeremy Huddleston jeremyhu at apple.com
Sun Oct 16 13:50:30 PDT 2011


On Oct 16, 2011, at 9:32 AM, walter harms wrote:
>> @@ -487,8 +487,7 @@ int ScreenNum;
>> 	     * actually mmap the start of the page, then the start of video
>> 	     * I/O space is added as an internal offset.
>> 	     */
>> -	    IOPortBase = (unsigned int)xf86MapInfoMap(memInfoP,
>> -						      (caddr_t)0x0, 0L) 
>> +	    IOPortBase = (pointer)xf86MapInfoMap(memInfoP, (caddr_t)0x0, 0L)
>> 		- memInfoP->memInfo.u.map_info_mmap.internal_offset;
> 
> I would expect a NULL instead of (caddr_t)0x0.

Yeah, well... that's what was there to begin with, and if I go down that road, I'll want to change 1000 other things in those files.



More information about the xorg-devel mailing list