linux-kernel - Re: [PATCH 3/4] thermal_sys: Kernel style cleanups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LNX.2.00.1201250038070.27508@swampdragon.chaosbits.net>
Date:	Wed, 25 Jan 2012 00:39:32 +0100 (CET)
From:	Jesper Juhl <jj@...osbits.net>
To:	Joe Perches <joe@...ches.com>
cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Len Brown <len.brown@...el.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/4] thermal_sys: Kernel style cleanups

On Tue, 24 Jan 2012, Joe Perches wrote:

> Just a few tidies to make it more like most
> kernel sources.
> 
> A couple of long lines still remain.
> 
> Signed-off-by: Joe Perches <joe@...ches.com>
> ---
>  drivers/thermal/thermal_sys.c |   30 +++++++++++++-----------------
>  1 files changed, 13 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
> index 1c87167..68df616 100644
> --- a/drivers/thermal/thermal_sys.c
> +++ b/drivers/thermal/thermal_sys.c
> @@ -66,7 +66,7 @@ static int get_idr(struct idr *idr, struct mutex *lock, int *id)
>  {
>  	int err;
>  
> -      again:
> +again:
>  	if (unlikely(idr_pre_get(idr, GFP_KERNEL) == 0))
>  		return -ENOMEM;
>  
> @@ -818,15 +818,14 @@ int thermal_zone_bind_cooling_device(struct thermal_zone_device *tz,
>  		return 0;
>  
>  	device_remove_file(&tz->device, &dev->attr);
> -      remove_symbol_link:
> +remove_symbol_link:
>  	sysfs_remove_link(&tz->device.kobj, dev->name);
> -      release_idr:
> +release_idr:
>  	release_idr(&tz->idr, &tz->lock, dev->id);
> -      free_mem:
> +free_mem:
>  	kfree(dev);
>  	return result;
>  }
> -
>  EXPORT_SYMBOL(thermal_zone_bind_cooling_device);
>  
>  /**
> @@ -856,14 +855,13 @@ int thermal_zone_unbind_cooling_device(struct thermal_zone_device *tz,
>  
>  	return -ENODEV;
>  
> -      unbind:
> +unbind:
>  	device_remove_file(&tz->device, &pos->attr);
>  	sysfs_remove_link(&tz->device.kobj, pos->name);
>  	release_idr(&tz->idr, &tz->lock, pos->id);
>  	kfree(pos);
>  	return 0;
>  }
> -
>  EXPORT_SYMBOL(thermal_zone_unbind_cooling_device);
>  
>  static void thermal_release(struct device *dev)
> @@ -871,7 +869,8 @@ static void thermal_release(struct device *dev)
>  	struct thermal_zone_device *tz;
>  	struct thermal_cooling_device *cdev;
>  
> -	if (!strncmp(dev_name(dev), "thermal_zone", sizeof "thermal_zone" - 1)) {
> +	if (!strncmp(dev_name(dev), "thermal_zone",
> +		     sizeof("thermal_zone") - 1)) {
>  		tz = to_thermal_zone(dev);
>  		kfree(tz);
>  	} else {
> @@ -891,8 +890,9 @@ static struct class thermal_class = {
>   * @devdata:	device private data.
>   * @ops:		standard thermal cooling devices callbacks.
>   */
> -struct thermal_cooling_device *thermal_cooling_device_register(
> -     char *type, void *devdata, const struct thermal_cooling_device_ops *ops)
> +struct thermal_cooling_device *
> +thermal_cooling_device_register(char *type, void *devdata,
> +				const struct thermal_cooling_device_ops *ops)
>  {
>  	struct thermal_cooling_device *cdev;
>  	struct thermal_zone_device *pos;
> @@ -957,12 +957,11 @@ struct thermal_cooling_device *thermal_cooling_device_register(
>  	if (!result)
>  		return cdev;
>  
> -      unregister:
> +unregister:
>  	release_idr(&thermal_cdev_idr, &thermal_idr_lock, cdev->id);
>  	device_unregister(&cdev->device);
>  	return ERR_PTR(result);
>  }
> -
>  EXPORT_SYMBOL(thermal_cooling_device_register);
>  
>  /**
> @@ -1007,7 +1006,6 @@ void thermal_cooling_device_unregister(struct
>  	device_unregister(&cdev->device);
>  	return;
>  }
> -
>  EXPORT_SYMBOL(thermal_cooling_device_unregister);
>  
>  /**
> @@ -1083,7 +1081,7 @@ void thermal_zone_device_update(struct thermal_zone_device *tz)
>  
>  	tz->last_temperature = temp;
>  
> -      leave:
> +leave:
>  	if (tz->passive)
>  		thermal_zone_device_set_polling(tz, tz->passive_delay);
>  	else if (tz->polling_delay)
> @@ -1223,12 +1221,11 @@ struct thermal_zone_device *thermal_zone_device_register(char *type,
>  	if (!result)
>  		return tz;
>  
> -      unregister:
> +unregister:
>  	release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
>  	device_unregister(&tz->device);
>  	return ERR_PTR(result);
>  }
> -
>  EXPORT_SYMBOL(thermal_zone_device_register);
>  
>  /**
> @@ -1280,7 +1277,6 @@ void thermal_zone_device_unregister(struct thermal_zone_device *tz)
>  	device_unregister(&tz->device);
>  	return;
>  }
> -
>  EXPORT_SYMBOL(thermal_zone_device_unregister);
>  
>  #ifdef CONFIG_NET
> 

Looks like a sane set of changes to me.
Reviewed-by: Jesper Juhl <jj@...osbits.net>


-- 
Jesper Juhl <jj@...osbits.net>       http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/