All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH xf86-video-ati] Require xserver 1.10 or newer
@ 2016-10-27  2:48 Michel Dänzer
       [not found] ` <20161027024858.30080-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Michel Dänzer @ 2016-10-27  2:48 UTC (permalink / raw)
  To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

From: Michel Dänzer <michel.daenzer@amd.com>

1.10.0 was released in February 2011.

We've been accidentally requiring 1.10 or newer since 121a6de72da5 ("Keep
track of damage event related flushes per-client v2").

Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
---
 configure.ac          |  2 +-
 src/drmmode_display.c |  7 -------
 src/drmmode_display.h |  2 --
 src/radeon.h          |  2 --
 src/radeon_kms.c      | 12 ------------
 5 files changed, 1 insertion(+), 24 deletions(-)

diff --git a/configure.ac b/configure.ac
index 3fc967e..2f76945 100644
--- a/configure.ac
+++ b/configure.ac
@@ -73,7 +73,7 @@ PKG_CHECK_MODULES(LIBDRM, [libdrm >= 2.4.58])
 PKG_CHECK_MODULES(LIBDRM_RADEON, [libdrm_radeon])
 
 # Obtain compiler/linker options for the driver dependencies
-PKG_CHECK_MODULES(XORG, [xorg-server >= 1.9 xproto fontsproto xf86driproto $REQUIRED_MODULES])
+PKG_CHECK_MODULES(XORG, [xorg-server >= 1.10 xproto fontsproto xf86driproto $REQUIRED_MODULES])
 PKG_CHECK_MODULES(XEXT, [xextproto >= 7.0.99.1],
                   HAVE_XEXTPROTO_71="yes"; AC_DEFINE(HAVE_XEXTPROTO_71, 1, [xextproto 7.1 available]),
                   HAVE_XEXTPROTO_71="no")
diff --git a/src/drmmode_display.c b/src/drmmode_display.c
index 9cf4846..b95e1c9 100644
--- a/src/drmmode_display.c
+++ b/src/drmmode_display.c
@@ -388,8 +388,6 @@ drmmode_crtc_dpms(xf86CrtcPtr crtc, int mode)
 					    crtc->x, crtc->y);
 }
 
-#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
-
 static PixmapPtr
 create_pixmap_for_fbcon(drmmode_ptr drmmode,
 			ScrnInfoPtr pScrn, int fbcon_id)
@@ -508,8 +506,6 @@ void drmmode_copy_fb(ScrnInfoPtr pScrn, drmmode_ptr drmmode)
 	return;
 }
 
-#endif /* GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10 */
-
 static void
 drmmode_crtc_scanout_destroy(drmmode_ptr drmmode,
 			     struct drmmode_scanout *scanout)
@@ -2149,9 +2145,6 @@ drmmode_xf86crtc_resize (ScrnInfoPtr scrn, int width, int height)
 					   width, height, -1, -1, pitch,
 					   info->fb_shadow);
 	}
-#if XORG_VERSION_CURRENT < XORG_VERSION_NUMERIC(1,9,99,1,0)
-	scrn->pixmapPrivate.ptr = ppix->devPrivate.ptr;
-#endif
 
 	if (info->use_glamor)
 		radeon_glamor_create_screen_resources(scrn->pScreen);
diff --git a/src/drmmode_display.h b/src/drmmode_display.h
index ade4a0b..85bcad9 100644
--- a/src/drmmode_display.h
+++ b/src/drmmode_display.h
@@ -141,9 +141,7 @@ extern void drmmode_set_cursor(ScrnInfoPtr scrn, drmmode_ptr drmmode, int id, st
 void drmmode_adjust_frame(ScrnInfoPtr pScrn, drmmode_ptr drmmode, int x, int y);
 extern Bool drmmode_set_desired_modes(ScrnInfoPtr pScrn, drmmode_ptr drmmode,
 				      Bool set_hw);
-#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
 extern void drmmode_copy_fb(ScrnInfoPtr pScrn, drmmode_ptr drmmode);
-#endif
 extern Bool drmmode_setup_colormap(ScreenPtr pScreen, ScrnInfoPtr pScrn);
 
 extern void drmmode_scanout_free(ScrnInfoPtr scrn);
diff --git a/src/radeon.h b/src/radeon.h
index 0bf6d37..ad7e69c 100644
--- a/src/radeon.h
+++ b/src/radeon.h
@@ -519,9 +519,7 @@ typedef struct {
     DisplayModePtr currentMode;
 
     CreateScreenResourcesProcPtr CreateScreenResources;
-#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
     CreateWindowProcPtr CreateWindow;
-#endif
 
     Bool              IsSecondary;
 
diff --git a/src/radeon_kms.c b/src/radeon_kms.c
index 68c7837..872150d 100644
--- a/src/radeon_kms.c
+++ b/src/radeon_kms.c
@@ -1402,13 +1402,8 @@ static int radeon_get_drm_master_fd(ScrnInfoPtr pScrn)
     }
 #endif
 
-#if XORG_VERSION_CURRENT >= XORG_VERSION_NUMERIC(1,9,99,901,0)
     XNFasprintf(&busid, "pci:%04x:%02x:%02x.%d",
                 dev->domain, dev->bus, dev->dev, dev->func);
-#else
-    busid = XNFprintf("pci:%04x:%02x:%02x.%d",
-		      dev->domain, dev->bus, dev->dev, dev->func);
-#endif
 
     fd = drmOpen(NULL, busid);
     if (fd == -1)
@@ -1597,8 +1592,6 @@ static void RADEONSetupCapabilities(ScrnInfoPtr pScrn)
 #endif
 }
 
-#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
-
 /* When the root window is created, initialize the screen contents from
  * console if -background none was specified on the command line
  */
@@ -1623,8 +1616,6 @@ static Bool RADEONCreateWindow_oneshot(WindowPtr pWin)
     return ret;
 }
 
-#endif
-
 Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int flags)
 {
     RADEONInfoPtr     info;
@@ -1714,7 +1705,6 @@ Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int flags)
     /* don't enable tiling if accel is not enabled */
     if (!info->r600_shadow_fb) {
 	Bool colorTilingDefault =
-	    xorgGetVersion() >= XORG_VERSION_NUMERIC(1,9,4,901,0) &&
 	    info->ChipFamily >= CHIP_FAMILY_R300 &&
 	    /* this check could be removed sometime after a big mesa release
 	     * with proper bit, in the meantime you need to set tiling option in
@@ -2303,12 +2293,10 @@ Bool RADEONScreenInit_KMS(SCREEN_INIT_ARGS_DECL)
     }
     pScrn->pScreen = pScreen;
 
-#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
     if (serverGeneration == 1 && bgNoneRoot && info->accelOn) {
 	info->CreateWindow = pScreen->CreateWindow;
 	pScreen->CreateWindow = RADEONCreateWindow_oneshot;
     }
-#endif
 
     /* Provide SaveScreen & wrap BlockHandler and CloseScreen */
     /* Wrap CloseScreen */
-- 
2.10.1

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

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* RE: [PATCH xf86-video-ati] Require xserver 1.10 or newer
       [not found] ` <20161027024858.30080-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2016-10-27 14:09   ` Deucher, Alexander
  0 siblings, 0 replies; 2+ messages in thread
From: Deucher, Alexander @ 2016-10-27 14:09 UTC (permalink / raw)
  To: 'Michel Dänzer', amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On Behalf
> Of Michel Dänzer
> Sent: Wednesday, October 26, 2016 10:49 PM
> To: amd-gfx@lists.freedesktop.org
> Subject: [PATCH xf86-video-ati] Require xserver 1.10 or newer
> 
> From: Michel Dänzer <michel.daenzer@amd.com>
> 
> 1.10.0 was released in February 2011.
> 
> We've been accidentally requiring 1.10 or newer since 121a6de72da5 ("Keep
> track of damage event related flushes per-client v2").
> 
> Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  configure.ac          |  2 +-
>  src/drmmode_display.c |  7 -------
>  src/drmmode_display.h |  2 --
>  src/radeon.h          |  2 --
>  src/radeon_kms.c      | 12 ------------
>  5 files changed, 1 insertion(+), 24 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 3fc967e..2f76945 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -73,7 +73,7 @@ PKG_CHECK_MODULES(LIBDRM, [libdrm >= 2.4.58])
>  PKG_CHECK_MODULES(LIBDRM_RADEON, [libdrm_radeon])
> 
>  # Obtain compiler/linker options for the driver dependencies
> -PKG_CHECK_MODULES(XORG, [xorg-server >= 1.9 xproto fontsproto
> xf86driproto $REQUIRED_MODULES])
> +PKG_CHECK_MODULES(XORG, [xorg-server >= 1.10 xproto fontsproto
> xf86driproto $REQUIRED_MODULES])
>  PKG_CHECK_MODULES(XEXT, [xextproto >= 7.0.99.1],
>                    HAVE_XEXTPROTO_71="yes"; AC_DEFINE(HAVE_XEXTPROTO_71,
> 1, [xextproto 7.1 available]),
>                    HAVE_XEXTPROTO_71="no")
> diff --git a/src/drmmode_display.c b/src/drmmode_display.c
> index 9cf4846..b95e1c9 100644
> --- a/src/drmmode_display.c
> +++ b/src/drmmode_display.c
> @@ -388,8 +388,6 @@ drmmode_crtc_dpms(xf86CrtcPtr crtc, int mode)
>  					    crtc->x, crtc->y);
>  }
> 
> -#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
> -
>  static PixmapPtr
>  create_pixmap_for_fbcon(drmmode_ptr drmmode,
>  			ScrnInfoPtr pScrn, int fbcon_id)
> @@ -508,8 +506,6 @@ void drmmode_copy_fb(ScrnInfoPtr pScrn,
> drmmode_ptr drmmode)
>  	return;
>  }
> 
> -#endif /* GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10 */
> -
>  static void
>  drmmode_crtc_scanout_destroy(drmmode_ptr drmmode,
>  			     struct drmmode_scanout *scanout)
> @@ -2149,9 +2145,6 @@ drmmode_xf86crtc_resize (ScrnInfoPtr scrn, int
> width, int height)
>  					   width, height, -1, -1, pitch,
>  					   info->fb_shadow);
>  	}
> -#if XORG_VERSION_CURRENT < XORG_VERSION_NUMERIC(1,9,99,1,0)
> -	scrn->pixmapPrivate.ptr = ppix->devPrivate.ptr;
> -#endif
> 
>  	if (info->use_glamor)
>  		radeon_glamor_create_screen_resources(scrn->pScreen);
> diff --git a/src/drmmode_display.h b/src/drmmode_display.h
> index ade4a0b..85bcad9 100644
> --- a/src/drmmode_display.h
> +++ b/src/drmmode_display.h
> @@ -141,9 +141,7 @@ extern void drmmode_set_cursor(ScrnInfoPtr scrn,
> drmmode_ptr drmmode, int id, st
>  void drmmode_adjust_frame(ScrnInfoPtr pScrn, drmmode_ptr drmmode,
> int x, int y);
>  extern Bool drmmode_set_desired_modes(ScrnInfoPtr pScrn,
> drmmode_ptr drmmode,
>  				      Bool set_hw);
> -#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
>  extern void drmmode_copy_fb(ScrnInfoPtr pScrn, drmmode_ptr
> drmmode);
> -#endif
>  extern Bool drmmode_setup_colormap(ScreenPtr pScreen, ScrnInfoPtr
> pScrn);
> 
>  extern void drmmode_scanout_free(ScrnInfoPtr scrn);
> diff --git a/src/radeon.h b/src/radeon.h
> index 0bf6d37..ad7e69c 100644
> --- a/src/radeon.h
> +++ b/src/radeon.h
> @@ -519,9 +519,7 @@ typedef struct {
>      DisplayModePtr currentMode;
> 
>      CreateScreenResourcesProcPtr CreateScreenResources;
> -#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
>      CreateWindowProcPtr CreateWindow;
> -#endif
> 
>      Bool              IsSecondary;
> 
> diff --git a/src/radeon_kms.c b/src/radeon_kms.c
> index 68c7837..872150d 100644
> --- a/src/radeon_kms.c
> +++ b/src/radeon_kms.c
> @@ -1402,13 +1402,8 @@ static int radeon_get_drm_master_fd(ScrnInfoPtr
> pScrn)
>      }
>  #endif
> 
> -#if XORG_VERSION_CURRENT >= XORG_VERSION_NUMERIC(1,9,99,901,0)
>      XNFasprintf(&busid, "pci:%04x:%02x:%02x.%d",
>                  dev->domain, dev->bus, dev->dev, dev->func);
> -#else
> -    busid = XNFprintf("pci:%04x:%02x:%02x.%d",
> -		      dev->domain, dev->bus, dev->dev, dev->func);
> -#endif
> 
>      fd = drmOpen(NULL, busid);
>      if (fd == -1)
> @@ -1597,8 +1592,6 @@ static void RADEONSetupCapabilities(ScrnInfoPtr
> pScrn)
>  #endif
>  }
> 
> -#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
> -
>  /* When the root window is created, initialize the screen contents from
>   * console if -background none was specified on the command line
>   */
> @@ -1623,8 +1616,6 @@ static Bool
> RADEONCreateWindow_oneshot(WindowPtr pWin)
>      return ret;
>  }
> 
> -#endif
> -
>  Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int flags)
>  {
>      RADEONInfoPtr     info;
> @@ -1714,7 +1705,6 @@ Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int
> flags)
>      /* don't enable tiling if accel is not enabled */
>      if (!info->r600_shadow_fb) {
>  	Bool colorTilingDefault =
> -	    xorgGetVersion() >= XORG_VERSION_NUMERIC(1,9,4,901,0) &&
>  	    info->ChipFamily >= CHIP_FAMILY_R300 &&
>  	    /* this check could be removed sometime after a big mesa release
>  	     * with proper bit, in the meantime you need to set tiling option in
> @@ -2303,12 +2293,10 @@ Bool
> RADEONScreenInit_KMS(SCREEN_INIT_ARGS_DECL)
>      }
>      pScrn->pScreen = pScreen;
> 
> -#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) >= 10
>      if (serverGeneration == 1 && bgNoneRoot && info->accelOn) {
>  	info->CreateWindow = pScreen->CreateWindow;
>  	pScreen->CreateWindow = RADEONCreateWindow_oneshot;
>      }
> -#endif
> 
>      /* Provide SaveScreen & wrap BlockHandler and CloseScreen */
>      /* Wrap CloseScreen */
> --
> 2.10.1
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-10-27 14:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-27  2:48 [PATCH xf86-video-ati] Require xserver 1.10 or newer Michel Dänzer
     [not found] ` <20161027024858.30080-1-michel-otUistvHUpPR7s880joybQ@public.gmane.org>
2016-10-27 14:09   ` Deucher, Alexander

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.