[Mesa-dev] [PATCH 4/7] gallium/targets/xorg: drop set but unused variable entity

Tom Stellard tom at stellard.net
Mon Oct 7 08:52:24 PDT 2013


On Sat, Sep 28, 2013 at 03:46:18PM +0100, Emil Velikov wrote:
> The function xf86GetEntityInfo() retrieves the entity rather than
> doing any changes. Remove this no-op code.
> 
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> ---
>  src/gallium/targets/r600/xorg/xorg.c            | 3 ---
>  src/gallium/targets/radeonsi/xorg/xorg.c        | 3 ---
>  src/gallium/targets/xorg-i915/intel_xorg.c      | 3 ---
>  src/gallium/targets/xorg-nouveau/nouveau_xorg.c | 3 ---
>  4 files changed, 12 deletions(-)
> 
> diff --git a/src/gallium/targets/r600/xorg/xorg.c b/src/gallium/targets/r600/xorg/xorg.c
> index 83610fd..b0b0ef0 100644
> --- a/src/gallium/targets/r600/xorg/xorg.c
> +++ b/src/gallium/targets/r600/xorg/xorg.c
> @@ -129,7 +129,6 @@ r600_xorg_pci_probe(DriverPtr driver,
>  	  int entity_num, struct pci_device *device, intptr_t match_data)
>  {
>      ScrnInfoPtr scrn = NULL;
> -    EntityInfoPtr entity;
>  
>      scrn = xf86ConfigPciEntity(scrn, 0, entity_num, r600_xorg_pci_devices,
>  			       NULL, NULL, NULL, NULL, NULL);
> @@ -139,8 +138,6 @@ r600_xorg_pci_probe(DriverPtr driver,
>  	scrn->name = "R600G";
>  	scrn->Probe = NULL;
>  
> -	entity = xf86GetEntityInfo(entity_num);
> -
>  	/* Use all the functions from the xorg tracker */
>  	xorg_tracker_set_functions(scrn);
>      }
> diff --git a/src/gallium/targets/radeonsi/xorg/xorg.c b/src/gallium/targets/radeonsi/xorg/xorg.c
> index cce7926..7858f03 100644
> --- a/src/gallium/targets/radeonsi/xorg/xorg.c
> +++ b/src/gallium/targets/radeonsi/xorg/xorg.c
> @@ -129,7 +129,6 @@ radeonsi_xorg_pci_probe(DriverPtr driver,
>  	  int entity_num, struct pci_device *device, intptr_t match_data)
>  {
>      ScrnInfoPtr scrn = NULL;
> -    EntityInfoPtr entity;
>  
>      scrn = xf86ConfigPciEntity(scrn, 0, entity_num, radeonsi_xorg_pci_devices,
>  			       NULL, NULL, NULL, NULL, NULL);
> @@ -139,8 +138,6 @@ radeonsi_xorg_pci_probe(DriverPtr driver,
>  	scrn->name = "RADEONSI";
>  	scrn->Probe = NULL;
>  
> -	entity = xf86GetEntityInfo(entity_num);
> -
>  	/* Use all the functions from the xorg tracker */
>  	xorg_tracker_set_functions(scrn);
>      }
> diff --git a/src/gallium/targets/xorg-i915/intel_xorg.c b/src/gallium/targets/xorg-i915/intel_xorg.c
> index 32d4a54..d8eb258 100644
> --- a/src/gallium/targets/xorg-i915/intel_xorg.c
> +++ b/src/gallium/targets/xorg-i915/intel_xorg.c
> @@ -128,7 +128,6 @@ intel_xorg_pci_probe(DriverPtr driver,
>  	  int entity_num, struct pci_device *device, intptr_t match_data)
>  {
>      ScrnInfoPtr scrn = NULL;
> -    EntityInfoPtr entity;
>  
>      scrn = xf86ConfigPciEntity(scrn, 0, entity_num, intel_xorg_pci_devices,
>  			       NULL, NULL, NULL, NULL, NULL);
> @@ -138,8 +137,6 @@ intel_xorg_pci_probe(DriverPtr driver,
>  	scrn->name = "modesetting";
>  	scrn->Probe = NULL;
>  
> -	entity = xf86GetEntityInfo(entity_num);
> -
>  	/* Use all the functions from the xorg tracker */
>  	xorg_tracker_set_functions(scrn);
>      }
> diff --git a/src/gallium/targets/xorg-nouveau/nouveau_xorg.c b/src/gallium/targets/xorg-nouveau/nouveau_xorg.c
> index 2324d24..454aea6 100644
> --- a/src/gallium/targets/xorg-nouveau/nouveau_xorg.c
> +++ b/src/gallium/targets/xorg-nouveau/nouveau_xorg.c
> @@ -125,7 +125,6 @@ nouveau_xorg_pci_probe(DriverPtr driver,
>  	  int entity_num, struct pci_device *device, intptr_t match_data)
>  {
>      ScrnInfoPtr scrn = NULL;
> -    EntityInfoPtr entity;
>      struct nouveau_device *dev = NULL;
>      char *busid;
>      int chipset, ret;
> @@ -185,8 +184,6 @@ nouveau_xorg_pci_probe(DriverPtr driver,
>  	scrn->name = "nouveau2";
>  	scrn->Probe = NULL;
>  
> -	entity = xf86GetEntityInfo(entity_num);
> -
>  	/* Use all the functions from the xorg tracker */
>  	xorg_tracker_set_functions(scrn);
>      }
> -- 
> 1.8.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list