On Tue, 7 Jul 2026 at 00:01, Titus Rwantare <[email protected]> wrote:
>
> Signed-off-by: Titus Rwantare <[email protected]>
> ---
> hw/i2c/pmbus_device.c | 12 +++++++++
> include/hw/i2c/pmbus_device.h | 46 +++++++++++++++++++++++++++++++++++
> 2 files changed, 58 insertions(+)
> +uint16_t pmbus_data2linear11(uint16_t value, int exp)
> +{
> + return (uint16_t)((exp & 0x1F) << 11 |
> + (pmbus_data2linear_mode(value, exp) & 0x7FF));
> +}
> +
> uint16_t pmbus_milliunits2linear_mode(uint32_t value, int exp)
> {
> uint32_t ret;
> @@ -58,6 +64,12 @@ uint16_t pmbus_milliunits2linear_mode(uint32_t value, int
> exp)
> return ret;
> }
>
> +uint16_t pmbus_milliunits2linear11(uint32_t value, int exp)
> +{
> + return (uint16_t)((exp & 0x1F) << 11 |
> + (pmbus_milliunits2linear_mode(value, exp) & 0x7FF));
> +}
Nothing calls these functions, and they are the only users of
the new functions added in patch 2. We should add these when
we have some actual callers, otherwise this is all dead code.
thanks
-- PMM
