Re: [PATCH 6/7] [RFC] ds2760 battery driver

[Matt Reimer]

On 4/15/07, Pavel Machek <pavel@xxxxxx> wrote:
> > +     di->update_time = jiffies;
> > +
> > +     /* DS2760 reports voltage in units of 4.88mV, but the battery class
> > +      * reports in units of mV, so convert by multiplying by 4.875.
> > +      * We approximate because integer math is cheap, and close enough. */
> > +     di->voltage_raw = (di->raw[DS2760_VOLTAGE_MSB] << 3) |
> > +                       (di->raw[DS2760_VOLTAGE_LSB] >> 5);
> > +     di->voltage_mV = (di->voltage_raw * 5) - (di->voltage_raw / 8);
>
> Hmm, not sure if such tricks re really worth it... should not compiler
> be doing this?

The shifts (<< 3 and >> 5) are just to get the bits reassembled in the
right positions. The multiplication by 5 and subtracting 1/8 is
because (AFAIK) we can't do floating point multiplication in the
kernel. I'm open to suggestions.

> > +     /* Calculate the empty level at the present temperature. */
> > +     scale[4] = di->raw[DS2760_ACTIVE_EMPTY + 4];
> > +     for (i = 3; i >= 0; i--)
> > +             scale[i] = scale[i + 1] + di->raw[DS2760_ACTIVE_EMPTY + i];
> > +
> > +     di->empty_mAh = battery_interpolate(scale, di->temp_C / 10);
>
> Wow.

Yeah, mea culpa. I don't like its obtuseness either, and haven't
revisited it since I got it working. Basically the battery has an
array stored in its EEPROM that represents the empty level at various
temperature thresholds. What makes this complicated is that the array
does not contain all absolute values; just the first is absolute, and
the others are relative to it, in reverse order. The above code
converts it to an array of absolute values from low to high. I'm open
to suggestions about this one too.

Matt
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/