[Bug 15151] Compile warning fixes.

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=15151


Paulo César Pereira de Andrade <pcpa at mandriva.com.br> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
           Keywords|janitor, patch              |
           Priority|medium                      |lowest




--- Comment #6 from Paulo César Pereira de Andrade <pcpa at mandriva.com.br>  2008-07-23 12:50:54 PST ---
  This patch should not apply anymore, but I will keep
it opened because there are some issues that should
really be addressed:

o AtomBios should be a shared module for radeon and
  radeonhd drivers.
  This patch corrected all compiler warning fixes and
  also made most of AtomBios static functions, so it
  could be help in starting point for such shared module.
o It should be possible to load variants of the theathre*_drv.so
  modules, but what happens is that they have symbols with
  the same name, and due to a lot of "LoaderSymbol" magic,
  it is required to make most of them visible (the corrections
  for the "LoaderSymbol" magic are also avaiable in my
  patches to allow compiling the XServer and drivers with
  -fvisibility=hidden).

  I think there may existed some misunderstanding that
prevented this patch from being applied, when I said that
math of void* and char* should give the same pointer. But
the fact is that the patch changes the driver to use
char* math, and so, that change should not cause unwanted
side effects.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.