extcon: gpio: Fix minor coding style and remove the unused fields.
authorChanwoo Choi <cw00.choi@samsung.com>
Tue, 29 Sep 2015 13:59:55 +0000 (22:59 +0900)
committerChanwoo Choi <cw00.choi@samsung.com>
Wed, 30 Sep 2015 10:22:50 +0000 (19:22 +0900)
This patch fixes the minor coding style about indentation and removes the
unused fields from struct gpio_extcon_platform_data.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
drivers/extcon/extcon-gpio.c
include/linux/extcon/extcon-gpio.h

index ba927df..27de6ae 100644 (file)
@@ -1,7 +1,5 @@
 /*
- *  drivers/extcon/extcon_gpio.c
- *
- *  Single-state GPIO extcon driver based on extcon class
+ * extcon_gpio.c - Single-state GPIO extcon driver based on extcon class
  *
  * Copyright (C) 2008 Google, Inc.
  * Author: Mike Lockwood <lockwood@android.com>
@@ -17,8 +15,7 @@
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- *
-*/
+ */
 
 #include <linux/extcon.h>
 #include <linux/extcon/extcon-gpio.h>
@@ -37,7 +34,7 @@ struct gpio_extcon_data {
        struct delayed_work work;
        unsigned long debounce_jiffies;
 
-       struct gpio_extcon_platform_data *pdata;
+       struct gpio_extcon_pdata *pdata;
 };
 
 static void gpio_extcon_work(struct work_struct *work)
@@ -64,7 +61,7 @@ static irqreturn_t gpio_irq_handler(int irq, void *dev_id)
 
 static int gpio_extcon_probe(struct platform_device *pdev)
 {
-       struct gpio_extcon_platform_data *pdata = dev_get_platdata(&pdev->dev);
+       struct gpio_extcon_pdata *pdata = dev_get_platdata(&pdev->dev);
        struct gpio_extcon_data *data;
        int ret;
 
index 232bb8f..7cacafb 100644 (file)
@@ -1,5 +1,5 @@
 /*
- *  External connector (extcon) class generic GPIO driver
+ * Single-state GPIO extcon driver based on extcon class
  *
  * Copyright (C) 2012 Samsung Electronics
  * Author: MyungJoo Ham <myungjoo.ham@samsung.com>
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- *
-*/
+ */
 #ifndef __EXTCON_GPIO_H__
 #define __EXTCON_GPIO_H__ __FILE__
 
 #include <linux/extcon.h>
 
 /**
- * struct gpio_extcon_platform_data - A simple GPIO-controlled extcon device.
- * @name:              The name of this GPIO extcon device.
+ * struct gpio_extcon_pdata - A simple GPIO-controlled extcon device.
  * @extcon_id:         The unique id of specific external connector.
  * @gpio:              Corresponding GPIO.
  * @gpio_active_low:   Boolean describing whether gpio active state is 1 or 0
  *                     If false, high state of gpio means active.
  * @debounce:          Debounce time for GPIO IRQ in ms.
  * @irq_flags:         IRQ Flags (e.g., IRQF_TRIGGER_LOW).
- * @state_on:          print_state is overriden with state_on if attached.
- *                     If NULL, default method of extcon class is used.
- * @state_off:         print_state is overriden with state_off if detached.
- *                     If NUll, default method of extcon class is used.
  * @check_on_resume:   Boolean describing whether to check the state of gpio
  *                     while resuming from sleep.
- *
- * Note that in order for state_on or state_off to be valid, both state_on
- * and state_off should be not NULL. If at least one of them is NULL,
- * the print_state is not overriden.
  */
-struct gpio_extcon_platform_data {
-       const char *name;
+struct gpio_extcon_pdata {
        unsigned int extcon_id;
        unsigned gpio;
        bool gpio_active_low;
        unsigned long debounce;
        unsigned long irq_flags;
 
-       /* if NULL, "0" or "1" will be printed */
-       const char *state_on;
-       const char *state_off;
        bool check_on_resume;
 };