[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/5] hw/i2c: pmbus: add vout mode bitfields
From: |
Corey Minyard |
Subject: |
Re: [PATCH 2/5] hw/i2c: pmbus: add vout mode bitfields |
Date: |
Thu, 30 Mar 2023 11:20:10 -0500 |
I almost never say this, as patches are usually too large :), but it
would be nice if you combined this with the patch that uses the
structure so we can see what it's used for. Especially since that patch
is several patches down the line.
-corey
On Wed, Mar 22, 2023 at 05:55:10PM +0000, Titus Rwantare wrote:
> Reviewed-by: Hao Wu <wuhaotsh@google.com>
> Signed-off-by: Titus Rwantare <titusr@google.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.rc1.284.g88254d51c5-goog
>
- [PATCH 0/5] PMBus fixes and new functions, Titus Rwantare, 2023/03/22
- [PATCH 1/5] hw/i2c: pmbus add support for block receive, Titus Rwantare, 2023/03/22
- [PATCH 2/5] hw/i2c: pmbus: add vout mode bitfields, Titus Rwantare, 2023/03/22
- Re: [PATCH 2/5] hw/i2c: pmbus: add vout mode bitfields,
Corey Minyard <=
- [PATCH 3/5] hw/i2c: pmbus: add fan support, Titus Rwantare, 2023/03/22
- [PATCH 4/5] hw/i2c: pmbus: block uninitialised string reads, Titus Rwantare, 2023/03/22
- [PATCH 5/5] hw/i2c: pmbus: add VCAP register, Titus Rwantare, 2023/03/22
- Re: [PATCH 0/5] PMBus fixes and new functions, Philippe Mathieu-Daudé, 2023/03/30