monitor-parse-edid: add support for standard timing descriptors in monitor descriptors

2 files changed, 4 insertions, 1 deletions

diff --git a/NEWS b/NEWS
index 9e3576c..2b699c5 100644
--- a/NEWS
+++ b/NEWS
@@ -20,6 +20,7 @@
     Descriptor specifies field vertical parameters instead of the frame
     parameters (at least Fujitsu-Siemens Myrica VQ32-1T testcase)
   o ignore 1x1 modes (at least Nokia Valuegraph 447w testcase)
+  o add support for standard timing descriptors in monitor descriptors
 
 Version 2.5 - 18 October 2009, by Anssi Hannula
 
diff --git a/monitor-parse-edid b/monitor-parse-edid
index dacc8ad..a8e9e74 100755
--- a/monitor-parse-edid
+++ b/monitor-parse-edid
@@ -680,7 +680,7 @@ sub add_standard_timing_modes {
 	    }
 	    $h;
 	} else { () }
-    } unpack('a2' x 8, $v) ];
+    } unpack('a2' x (length($v) / 2), $v) ];
     $v;
 }
 
@@ -761,6 +761,8 @@ sub parse_edid {
 			}
 			$edid{$valid ? 'monitor_range' : 'manufacturer_specified_range_timing'} = $range;
 
+		    } elsif ($flag == 0xfa) {
+			push @{$edid{standard_timings}}, add_standard_timing_modes(\%edid, unpack('a12', $vv));
 		    } elsif ($flag == 0xfc) {
 			my $prev = $edid{monitor_name};
 			$edid{monitor_name} = ($prev ? "$prev " : '') . unpack('A13', $vv);