Re: [PATCH libdrm] tests: remove missleading comments

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

 



On 12/18/2015 12:14 AM, Stefan Agner wrote:
> The comment has been copied from modetest and is not applicable
> for vbltest.
> 
> Signed-off-by: Stefan Agner <stefan@xxxxxxxx>
> ---
>  tests/vbltest/vbltest.c | 15 +--------------
>  1 file changed, 1 insertion(+), 14 deletions(-)
> 
> diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
> index 531196c..5c61b43 100644
> --- a/tests/vbltest/vbltest.c
> +++ b/tests/vbltest/vbltest.c
> @@ -1,5 +1,5 @@
>  /*
> - * DRM based mode setting test program
> + * DRM based vblank test program
>   * Copyright 2008 Tungsten Graphics
>   *   Jakob Bornecrantz <jakob@xxxxxxxxxxxxxxxxxxxx>
>   * Copyright 2008 Intel Corporation
> @@ -24,19 +24,6 @@
>   * IN THE SOFTWARE.
>   */
>  
> -/*
> - * This fairly simple test program dumps output in a similar format to the
> - * "xrandr" tool everyone knows & loves.  It's necessarily slightly different
> - * since the kernel separates outputs into encoder and connector structures,
> - * each with their own unique ID.  The program also allows test testing of the
> - * memory management and mode setting APIs by allowing the user to specify a
> - * connector and mode to use for mode setting.  If all works as expected, a
> - * blue background should be painted on the monitor attached to the specified
> - * connector after the selected mode is set.
> - *
> - * TODO: use cairo to write the mode info on the selected output once
> - *       the mode has been programmed, along with possible test patterns.
> - */
>  #ifdef HAVE_CONFIG_H
>  #include "config.h"
>  #endif

Yep, thanks.

Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux