diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-10-17 09:21:06 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-10-17 09:24:22 +0100 |
| commit | 33133a1d38a4dce60d2e135663b545895fc85abc (patch) | |
| tree | 970583681653d676a202f11614c413b492fb6686 /src/intel_display.c | |
| parent | 8507ef50ba08061ac158848a7c4c136e1755d87e (diff) | |
display: do not report failure for setting unrecognised properties
We need to accept any changes to properties not handled by ourselves -- we
can't validate the changes ourselves. Denying those changes breaks EDID
reporting, for example.
Reported-by: Elvis Pranskevichus <el@prans.net>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'src/intel_display.c')
| -rw-r--r-- | src/intel_display.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/intel_display.c b/src/intel_display.c index 74ce7362..d32224ec 100644 --- a/src/intel_display.c +++ b/src/intel_display.c @@ -1161,7 +1161,11 @@ intel_output_set_property(xf86OutputPtr output, Atom property, } } - return FALSE; + /* We didn't recognise this property, just report success in order + * to allow the set to continue, otherwise we break setting of + * common properties like EDID. + */ + return TRUE; } static Bool |