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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f383264b0704161214x168b36f4tdd9a074d604b36cb@mail.gmail.com>
Date:	Mon, 16 Apr 2007 12:14:27 -0700
From:	"Matt Reimer" <mattjreimer@...il.com>
To:	"Pavel Machek" <pavel@....cz>
Cc:	"Anton Vorontsov" <cbou@...l.ru>, linux-kernel@...r.kernel.org,
	kernel-discuss@...dhelds.org, dwmw2@...radead.org
Subject: Re: [PATCH 6/7] [RFC] ds2760 battery driver

On 4/15/07, Pavel Machek <pavel@....cz> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/