From nobody Fri May 3 01:59:15 2024 Delivered-To: importer2@patchew.org Received-SPF: pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; envelope-from=linux-kernel-owner@vger.kernel.org; helo=vger.kernel.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail(p=none dis=none) header.from=cesnet.cz ARC-Seal: i=1; a=rsa-sha256; t=1618387238; cv=none; d=zohomail.com; s=zohoarc; b=J9VnYVu8sz/S/dOO45oQdn9sfDGmVmIPhWqlp+auFBF4+wlLeMfNUlL7WIzlaXt3AKalHAi3wkVgM234QFDKPmCpiQlE6GSltGjVspVpo5353WQzz82DcXkA0gU7WZMkxd3PmtC55zwgPbOX/eOkHJqXSVF05wTenEGO0dGnGak= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1618387238; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:MIME-Version:Message-ID:References:Subject:To; bh=wDJ0gI8SIsM7RtEkdnPpXcZUZl6+Vj8Yk5cF33BynzE=; b=mX4G8sR0pEI5D6T4wZOiLylXJdW0Cx7dJ5t7IvPftDZPJR7tPj3Sr3r+wrp7bCqQWywPIRR69A5xURzxrN3Ir2lsOrZ2dhj6WukcPq8o57Auypu7+vnyfh0kdO4Y5v5FMcik0YK1v9ee1wGwCQXb71ANMwG7ohc5IzymtXNECZc= ARC-Authentication-Results: i=1; mx.zohomail.com; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mx.zohomail.com with SMTP id 1618387238431497.47918630735353; Wed, 14 Apr 2021 01:00:38 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349818AbhDNIA5 (ORCPT ); Wed, 14 Apr 2021 04:00:57 -0400 Received: from mail-ej1-f42.google.com ([209.85.218.42]:37749 "EHLO mail-ej1-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232940AbhDNIAz (ORCPT ); Wed, 14 Apr 2021 04:00:55 -0400 Received: by mail-ej1-f42.google.com with SMTP id w3so30027562ejc.4; Wed, 14 Apr 2021 01:00:31 -0700 (PDT) Received: from localhost ([2a02:8308:387:c900:a7b5:b859:9449:c07b]) by smtp.gmail.com with ESMTPSA id r25sm11386617edv.78.2021.04.14.01.00.28 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 14 Apr 2021 01:00:28 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=wDJ0gI8SIsM7RtEkdnPpXcZUZl6+Vj8Yk5cF33BynzE=; b=c/1rixrAGUAcvav1AVO4T7Ilzl1e0OkNrRyuoorA028yD5gjD3LAN3f9vW5FP6GLP9 rCN0bgyH1rWe7HVs9eXoGvQvx5fpO91wmbFqYoap28xaAh6HbqQUllK+wNzx7mLN+cDM FRSU6fAgtdOScCekPv3BOYSBsYOUtkpgO5tlBxyKRViCi1DN/AXDTsk9D447pkIdLYqm McTeB7CyLc8WMPC1P4sROAOw66vEr2uiajMDbxkM0X/vlBNz6+OJQHUQ8NcU9KGnjYaH 4EUi3hVvhZz6rJSKw6AUOX5JvA4wiEMP5kIQ+/DmSOFdw+ox1anlJQNFe3gLoQ9XD4Zj AB+Q== X-Gm-Message-State: AOAM530PxJ1IxOpWjAfvPm/gjMTNZxv4IhmwEPmKwG7Z3WPiFXgS+Ga4 NzlgUphDgeDK0BELZbcVUOj9wwjCA2Wusg== X-Google-Smtp-Source: ABdhPJyi3Sp2WP7nXyA3iVBF3mQUtCnAOvSeCwc7aKgtI/D1k0s11HwU4TPzkFdg/jmJWTUUHij+oA== X-Received: by 2002:a17:906:4d17:: with SMTP id r23mr15748790eju.134.1618387229145; Wed, 14 Apr 2021 01:00:29 -0700 (PDT) From: =?UTF-8?q?V=C3=A1clav=20Kubern=C3=A1t?= Cc: =?UTF-8?q?V=C3=A1clav=20Kubern=C3=A1t?= , Jean Delvare , Guenter Roeck , Jonathan Corbet , linux-hwmon@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v6] hwmon: Add driver for fsp-3y PSUs and PDUs Date: Wed, 14 Apr 2021 10:00:17 +0200 Message-Id: <20210414080019.3530794-1-kubernat@cesnet.cz> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210329143833.1047539-1-kubernat@cesnet.cz> References: <20210329143833.1047539-1-kubernat@cesnet.cz> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch adds support for these devices: - YH-5151E - the PDU - YM-2151E - the PSU The device datasheet says that the devices support PMBus 1.2, but in my testing, a lot of the commands aren't supported and if they are, they sometimes behave strangely or inconsistently. For example, writes to the PAGE command requires using PEC, otherwise the write won't work and the page won't switch, even though, the standard says that PEC is optional. On the other hand, writes to SMBALERT don't require PEC. Because of this, the driver is mostly reverse engineered with the help of a tool called pmbus_peek written by David Brownell (and later adopted by my colleague Jan Kundr=C3=A1t). The device also has some sort of a timing issue when switching pages, which is explained further in the code. Because of this, the driver support is limited. It exposes only the values, that have been tested to work correctly. Signed-off-by: V=C3=A1clav Kubern=C3=A1t --- Documentation/hwmon/fsp-3y.rst | 28 ++++ Documentation/hwmon/index.rst | 1 + drivers/hwmon/pmbus/Kconfig | 10 ++ drivers/hwmon/pmbus/Makefile | 1 + drivers/hwmon/pmbus/fsp-3y.c | 253 +++++++++++++++++++++++++++++++++ 5 files changed, 293 insertions(+) create mode 100644 Documentation/hwmon/fsp-3y.rst create mode 100644 drivers/hwmon/pmbus/fsp-3y.c diff --git a/Documentation/hwmon/fsp-3y.rst b/Documentation/hwmon/fsp-3y.rst new file mode 100644 index 000000000000..5693d83a2035 --- /dev/null +++ b/Documentation/hwmon/fsp-3y.rst @@ -0,0 +1,28 @@ +.. SPDX-License-Identifier: GPL-2.0 + +Kernel driver fsp3y +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +Supported devices: + * 3Y POWER YH-5151E + * 3Y POWER YM-2151E + +Author: V=C3=A1clav Kubern=C3=A1t + +Description +----------- +This driver implements limited support for two 3Y POWER devices. + +Sysfs entries +------------- + * in1_input input voltage + * in2_input 12V output voltage + * in3_input 5V output voltage + * curr1_input input current + * curr2_input 12V output current + * curr3_input 5V output current + * fan1_input fan rpm + * temp1_input temperature 1 + * temp2_input temperature 2 + * temp3_input temperature 3 + * power1_input input power + * power2_input output power diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst index fcb870ce6286..55c9f014c248 100644 --- a/Documentation/hwmon/index.rst +++ b/Documentation/hwmon/index.rst @@ -63,6 +63,7 @@ Hardware Monitoring Kernel Drivers f71805f f71882fg fam15h_power + fsp-3y ftsteutates g760a g762 diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig index 03606d4298a4..9d12d446396c 100644 --- a/drivers/hwmon/pmbus/Kconfig +++ b/drivers/hwmon/pmbus/Kconfig @@ -56,6 +56,16 @@ config SENSORS_BEL_PFE This driver can also be built as a module. If so, the module will be called bel-pfe. =20 +config SENSORS_FSP_3Y + tristate "FSP/3Y-Power power supplies" + help + If you say yes here you get hardware monitoring support for + FSP/3Y-Power hot-swap power supplies. + Supported models: YH-5151E, YM-2151E + + This driver can also be built as a module. If so, the module will + be called fsp-3y. + config SENSORS_IBM_CFFPS tristate "IBM Common Form Factor Power Supply" depends on LEDS_CLASS diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile index 6a4ba0fdc1db..bfe218ad898f 100644 --- a/drivers/hwmon/pmbus/Makefile +++ b/drivers/hwmon/pmbus/Makefile @@ -8,6 +8,7 @@ obj-$(CONFIG_SENSORS_PMBUS) +=3D pmbus.o obj-$(CONFIG_SENSORS_ADM1266) +=3D adm1266.o obj-$(CONFIG_SENSORS_ADM1275) +=3D adm1275.o obj-$(CONFIG_SENSORS_BEL_PFE) +=3D bel-pfe.o +obj-$(CONFIG_SENSORS_FSP_3Y) +=3D fsp-3y.o obj-$(CONFIG_SENSORS_IBM_CFFPS) +=3D ibm-cffps.o obj-$(CONFIG_SENSORS_INSPUR_IPSPS) +=3D inspur-ipsps.o obj-$(CONFIG_SENSORS_IR35221) +=3D ir35221.o diff --git a/drivers/hwmon/pmbus/fsp-3y.c b/drivers/hwmon/pmbus/fsp-3y.c new file mode 100644 index 000000000000..564649e87e34 --- /dev/null +++ b/drivers/hwmon/pmbus/fsp-3y.c @@ -0,0 +1,253 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * Hardware monitoring driver for FSP 3Y-Power PSUs + * + * Copyright (c) 2021 V=C3=A1clav Kubern=C3=A1t, CESNET + * + * This driver is mostly reverse engineered with the help of a tool called= pmbus_peek written by + * David Brownell (and later adopted by Jan Kundr=C3=A1t). The device has = some sort of a timing issue + * when switching pages, details are explained in the code. The driver sup= port is limited. It + * exposes only the values, that have been tested to work correctly. Unsup= ported values either + * aren't supported by the devices or their encondings are unknown. + */ + +#include +#include +#include +#include +#include "pmbus.h" + +#define YM2151_PAGE_12V_LOG 0x00 +#define YM2151_PAGE_12V_REAL 0x00 +#define YM2151_PAGE_5VSB_LOG 0x01 +#define YM2151_PAGE_5VSB_REAL 0x20 +#define YH5151E_PAGE_12V_LOG 0x00 +#define YH5151E_PAGE_12V_REAL 0x00 +#define YH5151E_PAGE_5V_LOG 0x01 +#define YH5151E_PAGE_5V_REAL 0x10 +#define YH5151E_PAGE_3V3_LOG 0x02 +#define YH5151E_PAGE_3V3_REAL 0x11 + +enum chips { + ym2151e, + yh5151e +}; + +struct fsp3y_data { + struct pmbus_driver_info info; + int chip; + int page; +}; + +#define to_fsp3y_data(x) container_of(x, struct fsp3y_data, info) + +static int page_log_to_page_real(int page_log, enum chips chip) +{ + switch (chip) { + case ym2151e: + switch (page_log) { + case YM2151_PAGE_12V_LOG: + return YM2151_PAGE_12V_REAL; + case YM2151_PAGE_5VSB_LOG: + return YM2151_PAGE_5VSB_REAL; + } + return -EINVAL; + case yh5151e: + switch (page_log) { + case YH5151E_PAGE_12V_LOG: + return YH5151E_PAGE_12V_REAL; + case YH5151E_PAGE_5V_LOG: + return YH5151E_PAGE_5V_LOG; + case YH5151E_PAGE_3V3_LOG: + return YH5151E_PAGE_3V3_REAL; + } + return -EINVAL; + } + + return -EINVAL; +} + +static int set_page(struct i2c_client *client, int page_log) +{ + const struct pmbus_driver_info *info =3D pmbus_get_driver_info(client); + struct fsp3y_data *data =3D to_fsp3y_data(info); + int rv; + int page_real; + + if (page_log < 0) + return 0; + + page_real =3D page_log_to_page_real(page_log, data->chip); + if (page_real < 0) + return page_real; + + if (data->page !=3D page_real) { + rv =3D i2c_smbus_write_byte_data(client, PMBUS_PAGE, page_real); + if (rv < 0) + return rv; + + data->page =3D page_real; + + /* + * Testing showed that the device has a timing issue. After + * setting a page, it takes a while, before the device actually + * gives the correct values from the correct page. 20 ms was + * tested to be enough to not give wrong values (15 ms wasn't + * enough). + */ + usleep_range(20000, 30000); + } + + return 0; +} + +static int fsp3y_read_byte_data(struct i2c_client *client, int page, int r= eg) +{ + int rv; + + rv =3D set_page(client, page); + if (rv < 0) + return rv; + + return i2c_smbus_read_byte_data(client, reg); +} + +static int fsp3y_read_word_data(struct i2c_client *client, int page, int p= hase, int reg) +{ + int rv; + + /* + * This masks commands which weren't tested to work correctly. Some of + * the masked commands return 0xFFFF. These would probably get tagged as + * invalid by pmbus_core. Other ones do return values which might be + * useful (that is, they are not 0xFFFF), but their encoding is unknown, + * and so they are unsupported. + */ + switch (reg) { + case PMBUS_READ_FAN_SPEED_1: + case PMBUS_READ_IIN: + case PMBUS_READ_IOUT: + case PMBUS_READ_PIN: + case PMBUS_READ_POUT: + case PMBUS_READ_TEMPERATURE_1: + case PMBUS_READ_TEMPERATURE_2: + case PMBUS_READ_TEMPERATURE_3: + case PMBUS_READ_VIN: + case PMBUS_READ_VOUT: + case PMBUS_STATUS_WORD: + break; + default: + return -ENXIO; + } + + rv =3D set_page(client, page); + if (rv < 0) + return rv; + + return i2c_smbus_read_word_data(client, reg); +} + +struct pmbus_driver_info fsp3y_info[] =3D { + [ym2151e] =3D { + .pages =3D 2, + .func[YM2151_PAGE_12V_LOG] =3D + PMBUS_HAVE_VOUT | PMBUS_HAVE_IOUT | + PMBUS_HAVE_PIN | PMBUS_HAVE_POUT | + PMBUS_HAVE_TEMP | PMBUS_HAVE_TEMP2 | + PMBUS_HAVE_VIN | PMBUS_HAVE_IIN | + PMBUS_HAVE_FAN12, + .func[YM2151_PAGE_5VSB_LOG] =3D + PMBUS_HAVE_VOUT | PMBUS_HAVE_IOUT, + PMBUS_HAVE_IIN, + .read_word_data =3D fsp3y_read_word_data, + .read_byte_data =3D fsp3y_read_byte_data, + }, + [yh5151e] =3D { + .pages =3D 3, + .func[YH5151E_PAGE_12V_LOG] =3D + PMBUS_HAVE_VOUT | PMBUS_HAVE_IOUT | + PMBUS_HAVE_POUT | + PMBUS_HAVE_TEMP | PMBUS_HAVE_TEMP2 | PMBUS_HAVE_TEMP3, + .func[YH5151E_PAGE_5V_LOG] =3D + PMBUS_HAVE_VOUT | PMBUS_HAVE_IOUT | + PMBUS_HAVE_POUT, + .func[YH5151E_PAGE_3V3_LOG] =3D + PMBUS_HAVE_VOUT | PMBUS_HAVE_IOUT | + PMBUS_HAVE_POUT, + .read_word_data =3D fsp3y_read_word_data, + .read_byte_data =3D fsp3y_read_byte_data, + } +}; + +static int fsp3y_detect(struct i2c_client *client) +{ + int rv; + u8 buf[I2C_SMBUS_BLOCK_MAX + 1]; + + rv =3D i2c_smbus_read_block_data(client, PMBUS_MFR_MODEL, buf); + if (rv < 0) + return rv; + + buf[rv] =3D '\0'; + + if (rv =3D=3D 8) { + if (!strcmp(buf, "YM-2151E")) + return ym2151e; + else if (!strcmp(buf, "YH-5151E")) + return yh5151e; + } + + dev_err(&client->dev, "Unsupported model %.*s\n", rv, buf); + return -ENODEV; +} + +static const struct i2c_device_id fsp3y_id[] =3D { + {"ym2151e", ym2151e}, + {"yh5151e", yh5151e}, + {0} +}; + +static int fsp3y_probe(struct i2c_client *client) +{ + struct fsp3y_data *data; + const struct i2c_device_id *id; + int rv; + + data =3D devm_kzalloc(&client->dev, sizeof(struct fsp3y_data), GFP_KERNEL= ); + if (!data) + return -ENOMEM; + + data->chip =3D fsp3y_detect(client); + if (data->chip < 0) + return data->chip; + + id =3D i2c_match_id(fsp3y_id, client); + if (data->chip !=3D id->driver_data) + dev_warn(&client->dev, "Device mismatch: Configured %s (%d), detected %d= \n", + id->name, (int)id->driver_data, data->chip); + + rv =3D i2c_smbus_read_byte_data(client, PMBUS_PAGE); + if (rv < 0) + return rv; + data->page =3D rv; + + data->info =3D fsp3y_info[data->chip]; + + return pmbus_do_probe(client, &data->info); +} + +MODULE_DEVICE_TABLE(i2c, fsp3y_id); + +static struct i2c_driver fsp3y_driver =3D { + .driver =3D { + .name =3D "fsp3y", + }, + .probe_new =3D fsp3y_probe, + .id_table =3D fsp3y_id +}; + +module_i2c_driver(fsp3y_driver); + +MODULE_AUTHOR("V=C3=A1clav Kubern=C3=A1t"); +MODULE_DESCRIPTION("PMBus driver for FSP/3Y-Power power supplies"); +MODULE_LICENSE("GPL"); --=20 2.31.1