On Mon, Aug 23, 2010 at 12:24 AM, Benjamin Franzke
<benjaminfran...@googlemail.com> wrote:
> st/egl/x11/x11_screen.c requests a driver named r300 not radeon
>
> KNOWN ISSUE: breaks st/egl/kms/
> st/egl/kms requests a pipe named "radeon"
> that will not be found now
>
> so why not leaving pipe_radeon there?
> that was possible as long we have only r300g.
> now there is also r600g for which st/egl/kms also
> requests a pipe named "radeon"
> (possible solution in later commit)
I had a look at xorg-video-ati and r300 should be the right name. st/egl/kms
should duplicate the logic in the DDX driver and request r300 or r600 when the
kernel module is radeon.
But I am now wondering why targets/dri-radeong builds radeon_dri instead of
r300_dri?
> ---
> src/gallium/targets/egl/Makefile | 14 +++++++-------
> src/gallium/targets/egl/pipe_r300.c | 27 +++++++++++++++++++++++++++
> src/gallium/targets/egl/pipe_radeon.c | 27 ---------------------------
> 3 files changed, 34 insertions(+), 34 deletions(-)
> create mode 100644 src/gallium/targets/egl/pipe_r300.c
> delete mode 100644 src/gallium/targets/egl/pipe_radeon.c
>
> diff --git a/src/gallium/targets/egl/Makefile
> b/src/gallium/targets/egl/Makefile
> index 1585e2d..636fceb 100644
> --- a/src/gallium/targets/egl/Makefile
> +++ b/src/gallium/targets/egl/Makefile
> @@ -90,10 +90,10 @@ nouveau_LIBS := \
> $(TOP)/src/gallium/drivers/nv50/libnv50.a \
> $(TOP)/src/gallium/drivers/nouveau/libnouveau.a
>
> -# radeon pipe driver
> -radeon_CPPFLAGS :=
> -radeon_SYS := -ldrm -ldrm_radeon
> -radeon_LIBS := \
> +# r300 pipe driver
> +r300_CPPFLAGS :=
> +r300_SYS := -ldrm -ldrm_radeon
> +r300_LIBS := \
> $(TOP)/src/gallium/winsys/radeon/drm/libradeonwinsys.a \
> $(TOP)/src/gallium/drivers/r300/libr300.a
>
> @@ -151,7 +151,7 @@ ifneq ($(findstring nouveau/drm,$(GALLIUM_WINSYS_DIRS)),)
> OUTPUTS += nouveau
> endif
> ifneq ($(findstring radeon/drm,$(GALLIUM_WINSYS_DIRS)),)
> -OUTPUTS += radeon
> +OUTPUTS += r300
> endif
> ifneq ($(findstring svga/drm,$(GALLIUM_WINSYS_DIRS)),)
> OUTPUTS += vmwgfx
> @@ -188,8 +188,8 @@ $(OUTPUT_PATH)/$(PIPE_PREFIX)i965.so: pipe_i965.o
> $(i965_LIBS)
> $(OUTPUT_PATH)/$(PIPE_PREFIX)nouveau.so: pipe_nouveau.o $(nouveau_LIBS)
> $(call mklib,nouveau)
>
> -$(OUTPUT_PATH)/$(PIPE_PREFIX)radeon.so: pipe_radeon.o $(radeon_LIBS)
> - $(call mklib,radeon)
> +$(OUTPUT_PATH)/$(PIPE_PREFIX)r300.so: pipe_r300.o $(r300_LIBS)
> + $(call mklib,r300)
>
> $(OUTPUT_PATH)/$(PIPE_PREFIX)vmwgfx.so: pipe_vmwgfx.o $(vmwgfx_LIBS)
> $(call mklib,vmwgfx)
> diff --git a/src/gallium/targets/egl/pipe_r300.c
> b/src/gallium/targets/egl/pipe_r300.c
> new file mode 100644
> index 0000000..2fa495e
> --- /dev/null
> +++ b/src/gallium/targets/egl/pipe_r300.c
> @@ -0,0 +1,27 @@
> +
> +#include "target-helpers/inline_debug_helper.h"
> +#include "state_tracker/drm_driver.h"
> +#include "radeon/drm/radeon_drm_public.h"
> +#include "r300/r300_public.h"
> +
> +static struct pipe_screen *
> +create_screen(int fd)
> +{
> + struct r300_winsys_screen *sws;
> + struct pipe_screen *screen;
> +
> + sws = r300_drm_winsys_screen_create(fd);
> + if (!sws)
> + return NULL;
> +
> + screen = r300_screen_create(sws);
> + if (!screen)
> + return NULL;
> +
> + screen = debug_screen_wrap(screen);
> +
> + return screen;
> +}
> +
> +PUBLIC
> +DRM_DRIVER_DESCRIPTOR("r300", "r300", create_screen)
> diff --git a/src/gallium/targets/egl/pipe_radeon.c
> b/src/gallium/targets/egl/pipe_radeon.c
> deleted file mode 100644
> index 35550bc..0000000
> --- a/src/gallium/targets/egl/pipe_radeon.c
> +++ /dev/null
> @@ -1,27 +0,0 @@
> -
> -#include "target-helpers/inline_debug_helper.h"
> -#include "state_tracker/drm_driver.h"
> -#include "radeon/drm/radeon_drm_public.h"
> -#include "r300/r300_public.h"
> -
> -static struct pipe_screen *
> -create_screen(int fd)
> -{
> - struct r300_winsys_screen *sws;
> - struct pipe_screen *screen;
> -
> - sws = r300_drm_winsys_screen_create(fd);
> - if (!sws)
> - return NULL;
> -
> - screen = r300_screen_create(sws);
> - if (!screen)
> - return NULL;
> -
> - screen = debug_screen_wrap(screen);
> -
> - return screen;
> -}
> -
> -PUBLIC
> -DRM_DRIVER_DESCRIPTOR("radeon", "radeon", create_screen)
> --
> 1.7.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
--
o...@lunarg.com
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
