diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-23 11:17:33 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-23 12:13:51 -0400 |
commit | cf9a2f3afc75d41a61cbc91e725b9ce0746c400a (patch) | |
tree | cf8fe7cc483b5f301f4d691260a0f6fa6ece3f6e /drivers/gpu/drm/i915 | |
parent | 123d5c0197d8333c3f5cb9572a0c8299dc611233 (diff) |
drm/i915/sdvo: Handle unsupported GET_SUPPORTED_ENHANCEMENTS gracefully
In the event that the external chipset doesn't implement the
GET_SUPPORTED_ENHANCEMENTS commands, gracefully treat it as having no
enhancments rather than bailing.
Reported-and-tested-by: Woody Suwalski <terraluna977@gmail.com>
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=18342
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/i915')
-rw-r--r-- | drivers/gpu/drm/i915/intel_sdvo.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c index 60fcef700be6..ee73e428a84a 100644 --- a/drivers/gpu/drm/i915/intel_sdvo.c +++ b/drivers/gpu/drm/i915/intel_sdvo.c | |||
@@ -2520,11 +2520,10 @@ static bool intel_sdvo_create_enhance_property(struct intel_sdvo *intel_sdvo, | |||
2520 | uint16_t response; | 2520 | uint16_t response; |
2521 | } enhancements; | 2521 | } enhancements; |
2522 | 2522 | ||
2523 | if (!intel_sdvo_get_value(intel_sdvo, | 2523 | enhancements.response = 0; |
2524 | SDVO_CMD_GET_SUPPORTED_ENHANCEMENTS, | 2524 | intel_sdvo_get_value(intel_sdvo, |
2525 | &enhancements, sizeof(enhancements))) | 2525 | SDVO_CMD_GET_SUPPORTED_ENHANCEMENTS, |
2526 | return false; | 2526 | &enhancements, sizeof(enhancements)); |
2527 | |||
2528 | if (enhancements.response == 0) { | 2527 | if (enhancements.response == 0) { |
2529 | DRM_DEBUG_KMS("No enhancement is supported\n"); | 2528 | DRM_DEBUG_KMS("No enhancement is supported\n"); |
2530 | return true; | 2529 | return true; |