extcon: Fix indentation coding style to improve readability

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: Myungjoo Ham <myungjoo.ham@samsung.com>

3 files changed, 64 insertions, 61 deletions

diff --git a/include/linux/extcon.h b/include/linux/extcon.h
index fcb51c88319f..c2b652dde72c 100644
--- a/include/linux/extcon.h
+++ b/include/linux/extcon.h
@@ -51,10 +51,10 @@
 enum extcon_cable_name {
         EXTCON_USB = 0,
         EXTCON_USB_HOST,
-        EXTCON_TA, /* Travel Adaptor */
+        EXTCON_TA,                      /* Travel Adaptor */
         EXTCON_FAST_CHARGER,
         EXTCON_SLOW_CHARGER,
-        EXTCON_CHARGE_DOWNSTREAM, /* Charging an external device */
+        EXTCON_CHARGE_DOWNSTREAM,       /* Charging an external device */
         EXTCON_HDMI,
         EXTCON_MHL,
         EXTCON_DVI,
@@ -76,8 +76,8 @@ struct extcon_cable;
 
 /**
  * struct extcon_dev - An extcon device represents one external connector.
- * @name:       The name of this extcon device. Parent device name is used
- *              if NULL.
+ * @name:               The name of this extcon device. Parent device name is
+ *                      used if NULL.
  * @supported_cable:    Array of supported cable names ending with NULL.
  *                      If supported_cable is NULL, cable name related APIs
  *                      are disabled.
@@ -89,21 +89,21 @@ struct extcon_cable;
  *                      be attached simulataneously. {0x7, 0} is equivalent to
  *                      {0x3, 0x6, 0x5, 0}. If it is {0xFFFFFFFF, 0}, there
  *                      can be no simultaneous connections.
- * @print_name: An optional callback to override the method to print the
- *              name of the extcon device.
+ * @print_name:         An optional callback to override the method to print the
+ *                      name of the extcon device.
  * @print_state:        An optional callback to override the method to print the
- *              status of the extcon device.
- * @dev:        Device of this extcon. Do not provide at register-time.
- * @state:      Attach/detach state of this extcon. Do not provide at
- *              register-time
- * @nh: Notifier for the state change events from this extcon
- * @entry:      To support list of extcon devices so that users can search
- *              for extcon devices based on the extcon name.
+ *                      status of the extcon device.
+ * @dev:                Device of this extcon. Do not provide at register-time.
+ * @state:              Attach/detach state of this extcon. Do not provide at
+ *                      register-time.
+ * @nh:                 Notifier for the state change events from this extcon
+ * @entry:              To support list of extcon devices so that users can search
+ *                      for extcon devices based on the extcon name.
  * @lock:
  * @max_supported:      Internal value to store the number of cables.
  * @extcon_dev_type:    Device_type struct to provide attribute_groups
  *                      customized for each extcon device.
- * @cables:     Sysfs subdirectories. Each represents one cable.
+ * @cables:             Sysfs subdirectories. Each represents one cable.
  *
  * In most cases, users only need to provide "User initializing data" of
  * this struct when registering an extcon. In some exceptional cases,
@@ -111,26 +111,27 @@ struct extcon_cable;
  * are overwritten by register function.
  */
 struct extcon_dev {
-        /* --- Optional user initializing data --- */
-        const char      *name;
+        /* Optional user initializing data */
+        const char *name;
         const char **supported_cable;
-        const u32       *mutually_exclusive;
+        const u32 *mutually_exclusive;
 
-        /* --- Optional callbacks to override class functions --- */
+        /* Optional callbacks to override class functions */
         ssize_t (*print_name)(struct extcon_dev *edev, char *buf);
         ssize_t (*print_state)(struct extcon_dev *edev, char *buf);
 
-        /* --- Internal data. Please do not set. --- */
-        struct device   *dev;
-        u32             state;
+        /* Internal data. Please do not set. */
+        struct device *dev;
         struct raw_notifier_head nh;
         struct list_head entry;
-        spinlock_t lock; /* could be called by irq handler */
         int max_supported;
+        spinlock_t lock;        /* could be called by irq handler */
+        u32 state;
 
         /* /sys/class/extcon/.../cable.n/... */
         struct device_type extcon_dev_type;
         struct extcon_cable *cables;
+
         /* /sys/class/extcon/.../mutually_exclusive/... */
         struct attribute_group attr_g_muex;
         struct attribute **attrs_muex;
@@ -138,13 +139,13 @@ struct extcon_dev {
 };
 
 /**
- * struct extcon_cable  - An internal data for each cable of extcon device.
- * @edev:       The extcon device
+ * struct extcon_cable - An internal data for each cable of extcon device.
+ * @edev:               The extcon device
  * @cable_index:        Index of this cable in the edev
- * @attr_g:     Attribute group for the cable
- * @attr_name:  "name" sysfs entry
- * @attr_state: "state" sysfs entry
- * @attrs:      Array pointing to attr_name and attr_state for attr_g
+ * @attr_g:             Attribute group for the cable
+ * @attr_name:          "name" sysfs entry
+ * @attr_state:         "state" sysfs entry
+ * @attrs:              Array pointing to attr_name and attr_state for attr_g
  */
 struct extcon_cable {
         struct extcon_dev *edev;
@@ -159,11 +160,13 @@ struct extcon_cable {
 
 /**
  * struct extcon_specific_cable_nb - An internal data for
- *                              extcon_register_interest().
- * @internal_nb:        a notifier block bridging extcon notifier and cable notifier.
- * @user_nb:    user provided notifier block for events from a specific cable.
+ *                                   extcon_register_interest().
+ * @internal_nb:        A notifier block bridging extcon notifier
+ *                      and cable notifier.
+ * @user_nb:            user provided notifier block for events from
+ *                      a specific cable.
  * @cable_index:        the target cable.
- * @edev:       the target extcon device.
+ * @edev:               the target extcon device.
  * @previous_value:     the saved previous event value.
  */
 struct extcon_specific_cable_nb {
diff --git a/include/linux/extcon/extcon-adc-jack.h b/include/linux/extcon/extcon-adc-jack.h
index 20e9eef25d4c..9ca958c4e94c 100644
--- a/include/linux/extcon/extcon-adc-jack.h
+++ b/include/linux/extcon/extcon-adc-jack.h
@@ -20,10 +20,10 @@
 
 /**
  * struct adc_jack_cond - condition to use an extcon state
- * @state      - the corresponding extcon state (if 0, this struct denotes
- *             the last adc_jack_cond element among the array)
- * @min_adc    - min adc value for this condition
- * @max_adc    - max adc value for this condition
+ * @state:              the corresponding extcon state (if 0, this struct
+ *                      denotes the last adc_jack_cond element among the array)
+ * @min_adc:            min adc value for this condition
+ * @max_adc:            max adc value for this condition
  *
  * For example, if { .state = 0x3, .min_adc = 100, .max_adc = 200}, it means
  * that if ADC value is between (inclusive) 100 and 200, than the cable 0 and
@@ -33,34 +33,34 @@
  * because when no adc_jack_cond is met, state = 0 is automatically chosen.
  */
 struct adc_jack_cond {
-        u32 state; /* extcon state value. 0 if invalid */
+        u32 state;      /* extcon state value. 0 if invalid */
         u32 min_adc;
         u32 max_adc;
 };
 
 /**
  * struct adc_jack_pdata - platform data for adc jack device.
- * @name       - name of the extcon device. If null, "adc-jack" is used.
- * @consumer_channel - Unique name to identify the channel on the consumer
- *                   side. This typically describes the channels used within
- *                   the consumer. E.g. 'battery_voltage'
- * @cable_names        - array of cable names ending with null.
- * @adc_contitions     - array of struct adc_jack_cond conditions ending
- *                     with .state = 0 entry. This describes how to decode
- *                     adc values into extcon state.
- * @irq_flags  - irq flags used for the @irq
- * @handling_delay_ms  - in some devices, we need to read ADC value some
- *                     milli-seconds after the interrupt occurs. You may
- *                     describe such delays with @handling_delay_ms, which
- *                     is rounded-off by jiffies.
+ * @name:               name of the extcon device. If null, "adc-jack" is used.
+ * @consumer_channel:   Unique name to identify the channel on the consumer
+ *                      side. This typically describes the channels used within
+ *                      the consumer. E.g. 'battery_voltage'
+ * @cable_names:        array of cable names ending with null.
+ * @adc_contitions:     array of struct adc_jack_cond conditions ending
+ *                      with .state = 0 entry. This describes how to decode
+ *                      adc values into extcon state.
+ * @irq_flags:          irq flags used for the @irq
+ * @handling_delay_ms:  in some devices, we need to read ADC value some
+ *                      milli-seconds after the interrupt occurs. You may
+ *                      describe such delays with @handling_delay_ms, which
+ *                      is rounded-off by jiffies.
  */
 struct adc_jack_pdata {
         const char *name;
         const char *consumer_channel;
-        /*
-         * The last entry should be NULL
-         */
+
+        /* The last entry should be NULL */
         const char **cable_names;
+
         /* The last entry's state should be 0 */
         struct adc_jack_cond *adc_conditions;
 
diff --git a/include/linux/extcon/extcon-gpio.h b/include/linux/extcon/extcon-gpio.h
index 2d8307f7d67d..4ce1aa7d29bd 100644
--- a/include/linux/extcon/extcon-gpio.h
+++ b/include/linux/extcon/extcon-gpio.h
@@ -25,14 +25,14 @@
 
 /**
  * struct gpio_extcon_platform_data - A simple GPIO-controlled extcon device.
- * @name        The name of this GPIO extcon device.
- * @gpio        Corresponding GPIO.
- * @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_on if detached. If Null,
- *              default method of extcon class is used.
+ * @name:               The name of this GPIO extcon device.
+ * @gpio:               Corresponding GPIO.
+ * @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_on if detached.
+ *                      If NUll, default method of extcon class is used.
  *
  * 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,