[coreboot] [PATCH] Remove warnings for s2892

Myles Watson mylesgw at gmail.com
Fri May 1 02:11:51 CEST 2009



> -----Original Message-----
> From: coreboot-bounces at coreboot.org [mailto:coreboot-bounces at coreboot.org]
> On Behalf Of Peter Stuge
> Sent: Thursday, April 30, 2009 5:47 PM
> To: coreboot at coreboot.org
> Subject: Re: [coreboot] [PATCH] Remove warnings for s2892
> 
> Myles Watson wrote:
> > +++ cbv2/src/pc80/keyboard.c	2009-04-30 15:47:16.000000000 -0600
> > @@ -112,11 +112,11 @@
> >  		outb(0x60, 0x64);
> >  		if (!kbc_input_buffer_empty()) return;
> >  		outb(0x20, 0x60);	/* send cmd: enable keyboard and IRQ
1 */
> > -		u8 resend = 10;
> > +		u8 broken_resend = 10;
> >  		if ((inb(0x64) & 0x01)) {
> >  			regval = inb(0x60);
> >  		}
> > -		--resend;
> > +		--broken_resend;
> >  	} while (regval == 0xFE && resend > 0);
> 
> This does not look right to me. I think the good fix is to not rename
> things but only remove the u8 (re)declaration on line 115. 
I didn't do that because it would change the behavior and I wouldn't be able
to tell breakage by building.  I don't know enough about keyboard
controllers to test this part of the code.

> Very good
> find!
Luckily the compiler did most of the work :)

Thanks,
Myles






More information about the coreboot mailing list