Re: [PATCH] result parameter for ScreenWakeupHandlerProcPtr should be unsigned long

2019-01-09 Thread Matt Turner
On Wed, Jan 9, 2019 at 10:03 AM Kevin Brace wrote: > > Hi Dave, > > I do see what is going on, but was there a mistake in the X Server header > file when the ABI was changed? > Who else uses ScreenWakeupHandlerProcPtr and does another device driver have > this issue? Can we please try to keep

Re: [PATCH] result parameter for ScreenWakeupHandlerProcPtr should be unsigned long

2019-01-09 Thread Kevin Brace
ent: Thursday, January 03, 2019 at 3:14 PM > From: "Dave Airlie" > To: "Kevin Brace" > Cc: xorg-devel > Subject: Re: [PATCH] result parameter for ScreenWakeupHandlerProcPtr should > be unsigned long > > On Fri, 14 Dec 2018 at 06:47, Kevin Brace wrote:

Re: [PATCH] result parameter for ScreenWakeupHandlerProcPtr should be unsigned long

2019-01-03 Thread Dave Airlie
On Fri, 14 Dec 2018 at 06:47, Kevin Brace wrote: > > The result parameter defined inside scrnintstr.h for > ScreenWakeupHandlerProcPtr should be unsigned long type not int type. > This led to wrong pointer type compilation warnings when compiling S3 > Savage DDX Version 2.3.9 for

[PATCH] result parameter for ScreenWakeupHandlerProcPtr should be unsigned long

2018-12-13 Thread Kevin Brace
The result parameter defined inside scrnintstr.h for ScreenWakeupHandlerProcPtr should be unsigned long type not int type. This led to wrong pointer type compilation warnings when compiling S3 Savage DDX Version 2.3.9 for ABI_VIDEODRV_VERSION >= 23. Signed-off-by: Kevin Brace ---