From nobody Mon Sep 8 10:04:05 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=1756850703; cv=none; d=zohomail.com; s=zohoarc; b=hNq2WH9dTxH5KQMP85KUkqTwv+SbSz+bnYtI+f3NCQhssOxRXVT5nU7gXWqgzk+W4SW3F74IPa7vJ4zBbkcVJzGLjrVQpt3mcdF+qEEngDmGB3DMFVom5AfbPm+12iDGbQ3GayLul5wOzh27+PmH0AY/b/m4lvpn/LFT8fFBte8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1756850703; 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=ziP9B54S2Bq5Q45jsFa6fAV/MYu3JZEJEdecaLugvHc=; b=WaVo8TIALyGhu/J6Ra4KcbSq1I24DL2v+psYuewkotT3WPmj6ketNtYA12rT9rjOFrqP+ea6+/NwBuhAKzYBCUY+JDq7d5eg9RPdMOwUDNvwvYxjtTYEo5rvt24BI3y+/5hoyp/PUTxT21HetAIkmgcs3VG7EfM5kH8VDxMPgyg= 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 1756850703799814.7449554318761; Tue, 2 Sep 2025 15:05:03 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1utZ76-0005Mx-Ck; Tue, 02 Sep 2025 18:04:28 -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 <3zGm3aAwKCsk2pqxwt78tup2v33v0t.r315t19-stAt0232v29.36v@flex--nabihestefan.bounces.google.com>) id 1utZ6p-0005If-LI for qemu-devel@nongnu.org; Tue, 02 Sep 2025 18:04:11 -0400 Received: from mail-yw1-x1149.google.com ([2607:f8b0:4864:20::1149]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from <3zGm3aAwKCsk2pqxwt78tup2v33v0t.r315t19-stAt0232v29.36v@flex--nabihestefan.bounces.google.com>) id 1utZ6l-0007W2-MJ for qemu-devel@nongnu.org; Tue, 02 Sep 2025 18:04:11 -0400 Received: by mail-yw1-x1149.google.com with SMTP id 00721157ae682-71d605205a0so74672277b3.0 for ; Tue, 02 Sep 2025 15:04:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1756850646; x=1757455446; darn=nongnu.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=ziP9B54S2Bq5Q45jsFa6fAV/MYu3JZEJEdecaLugvHc=; b=VVFXmPs49iBA7nOyzphaxcVM0s72dV/hUxwR0jJ5fGmoX8qBf6KLJypJrB0/qc0I87 gABTYCR9PGRiJFoY/PCAcOIn1SYgLCX78r64CuzoIH2Ch09O6dmIgeRRClG9x6N95eXD uwhGvOnTTVS3DxivcFJLzBB8qNrYtWjC3gbVyPTpNWgcJVAQ1gJn79Xp6T3jLT2wFf3+ t4WroW141VQyL4lkN4pZznEFlSGPXuMPLGFlGRuis3w3nC2B3O2oDMf+/X6AIMCcKrqp 109WSbX9G9nwFpxVuXE8R7gcieI0eOtebQHZNFEOs+ZkKPf5led3G4aBGgDZib+c1Qby 2SUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756850646; x=1757455446; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=ziP9B54S2Bq5Q45jsFa6fAV/MYu3JZEJEdecaLugvHc=; b=RwvDE6cUDLrJ1BK7TG94RkK5kPQw/gsjY8nCZkuZil1mHgUojnQD3WLAH0X+er9bXw WHVPF/HexY6HRLzvBnaO4J4g2BUN0JwdTyp0aMHIoAQT8VJPH3kcT9NlaLcBTkubvTT6 JlxUM9EeNLNXGXJgfgMcQuldhqD9uTl/6hiGLmaEaab0341UpFhsKLWj0lb97T2mcghz r2h1Fzs/NigI3JhgsbRyyNsSSRhcs17nSOxG7OC5m6DqQ1TNH/Ua9KaRlKTxmjaHnJJV llwDiBKMw940VUCWqKttZE34ewg8lkXpkYSwp1yTR7NcjVpEPLm0w3RR2bGpfQnOXGw9 xFiQ== X-Gm-Message-State: AOJu0Yx1ipKdWBF9wLvKObJbzFKj5VLDHZ/7cKSwzV6nsCiCjSfXl7sR MeoHED3gTsdiTMhD5+Db2u0LjVpiOWxwV7cgGdxSPSjNYtb4rXLCre5Q2BlkZfuwXQ6NtIL95JI 2CdH2bq2vOLD4I6tX6q8FSD0ghhr9mOWgoLrISTVl4pTTqY8M1Rr97HvAB3neQsUK81zXf12Pay EW9pLpLts5Qs/Mt4L7SkIhBswwVgw5DCH9qeC61kw+ib1g8czOw/SUFfF97uXUib5OQaA= X-Google-Smtp-Source: AGHT+IEKl6GSoUL98Z+nsdOBXIM/HPcCjhjsgEoy/AhsshwXZQmUjpWCvul5jYomN7UQcl5nYK6uWqI2MFXhK7KKk7E= X-Received: from ywbdo4.prod.google.com ([2002:a05:690c:23c4:b0:723:ac0b:257f]) (user=nabihestefan job=prod-delivery.src-stubby-dispatcher) by 2002:a05:690c:dcd:b0:71f:c5f0:3397 with SMTP id 00721157ae682-7227636ebf1mr151953587b3.18.1756850636443; Tue, 02 Sep 2025 15:03:56 -0700 (PDT) Date: Tue, 2 Sep 2025 22:03:41 +0000 Mime-Version: 1.0 X-Mailer: git-send-email 2.51.0.355.g5224444f11-goog Message-ID: <20250902220341.1460194-1-nabihestefan@google.com> Subject: [PATCH v2] 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::1149; envelope-from=3zGm3aAwKCsk2pqxwt78tup2v33v0t.r315t19-stAt0232v29.36v@flex--nabihestefan.bounces.google.com; helo=mail-yw1-x1149.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=unavailable 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: 1756850705047124100 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Shengtan Mao 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