On Tuesday, December 09, 2014 04:15:40 PM Srinivas Pandruvada wrote: > The _ART (Active Cooling Relationship Table), specifies relationship > among heat generating sources to a target active cooling device like > fan. The _ART table refers to actual bus id name for specifying relationship. > Naming "Fan" is not enough as name in the _ART table can change on every > platform, to establish relationship for user space thermal controllers. > > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx> I can apply this or do you have any thermal material depending on it? > --- > drivers/acpi/fan.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c > index caf9b76..7a36f02 100644 > --- a/drivers/acpi/fan.c > +++ b/drivers/acpi/fan.c > @@ -325,6 +325,7 @@ static int acpi_fan_probe(struct platform_device *pdev) > struct thermal_cooling_device *cdev; > struct acpi_fan *fan; > struct acpi_device *device = ACPI_COMPANION(&pdev->dev); > + char *name; > > fan = devm_kzalloc(&pdev->dev, sizeof(*fan), GFP_KERNEL); > if (!fan) { > @@ -346,7 +347,12 @@ static int acpi_fan_probe(struct platform_device *pdev) > } > } > > - cdev = thermal_cooling_device_register("Fan", device, > + if (!strncmp(pdev->name, "PNP0C0B", strlen("PNP0C0B"))) > + name = "Fan"; > + else > + name = acpi_device_bid(device); > + > + cdev = thermal_cooling_device_register(name, device, > &fan_cooling_ops); > if (IS_ERR(cdev)) { > result = PTR_ERR(cdev); > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html