On Thu, Nov 22, 2007 at 11:13:32PM +0100, Luc Verhaegen wrote:
On Thu, Nov 22, 2007 at 09:15:38PM +0100, Hermann Gausterer wrote:
hi
i think this git commit looks a little bit strange:
http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commitdif...
+ /* R600 adds an undocumented register at 0x7AD8, + * shifting all subsequent registers by exactly one. */ .....SNIP..... + LVTMA_R600_REG_TEST_OUTPUT = 0x7B18, + + LVTMA_RS690_NOT_DOCUMENTED = 0x7B18,
the comment tells that the undocumented register is at 0x7AD8 and the code defines the undocumented register to another offset which allready defined; i think this is a type!
mfg hermann
You are absolutely correct. This is not a typo, this is just badly thought through. This is a documented register of course, so i badly blame ATI for my own stupidity.
There are no adverse side effects, the end result is the same for rs690, but it will be fixed asap.
Luc Verhaegen.
Fixed in d9cd95f1fe4 Thanks :) Luc Verhaegen. SUSE/Novell X Driver Developer. -- To unsubscribe, e-mail: radeonhd+unsubscribe@opensuse.org For additional commands, e-mail: radeonhd+help@opensuse.org