August 2008 - Radeon HD - openSUSE Mailing Lists

Build failures with and without DRI on master
by Nico -telmich- Schottelius 07 Aug '08

07 Aug '08

Hello! I am unable to build radeon with DRI [0] and without DRI [1]. I thought the master branch should always work (as in compile and run)? Can you otherwise open a "stable" branch, so end-testers can use that branch for testing the latest running version? Sincerly, Nico [0]: https://nopaste.eof.name/750 [1]: https://nopaste.eof.name/751 -- Think about Free and Open Source Software (FOSS). http://nico.schottelius.org/documentations/foss/the-term-foss/ PGP: BFE4 C736 ABE5 406F 8F42 F7CF B8BE F92A 9885 188C

3 5

xf86-video-radeonhd:master: 1 commit(s)
by agd5f＠kemper.freedesktop.org 07 Aug '08

07 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd Commit against master at 3ab4a14e...: commit c3541cf5b8bcdfe99cad42230703665814dc6905 Author: Alex Deucher <alexdeucher(a)gmail.com> Date: Thu Aug 7 00:17:00 2008 -0400 Fix compilation againt ancient versions of exa diffstat: src/radeon_textured_video.c | 11 +++++++++++ 1 file changed, 11 insertions(+) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0

Compile error in git head
by Randall Nortman 06 Aug '08

06 Aug '08

I'm getting a compile error in recent versions from git; I haven't tracked down exactly which revision introduced the problem yet. I suspect I'm seeing it because I have an older version of the xorg headers installed than most people pulling radeonhd from git; I've got Debian's xorg-dev-7.1.0-19 package installed on etch. This is the error: radeon_textured_video.c -fPIC -DPIC -o .libs/radeonhd_drv_la-radeon_textured_video.o radeon_textured_video.c: In function 'RADEONPutImageTextured': radeon_textured_video.c:236: error: implicit declaration of function 'exaMoveInPixmap' radeon_textured_video.c:236: warning: nested extern declaration of 'exaMoveInPixmap' Am I right that my headers are just too old? If so, we probably ought to determine the minimum xorg version required and put that in the readme. TIA, -- Randall -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

3 3

xf86-video-radeonhd:master: 1 commit(s)
by agd5f＠kemper.freedesktop.org 06 Aug '08

06 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd Commit against master at 6bfbdc07...: commit 3ab4a14ee294bd8b3a3b6d8134af38a199502cdd Author: Alex Deucher <alexdeucher(a)gmail.com> Date: Wed Aug 6 13:19:46 2008 -0400 Fix last commit for r5xx dri - want RADEONEngineInit() not RADEONInit3DEngine() - both functions rely on accel_state struct with is not available if neither exa nor xaa are active diffstat: src/rhd_dri.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0

xf86-video-radeonhd:master: 1 commit(s)
by mhopf＠kemper.freedesktop.org 06 Aug '08

06 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd Commit against master at 16c65c72...: commit 6bfbdc0776d7b7876f7915e40e6f0c240d0f988b Author: Matthias Hopf <mhopf(a)suse.de> Date: Wed Aug 6 18:09:35 2008 +0200 Allow DRI initialization without 2D acceleration being active, again. Not completely correct this way (probably doesn't survive VT switches), but helps 3D bringup. diffstat: src/rhd_cp.h | 3 ++- src/rhd_dri.c | 25 ++++++++++--------------- 2 files changed, 12 insertions(+), 16 deletions(-) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0

xf86-video-radeonhd:master: 1 commit(s)
by eich＠kemper.freedesktop.org 06 Aug '08

06 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd Commit against master at 11e0a573...: commit 16c65c721b4098ec213bee537de8959291f34d4a Author: Egbert Eich <eich(a)freedesktop.org> Date: Wed Aug 6 17:30:38 2008 +0200 2DAccel: Build fixes. Include xf86.h before any other DDX header files. Make uint*_t a define when not building with DRI support. diffstat: src/radeon_accel.c | 10 +++++----- src/radeon_exa.c | 10 ++++------ src/radeon_textured_video.c | 9 ++++----- src/radeon_video.c | 14 +++++++------- src/rhd_dri.c | 4 ++++ src/rhd_driver.c | 4 ++++ 6 files changed, 28 insertions(+), 23 deletions(-) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0

Re: [radeonhd] Build failures with and without DRI on master
by Egbert Eich 06 Aug '08

06 Aug '08

Alex Deucher writes: > On Wed, Aug 6, 2008 at 5:33 AM, Nico -telmich- Schottelius > <nico-xorg-radeonhd(a)schottelius.org> wrote: > > Hello! > > > > I am unable to build radeon with DRI [0] and without DRI [1]. > > > > I thought the master branch should always work (as in compile and run)? > > > > Can you otherwise open a "stable" branch, so end-testers can use that > > branch for testing the latest running version? > > I can't reproduce any build problems here. Does this patch help? > Hi Alex! I thought about this solution, too. But then I looked at some header files which include stdint.h for us when we use DRI. They do some sort fo voodoo to figure out if to include this file or another. I've got a patch prepared which uses simple defines for the types. This seems to be saver than typedefs. Cheers, Egbert. > Alex > diff --git a/src/radeon_accel.c b/src/radeon_accel.c > index 5ab4a45..d5ffb8f 100644 > --- a/src/radeon_accel.c > +++ b/src/radeon_accel.c > @@ -56,6 +56,7 @@ > > #include <errno.h> > #include <string.h> > +#include <stdint.h> > /* Driver data structures */ > #ifdef USE_XAA > #include "xaa.h" > @@ -73,10 +74,6 @@ > #ifdef USE_DRI > # include "rhd_dri.h" > # include "rhd_cp.h" > -#else > -typedef CARD8 uint8_t; > -typedef CARD16 uint16_t; > -typedef CARD32 uint32_t; > #endif > > #include "radeon_accel.h" > diff --git a/src/radeon_exa.c b/src/radeon_exa.c > index c8bd434..ac5bc8d 100644 > --- a/src/radeon_exa.c > +++ b/src/radeon_exa.c > @@ -33,6 +33,8 @@ > #include "config.h" > #endif > > +#include <stdint.h> > + > #ifdef USE_XAA > #include "xaa.h" > #endif > @@ -48,10 +50,6 @@ > #ifdef USE_DRI > # include "rhd_dri.h" > # include "rhd_cp.h" > -#else > -typedef CARD8 uint8_t; > -typedef CARD16 uint16_t; > -typedef CARD32 uint32_t; > #endif > > #include "radeon_accel.h" > diff --git a/src/radeon_textured_video.c b/src/radeon_textured_video.c > index 46c2f45..79651a1 100644 > --- a/src/radeon_textured_video.c > +++ b/src/radeon_textured_video.c > @@ -30,6 +30,7 @@ > #endif > > #include <stdlib.h> > +#include <stdint.h> > #include <string.h> > #include <stdio.h> > #include <math.h> > @@ -54,9 +55,6 @@ > #ifdef USE_DRI > # include "rhd_cp.h" > #else > -typedef CARD8 uint8_t; > -typedef CARD16 uint16_t; > -typedef CARD32 uint32_t; > #include "damage.h" > #endif > > diff --git a/src/radeon_video.c b/src/radeon_video.c > index d09144c..5921a95 100644 > --- a/src/radeon_video.c > +++ b/src/radeon_video.c > @@ -3,6 +3,7 @@ > #endif > > #include <stdlib.h> > +#include <stdint.h> > #include <string.h> > #include <stdio.h> > #include <math.h> > @@ -24,10 +25,6 @@ > #ifdef USE_DRI > # include "rhd_dri.h" > # include "rhd_cp.h" > -#else > -typedef CARD8 uint8_t; > -typedef CARD16 uint16_t; > -typedef CARD32 uint32_t; > #endif > > #include "radeon_reg.h" > diff --git a/src/radeon_xaa.c b/src/radeon_xaa.c > index c3325ac..1f9c381 100644 > --- a/src/radeon_xaa.c > +++ b/src/radeon_xaa.c > @@ -66,6 +66,7 @@ > > #include <errno.h> > #include <string.h> > +#include <stdint.h> > /* Driver data structures */ > #ifdef USE_XAA > #include "xaa.h" > @@ -83,10 +84,6 @@ > #ifdef USE_DRI > # include "rhd_dri.h" > # include "rhd_cp.h" > -#else > -typedef CARD8 uint8_t; > -typedef CARD16 uint16_t; > -typedef CARD32 uint32_t; > #endif > > #include "radeon_accel.h" -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

2 1

Re: [radeonhd] Non-DRI build broken by 2cc9378e ("DRI/2D: Fix build for --disable-dri.")
by Egbert Eich 06 Aug '08

06 Aug '08

Hans Ulrich Niedermann writes: > Hi, > > 2cc9378e contains changes like the following > > -#include "rhd_dri.h" > -#include "rhd_cp.h" > +#ifdef USE_DRI > +# include "rhd_dri.h" > +# include "rhd_cp.h" > +#else > +typedef CARD8 uint8_t; > +typedef CARD16 uint16_t; > +typedef CARD32 uint32_t; > +#endif > > without checking that uintNN_t are actually undefined. > > This makes a build after "configure --disable-dri" fail as > follows (on my Fedora 9 system), just like IRC user "rbmorse" > reported about his system: > > src/radeon_accel.c:67: error: conflicting types for 'uint32_t' > /usr/include/stdint.h:52: error: previous declaration of 'uint32_t' was here > > Egbert, for what kinds of systems were those typedefs required, > and could there be a different way to define uintNN_t for these > systems? Honestly I don't know. I've checked the header file which normally includes /usr/include/stdint.h. It does quite some voodoo to find out on which system which file is required. I wanted to stay away from this so I wrote up those simple typedefs. I don't know if there is a configure rule to test this and frankly I don't have the time to do any research on this. I personally would do a string replacement on all those types and replace them with well known X types. Cheers, Egbert. -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

2 1

xf86-video-radeonhd:master: 1 commit(s)
by ndim＠kemper.freedesktop.org 06 Aug '08

06 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd Commit against master at 3ca7c8c3...: commit 11e0a573ae66cfa24b8d8e7067dfbea54afba100 Author: Hans Ulrich Niedermann <hun(a)n-dimensional.de> Date: Wed Aug 6 15:21:36 2008 +0200 man page: TVMode option is not implemented yet diffstat: man/radeonhd.man | 1 + 1 file changed, 1 insertion(+) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0

xf86-video-radeonhd:master: 4 commit(s)
by ndim＠kemper.freedesktop.org 06 Aug '08

06 Aug '08

Reporting from xf86-video-radeonhd: Radeon HD video driver Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd 4 commits against master at 87f05fd9...: man/radeonhd.man | 48 ++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 46 insertions(+), 2 deletions(-) commit 3ca7c8c379f6afdd4ca5d3f9545a999ab8b442ab Author: Hans Ulrich Niedermann <hun(a)n-dimensional.de> Date: Wed Aug 6 12:47:26 2008 +0200 man page: Document Option "TVMode" diffstat: man/radeonhd.man | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… commit 7b2ce7b0507b7df35320e3f50df5ad445d479574 Author: Hans Ulrich Niedermann <hun(a)n-dimensional.de> Date: Wed Aug 6 12:45:00 2008 +0200 man page: Document Option "UnverifiedFeatures" diffstat: man/radeonhd.man | 7 +++++++ 1 file changed, 7 insertions(+) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… commit d676619cf079cdfddd165d61aa46096a777aab73 Author: Hans Ulrich Niedermann <hun(a)n-dimensional.de> Date: Wed Aug 6 12:42:34 2008 +0200 man page: Document Option "DRI" diffstat: man/radeonhd.man | 7 +++++++ 1 file changed, 7 insertions(+) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… commit 4a70bdced511e80a5898d3fd32b5f7f43dfd8aad Author: Hans Ulrich Niedermann <hun(a)n-dimensional.de> Date: Thu Jul 24 11:24:30 2008 +0200 man page: Name the xorg.conf section for options diffstat: man/radeonhd.man | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h… -- To unsubscribe, e-mail: radeonhd+unsubscribe(a)opensuse.org For additional commands, e-mail: radeonhd+help(a)opensuse.org

1 0