[PATCH] Add Option "TearFree"

[Richard Wilbur]

On Mon, Apr 20, 2015 at 3:54 AM, Michel Dänzer <michel at daenzer.net> wrote:
[...]
> diff --git a/src/radeon_kms.c b/src/radeon_kms.c
> index 64593ab..e18f85a 100644
> --- a/src/radeon_kms.c
> +++ b/src/radeon_kms.c
> @@ -1092,15 +1157,22 @@ Bool RADEONPreInit_KMS(ScrnInfoPtr pScrn, int flags)
>      }
>  #endif
>
> +    info->tear_free = xf86ReturnOptValBool(info->Options, OPTION_TEAR_FREE,
> +                                          FALSE);
> +
> +    if (info->shadow_primary)
> +       xf86DrvMsg(pScrn->scrnIndex, X_CONFIG, "TearFree enabled\n");
> +

It seems likely that I don't understand the full implications of
info->tear_free and info->shadow_primary.  I would have expected the
message about "TearFree enabled" to be predicated on info->tear_free,
since that is what we are adding with this patch, as opposed to
info->shadow_primary, which doesn't seem to be touched by the new
option.

Sincerely,
Richard