forked from Minki/linux
8bdadac13f
The duty cycle (% of quiet duration) is used to put the device in quiet mode for the given period. Currently the quiet duration is wrongly calculated which results in not enabling quiet mode. Fix the calculation as below duration = (period * duty cycle) / 100 Signed-off-by: Rajkumar Manoharan <rmanohar@qti.qualcomm.com> Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
244 lines
6.3 KiB
C
244 lines
6.3 KiB
C
/*
|
|
* Copyright (c) 2014 Qualcomm Atheros, Inc.
|
|
*
|
|
* Permission to use, copy, modify, and/or distribute this software for any
|
|
* purpose with or without fee is hereby granted, provided that the above
|
|
* copyright notice and this permission notice appear in all copies.
|
|
*
|
|
* THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
|
|
* WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
|
|
* MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
|
|
* ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
|
|
* WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
|
|
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
|
|
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
|
|
*/
|
|
|
|
#include <linux/device.h>
|
|
#include <linux/sysfs.h>
|
|
#include <linux/thermal.h>
|
|
#include <linux/hwmon.h>
|
|
#include <linux/hwmon-sysfs.h>
|
|
#include "core.h"
|
|
#include "debug.h"
|
|
#include "wmi-ops.h"
|
|
|
|
static int ath10k_thermal_get_active_vifs(struct ath10k *ar,
|
|
enum wmi_vdev_type type)
|
|
{
|
|
struct ath10k_vif *arvif;
|
|
int count = 0;
|
|
|
|
lockdep_assert_held(&ar->conf_mutex);
|
|
|
|
list_for_each_entry(arvif, &ar->arvifs, list) {
|
|
if (!arvif->is_started)
|
|
continue;
|
|
|
|
if (!arvif->is_up)
|
|
continue;
|
|
|
|
if (arvif->vdev_type != type)
|
|
continue;
|
|
|
|
count++;
|
|
}
|
|
return count;
|
|
}
|
|
|
|
static int ath10k_thermal_get_max_dutycycle(struct thermal_cooling_device *cdev,
|
|
unsigned long *state)
|
|
{
|
|
*state = ATH10K_QUIET_DUTY_CYCLE_MAX;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int ath10k_thermal_get_cur_dutycycle(struct thermal_cooling_device *cdev,
|
|
unsigned long *state)
|
|
{
|
|
struct ath10k *ar = cdev->devdata;
|
|
|
|
mutex_lock(&ar->conf_mutex);
|
|
*state = ar->thermal.duty_cycle;
|
|
mutex_unlock(&ar->conf_mutex);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int ath10k_thermal_set_cur_dutycycle(struct thermal_cooling_device *cdev,
|
|
unsigned long duty_cycle)
|
|
{
|
|
struct ath10k *ar = cdev->devdata;
|
|
u32 period, duration, enabled;
|
|
int num_bss, ret = 0;
|
|
|
|
mutex_lock(&ar->conf_mutex);
|
|
if (ar->state != ATH10K_STATE_ON) {
|
|
ret = -ENETDOWN;
|
|
goto out;
|
|
}
|
|
|
|
if (duty_cycle > ATH10K_QUIET_DUTY_CYCLE_MAX) {
|
|
ath10k_warn(ar, "duty cycle %ld is exceeding the limit %d\n",
|
|
duty_cycle, ATH10K_QUIET_DUTY_CYCLE_MAX);
|
|
ret = -EINVAL;
|
|
goto out;
|
|
}
|
|
/* TODO: Right now, thermal mitigation is handled only for single/multi
|
|
* vif AP mode. Since quiet param is not validated in STA mode, it needs
|
|
* to be investigated further to handle multi STA and multi-vif (AP+STA)
|
|
* mode properly.
|
|
*/
|
|
num_bss = ath10k_thermal_get_active_vifs(ar, WMI_VDEV_TYPE_AP);
|
|
if (!num_bss) {
|
|
ath10k_warn(ar, "no active AP interfaces\n");
|
|
ret = -ENETDOWN;
|
|
goto out;
|
|
}
|
|
period = max(ATH10K_QUIET_PERIOD_MIN,
|
|
(ATH10K_QUIET_PERIOD_DEFAULT / num_bss));
|
|
duration = (period * duty_cycle) / 100;
|
|
enabled = duration ? 1 : 0;
|
|
|
|
ret = ath10k_wmi_pdev_set_quiet_mode(ar, period, duration,
|
|
ATH10K_QUIET_START_OFFSET,
|
|
enabled);
|
|
if (ret) {
|
|
ath10k_warn(ar, "failed to set quiet mode period %u duarion %u enabled %u ret %d\n",
|
|
period, duration, enabled, ret);
|
|
goto out;
|
|
}
|
|
ar->thermal.duty_cycle = duty_cycle;
|
|
out:
|
|
mutex_unlock(&ar->conf_mutex);
|
|
return ret;
|
|
}
|
|
|
|
static struct thermal_cooling_device_ops ath10k_thermal_ops = {
|
|
.get_max_state = ath10k_thermal_get_max_dutycycle,
|
|
.get_cur_state = ath10k_thermal_get_cur_dutycycle,
|
|
.set_cur_state = ath10k_thermal_set_cur_dutycycle,
|
|
};
|
|
|
|
static ssize_t ath10k_thermal_show_temp(struct device *dev,
|
|
struct device_attribute *attr,
|
|
char *buf)
|
|
{
|
|
struct ath10k *ar = dev_get_drvdata(dev);
|
|
int ret, temperature;
|
|
|
|
mutex_lock(&ar->conf_mutex);
|
|
|
|
/* Can't get temperature when the card is off */
|
|
if (ar->state != ATH10K_STATE_ON) {
|
|
ret = -ENETDOWN;
|
|
goto out;
|
|
}
|
|
|
|
reinit_completion(&ar->thermal.wmi_sync);
|
|
ret = ath10k_wmi_pdev_get_temperature(ar);
|
|
if (ret) {
|
|
ath10k_warn(ar, "failed to read temperature %d\n", ret);
|
|
goto out;
|
|
}
|
|
|
|
if (test_bit(ATH10K_FLAG_CRASH_FLUSH, &ar->dev_flags)) {
|
|
ret = -ESHUTDOWN;
|
|
goto out;
|
|
}
|
|
|
|
ret = wait_for_completion_timeout(&ar->thermal.wmi_sync,
|
|
ATH10K_THERMAL_SYNC_TIMEOUT_HZ);
|
|
if (ret == 0) {
|
|
ath10k_warn(ar, "failed to synchronize thermal read\n");
|
|
ret = -ETIMEDOUT;
|
|
goto out;
|
|
}
|
|
|
|
spin_lock_bh(&ar->data_lock);
|
|
temperature = ar->thermal.temperature;
|
|
spin_unlock_bh(&ar->data_lock);
|
|
|
|
ret = snprintf(buf, PAGE_SIZE, "%d", temperature);
|
|
out:
|
|
mutex_unlock(&ar->conf_mutex);
|
|
return ret;
|
|
}
|
|
|
|
void ath10k_thermal_event_temperature(struct ath10k *ar, int temperature)
|
|
{
|
|
spin_lock_bh(&ar->data_lock);
|
|
ar->thermal.temperature = temperature;
|
|
spin_unlock_bh(&ar->data_lock);
|
|
complete(&ar->thermal.wmi_sync);
|
|
}
|
|
|
|
static SENSOR_DEVICE_ATTR(temp1_input, S_IRUGO, ath10k_thermal_show_temp,
|
|
NULL, 0);
|
|
|
|
static struct attribute *ath10k_hwmon_attrs[] = {
|
|
&sensor_dev_attr_temp1_input.dev_attr.attr,
|
|
NULL,
|
|
};
|
|
ATTRIBUTE_GROUPS(ath10k_hwmon);
|
|
|
|
int ath10k_thermal_register(struct ath10k *ar)
|
|
{
|
|
struct thermal_cooling_device *cdev;
|
|
struct device *hwmon_dev;
|
|
int ret;
|
|
|
|
cdev = thermal_cooling_device_register("ath10k_thermal", ar,
|
|
&ath10k_thermal_ops);
|
|
|
|
if (IS_ERR(cdev)) {
|
|
ath10k_err(ar, "failed to setup thermal device result: %ld\n",
|
|
PTR_ERR(cdev));
|
|
return -EINVAL;
|
|
}
|
|
|
|
ret = sysfs_create_link(&ar->dev->kobj, &cdev->device.kobj,
|
|
"cooling_device");
|
|
if (ret) {
|
|
ath10k_err(ar, "failed to create thermal symlink\n");
|
|
goto err_cooling_destroy;
|
|
}
|
|
|
|
ar->thermal.cdev = cdev;
|
|
|
|
/* Do not register hwmon device when temperature reading is not
|
|
* supported by firmware
|
|
*/
|
|
if (ar->wmi.op_version != ATH10K_FW_WMI_OP_VERSION_10_2_4)
|
|
return 0;
|
|
|
|
/* Avoid linking error on devm_hwmon_device_register_with_groups, I
|
|
* guess linux/hwmon.h is missing proper stubs. */
|
|
if (!config_enabled(CONFIG_HWMON))
|
|
return 0;
|
|
|
|
hwmon_dev = devm_hwmon_device_register_with_groups(ar->dev,
|
|
"ath10k_hwmon", ar,
|
|
ath10k_hwmon_groups);
|
|
if (IS_ERR(hwmon_dev)) {
|
|
ath10k_err(ar, "failed to register hwmon device: %ld\n",
|
|
PTR_ERR(hwmon_dev));
|
|
ret = -EINVAL;
|
|
goto err_remove_link;
|
|
}
|
|
return 0;
|
|
|
|
err_remove_link:
|
|
sysfs_remove_link(&ar->dev->kobj, "thermal_sensor");
|
|
err_cooling_destroy:
|
|
thermal_cooling_device_unregister(cdev);
|
|
return ret;
|
|
}
|
|
|
|
void ath10k_thermal_unregister(struct ath10k *ar)
|
|
{
|
|
thermal_cooling_device_unregister(ar->thermal.cdev);
|
|
sysfs_remove_link(&ar->dev->kobj, "cooling_device");
|
|
}
|