[PATCH linux dev-5.4 v1 0/1] hwmon: (pmbus) Add Infineon IR38164, Renesas RAA228006, and TI SN1701022 driver
Joel Stanley
joel at jms.id.au
Mon Jan 6 17:03:15 AEDT 2020
On Thu, 26 Dec 2019 at 17:13, Guenter Roeck <linux at roeck-us.net> wrote:
>
> On 12/26/19 12:57 AM, Andrew Peng wrote:
> > Add the pmbus driver for Infineon IR38164 Voltage Regulator driver.
> > Add the pmbus driver for Renesas RAA228006 Digital PWM Controller.
> > Add the pmbus driver for TI SN1701022 driver.
> >
> > Signed-off-by: Andrew Peng <pengms1 at lenovo.com>
> > Signed-off-by: Derek Lin <dlin23 at lenovo.com>
> > Signed-off-by: Yonghui Liu <liuyh21 at lenovo.com>
> > ---
>
> This is annoying, and each time it happens it is more annoying.
> Can you folks please stop sending me patches for non-upstream kernel branches ?
Andrew, when submitting the patch for inclusion in OpenBMC you should
only send it to the openbmc list, and openbmc reviewers. You can use
--suppress-cc to stop git-send-email from picking up other addresses
in the commit log.
Cheers,
Joel
>
> Thanks,
> Guenter
>
> > v1: initial version
> >
> > drivers/hwmon/pmbus/Kconfig | 25 ++++++++
> > drivers/hwmon/pmbus/Makefile | 3 +
> > drivers/hwmon/pmbus/ir38164.c | 72 +++++++++++++++++++++++
> > drivers/hwmon/pmbus/raa228006.c | 127 ++++++++++++++++++++++++++++++++++++++++
> > drivers/hwmon/pmbus/sn1701022.c | 80 +++++++++++++++++++++++++
> > 5 files changed, 307 insertions(+)
> > create mode 100644 drivers/hwmon/pmbus/ir38164.c
> > create mode 100644 drivers/hwmon/pmbus/raa228006.c
> > create mode 100644 drivers/hwmon/pmbus/sn1701022.c
> >
> > diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
> > index 6106263..2312b58 100644
> > --- a/drivers/hwmon/pmbus/Kconfig
> > +++ b/drivers/hwmon/pmbus/Kconfig
> > @@ -73,6 +73,15 @@ config SENSORS_IR38064
> > This driver can also be built as a module. If so, the module will
> > be called ir38064.
> >
> > +config SENSORS_IR38164
> > + tristate "Infineon IR38164"
> > + help
> > + If you say yes here you get hardware monitoring support for Infineon
> > + IR38164.
> > +
> > + This driver can also be built as a module. If so, the module will
> > + be called ir38164.
> > +
> > config SENSORS_IRPS5401
> > tristate "Infineon IRPS5401"
> > help
> > @@ -230,4 +239,20 @@ config SENSORS_ZL6100
> > This driver can also be built as a module. If so, the module will
> > be called zl6100.
> >
> > +config SENSORS_RAA228006
> > + tristate "RAA228006 and compatibles"
> > + help
> > + If you say yes here you get hardware monitoring support for RAA228006.
> > +
> > + This driver can also be built as a module. If so, the module will
> > + be called RAA228006.
> > +
> > +config SENSORS_SN1701022
> > + tristate "SN1701022"
> > + help
> > + If you say yes here you get hardware monitoring support for SN1701022.
> > +
> > + This driver can also be built as a module. If so, the module will
> > + be called sn1701022.
> > +
> > endif # PMBUS
> > diff --git a/drivers/hwmon/pmbus/Makefile b/drivers/hwmon/pmbus/Makefile
> > index 03bacfc..ba3aed5 100644
> > --- a/drivers/hwmon/pmbus/Makefile
> > +++ b/drivers/hwmon/pmbus/Makefile
> > @@ -10,6 +10,7 @@ obj-$(CONFIG_SENSORS_IBM_CFFPS) += ibm-cffps.o
> > obj-$(CONFIG_SENSORS_INSPUR_IPSPS) += inspur-ipsps.o
> > obj-$(CONFIG_SENSORS_IR35221) += ir35221.o
> > obj-$(CONFIG_SENSORS_IR38064) += ir38064.o
> > +obj-$(CONFIG_SENSORS_IR38164) += ir38164.o
> > obj-$(CONFIG_SENSORS_IRPS5401) += irps5401.o
> > obj-$(CONFIG_SENSORS_ISL68137) += isl68137.o
> > obj-$(CONFIG_SENSORS_LM25066) += lm25066.o
> > @@ -26,3 +27,5 @@ obj-$(CONFIG_SENSORS_TPS53679) += tps53679.o
> > obj-$(CONFIG_SENSORS_UCD9000) += ucd9000.o
> > obj-$(CONFIG_SENSORS_UCD9200) += ucd9200.o
> > obj-$(CONFIG_SENSORS_ZL6100) += zl6100.o
> > +obj-$(CONFIG_SENSORS_RAA228006) += raa228006.o
> > +obj-$(CONFIG_SENSORS_SN1701022) += sn1701022.o
> > diff --git a/drivers/hwmon/pmbus/ir38164.c b/drivers/hwmon/pmbus/ir38164.c
> > new file mode 100644
> > index 0000000..db89473
> > --- /dev/null
> > +++ b/drivers/hwmon/pmbus/ir38164.c
> > @@ -0,0 +1,72 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Hardware monitoring driver for Infineon ir38164
> > + *
> > + * Copyright (C) 2019-present Lenovo
> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License
> > + * as published by the Free Software Foundation; either version 2
> > + * of the License, or (at your option) any later version.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU General Public License
> > + * along with this program; if not, write to the Free Software
> > + * Foundation, Inc., 51 Franklin Street, Fifth Floor,
> > + * Boston, MA 02110-1301, USA.
> > + */
> > +
> > +#include <linux/err.h>
> > +#include <linux/i2c.h>
> > +#include <linux/init.h>
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include "pmbus.h"
> > +
> > +static struct pmbus_driver_info ir38164_info = {
> > + .pages = 1,
> > + .format[PSC_VOLTAGE_IN] = linear,
> > + .format[PSC_VOLTAGE_OUT] = linear,
> > + .format[PSC_CURRENT_OUT] = linear,
> > + .format[PSC_POWER] = linear,
> > + .format[PSC_TEMPERATURE] = linear,
> > +
> > + .func[0] = PMBUS_HAVE_VIN | PMBUS_HAVE_STATUS_INPUT
> > + | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP
> > + | PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT
> > + | PMBUS_HAVE_IOUT | PMBUS_HAVE_STATUS_IOUT
> > + | PMBUS_HAVE_POUT,
> > +};
> > +
> > +static int ir38164_probe(struct i2c_client *client,
> > + const struct i2c_device_id *id)
> > +{
> > + return pmbus_do_probe(client, id, &ir38164_info);
> > +}
> > +
> > +static const struct i2c_device_id ir38164_id[] = {
> > + {"ir38164", 0},
> > + {}
> > +};
> > +
> > +MODULE_DEVICE_TABLE(i2c, ir38164_id);
> > +
> > +/* This is the driver that will be inserted */
> > +static struct i2c_driver ir38164_driver = {
> > + .driver = {
> > + .name = "ir38164",
> > + },
> > + .probe = ir38164_probe,
> > + .remove = pmbus_do_remove,
> > + .id_table = ir38164_id,
> > +};
> > +
> > +module_i2c_driver(ir38164_driver);
> > +
> > +MODULE_AUTHOR("Yonghui Liu <liuyh21 at lenovo.com>");
> > +MODULE_DESCRIPTION("PMBus driver for Infineon ir38164");
> > +MODULE_LICENSE("GPL");
> > diff --git a/drivers/hwmon/pmbus/raa228006.c b/drivers/hwmon/pmbus/raa228006.c
> > new file mode 100644
> > index 0000000..9aed481
> > --- /dev/null
> > +++ b/drivers/hwmon/pmbus/raa228006.c
> > @@ -0,0 +1,127 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Hardware monitoring driver for raa228006
> > + *
> > + * Copyright (C) 2019-present Lenovo
> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License
> > + * as published by the Free Software Foundation; either version 2
> > + * of the License, or (at your option) any later version.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU General Public License
> > + * along with this program; if not, write to the Free Software
> > + * Foundation, Inc., 51 Franklin Street, Fifth Floor,
> > + * Boston, MA 02110-1301, USA.
> > + */
> > +
> > +#include <linux/err.h>
> > +#include <linux/i2c.h>
> > +#include <linux/init.h>
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include "pmbus.h"
> > +
> > +static int raa228006_probe(struct i2c_client *client,
> > + const struct i2c_device_id *id)
> > +{
> > + struct pmbus_driver_info *info;
> > + u8 buf[I2C_SMBUS_BLOCK_MAX];
> > + int ret;
> > +
> > + if (!i2c_check_functionality(client->adapter,
> > + I2C_FUNC_SMBUS_READ_BYTE_DATA
> > + | I2C_FUNC_SMBUS_READ_WORD_DATA
> > + | I2C_FUNC_SMBUS_READ_BLOCK_DATA))
> > + return -ENODEV;
> > +
> > + /* By default this device doesn't boot to page 0, so set page 0
> > + * to access all pmbus registers.
> > + */
> > +
> > + i2c_smbus_write_byte_data(client, 0, 0);
> > +
> > + /* Read Manufacturer id */
> > + ret = i2c_smbus_read_block_data(client, PMBUS_MFR_ID, buf);
> > + if (ret < 0) {
> > + dev_err(&client->dev, "Failed to read PMBUS_MFR_ID\n");
> > + return ret;
> > + }
> > +
> > + info = devm_kzalloc(&client->dev, sizeof(struct pmbus_driver_info),
> > + GFP_KERNEL);
> > + if (!info)
> > + return -ENOMEM;
> > +
> > + info->format[PSC_VOLTAGE_IN] = direct;
> > + info->format[PSC_VOLTAGE_OUT] = direct;
> > + info->format[PSC_CURRENT_IN] = direct;
> > + info->format[PSC_CURRENT_OUT] = direct;
> > + info->format[PSC_POWER] = direct;
> > + info->format[PSC_TEMPERATURE] = direct;
> > + info->vrm_version = vr13;
> > +
> > + info->m[PSC_VOLTAGE_IN] = 1,
> > + info->b[PSC_VOLTAGE_IN] = 0,
> > + info->R[PSC_VOLTAGE_IN] = 3,
> > +
> > + info->m[PSC_VOLTAGE_OUT] = 1,
> > + info->b[PSC_VOLTAGE_OUT] = 0,
> > + info->R[PSC_VOLTAGE_OUT] = 0,
> > +
> > + info->m[PSC_CURRENT_IN] = 1,
> > + info->b[PSC_CURRENT_IN] = 0,
> > + info->R[PSC_CURRENT_IN] = 1,
> > +
> > + info->m[PSC_CURRENT_OUT] = 1,
> > + info->b[PSC_CURRENT_OUT] = 0,
> > + info->R[PSC_CURRENT_OUT] = 2,
> > +
> > + info->m[PSC_POWER] = 1,
> > + info->b[PSC_POWER] = 0,
> > + info->R[PSC_POWER] = 3,
> > +
> > + info->m[PSC_TEMPERATURE] = 1,
> > + info->b[PSC_TEMPERATURE] = 0,
> > + info->R[PSC_TEMPERATURE] = 3,
> > +
> > + info->func[0] = PMBUS_HAVE_VIN
> > + | PMBUS_HAVE_VOUT | PMBUS_HAVE_IIN
> > + | PMBUS_HAVE_IOUT | PMBUS_HAVE_PIN
> > + | PMBUS_HAVE_POUT | PMBUS_HAVE_TEMP
> > + | PMBUS_HAVE_TEMP2 | PMBUS_HAVE_TEMP3
> > + | PMBUS_HAVE_STATUS_VOUT | PMBUS_HAVE_STATUS_IOUT
> > + | PMBUS_HAVE_STATUS_INPUT | PMBUS_HAVE_STATUS_TEMP;
> > +
> > + info->pages = id->driver_data;
> > +
> > + return pmbus_do_probe(client, id, info);
> > +}
> > +
> > +static const struct i2c_device_id raa228006_id[] = {
> > + {"raa228006", 1},
> > + {}
> > +};
> > +
> > +MODULE_DEVICE_TABLE(i2c, raa228006_id);
> > +
> > +/* This is the driver that will be inserted */
> > +static struct i2c_driver raa228006_driver = {
> > + .driver = {
> > + .name = "raa228006",
> > + },
> > + .probe = raa228006_probe,
> > + .remove = pmbus_do_remove,
> > + .id_table = raa228006_id,
> > +};
> > +
> > +module_i2c_driver(raa228006_driver);
> > +
> > +MODULE_AUTHOR("Yonghui Liu <liuyh21 at lenovo.com>");
> > +MODULE_DESCRIPTION("PMBus driver for raa228006");
> > +MODULE_LICENSE("GPL");
> > diff --git a/drivers/hwmon/pmbus/sn1701022.c b/drivers/hwmon/pmbus/sn1701022.c
> > new file mode 100644
> > index 0000000..0ba45ac
> > --- /dev/null
> > +++ b/drivers/hwmon/pmbus/sn1701022.c
> > @@ -0,0 +1,80 @@
> > +// SPDX-License-Identifier: GPL-2.0-or-later
> > +/*
> > + * Hardware monitoring driver for sn1701022
> > + *
> > + * Copyright (C) 2019-present Lenovo
> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License
> > + * as published by the Free Software Foundation; either version 2
> > + * of the License, or (at your option) any later version.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU General Public License
> > + * along with this program; if not, write to the Free Software
> > + * Foundation, Inc., 51 Franklin Street, Fifth Floor,
> > + * Boston, MA 02110-1301, USA.
> > + */
> > +
> > +#include <linux/err.h>
> > +#include <linux/i2c.h>
> > +#include <linux/init.h>
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include "pmbus.h"
> > +
> > +static struct pmbus_driver_info sn1701022_info = {
> > + .pages = 2,
> > + .format[PSC_VOLTAGE_IN] = linear,
> > + .format[PSC_VOLTAGE_OUT] = vid,
> > + .vrm_version = vr12,
> > + .format[PSC_CURRENT_OUT] = linear,
> > + .format[PSC_CURRENT_IN] = linear,
> > + .format[PSC_POWER] = linear,
> > + .format[PSC_TEMPERATURE] = linear,
> > +
> > + .func[0] = PMBUS_HAVE_VIN | PMBUS_HAVE_STATUS_INPUT
> > + | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP
> > + | PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT
> > + | PMBUS_HAVE_IOUT | PMBUS_HAVE_STATUS_IOUT
> > + | PMBUS_HAVE_POUT,
> > +
> > + .func[1] = PMBUS_HAVE_VIN | PMBUS_HAVE_STATUS_INPUT
> > + | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP
> > + | PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT
> > + | PMBUS_HAVE_IOUT | PMBUS_HAVE_STATUS_IOUT
> > + | PMBUS_HAVE_POUT,
> > +};
> > +
> > +static int sn1701022_probe(struct i2c_client *client,
> > + const struct i2c_device_id *id)
> > +{
> > + return pmbus_do_probe(client, id, &sn1701022_info);
> > +}
> > +
> > +static const struct i2c_device_id sn1701022_id[] = {
> > + {"sn1701022", 0},
> > + {}
> > +};
> > +
> > +MODULE_DEVICE_TABLE(i2c, sn1701022_id);
> > +
> > +/* This is the driver that will be inserted */
> > +static struct i2c_driver sn1701022_driver = {
> > + .driver = {
> > + .name = "sn1701022",
> > + },
> > + .probe = sn1701022_probe,
> > + .remove = pmbus_do_remove,
> > + .id_table = sn1701022_id,
> > +};
> > +
> > +module_i2c_driver(sn1701022_driver);
> > +
> > +MODULE_AUTHOR("Yonghui Liu <liuyh21 at lenovo.com>");
> > +MODULE_DESCRIPTION("PMBus driver for sn1701022");
> > +MODULE_LICENSE("GPL");
> >
>
More information about the openbmc
mailing list