On Thu, May 17, 2007 at 03:48:45PM -0400, Corey Osgood wrote:
> > Nope, please use spd_read_byte() everywhere.
> 
> spd_read_byte was removed to try an use a few less registers,

Rather use macros then, but both those functions should probably be
inline because they are so simple.


> >> +  for(i = 0; i < 4; i++)
> >> +  {
> >> +          spd_data = smbus_read_byte(ctrl->channel0[i], 126);
> >> +          if(spd_data == 0x66)
> >> +          {
> >> +                  i = 6; //just to make sure
> > 
> > Huh?
> 
> To make sure the loop is broken, I wasn't sure if break (which
> should be the next line) would break out of the if statement or the
> for loop,

break exits for(), while() and switch() but not if().


//Peter

-- 
linuxbios mailing list
linuxbios@linuxbios.org
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to