From nobody Mon Sep 8 09:48:03 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=reject dis=none) header.from=google.com ARC-Seal: i=1; a=rsa-sha256; t=1756850060; cv=none; d=zohomail.com; s=zohoarc; b=MI+RJausCKOn/M4vaUscRxOAx5F8WYu3eZygX+Q0IlAQ/Iftu4TAaabpnTwFUBuhUXeezkMzNIxoGOwOfUwtP9PBR7JY3kQI2SVSoX5yLJa3UmyjYdH91+yqE6S2jFGKeGAdBqDyT/+bEpmYQmxKgMrIBANAlNy0c5p+sZkMHK8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1756850060; h=Content-Type:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=BF63uFroTBuY4N1oFo9+Vq9UpGx1OE+9wgUSgn3GUyg=; b=nu7e67/9fada1nW93JKICweL1bCgVaS6QKGCd8ne4HX15fsCDkQ5yliPNQIq/PUvu7H6pL8+73kW/YfCVYWhkN4fXuHGRKXQm1rjZexyLstR6hu1Igz3zn2qsxJHn/Dzd+h0M2CN8X44HnSOPjLyz8ZnbEVNPMOw9sUkQI6Usnw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=reject dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1756850060784491.90211385700286; Tue, 2 Sep 2025 14:54:20 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1utYwc-0008Fb-MG; Tue, 02 Sep 2025 17:53:40 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <3U2e3aAwKCkw1opwvs67sto1u22uzs.q204s08-rs9sz121u18.25u@flex--nabihestefan.bounces.google.com>) id 1utYwV-0008Ev-6u for qemu-devel@nongnu.org; Tue, 02 Sep 2025 17:53:31 -0400 Received: from mail-pg1-x54a.google.com ([2607:f8b0:4864:20::54a]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from <3U2e3aAwKCkw1opwvs67sto1u22uzs.q204s08-rs9sz121u18.25u@flex--nabihestefan.bounces.google.com>) id 1utYwS-0005Zg-H0 for qemu-devel@nongnu.org; Tue, 02 Sep 2025 17:53:30 -0400 Received: by mail-pg1-x54a.google.com with SMTP id 41be03b00d2f7-b4d3ab49a66so6070571a12.3 for ; Tue, 02 Sep 2025 14:53:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1756850004; x=1757454804; darn=nongnu.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=BF63uFroTBuY4N1oFo9+Vq9UpGx1OE+9wgUSgn3GUyg=; b=rJ9T56OgY7SXI275OkvF+viACF3l8E9YuCV2iMenz+65o6d/YOifPjpqVy3HMqf0tl wEBPaV0Z0UU8p2bPese51Td1ji4yVYqscYl5MnmnbXAgie3+5+EfEwt+3BbIYZ8xwn26 6nFtrKj4rO2IKBspaxWoqsF7+yXL1rPxi3BnIgJ5S1COD94HdYU39C3+ks9dWFsW16jZ cIazmMFBiQqtGQakrosfDPpA9VLCYLGzkb/7yEGF8aTK/vFxOPxh+A6Xbfe9hSqmQ+m/ 8Tdl2U6l92+aXSPcSe89FrNEWdFQfBGc6nNKgnitpsl0mkmIGxA7xv/49tWAaIM0SHG6 iSGA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756850004; x=1757454804; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=BF63uFroTBuY4N1oFo9+Vq9UpGx1OE+9wgUSgn3GUyg=; b=GR+PdRgu5TqhJKu2BTUcbqQILKxnp41JAPUciS18I/d8tRVPEmkqzVW4xfZdqds743 inT8XGHYL1556pK+N3FKW978Rp+rBdCuTq1lh5ddmG6aFVWoBkVhsd+MDRwpStjTpiC1 qFemu76NwYj3lHu6n9tX/EhTIvAnK/VdG2ZO/Lf7MRbsspNTlfrPcnM3tRfO4fgoMmhl k7zqQmBAwcXmXMd1W0ZEQTExUT7/HazipaVquWzIpsMKpJU3I876II13cLXbfJM6qWrL Uv/wTP49Nw/OixA0yRf38ifDDZTy6fzArBJmxDm0rr/YA3S7QO++9b8QGr+8OtbGP2zx Gq5Q== X-Gm-Message-State: AOJu0YyjKD/0y7gRuKWR1bG7imJF3YmwLbehY5x/yzH9B/B9t9dGwEbq DZc2tkV/gwf1k35SCxGnKjAKcdKmmbJJH12G9OKqwwXRvUJhz0LkMuvth23ICgObRU1dTTBbAc4 FJJmsb24CVKbiHK9/FiKvTApXpRXiDWEkxSX5QgPt7EvdUh7B6opoDDD4aLC5s1C4gx+Sp9FvWS 2QkdVZA3KC1ABgJP2mTC+kDP2Om75ql5OIIwUcok1nEGskMjH+TxDfflUEv1f7iPRD9vA= X-Google-Smtp-Source: AGHT+IFtaMhs1/2TxyWQyYZ0P1vUEAhY8mBp0biU5ka987EDPUJUfAL+SpB6bULlfjHVsve53ELBh4pu5KMbm8D5r4E= X-Received: from pfuj14.prod.google.com ([2002:a05:6a00:130e:b0:772:2a20:b41a]) (user=nabihestefan job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a21:999c:b0:243:a27e:598a with SMTP id adf61e73a8af0-243d6e0084bmr17495209637.13.1756850003654; Tue, 02 Sep 2025 14:53:23 -0700 (PDT) Date: Tue, 2 Sep 2025 21:53:20 +0000 Mime-Version: 1.0 X-Mailer: git-send-email 2.51.0.355.g5224444f11-goog Message-ID: <20250902215320.1455816-1-nabihestefan@google.com> Subject: [PATCH] hw/sensor: added MAX16600 device model From: Nabih Estefan To: qemu-devel@nongnu.org Cc: pbonzini@redhat.com, peter.maydell@linaro.org, philmd@linaro.org, farosas@suse.de, lvivier@redhat.com, qemu-arm@nongnu.org, Shengtan Mao , Titus Rwantare , Nabih Estefan Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2607:f8b0:4864:20::54a; envelope-from=3U2e3aAwKCkw1opwvs67sto1u22uzs.q204s08-rs9sz121u18.25u@flex--nabihestefan.bounces.google.com; helo=mail-pg1-x54a.google.com X-Spam_score_int: -94 X-Spam_score: -9.5 X-Spam_bar: --------- X-Spam_report: (-9.5 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_MED=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FUZZY_MILLION=0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @google.com) X-ZM-MESSAGEID: 1756850063054124100 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Shengtan Mao Change-Id: I5947e071fc27fd0d01c2d365545a81fda8131d32 Signed-off-by: Shengtan Mao Signed-off-by: Titus Rwantare Signed-off-by: Nabih Estefan --- hw/arm/Kconfig | 1 + hw/sensor/Kconfig | 4 + hw/sensor/max16600.c | 196 ++++++++++++++++++++++++++++ hw/sensor/meson.build | 1 + include/hw/sensor/max16600.h | 45 +++++++ tests/qtest/max16600-test.c | 240 +++++++++++++++++++++++++++++++++++ tests/qtest/meson.build | 1 + 7 files changed, 488 insertions(+) create mode 100644 hw/sensor/max16600.c create mode 100644 include/hw/sensor/max16600.h create mode 100644 tests/qtest/max16600-test.c diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig index 2aa4b5d778..4ab0a93ba6 100644 --- a/hw/arm/Kconfig +++ b/hw/arm/Kconfig @@ -480,6 +480,7 @@ config NPCM7XX select AT24C # EEPROM select MAX34451 select ISL_PMBUS_VR + select MAX_16600 select PL310 # cache controller select PMBUS select SERIAL_MM diff --git a/hw/sensor/Kconfig b/hw/sensor/Kconfig index bc6331b4ab..ef7b3262a8 100644 --- a/hw/sensor/Kconfig +++ b/hw/sensor/Kconfig @@ -43,3 +43,7 @@ config ISL_PMBUS_VR config MAX31785 bool depends on PMBUS + +config MAX_16600 + bool + depends on I2C diff --git a/hw/sensor/max16600.c b/hw/sensor/max16600.c new file mode 100644 index 0000000000..0aaab436e0 --- /dev/null +++ b/hw/sensor/max16600.c @@ -0,0 +1,196 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * MAX16600 VR13.HC Dual-Output Voltage Regulator Chipset + * + * Copyright 2021 Google LLC + */ + +#include "qemu/osdep.h" +#include "hw/i2c/pmbus_device.h" +#include "qapi/visitor.h" +#include "qemu/log.h" +#include "hw/sensor/max16600.h" + +static uint8_t max16600_read_byte(PMBusDevice *pmdev) +{ + MAX16600State *s =3D MAX16600(pmdev); + + switch (pmdev->code) { + case PMBUS_IC_DEVICE_ID: + pmbus_send_string(pmdev, s->ic_device_id); + break; + + case MAX16600_PHASE_ID: + pmbus_send8(pmdev, s->phase_id); + break; + + default: + qemu_log_mask(LOG_GUEST_ERROR, + "%s: reading from unsupported register: 0x%02x\n", + __func__, pmdev->code); + break; + } + return 0xFF; +} + +static int max16600_write_data(PMBusDevice *pmdev, const uint8_t *buf, + uint8_t len) +{ + qemu_log_mask(LOG_GUEST_ERROR, + "%s: write to unsupported register: 0x%02x\n", __func__, + pmdev->code); + return 0xFF; +} + +static void max16600_exit_reset(Object *obj, ResetType type) +{ + PMBusDevice *pmdev =3D PMBUS_DEVICE(obj); + MAX16600State *s =3D MAX16600(obj); + + pmdev->capability =3D MAX16600_CAPABILITY_DEFAULT; + pmdev->page =3D 0; + + pmdev->pages[0].operation =3D MAX16600_OPERATION_DEFAULT; + pmdev->pages[0].on_off_config =3D MAX16600_ON_OFF_CONFIG_DEFAULT; + pmdev->pages[0].vout_mode =3D MAX16600_VOUT_MODE_DEFAULT; + + pmdev->pages[0].read_vin =3D + pmbus_data2linear_mode(MAX16600_READ_VIN_DEFAULT, max16600_exp.vin= ); + pmdev->pages[0].read_iin =3D + pmbus_data2linear_mode(MAX16600_READ_IIN_DEFAULT, max16600_exp.iin= ); + pmdev->pages[0].read_pin =3D + pmbus_data2linear_mode(MAX16600_READ_PIN_DEFAULT, max16600_exp.pin= ); + pmdev->pages[0].read_vout =3D MAX16600_READ_VOUT_DEFAULT; + pmdev->pages[0].read_iout =3D + pmbus_data2linear_mode(MAX16600_READ_IOUT_DEFAULT, max16600_exp.io= ut); + pmdev->pages[0].read_pout =3D + pmbus_data2linear_mode(MAX16600_READ_PIN_DEFAULT, max16600_exp.pou= t); + pmdev->pages[0].read_temperature_1 =3D + pmbus_data2linear_mode(MAX16600_READ_TEMP_DEFAULT, max16600_exp.te= mp); + + s->ic_device_id =3D "MAX16601"; + s->phase_id =3D MAX16600_PHASE_ID_DEFAULT; +} + +static void max16600_get(Object *obj, Visitor *v, const char *name, + void *opaque, Error **errp) +{ + uint16_t value; + + if (strcmp(name, "vin") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .vin); + } else if (strcmp(name, "iin") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .iin); + } else if (strcmp(name, "pin") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .pin); + } else if (strcmp(name, "iout") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .iout); + } else if (strcmp(name, "pout") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .pout); + } else if (strcmp(name, "temperature") =3D=3D 0) { + value =3D pmbus_linear_mode2data(*(uint16_t *)opaque, max16600_exp= .temp); + } else { + value =3D *(uint16_t *)opaque; + } + + /* scale to milli-units */ + if (strcmp(name, "pout") !=3D 0 && strcmp(name, "pin") !=3D 0) { + value *=3D 1000; + } + + visit_type_uint16(v, name, &value, errp); +} + +static void max16600_set(Object *obj, Visitor *v, const char *name, + void *opaque, Error **errp) +{ + PMBusDevice *pmdev =3D PMBUS_DEVICE(obj); + uint16_t *internal =3D opaque; + uint16_t value; + if (!visit_type_uint16(v, name, &value, errp)) { + return; + } + + /* inputs match kernel driver which scales to milliunits except power = */ + if (strcmp(name, "pout") !=3D 0 && strcmp(name, "pin") !=3D 0) { + value /=3D 1000; + } + + if (strcmp(name, "vin") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.vin); + } else if (strcmp(name, "iin") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.iin); + } else if (strcmp(name, "pin") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.pin); + } else if (strcmp(name, "iout") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.iout); + } else if (strcmp(name, "pout") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.pout); + } else if (strcmp(name, "temperature") =3D=3D 0) { + *internal =3D pmbus_data2linear_mode(value, max16600_exp.temp); + } else { + *internal =3D value; + } + + pmbus_check_limits(pmdev); +} + +static void max16600_init(Object *obj) +{ + PMBusDevice *pmdev =3D PMBUS_DEVICE(obj); + uint64_t flags =3D PB_HAS_VOUT_MODE | PB_HAS_VIN | PB_HAS_IIN | PB_HAS= _PIN | + PB_HAS_IOUT | PB_HAS_POUT | PB_HAS_VOUT | + PB_HAS_TEMPERATURE | PB_HAS_MFR_INFO; + pmbus_page_config(pmdev, 0, flags); + + object_property_add(obj, "vin", "uint16", max16600_get, max16600_set, = NULL, + &pmdev->pages[0].read_vin); + + object_property_add(obj, "iin", "uint16", max16600_get, max16600_set, = NULL, + &pmdev->pages[0].read_iin); + + object_property_add(obj, "pin", "uint16", max16600_get, max16600_set, = NULL, + &pmdev->pages[0].read_pin); + + object_property_add(obj, "vout", "uint16", max16600_get, max16600_set, + NULL, &pmdev->pages[0].read_vout); + + object_property_add(obj, "iout", "uint16", max16600_get, max16600_set, + NULL, &pmdev->pages[0].read_iout); + + object_property_add(obj, "pout", "uint16", max16600_get, max16600_set, + NULL, &pmdev->pages[0].read_pout); + + object_property_add(obj, "temperature", "uint16", + max16600_get, max16600_set, + NULL, &pmdev->pages[0].read_temperature_1); +} + +static void max16600_class_init(ObjectClass *klass, const void *data) +{ + ResettableClass *rc =3D RESETTABLE_CLASS(klass); + DeviceClass *dc =3D DEVICE_CLASS(klass); + PMBusDeviceClass *k =3D PMBUS_DEVICE_CLASS(klass); + + dc->desc =3D "MAX16600 Dual-Output Voltage Regulator"; + k->write_data =3D max16600_write_data; + k->receive_byte =3D max16600_read_byte; + k->device_num_pages =3D 1; + + rc->phases.exit =3D max16600_exit_reset; +} + +static const TypeInfo max16600_info =3D { + .name =3D TYPE_MAX16600, + .parent =3D TYPE_PMBUS_DEVICE, + .instance_size =3D sizeof(MAX16600State), + .instance_init =3D max16600_init, + .class_init =3D max16600_class_init, +}; + +static void max16600_register_types(void) +{ + type_register_static(&max16600_info); +} + +type_init(max16600_register_types) diff --git a/hw/sensor/meson.build b/hw/sensor/meson.build index 420fdc3359..85c2c73c99 100644 --- a/hw/sensor/meson.build +++ b/hw/sensor/meson.build @@ -8,3 +8,4 @@ system_ss.add(when: 'CONFIG_MAX34451', if_true: files('max3= 4451.c')) system_ss.add(when: 'CONFIG_LSM303DLHC_MAG', if_true: files('lsm303dlhc_ma= g.c')) system_ss.add(when: 'CONFIG_ISL_PMBUS_VR', if_true: files('isl_pmbus_vr.c'= )) system_ss.add(when: 'CONFIG_MAX31785', if_true: files('max31785.c')) +system_ss.add(when: 'CONFIG_MAX_16600', if_true: files('max16600.c')) diff --git a/include/hw/sensor/max16600.h b/include/hw/sensor/max16600.h new file mode 100644 index 0000000000..c8a1c097d1 --- /dev/null +++ b/include/hw/sensor/max16600.h @@ -0,0 +1,45 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * MAX16600 VR13.HC Dual-Output Voltage Regulator Chipset + * + * Copyright 2021 Google LLC + */ + +#include "hw/i2c/pmbus_device.h" + +#define TYPE_MAX16600 "max16600" +#define MAX16600(obj) OBJECT_CHECK(MAX16600State, (obj), TYPE_MAX16600) + +#define MAX16600_PHASE_ID 0xF3 +/* + * Packet error checking capability is disabled. + * Pending QEMU support + */ +#define MAX16600_CAPABILITY_DEFAULT 0x30 +#define MAX16600_OPERATION_DEFAULT 0x88 +#define MAX16600_ON_OFF_CONFIG_DEFAULT 0x17 +#define MAX16600_VOUT_MODE_DEFAULT 0x22 +#define MAX16600_PHASE_ID_DEFAULT 0x80 + +#define MAX16600_READ_VIN_DEFAULT 5 /* Volts */ +#define MAX16600_READ_IIN_DEFAULT 3 /* Amps */ +#define MAX16600_READ_PIN_DEFAULT 100 /* Watts */ +#define MAX16600_READ_VOUT_DEFAULT 5 /* Volts */ +#define MAX16600_READ_IOUT_DEFAULT 3 /* Amps */ +#define MAX16600_READ_POUT_DEFAULT 100 /* Watts */ +#define MAX16600_READ_TEMP_DEFAULT 40 /* Celsius */ + +typedef struct MAX16600State { + PMBusDevice parent; + const char *ic_device_id; + uint8_t phase_id; +} MAX16600State; + +/* + * determines the exponents used in linear conversion for CORE + * (iin, pin) may be (-4, 0) or (-3, 1) + * iout may be -2, -1, 0, 1 + */ +static const struct { + int vin, iin, pin, iout, pout, temp; +} max16600_exp =3D {-6, -4, 0, -2, -1, 0}; diff --git a/tests/qtest/max16600-test.c b/tests/qtest/max16600-test.c new file mode 100644 index 0000000000..b2058aecad --- /dev/null +++ b/tests/qtest/max16600-test.c @@ -0,0 +1,240 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * QTest for the MAX16600 VR13.HC Dual-Output Voltage Regulator Chipset + * + * Copyright 2021 Google LLC + */ + +#include "qemu/osdep.h" +#include "hw/i2c/pmbus_device.h" +#include "hw/sensor/max16600.h" +#include "libqtest-single.h" +#include "libqos/qgraph.h" +#include "libqos/i2c.h" +#include "qobject/qdict.h" +#include "qobject/qnum.h" +#include "qemu/bitops.h" + +#define TEST_ID "max16600-test" +#define TEST_ADDR (0x61) + +uint16_t pmbus_linear_mode2data(uint16_t value, int exp) +{ + /* D =3D L * 2^e */ + if (exp < 0) { + return value >> (-exp); + } + return value << exp; +} + +static uint16_t qmp_max16600_get(const char *id, const char *property) +{ + QDict *response; + uint64_t ret; + + response =3D qmp("{ 'execute': 'qom-get', 'arguments': { 'path': %s, " + "'property': %s } }", + id, property); + g_assert(qdict_haskey(response, "return")); + ret =3D qnum_get_uint(qobject_to(QNum, qdict_get(response, "return"))); + qobject_unref(response); + return ret; +} + +static void qmp_max16600_set(const char *id, const char *property, + uint16_t value) +{ + QDict *response; + + response =3D qmp("{ 'execute': 'qom-set', 'arguments': { 'path': %s, " + "'property': %s, 'value': %u } }", + id, property, value); + g_assert(qdict_haskey(response, "return")); + qobject_unref(response); +} + +static uint16_t max16600_i2c_get16(QI2CDevice *i2cdev, uint8_t reg) +{ + uint8_t resp[2]; + i2c_read_block(i2cdev, reg, resp, sizeof(resp)); + return (resp[1] << 8) | resp[0]; +} + +static void max16600_i2c_set16(QI2CDevice *i2cdev, uint8_t reg, uint16_t v= alue) +{ + uint8_t data[2]; + + data[0] =3D value & 255; + data[1] =3D value >> 8; + i2c_write_block(i2cdev, reg, data, sizeof(data)); +} + +/* test default values */ +static void test_defaults(void *obj, void *data, QGuestAllocator *alloc) +{ + uint16_t i2c_value, value; + QI2CDevice *i2cdev =3D (QI2CDevice *)obj; + + i2c_value =3D i2c_get8(i2cdev, PMBUS_CAPABILITY); + g_assert_cmphex(i2c_value, =3D=3D, MAX16600_CAPABILITY_DEFAULT); + + i2c_value =3D i2c_get8(i2cdev, PMBUS_OPERATION); + g_assert_cmphex(i2c_value, =3D=3D, MAX16600_OPERATION_DEFAULT); + + i2c_value =3D i2c_get8(i2cdev, PMBUS_ON_OFF_CONFIG); + g_assert_cmphex(i2c_value, =3D=3D, MAX16600_ON_OFF_CONFIG_DEFAULT); + + i2c_value =3D i2c_get8(i2cdev, PMBUS_VOUT_MODE); + g_assert_cmphex(i2c_value, =3D=3D, MAX16600_VOUT_MODE_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "vin") / 1000; + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_VIN_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "iin") / 1000; + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_IIN_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "pin"); + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_PIN_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "vout") / 1000; + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_VOUT_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "iout") / 1000; + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_IOUT_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "pout"); + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_POUT_DEFAULT); + + value =3D qmp_max16600_get(TEST_ID, "temperature") / 1000; + g_assert_cmpuint(value, =3D=3D, MAX16600_READ_TEMP_DEFAULT); +} + +/* test qmp access */ +static void test_tx_rx(void *obj, void *data, QGuestAllocator *alloc) +{ + uint16_t value, i2c_value; + QI2CDevice *i2cdev =3D (QI2CDevice *)obj; + + qmp_max16600_set(TEST_ID, "vin", 2000); + value =3D qmp_max16600_get(TEST_ID, "vin") / 1000; + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VIN); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.vin); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "iin", 3000); + value =3D qmp_max16600_get(TEST_ID, "iin") / 1000; + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IIN); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.iin); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "pin", 4); + value =3D qmp_max16600_get(TEST_ID, "pin"); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_PIN); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.pin); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "vout", 5000); + value =3D qmp_max16600_get(TEST_ID, "vout") / 1000; + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VOUT); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "iout", 6000); + value =3D qmp_max16600_get(TEST_ID, "iout") / 1000; + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IOUT); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.iout); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "pout", 7); + value =3D qmp_max16600_get(TEST_ID, "pout"); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_POUT); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.pout); + g_assert_cmpuint(value, =3D=3D, i2c_value); + + qmp_max16600_set(TEST_ID, "temperature", 8000); + value =3D qmp_max16600_get(TEST_ID, "temperature") / 1000; + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_TEMPERATURE_1); + i2c_value =3D pmbus_linear_mode2data(i2c_value, max16600_exp.temp); + g_assert_cmpuint(value, =3D=3D, i2c_value); +} + +/* test r/w registers */ +static void test_rw_regs(void *obj, void *data, QGuestAllocator *alloc) +{ + uint16_t i2c_value; + QI2CDevice *i2cdev =3D (QI2CDevice *)obj; + + max16600_i2c_set16(i2cdev, PMBUS_OPERATION, 0xA); + i2c_value =3D i2c_get8(i2cdev, PMBUS_OPERATION); + g_assert_cmphex(i2c_value, =3D=3D, 0xA); + + max16600_i2c_set16(i2cdev, PMBUS_ON_OFF_CONFIG, 0xB); + i2c_value =3D i2c_get8(i2cdev, PMBUS_ON_OFF_CONFIG); + g_assert_cmphex(i2c_value, =3D=3D, 0xB); + + max16600_i2c_set16(i2cdev, PMBUS_VOUT_MODE, 0xC); + i2c_value =3D i2c_get8(i2cdev, PMBUS_VOUT_MODE); + g_assert_cmphex(i2c_value, =3D=3D, 0xC); +} + +/* test read-only registers */ +static void test_ro_regs(void *obj, void *data, QGuestAllocator *alloc) +{ + uint16_t i2c_init_value, i2c_value; + QI2CDevice *i2cdev =3D (QI2CDevice *)obj; + + i2c_init_value =3D i2c_get8(i2cdev, PMBUS_CAPABILITY); + max16600_i2c_set16(i2cdev, PMBUS_CAPABILITY, 0xD); + i2c_value =3D i2c_get8(i2cdev, PMBUS_CAPABILITY); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VIN); + max16600_i2c_set16(i2cdev, PMBUS_READ_VIN, 0x1234); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VIN); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IIN); + max16600_i2c_set16(i2cdev, PMBUS_READ_IIN, 0x2234); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IIN); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_PIN); + max16600_i2c_set16(i2cdev, PMBUS_READ_PIN, 0x3234); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_PIN); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VOUT); + max16600_i2c_set16(i2cdev, PMBUS_READ_VOUT, 0x4234); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_VOUT); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IOUT); + max16600_i2c_set16(i2cdev, PMBUS_READ_IOUT, 0x5235); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_IOUT); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_POUT); + max16600_i2c_set16(i2cdev, PMBUS_READ_POUT, 0x6234); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_POUT); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); + + i2c_init_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_TEMPERATURE_1= ); + max16600_i2c_set16(i2cdev, PMBUS_READ_TEMPERATURE_1, 0x7236); + i2c_value =3D max16600_i2c_get16(i2cdev, PMBUS_READ_TEMPERATURE_1); + g_assert_cmphex(i2c_init_value, =3D=3D, i2c_value); +} + +static void max16600_register_nodes(void) +{ + QOSGraphEdgeOptions opts =3D {.extra_device_opts =3D + "id=3D" TEST_ID ",address=3D0x61"}; + add_qi2c_address(&opts, &(QI2CAddress){TEST_ADDR}); + + qos_node_create_driver("max16600", i2c_device_create); + qos_node_consumes("max16600", "i2c-bus", &opts); + + qos_add_test("test_defaults", "max16600", test_defaults, NULL); + qos_add_test("test_tx_rx", "max16600", test_tx_rx, NULL); + qos_add_test("test_rw_regs", "max16600", test_rw_regs, NULL); + qos_add_test("test_ro_regs", "max16600", test_ro_regs, NULL); +} +libqos_init(max16600_register_nodes); diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build index 669d07c06b..459cf41985 100644 --- a/tests/qtest/meson.build +++ b/tests/qtest/meson.build @@ -295,6 +295,7 @@ qos_test_ss.add( 'es1370-test.c', 'lsm303dlhc-mag-test.c', 'isl_pmbus_vr-test.c', + 'max16600-test.c', 'max34451-test.c', 'megasas-test.c', 'ne2000-test.c', --=20 2.51.0.355.g5224444f11-goog