linux-kernel - Re: [PATCH] Fixed coding style in drivers/acpi/ac.c

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] [day] [month] [year] [list]

Date:	Mon, 18 Jul 2011 10:52:41 -0700
From:	Joe Perches <joe@...ches.com>
To:	Angus Hammond <angusgh@...il.com>
Cc:	LKML <linux-kernel@...r.kernel.org>, Len Brown <lenb@...nel.org>,
	linux-acpi <linux-acpi@...r.kernel.org>
Subject: Re: [PATCH] Fixed coding style in drivers/acpi/ac.c

On Mon, 2011-07-18 at 18:14 +0100, Angus Hammond wrote:
> Changed lines causing errors from  scrips/checkpatch.pl.

Please don't send emails with only bcc's.
It makes following up / reply-all difficult.

Use ./scripts/get_maintainer.pl <patch> as a guide
to determine the best recipients for a patch.

> ---
>  drivers/acpi/ac.c |   18 +++++++++---------
>  1 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c
> index 58c3f74..e510828 100644
> --- a/drivers/acpi/ac.c
> +++ b/drivers/acpi/ac.c
> @@ -1,5 +1,5 @@
>  /*
> - *  acpi_ac.c - ACPI AC Adapter Driver ($Revision: 27 $)
> + *  acpi_ac.c - ACPI AC Adapter Driver
>   *
>   *  Copyright (C) 2001, 2002 Andy Grover <andrew.grover@...el.com>
>   *  Copyright (C) 2001, 2002 Paul Diefenbaugh <paul.s.diefenbaugh@...el.com>
> @@ -54,8 +54,8 @@ MODULE_DESCRIPTION("ACPI AC Adapter Driver");
>  MODULE_LICENSE("GPL");
>  
>  #ifdef CONFIG_ACPI_PROCFS_POWER
> -extern struct proc_dir_entry *acpi_lock_ac_dir(void);
> -extern void *acpi_unlock_ac_dir(struct proc_dir_entry *acpi_ac_dir);
> +struct proc_dir_entry *acpi_lock_ac_dir(void);
> +void *acpi_unlock_ac_dir(struct proc_dir_entry *acpi_ac_dir);
>  static int acpi_ac_open_fs(struct inode *inode, struct file *file);
>  #endif
>  
> @@ -85,7 +85,7 @@ static struct acpi_driver acpi_ac_driver = {
>  
>  struct acpi_ac {
>  	struct power_supply charger;
> -	struct acpi_device * device;
> +	struct acpi_device *device;
>  	unsigned long long state;
>  };
>  
> @@ -102,7 +102,7 @@ static const struct file_operations acpi_ac_fops = {
>  #endif
>  
>  /* --------------------------------------------------------------------------
> -                               AC Adapter Management
> +				AC Adapter Management
>     -------------------------------------------------------------------------- */
>  
>  static int acpi_ac_get_state(struct acpi_ac *ac)
> @@ -124,7 +124,7 @@ static int acpi_ac_get_state(struct acpi_ac *ac)
>  }
>  
>  /* --------------------------------------------------------------------------
> -                            sysfs I/F
> +				sysfs I/F
>     -------------------------------------------------------------------------- */
>  static int get_ac_property(struct power_supply *psy,
>  			   enum power_supply_property psp,
> @@ -154,7 +154,7 @@ static enum power_supply_property ac_props[] = {
>  
>  #ifdef CONFIG_ACPI_PROCFS_POWER
>  /* --------------------------------------------------------------------------
> -                              FS Interface (/proc)
> +				FS Interface (/proc)
>     -------------------------------------------------------------------------- */
>  
>  static struct proc_dir_entry *acpi_ac_dir;
> @@ -230,7 +230,7 @@ static int acpi_ac_remove_fs(struct acpi_device *device)
>  #endif
>  
>  /* --------------------------------------------------------------------------
> -                                   Driver Model
> +				Driver Model
>     -------------------------------------------------------------------------- */
>  
>  static void acpi_ac_notify(struct acpi_device *device, u32 event)
> @@ -298,7 +298,7 @@ static int acpi_ac_add(struct acpi_device *device)
>  	       acpi_device_name(device), acpi_device_bid(device),
>  	       ac->state ? "on-line" : "off-line");
>  
> -      end:
> +end:
>  	if (result) {
>  #ifdef CONFIG_ACPI_PROCFS_POWER
>  		acpi_ac_remove_fs(device);



--
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/