* [PATCH 5/7] ACPI video: check cap._DDC flag before getting EDID
@ 2010-12-06 7:04 Zhang Rui
2010-12-06 12:53 ` Matthew Garrett
0 siblings, 1 reply; 2+ messages in thread
From: Zhang Rui @ 2010-12-06 7:04 UTC (permalink / raw)
To: Brown, Len; +Cc: linux-acpi@vger.kernel.org, Matthew Garrett, Zhang, Rui
cap._DDC is defined but never used.
Check this flag now and don't try to get EDID for video output devices with this flag cleared.
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
---
drivers/acpi/video.c | 3 +++
1 file changed, 3 insertions(+)
Index: linux-2.6/drivers/acpi/video.c
===================================================================
--- linux-2.6.orig/drivers/acpi/video.c
+++ linux-2.6/drivers/acpi/video.c
@@ -1275,6 +1275,9 @@ int acpi_video_get_edid(struct acpi_devi
if (!video_device)
continue;
+ if (!video_device->cap._DDC)
+ continue;
+
if (type) {
switch (type) {
case ACPI_VIDEO_DISPLAY_CRT:
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 5/7] ACPI video: check cap._DDC flag before getting EDID
2010-12-06 7:04 [PATCH 5/7] ACPI video: check cap._DDC flag before getting EDID Zhang Rui
@ 2010-12-06 12:53 ` Matthew Garrett
0 siblings, 0 replies; 2+ messages in thread
From: Matthew Garrett @ 2010-12-06 12:53 UTC (permalink / raw)
To: Zhang Rui; +Cc: Brown, Len, linux-acpi@vger.kernel.org
On Mon, Dec 06, 2010 at 03:04:24PM +0800, Zhang Rui wrote:
>
> cap._DDC is defined but never used.
> Check this flag now and don't try to get EDID for video output devices with this flag cleared.
>
> Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Acked-by: Matthew Garrett <mjg@redhat.com>
--
Matthew Garrett | mjg59@srcf.ucam.org
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-12-06 12:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-06 7:04 [PATCH 5/7] ACPI video: check cap._DDC flag before getting EDID Zhang Rui
2010-12-06 12:53 ` Matthew Garrett
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox