Messages in this thread | | | From | (Dick Streefland) | Subject | Re: Linux 2.6.27.27 | Date | Wed, 22 Jul 2009 10:33:01 -0000 |
| |
Krzysztof Oledzki <olel@ans.pl> wrote: | Here is a diff between a good and a bad kernel: | | -edid_checksum debug: csum=0, all_null=255, err=1 | -edid_checksum debug: csum=0, all_null=255, err=1 | -Console: switching to colour frame buffer device 128x48 | +edid_checksum debug: csum=6400, all_null=255, err=0 | +Console: switching to colour frame buffer device 80x30 | | In the good one the function is called twice and it returns err=1 (==OK). | In the bad kernel it returns 0 because csum!=0x00 (==6400).
Linus accidentally dropped the "char" from the declaration of "csum" and "all_null":
> - unsigned char i, csum = 0, all_null = 0; > - int err = 0, fix = check_edid(edid); > + unsigned csum = 0, all_null = 0; > + int i, err = 0, fix = check_edid(edid);
-- Dick Streefland //// Altium BV dick.streefland@altium.nl (@ @) http://www.altium.com --------------------------------oOO--(_)--OOo---------------------------
| |