On Mon, 2008-11-17 at 16:33 +0800, [EMAIL PROTECTED] wrote: > From: MaLing<[EMAIL PROTECTED]>
> +struct hdmi {
> + Uchar Support_flags;
> + Uchar Max_TMDS_Clock;
> + Uchar Latency_Present;
> + Uchar Video_Latency;
> + Uchar Audio_Latency;
> + Uchar Interlaced_Video_Latency;
> + Uchar Interlaced_Audio_Latency;
> +};
Please, CamelCase or underscore_separated_names, but Not_Both.
> @@ -286,11 +381,11 @@ get_std_timing_section(Uchar *c, struct std_timings *r,
>
> static void
> get_dt_md_section(Uchar *c, struct edid_version *ver,
> - struct detailed_monitor_section *det_mon)
> + struct detailed_monitor_section *det_mon, int det_mon_num)
> {
> int i;
>
> - for (i=0;i<DET_TIMINGS;i++) {
> + for (i=0;i<det_mon_num;i++) {
> if (ver->version == 1 && ver->revision >= 1 && IS_MONITOR_DESC) {
>
> switch (MONITOR_DESC_TYPE) {
This just walks over the base block, walking from 0 to (det_mon_num-1)
is wrong since det_mon_num will count detailed blocks found in extension
blocks. Should just drop the changes to this function.
- ajax
signature.asc
Description: This is a digitally signed message part
_______________________________________________ xorg mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/xorg
