Created attachment 47892 [details] [review] Patch Since my email sent to xorg-driver-ati mailing list has not been accepted/published for more than a week (http://lists.x.org/archives/xorg-driver-ati/2011-June/020353.html): Further information: http://comments.gmane.org/gmane.linux.ide/46629 and http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=b429dd5998d 6037e0057564f3894f56aff64d661
This patch has come up a number of times in the past since gcc started warning about this (it didn't used to when the code was written). I'd rather seem a nicer, sustainable fix (converting to defines or something) than casting to ints everywhere.
Created attachment 47962 [details] [review] patch to silence compiler warings here you go
Created attachment 47964 [details] [review] Alternate patch to silence warnings The number of casts involved in Marc's patch got me to thinking that using a temporary variable might be cleaner. The attached patch also uses the opportunity to line up backslashes in the macros.
ping!
Thanks for the patches, guys, but I took Alex's suggestion and converted the register ranges to defines.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.