Re: [PATCH] platform: arm64: qcom-hamoa-ec: Register cooling devices with their name

[Ilpo Järvinen]

On Mon, 29 Jun 2026, Anvesh Jain P wrote:

> Each fan is registered as a thermal cooling device using
> devm_thermal_of_cooling_device_register(). When the thermal core gained
> cooling device ID support, the helper's prototype changed: its second
> argument became a 'u32 cdev_id' and its third a 'const char *type',
> replacing the previous 'struct device_node *np'.
> 
> The resulting semantic conflict between the EC driver and the reworked
> thermal helper was resolved while merging the drivers-x86 tree into
> linux-next by passing NULL as the 'type' argument, dropping the per-fan
> 'name' string. As a result every fan is registered as a cooling device
> with an empty name.
> 
> Pass 'name' as the 'type' argument so the cooling devices are registered
> with their intended "qcom_ec_fan_<n>" names.
> 
> Fixes: 5c44f48e91de ("platform: arm64: Add driver for EC found on Qualcomm reference devices")
> Signed-off-by: Anvesh Jain P <anvesh.p@xxxxxxxxxxxxxxxx>
> ---
>  drivers/platform/arm64/qcom-hamoa-ec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/platform/arm64/qcom-hamoa-ec.c b/drivers/platform/arm64/qcom-hamoa-ec.c
> index 45829f2786b7..12196d56b60e 100644
> --- a/drivers/platform/arm64/qcom-hamoa-ec.c
> +++ b/drivers/platform/arm64/qcom-hamoa-ec.c
> @@ -391,7 +391,7 @@ static int qcom_ec_probe(struct i2c_client *client)
>  		ec_cdev->fan_id = i + 1;
>  		ec_cdev->parent_dev = dev;
>  
> -		ec_cdev->cdev = devm_thermal_of_cooling_device_register(dev, 0, NULL, ec_cdev,
> +		ec_cdev->cdev = devm_thermal_of_cooling_device_register(dev, 0, name, ec_cdev,
>  									&qcom_ec_thermal_ops);
>  		if (IS_ERR(ec_cdev->cdev)) {
>  			return dev_err_probe(dev, PTR_ERR(ec_cdev->cdev),
> 
> ---
> base-commit: 948efecf22e49aa4bf55bb73ec79a0ddcfd38571

$ git log 948efecf22e49aa4bf55bb73ec79a0ddcfd38571
fatal: bad object 948efecf22e49aa4bf55bb73ec79a0ddcfd38571

Please don't base your submissions on some random commits. There has never 
been code with (dev, 0, NULL, ...) in platforms drivers tree (and neither 
in Linus' tree).

-- 
 i.