| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20201125141053.GA96791@roeck-us.net>
Date: Wed, 25 Nov 2020 06:10:53 -0800
From: Guenter Roeck <linux@...ck-us.net>
To: Shihlun Lin <shihlun.lin@...antech.com.tw>
Cc: "David S . Miller" <davem@...emloft.net>,
Rob Herring <robh@...nel.org>,
Lee Jones <lee.jones@...aro.org>,
Jean Delvare <jdelvare@...e.com>,
Wim Van Sebroeck <wim@...ux-watchdog.org>,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-hwmon@...r.kernel.org, linux-watchdog@...r.kernel.org,
Campion Kang <campion.kang@...antech.com.tw>,
AceLan Kao <chia-lin.kao@...onical.com>
Subject: Re: [RESEND PATCH v4 5/6] hwmon: ahc1ec0-hwmon: Add sub-device hwmon
for Advantech embedded controller
On Wed, Nov 25, 2020 at 03:07:43PM +0800, Shihlun Lin wrote:
> This is one of sub-device driver for Advantech embedded controller
> AHC1EC0. This driver provides sysfs ABI for Advantech related
> applications to monitor the system status.
>
> Signed-off-by: Shihlun Lin <shihlun.lin@...antech.com.tw>
> ---
> drivers/hwmon/Kconfig | 8 +
> drivers/hwmon/Makefile | 1 +
> drivers/hwmon/ahc1ec0-hwmon.c | 1504 +++++++++++++++++++++++++++++++++
Documentation and context is completely missing.
What does this driver have to do with Fintek chips ? We already have a driver
for those chips. Why would we want or need another driver for the same chips
under any circumstances ?
Also, new drivers shall use hwmon_device_register_with_info() to register
hwmon devices.
Guenter
> 3 files changed, 1513 insertions(+)
> create mode 100644 drivers/hwmon/ahc1ec0-hwmon.c
>
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index a850e4f0e0bd..577dd1dd60ee 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -2095,6 +2095,14 @@ config SENSORS_INTEL_M10_BMC_HWMON
> sensors monitor various telemetry data of different components on the
> card, e.g. board temperature, FPGA core temperature/voltage/current.
>
> +config SENSORS_AHC1EC0_HWMON
> + tristate "Advantech EC Hardware Monitor Function"
> + depends on MFD_AHC1EC0
> + help
> + This is sub-device for Advantech embedded controller AHC1EC0. This
> + driver provides the sysfs attributes for applications to monitor
> + the system status.
> +
> if ACPI
>
> comment "ACPI drivers"
> diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
> index 9db2903b61e5..e06ddc314b4a 100644
> --- a/drivers/hwmon/Makefile
> +++ b/drivers/hwmon/Makefile
> @@ -45,6 +45,7 @@ obj-$(CONFIG_SENSORS_ADT7411) += adt7411.o
> obj-$(CONFIG_SENSORS_ADT7462) += adt7462.o
> obj-$(CONFIG_SENSORS_ADT7470) += adt7470.o
> obj-$(CONFIG_SENSORS_ADT7475) += adt7475.o
> +obj-$(CONFIG_SENSORS_AHC1EC0_HWMON) += ahc1ec0-hwmon.o
> obj-$(CONFIG_SENSORS_AMD_ENERGY) += amd_energy.o
> obj-$(CONFIG_SENSORS_APPLESMC) += applesmc.o
> obj-$(CONFIG_SENSORS_ARM_SCMI) += scmi-hwmon.o
> diff --git a/drivers/hwmon/ahc1ec0-hwmon.c b/drivers/hwmon/ahc1ec0-hwmon.c
> new file mode 100644
> index 000000000000..d71eb8e01422
> --- /dev/null
> +++ b/drivers/hwmon/ahc1ec0-hwmon.c
> @@ -0,0 +1,1504 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * HWMON Driver for Advantech controlling EC chip AHC1EC0
> + *
> + * Copyright (C) 2020, Advantech Automation Corp.
> + *
> + * Change Log:
> + * Version 1.00 <11/05/2015> Jiangwei.Zhu
> + * - Initial version
> + * Version 1.01 <03/04/2016> Jiangwei.Zhu
> + * - Support UNO-1372G-E3AE, TPC-1782H-433AE, APAX-5580-433AE
> + * Version 1.02 <05/09/2016> Ji.Xu
> + * - Support APAX-5580-473AE/4C3AE
> + * - Modify the device name check method to fuzzy matching.
> + * Version 1.03 <05/09/2017> Ji.Xu
> + * - Support UNO-2271G-E2xAE
> + * - Support UNO-2271G-E02xAE
> + * - Support ECU-4784
> + * - Support UNO-2473G-JxAE
> + * Version 1.04 <09/20/2017> Ji.Xu
> + * - Support UNO-2484G-633xAE
> + * - Support UNO-2484G-653xAE
> + * - Support UNO-2484G-673xAE
> + * - Support UNO-2484G-733xAE
> + * - Support UNO-2484G-753xAE
> + * - Support UNO-2484G-773xAE
> + * Version 1.05 <10/26/2017> Ji.Xu
> + * - Support PR/VR4
> + * - Support UNO-3283G-674AE
> + * - Support UNO-3285G-674AE
> + * Version 1.06 <11/16/2017> Zhang.Yang
> + * - Support UNO-1372G-J021AE/J031AE
> + * - Support UNO-2372G
> + * Version 1.07 <02/02/2018> Ji.Xu
> + * - Convert the driver to use new hwmon API after kernel version 4.10.0
> + * - Support EC TPC-B500-6??AE
> + * - Support EC TPC-5???T-6??AE
> + * Version 1.08 <02/20/2019> Ji.Xu
> + * - Support EC UNO-420
> + * - Support EC TPC-B200-???AE
> + * - Support EC TPC-2???T-???AE
> + * - Support EC TPC-2???W-???AE
> + * Version 1.09 <04/24/2020> Yao.Kang
> + * - Support EC UNO-2473G
> + */
> +
> +#include <linux/init.h>
> +#include <linux/module.h>
> +#include <linux/types.h>
> +#include <linux/errno.h>
> +#include <linux/kernel.h>
> +#include <linux/delay.h>
> +#include <linux/interrupt.h>
> +#include <linux/ioport.h>
> +#include <linux/ioctl.h>
> +#include <linux/io.h>
> +#include <linux/wait.h>
> +#include <linux/sched.h>
> +#include <linux/fs.h>
> +#include <linux/uaccess.h>
> +#include <linux/string.h>
> +#include <linux/slab.h>
> +#include <linux/errno.h>
> +#include <asm/msr.h>
> +#include <asm/msr-index.h>
> +#include <linux/version.h>
> +#include <linux/i2c.h>
> +#include <linux/device.h>
> +#include <linux/platform_device.h>
> +#include <linux/hwmon.h>
> +#include <linux/hwmon-sysfs.h>
> +#include <linux/sysfs.h>
> +#include <linux/mfd/ahc1ec0.h>
> +
> +#define ADVANTECH_EC_HWMON_VER "1.09"
> +#define ADVANTECH_EC_HWMON_DATE "04/24/2020"
> +
> +/* Addresses to scan */
> +static const unsigned short normal_i2c[] = { 0x2d, 0x2e, I2C_CLIENT_END };
> +
> +enum chips { f75373, f75375, f75387 };
> +
> +/* Fintek F75375 registers */
> +#define F75375_REG_CONFIG0 0x0
> +#define F75375_REG_CONFIG1 0x1
> +#define F75375_REG_CONFIG2 0x2
> +#define F75375_REG_CONFIG3 0x3
> +#define F75375_REG_ADDR 0x4
> +#define F75375_REG_INTR 0x31
> +#define F75375_CHIP_ID 0x5A
> +#define F75375_REG_VERSION 0x5C
> +#define F75375_REG_VENDOR 0x5D
> +
> +#define F75375_REG_TEMP(nr) (0x14 + (nr))
> +#define F75387_REG_TEMP11_LSB(nr) (0x1c + (nr))
> +#define F75375_REG_TEMP_HIGH(nr) (0x28 + (nr) * 2)
> +#define F75375_REG_TEMP_HYST(nr) (0x29 + (nr) * 2)
> +
> +/*
> + * Data structures and manipulation thereof
> + */
> +
> +struct f75375_data {
> + unsigned short addr;
> + struct device *hwmon_dev;
> +
> + const char *name;
> + int kind;
> + struct mutex update_lock; /* protect register access */
> + char valid;
> + unsigned long last_updated; /* In jiffies */
> + unsigned long last_limits; /* In jiffies */
> +
> + /* Register values */
> + /*
> + * f75387: For remote temperature reading, it uses signed 11-bit
> + * values with LSB = 0.125 degree Celsius, left-justified in 16-bit
> + * registers. For original 8-bit temp readings, the LSB just is 0.
> + */
> + s16 temp11[2];
> + s8 temp_high[2];
> + s8 temp_max_hyst[2];
> +};
> +
> +static int f75375_detect(struct i2c_client *client, struct i2c_board_info *info);
> +static int f75375_probe(struct i2c_client *client, const struct i2c_device_id *id);
> +static int f75375_remove(struct i2c_client *client);
> +static int adspname_detect(const char *bios_product_name, const char *standard_name);
> +
> +static const struct i2c_device_id f75375_id[] = {
> + { "f75387", f75387 },
> + { }
> +};
> +MODULE_DEVICE_TABLE(i2c, f75375_id);
> +
> +struct adv_hwmon_profile {
> + int offset;
> + unsigned long resolution, resolution_vin, resolution_sys, resolution_curr, resolution_power;
> + unsigned long r1, r1_vin, r1_sys, r1_curr, r1_power;
> + unsigned long r2, r2_vin, r2_sys, r2_curr, r2_power;
> + const struct attribute_group ec_hwmon_group;
> +};
> +
> +struct EC_HWMON_DATA {
> + uchar temperature[3];
> + uchar ec_current[5];
> + uchar power[5];
> + char *bios_product_name;
> + int voltage[7];
> +
> + struct device *dev, *hwmon_dev;
> + struct HW_PIN_TBL pin_tbl;
> + struct EC_SMBOEM0 ec_smboem0;
> + struct adv_hwmon_profile *profile;
> +};
> +struct EC_HWMON_DATA lmsensor_data;
> +
> +static struct i2c_driver f75375_driver = {
> + .class = I2C_CLASS_HWMON,
> + .driver = {
> + .name = "f75375",
> + },
> + .probe = f75375_probe,
> + .remove = f75375_remove,
> + .id_table = f75375_id,
> + .detect = f75375_detect,
> + .address_list = normal_i2c,
> +};
> +
> +static inline int f75375_read8(struct i2c_client *client, u8 reg)
> +{
> + return i2c_smbus_read_byte_data(client, reg);
> +}
> +
> +/* in most cases, should be called while holding update_lock */
> +static inline u16 f75375_read16(struct i2c_client *client, u8 reg)
> +{
> + return (i2c_smbus_read_byte_data(client, reg) << 8)
> + | i2c_smbus_read_byte_data(client, reg + 1);
> +}
> +
> +static inline void f75375_write8(struct i2c_client *client, u8 reg,
> + u8 value)
> +{
> + i2c_smbus_write_byte_data(client, reg, value);
> +}
> +
> +static inline void f75375_write16(struct i2c_client *client, u8 reg,
> + u16 value)
> +{
> + int err = i2c_smbus_write_byte_data(client, reg, (value >> 8));
> +
> + if (err)
> + return;
> + i2c_smbus_write_byte_data(client, reg + 1, (value & 0xFF));
> +}
> +
> +
> +static struct f75375_data *f75375_update_device(struct device *dev)
> +{
> + struct i2c_client *client = to_i2c_client(dev);
> + struct f75375_data *data = i2c_get_clientdata(client);
> + int nr;
> +
> + mutex_lock(&data->update_lock);
> +
> + /* Limit registers cache is refreshed after 60 seconds */
> + if (time_after(jiffies, data->last_limits + 60 * HZ)
> + || !data->valid) {
> + for (nr = 0; nr < 2; nr++) {
> + data->temp_high[nr] =
> + f75375_read8(client, F75375_REG_TEMP_HIGH(nr));
> + data->temp_max_hyst[nr] =
> + f75375_read8(client, F75375_REG_TEMP_HYST(nr));
> + }
> + data->last_limits = jiffies;
> + }
> +
> + /* Measurement registers cache is refreshed after 2 second */
> + if (time_after(jiffies, data->last_updated + 2 * HZ)
> + || !data->valid) {
> + for (nr = 0; nr < 2; nr++) {
> + /* assign MSB, therefore shift it by 8 bits */
> + data->temp11[nr] = f75375_read8(client, F75375_REG_TEMP(nr)) << 8;
> + if (data->kind == f75387)
> + /* merge F75387's temperature LSB (11-bit) */
> + data->temp11[nr] |= f75375_read8(client, F75387_REG_TEMP11_LSB(nr));
> + }
> + data->last_updated = jiffies;
> + data->valid = 1;
> + }
> +
> + mutex_unlock(&data->update_lock);
> + return data;
> +}
> +
> +#define TEMP_FROM_REG(val) ((val) * 1000)
> +#define TEMP_TO_REG(val) ((val) / 1000)
> +#define TEMP11_FROM_REG(reg) ((reg) / 32 * 125)
> +
> +static ssize_t show_temp11(struct device *dev, struct device_attribute *attr,
> + char *buf)
> +{
> + int nr = to_sensor_dev_attr(attr)->index;
> + struct f75375_data *data = f75375_update_device(dev);
> +
> + return sprintf(buf, "%d\n", TEMP11_FROM_REG(data->temp11[nr]));
> +}
> +
> +static ssize_t show_temp_max(struct device *dev, struct device_attribute *attr,
> + char *buf)
> +{
> + int nr = to_sensor_dev_attr(attr)->index;
> + struct f75375_data *data = f75375_update_device(dev);
> +
> + return sprintf(buf, "%d\n", TEMP_FROM_REG(data->temp_high[nr]));
> +}
> +
> +static ssize_t show_temp_max_hyst(struct device *dev,
> + struct device_attribute *attr, char *buf)
> +{
> + int nr = to_sensor_dev_attr(attr)->index;
> + struct f75375_data *data = f75375_update_device(dev);
> +
> + return sprintf(buf, "%d\n", TEMP_FROM_REG(data->temp_max_hyst[nr]));
> +}
> +
> +static ssize_t set_temp_max(struct device *dev, struct device_attribute *attr,
> + const char *buf, size_t count)
> +{
> + int nr = to_sensor_dev_attr(attr)->index;
> + struct i2c_client *client = to_i2c_client(dev);
> + struct f75375_data *data = i2c_get_clientdata(client);
> + unsigned long val;
> + int err;
> +
> + err = kstrtoul(buf, 10, &val);
> + if (err < 0)
> + return err;
> +
> + val = clamp_val(TEMP_TO_REG(val), 0, 127);
> + mutex_lock(&data->update_lock);
> + data->temp_high[nr] = val;
> + f75375_write8(client, F75375_REG_TEMP_HIGH(nr), data->temp_high[nr]);
> + mutex_unlock(&data->update_lock);
> + return count;
> +}
> +
> +static ssize_t set_temp_max_hyst(struct device *dev,
> + struct device_attribute *attr, const char *buf, size_t count)
> +{
> + int nr = to_sensor_dev_attr(attr)->index;
> + struct i2c_client *client = to_i2c_client(dev);
> + struct f75375_data *data = i2c_get_clientdata(client);
> + unsigned long val;
> + int err;
> +
> + err = kstrtoul(buf, 10, &val);
> + if (err < 0)
> + return err;
> +
> + val = clamp_val(TEMP_TO_REG(val), 0, 127);
> + mutex_lock(&data->update_lock);
> + data->temp_max_hyst[nr] = val;
> + f75375_write8(client, F75375_REG_TEMP_HYST(nr),
> + data->temp_max_hyst[nr]);
> + mutex_unlock(&data->update_lock);
> + return count;
> +}
> +
> +static ssize_t get_ec_hwmon_name(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "advhwmon\n");
> +}
> +
> +static ssize_t get_ec_in1_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "VBAT\n");
> +}
> +
> +static ssize_t get_ec_in2_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "5VSB\n");
> +}
> +
> +static ssize_t get_ec_in3_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "VIN\n");
> +}
> +
> +static ssize_t get_ec_in4_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "VCORE\n");
> +}
> +
> +static ssize_t get_ec_in5_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "VIN1\n");
> +}
> +
> +static ssize_t get_ec_in6_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "VIN2\n");
> +}
> +
> +static ssize_t get_ec_in7_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "System Voltage\n");
> +}
> +
> +static ssize_t get_ec_in8_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Vin\n");
> +}
> +
> +static ssize_t get_ec_curr1_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Current\n");
> +}
> +
> +static ssize_t get_ec_curr2_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Current\n");
> +}
> +
> +static ssize_t get_ec_power1_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Power\n");
> +}
> +
> +static ssize_t get_ec_temp1_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Temp Board\n");
> +}
> +
> +static ssize_t get_ec_temp2_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Temp CPU\n");
> +}
> +
> +static ssize_t get_ec_temp2_crit(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "100000\n");
> +}
> +
> +static ssize_t get_ec_temp2_crit_alarm(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "0\n");
> +}
> +
> +static ssize_t get_ec_temp3_label(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "Temp System\n");
> +}
> +
> +static ssize_t get_ec_temp3_crit(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "100000\n");
> +}
> +
> +static ssize_t get_ec_temp3_crit_alarm(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "0\n");
> +}
> +
> +static ssize_t get_ec_in1_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + uchar voltage = 0;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->vbat[0], ptbl->vbat[1]);
> +
> + if (profile->r2 != 0)
> + voltage = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + voltage = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[0] = 10*voltage;
> +
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[0]);
> +}
> +
> +static ssize_t get_ec_in2_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + uchar voltage = 0;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->v5[0], ptbl->v5[1]);
> +
> + if (profile->r2 != 0)
> + voltage = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + voltage = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[1] = 10*voltage;
> +
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[1]);
> +}
> +
> +static ssize_t get_ec_in3_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + uchar voltage = 0;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->v12[0], ptbl->v12[1]);
> + if (temp == -1)
> + temp = read_ad_value(ptbl->vdc[0], ptbl->vdc[1]);
> +
> + if (profile->r2 != 0)
> + voltage = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + voltage = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[2] = 10*voltage;
> +
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[2]);
> +}
> +
> +static ssize_t get_ec_in4_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + uchar voltage = 0;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->vcore[0], ptbl->vcore[1]);
> +
> + if (profile->r2 != 0)
> + voltage = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + voltage = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[3] = 10*voltage;
> +
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[3]);
> +}
> +
> +static ssize_t get_ec_curr1_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->ec_current[0], ptbl->ec_current[1]);
> +
> + if (profile->r2 != 0)
> + temp = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + temp = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + temp += (int)profile->offset * 100;
> +
> + lmsensor_data.ec_current[3] = temp;
> + return sprintf(buf, "%d\n", 10*lmsensor_data.ec_current[3]);
> +}
> +
> +static void get_temperaturedts(uchar *pvalue)
> +{
> + u32 eax, edx;
> + uchar temp;
> +
> + rdmsr_on_cpu(1, MSR_IA32_THERM_STATUS, &eax, &edx);
> + temp = 100000-((eax>>16)&0x7f) * 1000;
> + *pvalue = temp;
> +}
> +
> +static void ec_get_temperature_value_via_lm96163_ec_smbus(uchar *temperature)
> +{
> + int ret;
> + struct EC_SMBUS_READ_BYTE in_data = {
> + .Channel = NSLM96163_CHANNEL,
> + .Address = NSLM96163_ADDR,
> + .Register = NSLM96163_LOC_TEMP,
> + .Data = 0,
> + };
> +
> + ret = smbus_read_byte(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_byte error.");
> + else
> + *temperature = (unsigned long)in_data.Data * 10;
> +}
> +
> +static ssize_t ec_get_sys_temperature_value_via_f75387_ec_smbus(uchar *temperature)
> +{
> + int ret;
> + uchar Temp_MSB, Temp_LSB;
> +
> + struct EC_SMBUS_READ_BYTE in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = LMF75387_SMBUS_SLAVE_ADDRESS_5A,
> + .Register = F75387_REG_R_TEMP0_MSB,
> + .Data = 0,
> + };
> +
> + ret = smbus_read_byte(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_byte error.\n");
> +
> + Temp_MSB = in_data.Data;
> + if (Temp_MSB != 0xFF) {
> + in_data.Register = F75387_REG_R_TEMP0_LSB;
> + ret = smbus_read_byte(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_byte error.\n");
> +
> + Temp_LSB = in_data.Data;
> + } else {
> + Temp_MSB = 0;
> + Temp_LSB = 0;
> + }
> + *temperature = Temp_MSB + Temp_LSB/256;
> +
> + return 0;
> +}
> +
> +static int ec_get_voltage_v1_value_via_f75387_ec_smbus(uchar *voltage)
> +{
> + int ret;
> + struct EC_SMBUS_READ_BYTE in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = LMF75387_SMBUS_SLAVE_ADDRESS_5A,
> + .Register = F75387_REG_R_V1,
> + .Data = 0,
> + };
> +
> + ret = smbus_read_byte(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_byte error.\n");
> +
> + *voltage = in_data.Data;
> + return 0;
> +}
> +
> +static int ec_get_voltage_v2_value_via_f75387_ec_smbus(uchar *voltage)
> +{
> + int ret;
> + struct EC_SMBUS_READ_BYTE in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = LMF75387_SMBUS_SLAVE_ADDRESS_5A,
> + .Register = F75387_REG_R_V2,
> + .Data = 0,
> + };
> +
> + ret = smbus_read_byte(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_byte error.\n");
> +
> + *voltage = in_data.Data;
> + return 0;
> +}
> +
> +static int ec_get_voltage_system_value_via_ina226_ec_smbus(uchar *voltage)
> +{
> + int ret;
> + struct EC_SMBUS_WORD_DATA in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = INA266_SMBUS_SLAVE_ADDRESS_8A,
> + .Register = INA266_REG_VOLTAGE,
> + .Value = 0,
> + };
> +
> + ret = smbus_read_word(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_word error.\n");
> +
> + *voltage = in_data.Value;
> + return 0;
> +}
> +
> +static int ec_get_current_value_via_ina226_ec_smbus(uchar *curr)
> +{
> + int ret;
> + struct EC_SMBUS_WORD_DATA in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = INA266_SMBUS_SLAVE_ADDRESS_8A,
> + .Register = INA266_REG_CURRENT,
> + .Value = 0,
> + };
> +
> + ret = smbus_read_word(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_word error.\n");
> +
> + *curr = in_data.Value;
> + return 0;
> +}
> +
> +static int ec_get_power_value_via_ina226_ec_smbus(uchar *power)
> +{
> + int ret;
> + struct EC_SMBUS_WORD_DATA in_data = {
> + .Channel = lmsensor_data.ec_smboem0.HWPinNumber & 0x03,
> + .Address = INA266_SMBUS_SLAVE_ADDRESS_8A,
> + .Register = INA266_REG_POWER,
> + .Value = 0,
> + };
> +
> + ret = smbus_read_word(&in_data);
> + if (ret)
> + dev_err(lmsensor_data.dev, "smbus_read_word error.\n");
> +
> + *power = in_data.Value;
> + return 0;
> +}
> +
> +static ssize_t get_ec_temp2_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp = 0;
> + uchar value;
> + char *product = lmsensor_data.bios_product_name;
> +
> + if (!adspname_detect(product, "TPC-8100TR")) {
> + get_temperaturedts(&temp);
> + return sprintf(buf, "%d\n", temp);
> + } else if (!adspname_detect(product, "TPC-*51T-E??E")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if ((!adspname_detect(product, "TPC-*51WP-E?AE"))
> + || (!adspname_detect(product, "TPC-*81WP-4???E"))) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "TPC-1?82H-4???E")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if ((!adspname_detect(product, "TPC-B500-6??AE"))
> + || (!adspname_detect(product, "TPC-5???T-6??AE"))
> + || (!adspname_detect(product, "TPC-5???W-6??AE"))
> + || (!adspname_detect(product, "TPC-B200-???AE"))
> + || (!adspname_detect(product, "TPC-2???T-???AE"))
> + || (!adspname_detect(product, "TPC-2???W-???AE"))) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-1172A")) {
> + ec_get_temperature_value_via_lm96163_ec_smbus(&temp);
> + return sprintf(buf, "%d\n", 100*temp);
> + } else if (!adspname_detect(product, "UNO-1372G-E?AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-1372G-J0?1AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-1483G-4??AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2372G")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2473G")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2271G-E??AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2271G-E???AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-420")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2483G-4??AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2484G-6???AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-2484G-7???AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-3283G/3285G-674AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "UNO-3483G-3??AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "PR/VR4")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "ECU-4784")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + } else if (!adspname_detect(product, "APAX-5580-4??AE")) {
> + read_acpi_value(0x61, &value);
> + return sprintf(buf, "%d\n", 1000*value);
> + }
> +
> + return 0;
> +}
> +
> +static ssize_t get_ec_temp3_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp = 0;
> + char *product = lmsensor_data.bios_product_name;
> +
> + if (!adspname_detect(product, "APAX-5580-4??AE")) {
> + ec_get_sys_temperature_value_via_f75387_ec_smbus(&temp);
> + return sprintf(buf, "%d\n", 1000*temp);
> + } else if (!adspname_detect(product, "PR/VR4")) {
> + read_acpi_value(0x60, &temp);
> + return sprintf(buf, "%d\n", 1000*temp);
> + }
> +
> + return 0;
> +}
> +
> +static ssize_t get_ec_in5_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar voltage = 0;
> + char *product = lmsensor_data.bios_product_name;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + if (!adspname_detect(product, "APAX-5580-4??AE")) {
> + ec_get_voltage_v1_value_via_f75387_ec_smbus(&voltage);
> +
> + if (profile->r2_vin != 0)
> + voltage = voltage * (profile->r1_vin + profile->r2_vin) / profile->r2_vin;
> +
> + if (profile->resolution_vin != 0)
> + voltage = voltage * profile->resolution_vin / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[4] = voltage;
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[4]);
> + }
> +
> + return 0;
> +}
> +
> +static ssize_t get_ec_in6_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar voltage = 0;
> + char *product = lmsensor_data.bios_product_name;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + if (!adspname_detect(product, "APAX-5580-4??AE")) {
> + ec_get_voltage_v2_value_via_f75387_ec_smbus(&voltage);
> +
> + if (profile->r2_vin != 0)
> + voltage = voltage * (profile->r1_vin + profile->r2_vin) / profile->r2_vin;
> +
> + if (profile->resolution_vin != 0)
> + voltage = voltage * profile->resolution_vin / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[5] = voltage;
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[5]);
> + }
> +
> + return 0;
> +}
> +
> +static ssize_t get_ec_in7_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar voltage = 0;
> + char *product = lmsensor_data.bios_product_name;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + if (!adspname_detect(product, "APAX-5580-4??AE")) {
> + ec_get_voltage_system_value_via_ina226_ec_smbus(&voltage);
> +
> + if (profile->r2_sys != 0)
> + voltage = voltage * (profile->r1_sys + profile->r2_sys) / profile->r2_sys;
> +
> + if (profile->resolution_sys != 0)
> + voltage = voltage * profile->resolution_sys / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[5] = voltage;
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[5]);
> + }
> +
> + return 0;
> +}
> +
> +static ssize_t get_ec_in8_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp;
> + uchar voltage = 0;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + temp = read_ad_value(ptbl->v12[0], ptbl->v12[1]);
> + if (temp == -1)
> + temp = read_ad_value(ptbl->vdc[0], ptbl->vdc[1]);
> +
> + if (profile->r2 != 0)
> + voltage = temp * (profile->r1 + profile->r2) / profile->r2;
> +
> + if (profile->resolution != 0)
> + voltage = temp * profile->resolution / 1000 / 1000;
> +
> + if (profile->offset != 0)
> + voltage += (int)profile->offset * 100;
> +
> + lmsensor_data.voltage[2] = 10*voltage;
> + return sprintf(buf, "%d\n", lmsensor_data.voltage[2]);
> +}
> +
> +static ssize_t get_ec_curr2_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp = 0;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + ec_get_current_value_via_ina226_ec_smbus(&temp);
> + if (profile->r2_curr != 0)
> + temp = temp * (profile->r1_curr + profile->r2_curr) / profile->r2_curr;
> +
> + if (profile->resolution_curr != 0)
> + temp = temp * profile->resolution_curr / 1000;
> +
> + if (profile->offset != 0)
> + temp += (int)profile->offset * 100;
> +
> + lmsensor_data.ec_current[4] = temp;
> + return sprintf(buf, "%d\n", lmsensor_data.ec_current[4]);
> +}
> +
> +static ssize_t get_ec_power1_input(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> + uchar temp = 0;
> + struct adv_hwmon_profile *profile = lmsensor_data.profile;
> +
> + ec_get_power_value_via_ina226_ec_smbus(&temp);
> +
> + if (profile->r2_power != 0)
> + temp = temp * (profile->r1_power + profile->r2_power) / profile->r2_power;
> +
> + if (profile->resolution_power != 0)
> + temp = temp * profile->resolution_power / 1000;
> +
> + if (profile->offset != 0)
> + temp += (int)profile->offset * 100;
> +
> + lmsensor_data.power[1] = 1000*temp;
> + return sprintf(buf, "%d\n", lmsensor_data.power[1]);
> +}
> +
> +static SENSOR_DEVICE_ATTR(name, 0444, get_ec_hwmon_name, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in1_input, 0444, get_ec_in1_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in1_label, 0444, get_ec_in1_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in2_input, 0444, get_ec_in2_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in2_label, 0444, get_ec_in2_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in3_input, 0444, get_ec_in3_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in3_label, 0444, get_ec_in3_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in4_input, 0444, get_ec_in4_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in4_label, 0444, get_ec_in4_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in5_input, 0444, get_ec_in5_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in5_label, 0444, get_ec_in5_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in6_input, 0444, get_ec_in6_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in6_label, 0444, get_ec_in6_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in7_input, 0444, get_ec_in7_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in7_label, 0444, get_ec_in7_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in8_input, 0444, get_ec_in8_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(in8_label, 0444, get_ec_in8_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(curr1_label, 0444, get_ec_curr1_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(curr1_input, 0444, get_ec_curr1_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(curr2_label, 0444, get_ec_curr2_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(curr2_input, 0444, get_ec_curr2_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(power1_label, 0444, get_ec_power1_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(power1_input, 0444, get_ec_power1_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp1_label, 0444, get_ec_temp1_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp1_input, 0444, show_temp11, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp1_max_hyst, 0644, show_temp_max_hyst, set_temp_max_hyst, 0);
> +static SENSOR_DEVICE_ATTR(temp1_max, 0644, show_temp_max, set_temp_max, 0);
> +static SENSOR_DEVICE_ATTR(temp2_label, 0444, get_ec_temp2_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp2_input, 0444, get_ec_temp2_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp2_crit, 0400, get_ec_temp2_crit, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp2_crit_alarm, 0400, get_ec_temp2_crit_alarm, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp3_label, 0444, get_ec_temp3_label, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp3_input, 0444, get_ec_temp3_input, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp3_crit, 0400, get_ec_temp3_crit, NULL, 0);
> +static SENSOR_DEVICE_ATTR(temp3_crit_alarm, 0400, get_ec_temp3_crit_alarm, NULL, 0);
> +
> +/*
> + * Support list:
> + * TPC-8100TR, TPC-651T-E3AE, TPC-1251T-E3AE, TPC-1551T-E3AE,
> + * TPC-1751T-E3AE, TPC-1051WP-E3AE, TPC-1551WP-E3AE, TPC-1581WP-433AE
> + * TPC-1782H-433AE,
> + * UNO-1483G-434AE, UNO-2483G-434AE, UNO-3483G-374AE, UNO-2473G
> + * UNO-2484G-6???AE, UNO-2484G-7???AE, UNO-3283G-674AE, UNO-3285G-674AE
> + */
> +static const struct attribute *ec_hwmon_attrs_TEMPLATE[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in3_input.dev_attr.attr,
> + &sensor_dev_attr_in3_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_curr1_label.dev_attr.attr,
> + &sensor_dev_attr_curr1_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * TPC-B500-6??AE
> + * TPC-5???T-6??AE
> + * TPC-5???W-6??AE
> + * TPC-B200-???AE
> + * TPC-2???T-???AE
> + * TPC-2???W-???AE
> + */
> +static const struct attribute *ec_hwmon_attrs_TPC5XXX[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in3_input.dev_attr.attr,
> + &sensor_dev_attr_in3_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * PR/VR4
> + */
> +static const struct attribute *ec_hwmon_attrs_PRVR4[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in3_input.dev_attr.attr,
> + &sensor_dev_attr_in3_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + &sensor_dev_attr_temp3_label.dev_attr.attr,
> + &sensor_dev_attr_temp3_input.dev_attr.attr,
> + &sensor_dev_attr_temp3_crit.dev_attr.attr,
> + &sensor_dev_attr_temp3_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * UNO-2271G-E22AE/E23AE/E022AE/E023AE, UNO-420
> + */
> +static const struct attribute *ec_hwmon_attrs_UNO2271G[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in3_input.dev_attr.attr,
> + &sensor_dev_attr_in3_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * UNO-1172A
> + * ECU-4784
> + */
> +static const struct attribute *ec_hwmon_attrs_UNO1172A[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in3_input.dev_attr.attr,
> + &sensor_dev_attr_in3_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * UNO-1372G
> + */
> +static const struct attribute *ec_hwmon_attrs_UNO1372G[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_in8_input.dev_attr.attr,
> + &sensor_dev_attr_in8_label.dev_attr.attr,
> + &sensor_dev_attr_curr1_label.dev_attr.attr,
> + &sensor_dev_attr_curr1_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * UNO-2372G, UNO-1372G-J021AE/J031AE
> + */
> +static const struct attribute *ec_hwmon_attrs_UNO2372G[] = {
> + //sor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in4_input.dev_attr.attr,
> + &sensor_dev_attr_in4_label.dev_attr.attr,
> + &sensor_dev_attr_in8_input.dev_attr.attr,
> + &sensor_dev_attr_in8_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +/*
> + * Support list:
> + * APAX-5580-433AE/473AE/4C3AE
> + */
> +static const struct attribute *ec_hwmon_attrs_APAX5580[] = {
> + &sensor_dev_attr_name.dev_attr.attr,
> + &sensor_dev_attr_in1_input.dev_attr.attr,
> + &sensor_dev_attr_in1_label.dev_attr.attr,
> + &sensor_dev_attr_in2_input.dev_attr.attr,
> + &sensor_dev_attr_in2_label.dev_attr.attr,
> + &sensor_dev_attr_in5_input.dev_attr.attr,
> + &sensor_dev_attr_in5_label.dev_attr.attr,
> + &sensor_dev_attr_in6_input.dev_attr.attr,
> + &sensor_dev_attr_in6_label.dev_attr.attr,
> + &sensor_dev_attr_in7_input.dev_attr.attr,
> + &sensor_dev_attr_in7_label.dev_attr.attr,
> + &sensor_dev_attr_curr2_label.dev_attr.attr,
> + &sensor_dev_attr_curr2_input.dev_attr.attr,
> + &sensor_dev_attr_power1_label.dev_attr.attr,
> + &sensor_dev_attr_power1_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_label.dev_attr.attr,
> + &sensor_dev_attr_temp2_input.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit.dev_attr.attr,
> + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr,
> + &sensor_dev_attr_temp3_label.dev_attr.attr,
> + &sensor_dev_attr_temp3_input.dev_attr.attr,
> + &sensor_dev_attr_temp3_crit.dev_attr.attr,
> + &sensor_dev_attr_temp3_crit_alarm.dev_attr.attr,
> + NULL
> +};
> +
> +static struct attribute *f75375_attributes[] = {
> + &sensor_dev_attr_temp1_label.dev_attr.attr,
> + &sensor_dev_attr_temp1_input.dev_attr.attr,
> + &sensor_dev_attr_temp1_max.dev_attr.attr,
> + &sensor_dev_attr_temp1_max_hyst.dev_attr.attr,
> + NULL
> +};
> +
> +static const struct attribute_group f75375_group = {
> + .attrs = f75375_attributes,
> +};
> +
> +static struct adv_hwmon_profile advec_profile[] = {
> + /*
> + * TPC-8100TR, TPC-651T-E3AE, TPC-1251T-E3AE, TPC-1551T-E3AE,
> + * TPC-1751T-E3AE, TPC-1051WP-E3AE, TPC-1551WP-E3AE, TPC-1581WP-433AE,
> + * TPC-1782H-433AE, UNO-1483G-434AE, UNO-2483G-434AE, UNO-3483G-374AE,
> + * UNO-2473G, UNO-2484G-6???AE, UNO-2484G-7???AE, UNO-3283G-674AE,
> + * UNO-3285G-674AE
> + * (0)
> + */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_TEMPLATE,
> + },
> + /*
> + * TPC-B500-6??AE, TPC-5???T-6??AE, TPC-5???W-6??AE, TPC-B200-???AE,
> + * TPC-2???T-???AE, TPC-2???W-???AE
> + * (1)
> + */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_TPC5XXX,
> + },
> + /* PR/VR4 (2) */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_PRVR4,
> + },
> + /* UNO-2271G-E22AE/E23AE/E022AE/E023AE,UNO-420 (3) */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_UNO2271G,
> + },
> + /* UNO-1172A, ECU-4784 (4) */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_UNO1172A,
> + },
> + /* UNO-1372G (5) */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_UNO1372G,
> + },
> + /* UNO-2372G, UNO-1372G-J021AE/J031AE (6) */
> + {
> + .resolution = 2929,
> + .r1 = 1912,
> + .r2 = 1000,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_UNO2372G,
> + },
> + /* APAX-5580-433AE, APAX-5580-473AE, APAX-5580-4C3AE (7) */
> + {
> + .resolution = 2929,
> + .resolution_sys = 1250,
> + .resolution_curr = 1000,
> + .resolution_power = 25000,
> + .r1 = 0,
> + .r1_vin = 1120,
> + .r1_sys = 0,
> + .r1_curr = 0,
> + .r1_power = 0,
> + .r2 = 0,
> + .r2_vin = 56,
> + .r2_sys = 0,
> + .r2_curr = 0,
> + .r2_power = 0,
> + .offset = 0,
> + .ec_hwmon_group.attrs = (struct attribute **)ec_hwmon_attrs_APAX5580,
> + },
> +};
> +
> +static int adspname_detect(const char *bios_product_name, const char *standard_name)
> +{
> + int i, j;
> +
> + i = 0;
> +
> + for (j = 0; j < strlen(bios_product_name); j++) {
> + if (standard_name[i] == '*') {
> + if (i) {
> + if (bios_product_name[j] == standard_name[(i + 1)])
> + i += 2;
> +
> + if (i >= (strlen(standard_name) - 1))
> + return 0;
> + }
> + } else if (standard_name[i] == '?') {
> + if (i) {
> + i++;
> + if (i >= strlen(standard_name))
> + return 0;
> + }
> + } else if (bios_product_name[j] == standard_name[i]) {
> + i++;
> + if (i >= strlen(standard_name))
> + return 0;
> + }
> + }
> +
> + return 1;
> +}
> +
> +static void adv_ec_init_hwmon_profile(u32 profile, struct adv_ec_platform_data *plat_data)
> +{
> + int i;
> + struct HW_PIN_TBL *ptbl = &lmsensor_data.pin_tbl;
> + struct Dynamic_Tab *dym_tbl = plat_data->dym_tbl;
> +
> + lmsensor_data.bios_product_name = plat_data->bios_product_name;
> + lmsensor_data.profile = &advec_profile[profile];
> +
> + for (i = 0; i < EC_MAX_TBL_NUM ; i++) {
> + switch (dym_tbl[i].DeviceID) {
> + case EC_DID_CMOSBAT:
> + ptbl->vbat[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vbat[1] = 1;
> + break;
> + case EC_DID_CMOSBAT_X2:
> + ptbl->vbat[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vbat[1] = 2;
> + break;
> + case EC_DID_CMOSBAT_X10:
> + ptbl->vbat[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vbat[1] = 10;
> + break;
> + case EC_DID_5VS0:
> + case EC_DID_5VS5:
> + ptbl->v5[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v5[1] = 1;
> + break;
> + case EC_DID_5VS0_X2:
> + case EC_DID_5VS5_X2:
> + ptbl->v5[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v5[1] = 2;
> + break;
> + case EC_DID_5VS0_X10:
> + case EC_DID_5VS5_X10:
> + ptbl->v5[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v5[1] = 10;
> + break;
> + case EC_DID_12VS0:
> + ptbl->v12[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v12[1] = 1;
> + break;
> + case EC_DID_12VS0_X2:
> + ptbl->v12[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v12[1] = 2;
> + break;
> + case EC_DID_12VS0_X10:
> + ptbl->v12[0] = dym_tbl[i].HWPinNumber;
> + ptbl->v12[1] = 10;
> + break;
> + case EC_DID_VCOREA:
> + case EC_DID_VCOREB:
> + ptbl->vcore[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vcore[1] = 1;
> + break;
> + case EC_DID_VCOREA_X2:
> + case EC_DID_VCOREB_X2:
> + ptbl->vcore[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vcore[1] = 2;
> + break;
> + case EC_DID_VCOREA_X10:
> + case EC_DID_VCOREB_X10:
> + ptbl->vcore[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vcore[1] = 10;
> + break;
> + case EC_DID_DC:
> + ptbl->vdc[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vdc[1] = 1;
> + break;
> + case EC_DID_DC_X2:
> + ptbl->vdc[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vdc[1] = 2;
> + break;
> + case EC_DID_DC_X10:
> + ptbl->vdc[0] = dym_tbl[i].HWPinNumber;
> + ptbl->vdc[1] = 10;
> + break;
> + case EC_DID_CURRENT:
> + ptbl->ec_current[0] = dym_tbl[i].HWPinNumber;
> + ptbl->ec_current[1] = 1;
> + break;
> + case EC_DID_SMBOEM0:
> + lmsensor_data.ec_smboem0.HWPinNumber = dym_tbl[i].HWPinNumber;
> + break;
> + default:
> + break;
> + }
> + }
> +}
> +
> +static int f75375_probe(struct i2c_client *client, const struct i2c_device_id *id)
> +{
> + struct f75375_data *data;
> + int err;
> +
> + if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
> + return -EIO;
> + data = devm_kzalloc(&client->dev, sizeof(struct f75375_data), GFP_KERNEL);
> + if (!data)
> + return -ENOMEM;
> +
> + i2c_set_clientdata(client, data);
> + mutex_init(&data->update_lock);
> + data->kind = id->driver_data;
> +
> + err = sysfs_create_group(&client->dev.kobj, &f75375_group);
> + if (err)
> + return err;
> +
> + data->hwmon_dev = hwmon_device_register(&client->dev);
> + if (IS_ERR(data->hwmon_dev)) {
> + err = PTR_ERR(data->hwmon_dev);
> + goto exit_remove;
> + }
> + return 0;
> +
> +exit_remove:
> + sysfs_remove_group(&client->dev.kobj, &f75375_group);
> + return err;
> +}
> +
> +static int f75375_remove(struct i2c_client *client)
> +{
> + struct f75375_data *data = i2c_get_clientdata(client);
> +
> + hwmon_device_unregister(data->hwmon_dev);
> + sysfs_remove_group(&client->dev.kobj, &f75375_group);
> + return 0;
> +}
> +
> +/* Return 0 if detection is successful, -ENODEV otherwise */
> +static int f75375_detect(struct i2c_client *client, struct i2c_board_info *info)
> +{
> + struct i2c_adapter *adapter = client->adapter;
> + u16 vendid, chipid;
> + u8 version;
> + const char *name;
> +
> + vendid = f75375_read16(client, F75375_REG_VENDOR);
> + chipid = f75375_read16(client, F75375_CHIP_ID);
> + dev_info(&adapter->dev, "VendID: 0x%x, ChipID: 0x%x", vendid, chipid);
> +
> + if (vendid != 0x1934)
> + return -ENODEV;
> +
> + if (chipid == 0x0306)
> + name = "f75375";
> + else if (chipid == 0x0204)
> + name = "f75373";
> + else if (chipid == 0x0410)
> + name = "f75387";
> + else
> + return -ENODEV;
> +
> + version = f75375_read8(client, F75375_REG_VERSION);
> + dev_info(&adapter->dev, "found %s version: %02X\n", name, version);
> + strlcpy(info->type, name, I2C_NAME_SIZE);
> +
> + return 0;
> +}
> +
> +static int adv_ec_hwmon_probe(struct platform_device *pdev)
> +{
> + int ret = 0;
> + u32 profile;
> + struct adv_ec_platform_data *adv_ec_data = NULL;
> +
> + adv_ec_data = (struct adv_ec_platform_data *)dev_get_drvdata(pdev->dev.parent);
> +
> + ret = device_property_read_u32(pdev->dev.parent, "advantech,hwmon-profile", &profile);
> + if (ret < 0) {
> + dev_err(&pdev->dev, "get hwmon-profile failed! (%d)", ret);
> + return ret;
> + }
> +
> + adv_ec_init_hwmon_profile(profile, adv_ec_data);
> +
> + lmsensor_data.dev = &pdev->dev;
> + lmsensor_data.hwmon_dev = hwmon_device_register(NULL);
> +
> + if (IS_ERR(lmsensor_data.hwmon_dev)) {
> + ret = -ENOMEM;
> + dev_err(lmsensor_data.dev, "hwmon_dev register failed\n");
> + goto fail_hwmon_device_register;
> + }
> +
> + ret = sysfs_create_group(&lmsensor_data.hwmon_dev->kobj,
> + &lmsensor_data.profile->ec_hwmon_group);
> + if (ret) {
> + dev_err(lmsensor_data.dev, "failed to creat ec hwmon\n");
> + goto fail_create_group_hwmon;
> + }
> +
> + if (!adspname_detect(adv_ec_data->bios_product_name, "UNO-1172A")) {
> + ret = i2c_add_driver(&f75375_driver);
> + if (ret)
> + dev_err(lmsensor_data.dev, "failed to register driver f75375.\n");
> + }
> +
> + dev_info(&pdev->dev, "Ver:%s, Data:%s, probe done",
> + ADVANTECH_EC_HWMON_VER, ADVANTECH_EC_HWMON_DATE);
> +
> + return ret;
> +
> +fail_create_group_hwmon:
> + sysfs_remove_group(&lmsensor_data.hwmon_dev->kobj, &lmsensor_data.profile->ec_hwmon_group);
> +fail_hwmon_device_register:
> + hwmon_device_unregister(lmsensor_data.hwmon_dev);
> +
> + return ret;
> +}
> +
> +static int adv_ec_hwmon_remove(struct platform_device *pdev)
> +{
> + sysfs_remove_group(&lmsensor_data.hwmon_dev->kobj, &lmsensor_data.profile->ec_hwmon_group);
> + hwmon_device_unregister(lmsensor_data.hwmon_dev);
> + if (!adspname_detect(lmsensor_data.bios_product_name, "UNO-1172A"))
> + i2c_del_driver(&f75375_driver);
> +
> + return 0;
> +}
> +
> +static struct platform_driver adv_hwmon_drv = {
> + .driver = {
> + .name = "adv-ec-hwmon",
> + },
> + .probe = adv_ec_hwmon_probe,
> + .remove = adv_ec_hwmon_remove,
> +};
> +
> +module_platform_driver(adv_hwmon_drv);
> +
> +MODULE_DESCRIPTION("Advantech EC Hwmon Driver.");
> +MODULE_AUTHOR("Jiangwei.Zhu");
> +MODULE_LICENSE("Dual BSD/GPL");
> --
> 2.17.1
>
Powered by blists - more mailing lists