123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174 |
- From 15ae701189744d321d3a1264ff46f8871e8765ee Mon Sep 17 00:00:00 2001
- From: Christian Lamparter <[email protected]>
- Date: Sun, 17 Dec 2017 17:29:13 +0100
- Subject: [PATCH] hwmon: tc654: add thermal_cooling device
- This patch adds a thermaL_cooling device to the tc654 driver.
- This allows the chip to be used for DT-based cooling.
- Signed-off-by: Christian Lamparter <[email protected]>
- ---
- drivers/hwmon/tc654.c | 103 +++++++++++++++++++++++++++++++++++++++++---------
- 1 file changed, 86 insertions(+), 17 deletions(-)
- --- a/drivers/hwmon/tc654.c
- +++ b/drivers/hwmon/tc654.c
- @@ -24,6 +24,7 @@
- #include <linux/module.h>
- #include <linux/mutex.h>
- #include <linux/slab.h>
- +#include <linux/thermal.h>
- #include <linux/util_macros.h>
-
- enum tc654_regs {
- @@ -141,6 +142,9 @@ struct tc654_data {
- * writable register used to control the duty
- * cycle of the V OUT output.
- */
- +
- + /* optional cooling device */
- + struct thermal_cooling_device *cdev;
- };
-
- /* helper to grab and cache data, at most one time per second */
- @@ -376,36 +380,30 @@ static ssize_t set_pwm_mode(struct devic
- static const int tc654_pwm_map[16] = { 77, 88, 102, 112, 124, 136, 148, 160,
- 172, 184, 196, 207, 219, 231, 243, 255};
-
- +static int get_pwm(struct tc654_data *data)
- +{
- + if (data->config & TC654_REG_CONFIG_SDM)
- + return 0;
- + else
- + return tc654_pwm_map[data->duty_cycle];
- +}
- +
- static ssize_t show_pwm(struct device *dev, struct device_attribute *da,
- char *buf)
- {
- struct tc654_data *data = tc654_update_client(dev);
- - int pwm;
-
- if (IS_ERR(data))
- return PTR_ERR(data);
-
- - if (data->config & TC654_REG_CONFIG_SDM)
- - pwm = 0;
- - else
- - pwm = tc654_pwm_map[data->duty_cycle];
- -
- - return sprintf(buf, "%d\n", pwm);
- + return sprintf(buf, "%d\n", get_pwm(data));
- }
-
- -static ssize_t set_pwm(struct device *dev, struct device_attribute *da,
- - const char *buf, size_t count)
- +static int _set_pwm(struct tc654_data *data, unsigned long val)
- {
- - struct tc654_data *data = dev_get_drvdata(dev);
- struct i2c_client *client = data->client;
- - unsigned long val;
- int ret;
-
- - if (kstrtoul(buf, 10, &val))
- - return -EINVAL;
- - if (val > 255)
- - return -EINVAL;
- -
- mutex_lock(&data->update_lock);
-
- if (val == 0)
- @@ -425,6 +423,22 @@ static ssize_t set_pwm(struct device *de
-
- out:
- mutex_unlock(&data->update_lock);
- + return ret;
- +}
- +
- +static ssize_t set_pwm(struct device *dev, struct device_attribute *da,
- + const char *buf, size_t count)
- +{
- + struct tc654_data *data = dev_get_drvdata(dev);
- + unsigned long val;
- + int ret;
- +
- + if (kstrtoul(buf, 10, &val))
- + return -EINVAL;
- + if (val > 255)
- + return -EINVAL;
- +
- + ret = _set_pwm(data, val);
- return ret < 0 ? ret : count;
- }
-
- @@ -462,6 +476,47 @@ static struct attribute *tc654_attrs[] =
-
- ATTRIBUTE_GROUPS(tc654);
-
- +/* cooling device */
- +
- +static int tc654_get_max_state(struct thermal_cooling_device *cdev,
- + unsigned long *state)
- +{
- + *state = 255;
- + return 0;
- +}
- +
- +static int tc654_get_cur_state(struct thermal_cooling_device *cdev,
- + unsigned long *state)
- +{
- + struct tc654_data *data = tc654_update_client(cdev->devdata);
- +
- + if (IS_ERR(data))
- + return PTR_ERR(data);
- +
- + *state = get_pwm(data);
- + return 0;
- +}
- +
- +static int tc654_set_cur_state(struct thermal_cooling_device *cdev,
- + unsigned long state)
- +{
- + struct tc654_data *data = tc654_update_client(cdev->devdata);
- +
- + if (IS_ERR(data))
- + return PTR_ERR(data);
- +
- + if (state > 255)
- + return -EINVAL;
- +
- + return _set_pwm(data, state);
- +}
- +
- +static const struct thermal_cooling_device_ops tc654_fan_cool_ops = {
- + .get_max_state = tc654_get_max_state,
- + .get_cur_state = tc654_get_cur_state,
- + .set_cur_state = tc654_set_cur_state,
- +};
- +
- /*
- * device probe and removal
- */
- @@ -493,7 +548,21 @@ static int tc654_probe(struct i2c_client
- hwmon_dev =
- devm_hwmon_device_register_with_groups(dev, client->name, data,
- tc654_groups);
- - return PTR_ERR_OR_ZERO(hwmon_dev);
- + if (IS_ERR(hwmon_dev))
- + return PTR_ERR(hwmon_dev);
- +
- +#if IS_ENABLED(CONFIG_OF)
- + /* Optional cooling device register for Device tree platforms */
- + data->cdev = thermal_of_cooling_device_register(client->dev.of_node,
- + "tc654", hwmon_dev,
- + &tc654_fan_cool_ops);
- +#else /* CONFIG_OF */
- + /* Optional cooling device register for non Device tree platforms */
- + data->cdev = thermal_cooling_device_register("tc654", hwmon_dev,
- + &tc654_fan_cool_ops);
- +#endif /* CONFIG_OF */
- +
- + return PTR_ERR_OR_ZERO(data->cdev);
- }
-
- static const struct i2c_device_id tc654_id[] = {
|