Re: [PATCH 1/2] Make output name numbering 0-based

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Title should be "Make output name numbering 1-based"
> 
> The QXL driver names its outputs starting at 0 (e.g. Virtual-0,
> Virtual-1, etc). This code was presumably copy/pasted from a different
> driver, and is not necessary for the QXL driver. Other drivers simply
> use the kernel connector_type_id which starts at 1. For example, the
> modesetting driver changed from 1-based names to 1-based names for the

maybe " ... from 0-based names to 1-based names ....", from 1 to 1 does
not change much.

> same reason in xserver commit 139e36dd.
> 
> This will help to make it easier to identify which xrandr outputs belong
> to which drm connector without requiring as many driver-specific
> special-cases.
> 
> This change might effect custom xorg configurations that references a
> specific output name. But the same change was made in modesetting driver
> despite that possibility.
> 
> Signed-off-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
> Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

yes, acked.

> ---
>  src/qxl_drmmode.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/qxl_drmmode.c b/src/qxl_drmmode.c
> index a2f84b1..a814859 100644
> --- a/src/qxl_drmmode.c
> +++ b/src/qxl_drmmode.c
> @@ -765,8 +765,7 @@ drmmode_output_init(ScrnInfoPtr pScrn, drmmode_ptr
> drmmode, int num)
>  		}
>  	}
>  
> -	/* need to do smart conversion here for compat with non-kms ATI driver */
> -	snprintf(name, 32, "%s-%d", output_names[koutput->connector_type],
> koutput->connector_type_id - 1);
> +	snprintf(name, 32, "%s-%d", output_names[koutput->connector_type],
> koutput->connector_type_id);
>  	
>  
>  	output = xf86OutputCreate (pScrn, &drmmode_output_funcs, name);

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]