[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:25:42 -0500 |
On Thu, Mar 30, 2023 at 11:20:11AM -0500, Corey Minyard wrote:
> 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.
Actually, in re-reviewing, I don't see this used at all. Is there
something I'm missing?
>
> -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
- [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