On Fri, Mar 31, 2023 at 12:07:51AM +0000, Titus Rwantare wrote:
> The VOUT_MODE command is described in the PMBus Specification,
> Part II, Ver 1.3 Section 8.3
>
> VOUT_MODE has a three bit mode and 4 bit parameter, the three bit
> mode determines whether voltages are formatted as uint16, uint16,
> VID, and Direct modes. VID and Direct modes use the remaining 5 bits
> to scale the voltage readings.
>
> Reviewed-by: Hao Wu <wuhaotsh@google.com>
> Signed-off-by: Titus Rwantare <titusr@google.com>
Ok, I see the new sensor later.
Acked-by: Corey Minyard <cminyard@mvista.com>
> ---
> include/hw/i2c/pmbus_device.h | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/include/hw/i2c/pmbus_device.h b/include/hw/i2c/pmbus_device.h
> index 7dc00cc4d9..2e95164aa1 100644
> --- a/include/hw/i2c/pmbus_device.h
> +++ b/include/hw/i2c/pmbus_device.h
> @@ -444,6 +444,14 @@ typedef struct PMBusCoefficients {
> int32_t R; /* exponent */
> } PMBusCoefficients;
>
> +/**
> + * VOUT_Mode bit fields
> + */
> +typedef struct PMBusVoutMode {
> + uint8_t mode:3;
> + int8_t exp:5;
> +} PMBusVoutMode;
> +
> /**
> * Convert sensor values to direct mode format
> *
> --
> 2.40.0.423.gd6c402a77b-goog
>