[PATCH] xf86-video-ati: Fix non GLAMOR build

[PATCH] xf86-video-ati: Fix non GLAMOR build

[Joakim Tjernlund]

Non GLAMOR builds needs pRADEONEnt so move the USE_GLAMOR define
down to make pRADEONEnt defined in this case.

Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
---
 src/radeon.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/radeon.h b/src/radeon.h
index d54e4990..3e04101c 100644
--- a/src/radeon.h
+++ b/src/radeon.h
@@ -712,9 +712,9 @@ uint32_t radeon_get_pixmap_tiling(PixmapPtr pPix);
 
 static inline Bool radeon_set_pixmap_bo(PixmapPtr pPix, struct radeon_bo *bo)
 {
-#ifdef USE_GLAMOR
     ScrnInfoPtr scrn = xf86ScreenToScrn(pPix->drawable.pScreen);
     RADEONEntPtr pRADEONEnt = RADEONEntPriv(scrn);
+#ifdef USE_GLAMOR
     RADEONInfoPtr info = RADEONPTR(scrn);
 
     if (info->use_glamor) {
-- 
2.13.6

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: [PATCH] xf86-video-ati: Fix non GLAMOR build

[Michel Dänzer]

On 2017-11-24 05:27 PM, Joakim Tjernlund wrote:
> Non GLAMOR builds needs pRADEONEnt so move the USE_GLAMOR define
> down to make pRADEONEnt defined in this case.
> 
> Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
> ---
>  src/radeon.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/radeon.h b/src/radeon.h
> index d54e4990..3e04101c 100644
> --- a/src/radeon.h
> +++ b/src/radeon.h
> @@ -712,9 +712,9 @@ uint32_t radeon_get_pixmap_tiling(PixmapPtr pPix);
>  
>  static inline Bool radeon_set_pixmap_bo(PixmapPtr pPix, struct radeon_bo *bo)
>  {
> -#ifdef USE_GLAMOR
>      ScrnInfoPtr scrn = xf86ScreenToScrn(pPix->drawable.pScreen);
>      RADEONEntPtr pRADEONEnt = RADEONEntPriv(scrn);
> +#ifdef USE_GLAMOR
>      RADEONInfoPtr info = RADEONPTR(scrn);
>  
>      if (info->use_glamor) {
> 

Pushed (with the redundant prefix removed from the shortlog), thanks!


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx