APM Patch - review

[Nate Lawson]

On Thu, 27 May 2004, Liam J. Foy wrote:

> On Wed, 26 May 2004 22:08:26 -0700 (PDT)
> Nate Lawson <nate at root.org> wrote:
>
> > On Tue, 25 May 2004, Grover, Andrew wrote:
> > > Valid values for remaining battery life are 0xFF, and 0-100.
> > >
> > > -1, 255 and 0xFF are the SAME when looking at an 8 bit value. Since this
> > > 8 bit value was sign-extended to 32 bits to go into ai_batt_time, if it
> > > was 0xFF, it is now 0xFFFFFFFF, and you will never see 0x000000FF.
> >
> > Correct.  Liam, can you add a printf for the unknown case (%x) that gives
> > the actual value?
>
> Nate, to do you mean add a printf to the unknown case to print the value instead
> of have "invalud value (%x)" etc?

Oh, I missed that part where you said you get "-1" printed for the
"invalid value (%x) case.

-Nate