mirror of
https://github.com/torvalds/linux.git
synced 2026-07-27 09:36:22 +02:00
thermal/of: Rename the devm_thermal_of_cooling_device_register() function
To clarify that the function operates on child nodes, rename:
devm_thermal_of_cooling_device_register()
|
v
devm_thermal_of_child_cooling_device_register()
Used the command:
find . -type f -name '*.[ch]' -exec \
sed -i 's/devm_thermal_of_cooling_device_register/\
devm_thermal_of_child_cooling_device_register/g' {} \;
Did not used clang-format-diff because it does not indent correctly
and checkpatch complained. Manually reindented to make checkpatch
happy
This prepares for upcoming support of cooling devices identified by
an ID rather than device tree child nodes.
No functional change.
Signed-off-by: Daniel Lezcano <daniel.lezcano@oss.qualcomm.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@kernel.org>
Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
Acked-by: Rafael J. Wysocki (Intel) <rafael@kernel.org>
Link: https://patch.msgid.link/20260526140802.1059293-18-daniel.lezcano@oss.qualcomm.com
This commit is contained in:
parent
5fa8b4225b
commit
8e1529e793
|
|
@ -1076,7 +1076,7 @@ static int amc6821_probe(struct i2c_client *client)
|
|||
"Failed to initialize hwmon\n");
|
||||
|
||||
if (IS_ENABLED(CONFIG_THERMAL) && fan_np && data->fan_cooling_levels)
|
||||
return PTR_ERR_OR_ZERO(devm_thermal_of_cooling_device_register(dev,
|
||||
return PTR_ERR_OR_ZERO(devm_thermal_of_child_cooling_device_register(dev,
|
||||
fan_np, client->name, data, &amc6821_cooling_ops));
|
||||
|
||||
return 0;
|
||||
|
|
|
|||
|
|
@ -841,8 +841,9 @@ static int aspeed_create_pwm_cooling(struct device *dev,
|
|||
}
|
||||
snprintf(cdev->name, MAX_CDEV_NAME_LEN, "%pOFn%d", child, pwm_port);
|
||||
|
||||
cdev->tcdev = devm_thermal_of_cooling_device_register(dev, child,
|
||||
cdev->name, cdev, &aspeed_pwm_cool_ops);
|
||||
cdev->tcdev = devm_thermal_of_child_cooling_device_register(dev, child,
|
||||
cdev->name, cdev,
|
||||
&aspeed_pwm_cool_ops);
|
||||
if (IS_ERR(cdev->tcdev))
|
||||
return PTR_ERR(cdev->tcdev);
|
||||
|
||||
|
|
|
|||
|
|
@ -309,9 +309,9 @@ static int emc2305_set_single_tz(struct device *dev, struct device_node *fan_nod
|
|||
pwm = data->pwm_min[cdev_idx];
|
||||
|
||||
data->cdev_data[cdev_idx].cdev =
|
||||
devm_thermal_of_cooling_device_register(dev, fan_node,
|
||||
emc2305_fan_name[idx], data,
|
||||
&emc2305_cooling_ops);
|
||||
devm_thermal_of_child_cooling_device_register(dev, fan_node,
|
||||
emc2305_fan_name[idx], data,
|
||||
&emc2305_cooling_ops);
|
||||
|
||||
if (IS_ERR(data->cdev_data[cdev_idx].cdev)) {
|
||||
dev_err(dev, "Failed to register cooling device %s\n", emc2305_fan_name[idx]);
|
||||
|
|
|
|||
|
|
@ -592,8 +592,10 @@ static int gpio_fan_probe(struct platform_device *pdev)
|
|||
}
|
||||
|
||||
/* Optional cooling device register for Device tree platforms */
|
||||
fan_data->cdev = devm_thermal_of_cooling_device_register(dev, np,
|
||||
"gpio-fan", fan_data, &gpio_fan_cool_ops);
|
||||
fan_data->cdev = devm_thermal_of_child_cooling_device_register(dev, np,
|
||||
"gpio-fan",
|
||||
fan_data,
|
||||
&gpio_fan_cool_ops);
|
||||
|
||||
dev_info(dev, "GPIO fan initialized\n");
|
||||
|
||||
|
|
|
|||
|
|
@ -794,9 +794,9 @@ static int max6650_probe(struct i2c_client *client)
|
|||
return err;
|
||||
|
||||
if (IS_ENABLED(CONFIG_THERMAL)) {
|
||||
cooling_dev = devm_thermal_of_cooling_device_register(dev,
|
||||
dev->of_node, client->name,
|
||||
data, &max6650_cooling_ops);
|
||||
cooling_dev = devm_thermal_of_child_cooling_device_register(dev, dev->of_node,
|
||||
client->name, data,
|
||||
&max6650_cooling_ops);
|
||||
if (IS_ERR(cooling_dev)) {
|
||||
dev_warn(dev, "thermal cooling device register failed: %ld\n",
|
||||
PTR_ERR(cooling_dev));
|
||||
|
|
|
|||
|
|
@ -857,8 +857,10 @@ static int npcm7xx_create_pwm_cooling(struct device *dev,
|
|||
snprintf(cdev->name, THERMAL_NAME_LENGTH, "%pOFn%d", child,
|
||||
pwm_port);
|
||||
|
||||
cdev->tcdev = devm_thermal_of_cooling_device_register(dev, child,
|
||||
cdev->name, cdev, &npcm7xx_pwm_cool_ops);
|
||||
cdev->tcdev = devm_thermal_of_child_cooling_device_register(dev, child,
|
||||
cdev->name,
|
||||
cdev,
|
||||
&npcm7xx_pwm_cool_ops);
|
||||
if (IS_ERR(cdev->tcdev))
|
||||
return PTR_ERR(cdev->tcdev);
|
||||
|
||||
|
|
|
|||
|
|
@ -685,8 +685,9 @@ static int pwm_fan_probe(struct platform_device *pdev)
|
|||
|
||||
ctx->pwm_fan_state = ctx->pwm_fan_max_state;
|
||||
if (IS_ENABLED(CONFIG_THERMAL)) {
|
||||
cdev = devm_thermal_of_cooling_device_register(dev,
|
||||
dev->of_node, "pwm-fan", ctx, &pwm_fan_cooling_ops);
|
||||
cdev = devm_thermal_of_child_cooling_device_register(dev, dev->of_node,
|
||||
"pwm-fan", ctx,
|
||||
&pwm_fan_cooling_ops);
|
||||
if (IS_ERR(cdev)) {
|
||||
ret = PTR_ERR(cdev);
|
||||
dev_err(dev,
|
||||
|
|
|
|||
|
|
@ -337,9 +337,9 @@ static int qnap_mcu_hwmon_probe(struct platform_device *pdev)
|
|||
* levels and only succeed with either no or correct cooling levels.
|
||||
*/
|
||||
if (IS_ENABLED(CONFIG_THERMAL) && hwm->fan_cooling_levels) {
|
||||
cdev = devm_thermal_of_cooling_device_register(dev,
|
||||
to_of_node(hwm->fan_node), "qnap-mcu-hwmon",
|
||||
hwm, &qnap_mcu_hwmon_cooling_ops);
|
||||
cdev = devm_thermal_of_child_cooling_device_register(dev, to_of_node(hwm->fan_node),
|
||||
"qnap-mcu-hwmon", hwm,
|
||||
&qnap_mcu_hwmon_cooling_ops);
|
||||
if (IS_ERR(cdev))
|
||||
return dev_err_probe(dev, PTR_ERR(cdev),
|
||||
"Failed to register qnap-mcu-hwmon as cooling device\n");
|
||||
|
|
|
|||
|
|
@ -541,8 +541,9 @@ static int tc654_probe(struct i2c_client *client)
|
|||
if (IS_ENABLED(CONFIG_THERMAL)) {
|
||||
struct thermal_cooling_device *cdev;
|
||||
|
||||
cdev = devm_thermal_of_cooling_device_register(dev, dev->of_node, client->name,
|
||||
hwmon_dev, &tc654_fan_cool_ops);
|
||||
cdev = devm_thermal_of_child_cooling_device_register(dev, dev->of_node,
|
||||
client->name, hwmon_dev,
|
||||
&tc654_fan_cool_ops);
|
||||
return PTR_ERR_OR_ZERO(cdev);
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -1966,8 +1966,8 @@ static int tegra210_emc_probe(struct platform_device *pdev)
|
|||
|
||||
tegra210_emc_debugfs_init(emc);
|
||||
|
||||
cd = devm_thermal_of_cooling_device_register(emc->dev, np, "emc", emc,
|
||||
&tegra210_emc_cd_ops);
|
||||
cd = devm_thermal_of_child_cooling_device_register(emc->dev, np, "emc", emc,
|
||||
&tegra210_emc_cd_ops);
|
||||
if (IS_ERR(cd)) {
|
||||
err = PTR_ERR(cd);
|
||||
dev_err(emc->dev, "failed to register cooling device: %d\n",
|
||||
|
|
|
|||
|
|
@ -381,7 +381,7 @@ static int qmp_cooling_device_add(struct qmp *qmp,
|
|||
qmp_cdev->qmp = qmp;
|
||||
qmp_cdev->state = !qmp_cdev_max_state;
|
||||
qmp_cdev->name = cdev_name;
|
||||
qmp_cdev->cdev = devm_thermal_of_cooling_device_register
|
||||
qmp_cdev->cdev = devm_thermal_of_child_cooling_device_register
|
||||
(qmp->dev, node,
|
||||
cdev_name,
|
||||
qmp_cdev, &qmp_cooling_device_ops);
|
||||
|
|
|
|||
|
|
@ -90,9 +90,10 @@ static int khadas_mcu_fan_probe(struct platform_device *pdev)
|
|||
ctx->mcu = mcu;
|
||||
platform_set_drvdata(pdev, ctx);
|
||||
|
||||
cdev = devm_thermal_of_cooling_device_register(dev->parent,
|
||||
dev->parent->of_node, "khadas-mcu-fan", ctx,
|
||||
&khadas_mcu_fan_cooling_ops);
|
||||
cdev = devm_thermal_of_child_cooling_device_register(dev->parent,
|
||||
dev->parent->of_node,
|
||||
"khadas-mcu-fan", ctx,
|
||||
&khadas_mcu_fan_cooling_ops);
|
||||
if (IS_ERR(cdev)) {
|
||||
ret = PTR_ERR(cdev);
|
||||
dev_err(dev, "Failed to register khadas-mcu-fan as cooling device: %d\n",
|
||||
|
|
|
|||
|
|
@ -1707,9 +1707,9 @@ static void soctherm_init_hw_throt_cdev(struct platform_device *pdev)
|
|||
stc->init = true;
|
||||
} else {
|
||||
|
||||
tcd = devm_thermal_of_cooling_device_register(dev, np_stcc,
|
||||
(char *)name, ts,
|
||||
&throt_cooling_ops);
|
||||
tcd = devm_thermal_of_child_cooling_device_register(dev, np_stcc,
|
||||
(char *)name, ts,
|
||||
&throt_cooling_ops);
|
||||
if (IS_ERR_OR_NULL(tcd)) {
|
||||
dev_err(dev,
|
||||
"throttle-cfg: %s: failed to register cooling device\n",
|
||||
|
|
|
|||
|
|
@ -557,7 +557,7 @@ static void thermal_of_cooling_device_release(void *data)
|
|||
}
|
||||
|
||||
/**
|
||||
* devm_thermal_of_cooling_device_register() - register an OF thermal cooling
|
||||
* devm_thermal_of_child_cooling_device_register() - register an OF thermal cooling
|
||||
* device
|
||||
* @dev: a valid struct device pointer of a sensor device.
|
||||
* @np: a pointer to a device tree node.
|
||||
|
|
@ -570,14 +570,17 @@ static void thermal_of_cooling_device_release(void *data)
|
|||
* to /sys/class/thermal/ folder as cooling_device[0-*]. It tries to bind itself
|
||||
* to all the thermal zone devices registered at the same time.
|
||||
*
|
||||
* This function should be used when a cooling controller has child
|
||||
* nodes which are referenced in the thermal zone cooling map.
|
||||
*
|
||||
* Return: a pointer to the created struct thermal_cooling_device or an
|
||||
* ERR_PTR. Caller must check return value with IS_ERR*() helpers.
|
||||
*/
|
||||
struct thermal_cooling_device *
|
||||
devm_thermal_of_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops)
|
||||
devm_thermal_of_child_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops)
|
||||
{
|
||||
struct thermal_cooling_device *cdev;
|
||||
int ret;
|
||||
|
|
@ -592,4 +595,4 @@ devm_thermal_of_cooling_device_register(struct device *dev,
|
|||
|
||||
return cdev;
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(devm_thermal_of_cooling_device_register);
|
||||
EXPORT_SYMBOL_GPL(devm_thermal_of_child_cooling_device_register);
|
||||
|
|
|
|||
|
|
@ -205,10 +205,10 @@ thermal_of_cooling_device_register(struct device_node *np, const char *type, voi
|
|||
const struct thermal_cooling_device_ops *ops);
|
||||
|
||||
struct thermal_cooling_device *
|
||||
devm_thermal_of_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops);
|
||||
devm_thermal_of_child_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops);
|
||||
#else
|
||||
|
||||
static inline
|
||||
|
|
@ -232,10 +232,10 @@ thermal_of_cooling_device_register(struct device_node *np,
|
|||
}
|
||||
|
||||
static inline struct thermal_cooling_device *
|
||||
devm_thermal_of_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops)
|
||||
devm_thermal_of_child_cooling_device_register(struct device *dev,
|
||||
struct device_node *np,
|
||||
const char *type, void *devdata,
|
||||
const struct thermal_cooling_device_ops *ops)
|
||||
{
|
||||
return ERR_PTR(-ENODEV);
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user