2014-07-09 14:53:49

by Kamil Debski

[permalink] [raw]
Subject: [PATCH] hwmon: gpio-fan: Change name passed to hwmon_device_register_with_groups

Commit 648cd48c9e566f53c5df30d79857e0937ae13b09 (hwmon: Do not accept
invalid name attributes) added check for "-" in the name of hwmon device.
This patch changes the name supplied by gpio-fan so that it is accepted
by hwmon.

Signed-off-by: Kamil Debski <[email protected]>
---
drivers/hwmon/gpio-fan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hwmon/gpio-fan.c b/drivers/hwmon/gpio-fan.c
index ba35e4d..5acd555 100644
--- a/drivers/hwmon/gpio-fan.c
+++ b/drivers/hwmon/gpio-fan.c
@@ -538,7 +538,7 @@ static int gpio_fan_probe(struct platform_device *pdev)

/* Make this driver part of hwmon class. */
fan_data->hwmon_dev = hwmon_device_register_with_groups(&pdev->dev,
- "gpio-fan", fan_data,
+ "gpiofan", fan_data,
gpio_fan_groups);
if (IS_ERR(fan_data->hwmon_dev))
return PTR_ERR(fan_data->hwmon_dev);
--
1.7.9.5


2014-07-09 17:10:39

by Guenter Roeck

[permalink] [raw]
Subject: Re: [lm-sensors] [PATCH] hwmon: gpio-fan: Change name passed to hwmon_device_register_with_groups

On Wed, Jul 09, 2014 at 04:53:32PM +0200, Kamil Debski wrote:
> Commit 648cd48c9e566f53c5df30d79857e0937ae13b09 (hwmon: Do not accept
> invalid name attributes) added check for "-" in the name of hwmon device.
> This patch changes the name supplied by gpio-fan so that it is accepted
> by hwmon.
>
> Signed-off-by: Kamil Debski <[email protected]>
> ---
> drivers/hwmon/gpio-fan.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/gpio-fan.c b/drivers/hwmon/gpio-fan.c
> index ba35e4d..5acd555 100644
> --- a/drivers/hwmon/gpio-fan.c
> +++ b/drivers/hwmon/gpio-fan.c
> @@ -538,7 +538,7 @@ static int gpio_fan_probe(struct platform_device *pdev)
>
> /* Make this driver part of hwmon class. */
> fan_data->hwmon_dev = hwmon_device_register_with_groups(&pdev->dev,
> - "gpio-fan", fan_data,
> + "gpiofan", fan_data,

This has already been changed to "gpio_fan" in the latest kernel.
See commit d26e0da78 (hwmon: (gpio-fan) Change name used in
hwmon_device_register_with_groups).

Guenter