diff options
| author | Peter Delevoryas <pdel@fb.com> | 2022-07-14 16:24:38 +0200 |
|---|---|---|
| committer | Cédric Le Goater <clg@kaod.org> | 2022-07-14 16:24:38 +0200 |
| commit | e51ae82571746e40f36aa9bfc5d0924dcf2b0a5d (patch) | |
| tree | cedffb1c5e15c1b9265af6840ac1fe2bb728b868 /hw/sensor | |
| parent | d272d1410c91f399af15ef9ae4b561bc621de115 (diff) | |
| download | qemu-e51ae82571746e40f36aa9bfc5d0924dcf2b0a5d.zip qemu-e51ae82571746e40f36aa9bfc5d0924dcf2b0a5d.tar.gz qemu-e51ae82571746e40f36aa9bfc5d0924dcf2b0a5d.tar.bz2 | |
hw/sensor: Add IC_DEVICE_ID to ISL voltage regulators
This commit adds a passthrough for PMBUS_IC_DEVICE_ID to allow Renesas
voltage regulators to return the integrated circuit device ID if they
would like to.
The behavior is very device specific, so it hasn't been added to the
general PMBUS model. Additionally, if the device ID hasn't been set,
then the voltage regulator will respond with the error byte value. The
guest error message will change slightly for IC_DEVICE_ID with this
commit.
Signed-off-by: Peter Delevoryas <pdel@fb.com>
Reviewed-by: Titus Rwantare <titusr@google.com>
Message-Id: <20220701000626.77395-3-me@pjd.dev>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
Diffstat (limited to 'hw/sensor')
| -rw-r--r-- | hw/sensor/isl_pmbus_vr.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/hw/sensor/isl_pmbus_vr.c b/hw/sensor/isl_pmbus_vr.c index e11e028..799ea9d 100644 --- a/hw/sensor/isl_pmbus_vr.c +++ b/hw/sensor/isl_pmbus_vr.c @@ -15,6 +15,18 @@ static uint8_t isl_pmbus_vr_read_byte(PMBusDevice *pmdev) { + ISLState *s = ISL69260(pmdev); + + switch (pmdev->code) { + case PMBUS_IC_DEVICE_ID: + if (!s->ic_device_id_len) { + break; + } + pmbus_send(pmdev, s->ic_device_id, s->ic_device_id_len); + pmbus_idle(pmdev); + return 0; + } + qemu_log_mask(LOG_GUEST_ERROR, "%s: reading from unsupported register: 0x%02x\n", __func__, pmdev->code); |