All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yakir Yang <ykk@rock-chips.com>
To: Heiko Stuebner <heiko@sntech.de>,
	Russell King <rmk+kernel@arm.linux.org.uk>,
	Fabio Estevam <fabio.estevam@freescale.com>,
	Jingoo Han <jingoohan1@gmail.com>,
	Inki Dae <inki.dae@samsung.com>
Cc: djkurtz@google.com, dianders@google.com,
	Takashi Iwai <tiwai@suse.de>, Andrzej Hajda <a.hajda@samsung.com>,
	Thierry Reding <treding@nvidia.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	David Airlie <airlied@linux.ie>,
	Gustavo Padovan <gustavo.padovan@collabora.co.uk>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Kukjin Kim <kgene@kernel.org>,
	Ajay Kumar <ajaykumar.rs@samsung.com>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	Vincent Palatin <vpalatin@chromium.org>,
	Mark Yao <mark.yao@rock-chips.com>,
	Andy Yan <andy.yan@rock-chips.com>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 1/8] drm: exynos/dp: fix code style
Date: Thu, 6 Aug 2015 21:58:38 +0800	[thread overview]
Message-ID: <55C3680E.30705@rock-chips.com> (raw)
In-Reply-To: <1438868947-22132-1-git-send-email-ykk@rock-chips.com>


make checkpatch.pl script happy

Signed-off-by: Yakir Yang <ykk@rock-chips.com>
---
  drivers/gpu/drm/exynos/exynos_dp_core.c | 224 
++++++++++++++++----------------
  drivers/gpu/drm/exynos/exynos_dp_core.h |  53 ++++----
  drivers/gpu/drm/exynos/exynos_dp_reg.c  | 100 +++++++-------
  3 files changed, 185 insertions(+), 192 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c 
b/drivers/gpu/drm/exynos/exynos_dp_core.c
index 172b800..a8097a4 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
@@ -114,8 +114,8 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)

      /* Read Extension Flag, Number of 128-byte EDID extension blocks */
      retval = exynos_dp_read_byte_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
-                EDID_EXTENSION_FLAG,
-                &extend_block);
+                          EDID_EXTENSION_FLAG,
+                          &extend_block);
      if (retval)
          return retval;

@@ -123,10 +123,11 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          dev_dbg(dp->dev, "EDID data includes a single extension!\n");

          /* Read EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
-                        EDID_HEADER_PATTERN,
-                        EDID_BLOCK_LENGTH,
-                        &edid[EDID_HEADER_PATTERN]);
+        retval =
+        exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
+                          EDID_HEADER_PATTERN,
+                          EDID_BLOCK_LENGTH,
+                          &edid[EDID_HEADER_PATTERN]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -138,11 +139,11 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          }

          /* Read additional EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp,
-                I2C_EDID_DEVICE_ADDR,
-                EDID_BLOCK_LENGTH,
-                EDID_BLOCK_LENGTH,
-                &edid[EDID_BLOCK_LENGTH]);
+        retval =
+        exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
+                          EDID_BLOCK_LENGTH,
+                          EDID_BLOCK_LENGTH,
+                          &edid[EDID_BLOCK_LENGTH]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -154,24 +155,22 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          }

          exynos_dp_read_byte_from_dpcd(dp, DP_TEST_REQUEST,
-                    &test_vector);
+                          &test_vector);
          if (test_vector & DP_TEST_LINK_EDID_READ) {
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_EDID_CHECKSUM,
+            exynos_dp_write_byte_to_dpcd(
+                dp, DP_TEST_EDID_CHECKSUM,
                  edid[EDID_BLOCK_LENGTH + EDID_CHECKSUM]);
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_RESPONSE,
+            exynos_dp_write_byte_to_dpcd(
+                dp, DP_TEST_RESPONSE,
                  DP_TEST_EDID_CHECKSUM_WRITE);
          }
      } else {
          dev_info(dp->dev, "EDID data does not include any extensions.\n");

          /* Read EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp,
-                I2C_EDID_DEVICE_ADDR,
-                EDID_HEADER_PATTERN,
-                EDID_BLOCK_LENGTH,
-                &edid[EDID_HEADER_PATTERN]);
+        retval = exynos_dp_read_bytes_from_i2c(
+                dp, I2C_EDID_DEVICE_ADDR, EDID_HEADER_PATTERN,
+                EDID_BLOCK_LENGTH, &edid[EDID_HEADER_PATTERN]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -182,16 +181,15 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
              return -EIO;
          }

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TEST_REQUEST,
-            &test_vector);
+        exynos_dp_read_byte_from_dpcd(dp, DP_TEST_REQUEST,
+                          &test_vector);
          if (test_vector & DP_TEST_LINK_EDID_READ) {
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_EDID_CHECKSUM,
-                edid[EDID_CHECKSUM]);
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_RESPONSE,
-                DP_TEST_EDID_CHECKSUM_WRITE);
+            exynos_dp_write_byte_to_dpcd(
+                    dp, DP_TEST_EDID_CHECKSUM,
+                    edid[EDID_CHECKSUM]);
+            exynos_dp_write_byte_to_dpcd(
+                    dp, DP_TEST_RESPONSE,
+                    DP_TEST_EDID_CHECKSUM_WRITE);
          }
      }

@@ -206,8 +204,7 @@ static int exynos_dp_handle_edid(struct 
exynos_dp_device *dp)
      int retval;

      /* Read DPCD DP_DPCD_REV~RECEIVE_PORT1_CAP_1 */
-    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_DPCD_REV,
-                12, buf);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_DPCD_REV, 12, buf);
      if (retval)
          return retval;

@@ -222,19 +219,21 @@ static int exynos_dp_handle_edid(struct 
exynos_dp_device *dp)
  }

  static void exynos_dp_enable_rx_to_enhanced_mode(struct 
exynos_dp_device *dp,
-                        bool enable)
+                         bool enable)
  {
      u8 data;

      exynos_dp_read_byte_from_dpcd(dp, DP_LANE_COUNT_SET, &data);

      if (enable)
-        exynos_dp_write_byte_to_dpcd(dp, DP_LANE_COUNT_SET,
-            DP_LANE_COUNT_ENHANCED_FRAME_EN |
-            DPCD_LANE_COUNT_SET(data));
+        exynos_dp_write_byte_to_dpcd(
+                    dp, DP_LANE_COUNT_SET,
+                    DP_LANE_COUNT_ENHANCED_FRAME_EN |
+                    DPCD_LANE_COUNT_SET(data));
      else
-        exynos_dp_write_byte_to_dpcd(dp, DP_LANE_COUNT_SET,
-            DPCD_LANE_COUNT_SET(data));
+        exynos_dp_write_byte_to_dpcd(
+                    dp, DP_LANE_COUNT_SET,
+                    DPCD_LANE_COUNT_SET(data));
  }

  static int exynos_dp_is_enhanced_mode_available(struct 
exynos_dp_device *dp)
@@ -261,13 +260,12 @@ static void exynos_dp_training_pattern_dis(struct 
exynos_dp_device *dp)
  {
      exynos_dp_set_training_pattern(dp, DP_NONE);

-    exynos_dp_write_byte_to_dpcd(dp,
-        DP_TRAINING_PATTERN_SET,
-        DP_TRAINING_PATTERN_DISABLE);
+    exynos_dp_write_byte_to_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                     DP_TRAINING_PATTERN_DISABLE);
  }

  static void exynos_dp_set_lane_lane_pre_emphasis(struct 
exynos_dp_device *dp,
-                    int pre_emphasis, int lane)
+                         int pre_emphasis, int lane)
  {
      switch (lane) {
      case 0:
@@ -307,15 +305,14 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
      /* Setup RX configuration */
      buf[0] = dp->link_train.link_rate;
      buf[1] = dp->link_train.lane_count;
-    retval = exynos_dp_write_bytes_to_dpcd(dp, DP_LINK_BW_SET,
-                2, buf);
+    retval = exynos_dp_write_bytes_to_dpcd(dp, DP_LINK_BW_SET, 2, buf);
      if (retval)
          return retval;

      /* Set TX pre-emphasis to minimum */
      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_lane_pre_emphasis(dp,
-            PRE_EMPHASIS_LEVEL_0, lane);
+        exynos_dp_set_lane_lane_pre_emphasis(dp, PRE_EMPHASIS_LEVEL_0,
+                             lane);

      /* Wait for PLL lock */
      pll_tries = 0;
@@ -333,9 +330,9 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
      exynos_dp_set_training_pattern(dp, TRAINING_PTN1);

      /* Set RX training pattern */
-    retval = exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            DP_LINK_SCRAMBLING_DISABLE | DP_TRAINING_PATTERN_1);
+    retval = exynos_dp_write_byte_to_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          DP_LINK_SCRAMBLING_DISABLE |
+                          DP_TRAINING_PATTERN_1);
      if (retval)
          return retval;

@@ -344,7 +341,7 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
                  DP_TRAIN_VOLTAGE_SWING_LEVEL_0;

      retval = exynos_dp_write_bytes_to_dpcd(dp, DP_TRAINING_LANE0_SET,
-            lane_count, buf);
+                           lane_count, buf);

      return retval;
  }
@@ -352,7 +349,7 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
  static unsigned char exynos_dp_get_lane_status(u8 link_status[2], int 
lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = link_status[lane>>1];
+    u8 link_value = link_status[lane >> 1];

      return (link_value >> shift) & 0xf;
  }
@@ -371,7 +368,7 @@ static int exynos_dp_clock_recovery_ok(u8 
link_status[2], int lane_count)
  }

  static int exynos_dp_channel_eq_ok(u8 link_status[2], u8 link_align,
-                int lane_count)
+                   int lane_count)
  {
      int lane;
      u8 lane_status;
@@ -390,10 +387,10 @@ static int exynos_dp_channel_eq_ok(u8 
link_status[2], u8 link_align,
  }

  static unsigned char exynos_dp_get_adjust_request_voltage(u8 
adjust_request[2],
-                            int lane)
+                              int lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = adjust_request[lane>>1];
+    u8 link_value = adjust_request[lane >> 1];

      return (link_value >> shift) & 0x3;
  }
@@ -403,13 +400,13 @@ static unsigned char 
exynos_dp_get_adjust_request_pre_emphasis(
                      int lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = adjust_request[lane>>1];
+    u8 link_value = adjust_request[lane >> 1];

      return ((link_value >> shift) & 0xc) >> 2;
  }

  static void exynos_dp_set_lane_link_training(struct exynos_dp_device *dp,
-                    u8 training_lane_set, int lane)
+                         u8 training_lane_set, int lane)
  {
      switch (lane) {
      case 0:
@@ -465,7 +462,7 @@ static void exynos_dp_reduce_link_rate(struct 
exynos_dp_device *dp)
  }

  static void exynos_dp_get_adjust_training_lane(struct exynos_dp_device 
*dp,
-                    u8 adjust_request[2])
+                           u8 adjust_request[2])
  {
      int lane, lane_count;
      u8 voltage_swing, pre_emphasis, training_lane;
@@ -498,13 +495,13 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)

      lane_count = dp->link_train.lane_count;

-    retval =  exynos_dp_read_bytes_from_dpcd(dp,
-            DP_LANE0_1_STATUS, 2, link_status);
+    retval =  exynos_dp_read_bytes_from_dpcd(dp, DP_LANE0_1_STATUS,
+                         2, link_status);
      if (retval)
          return retval;

-    retval =  exynos_dp_read_bytes_from_dpcd(dp,
-            DP_ADJUST_REQUEST_LANE0_1, 2, adjust_request);
+    retval =  exynos_dp_read_bytes_from_dpcd(dp, DP_ADJUST_REQUEST_LANE0_1,
+                         2, adjust_request);
      if (retval)
          return retval;

@@ -512,8 +509,8 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
          /* set training pattern 2 for EQ */
          exynos_dp_set_training_pattern(dp, TRAINING_PTN2);

-        retval = exynos_dp_write_byte_to_dpcd(dp,
-                DP_TRAINING_PATTERN_SET,
+        retval = exynos_dp_write_byte_to_dpcd(
+                dp, DP_TRAINING_PATTERN_SET,
                  DP_LINK_SCRAMBLING_DISABLE |
                  DP_TRAINING_PATTERN_2);
          if (retval)
@@ -527,8 +524,9 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
                              dp, lane);
              voltage_swing = exynos_dp_get_adjust_request_voltage(
                              adjust_request, lane);
-            pre_emphasis = exynos_dp_get_adjust_request_pre_emphasis(
-                            adjust_request, lane);
+            pre_emphasis =
+                exynos_dp_get_adjust_request_pre_emphasis(
+                        adjust_request, lane);

              if (DPCD_VOLTAGE_SWING_GET(training_lane) ==
                      voltage_swing &&
@@ -551,11 +549,11 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
      exynos_dp_get_adjust_training_lane(dp, adjust_request);

      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_link_training(dp,
-            dp->link_train.training_lane[lane], lane);
+        exynos_dp_set_lane_link_training(
+                dp, dp->link_train.training_lane[lane], lane);

-    retval = exynos_dp_write_bytes_to_dpcd(dp,
-            DP_TRAINING_LANE0_SET, lane_count,
+    retval = exynos_dp_write_bytes_to_dpcd(
+            dp, DP_TRAINING_LANE0_SET, lane_count,
              dp->link_train.training_lane);
      if (retval)
          return retval;
@@ -573,8 +571,8 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)

      lane_count = dp->link_train.lane_count;

-    retval = exynos_dp_read_bytes_from_dpcd(dp,
-            DP_LANE0_1_STATUS, 2, link_status);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_LANE0_1_STATUS,
+                        2, link_status);
      if (retval)
          return retval;

@@ -583,13 +581,13 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)
          return -EIO;
      }

-    retval = exynos_dp_read_bytes_from_dpcd(dp,
-            DP_ADJUST_REQUEST_LANE0_1, 2, adjust_request);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_ADJUST_REQUEST_LANE0_1,
+                        2, adjust_request);
      if (retval)
          return retval;

-    retval = exynos_dp_read_byte_from_dpcd(dp,
-            DP_LANE_ALIGN_STATUS_UPDATED, &link_align);
+    retval = exynos_dp_read_byte_from_dpcd(
+            dp, DP_LANE_ALIGN_STATUS_UPDATED, &link_align);
      if (retval)
          return retval;

@@ -628,17 +626,18 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)
      }

      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_link_training(dp,
-            dp->link_train.training_lane[lane], lane);
+        exynos_dp_set_lane_link_training(
+                dp, dp->link_train.training_lane[lane], lane);

      retval = exynos_dp_write_bytes_to_dpcd(dp, DP_TRAINING_LANE0_SET,
-            lane_count, dp->link_train.training_lane);
+                           lane_count,
+                           dp->link_train.training_lane);

      return retval;
  }

  static void exynos_dp_get_max_rx_bandwidth(struct exynos_dp_device *dp,
-                    u8 *bandwidth)
+                       u8 *bandwidth)
  {
      u8 data;

@@ -651,7 +650,7 @@ static void exynos_dp_get_max_rx_bandwidth(struct 
exynos_dp_device *dp,
  }

  static void exynos_dp_get_max_rx_lane_count(struct exynos_dp_device *dp,
-                    u8 *lane_count)
+                        u8 *lane_count)
  {
      u8 data;

@@ -664,8 +663,8 @@ static void exynos_dp_get_max_rx_lane_count(struct 
exynos_dp_device *dp,
  }

  static void exynos_dp_init_training(struct exynos_dp_device *dp,
-            enum link_lane_count_type max_lane,
-            enum link_rate_type max_rate)
+                    enum link_lane_count_type max_lane,
+                    enum link_rate_type max_rate)
  {
      /*
       * MACRO_RST must be applied after the PLL_LOCK to avoid
@@ -678,7 +677,7 @@ static void exynos_dp_init_training(struct 
exynos_dp_device *dp,
      exynos_dp_get_max_rx_lane_count(dp, &dp->link_train.lane_count);

      if ((dp->link_train.link_rate != LINK_RATE_1_62GBPS) &&
-       (dp->link_train.link_rate != LINK_RATE_2_70GBPS)) {
+        (dp->link_train.link_rate != LINK_RATE_2_70GBPS)) {
          dev_err(dp->dev, "Rx Max Link Rate is abnormal :%x !\n",
              dp->link_train.link_rate);
          dp->link_train.link_rate = LINK_RATE_1_62GBPS;
@@ -738,8 +737,7 @@ static int exynos_dp_sw_link_training(struct 
exynos_dp_device *dp)
  }

  static int exynos_dp_set_link_train(struct exynos_dp_device *dp,
-                u32 count,
-                u32 bwtype)
+                    u32 count, u32 bwtype)
  {
      int i;
      int retval;
@@ -830,21 +828,19 @@ static void exynos_dp_enable_scramble(struct 
exynos_dp_device *dp, bool enable)
      if (enable) {
          exynos_dp_enable_scrambling(dp);

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            &data);
-        exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
+        exynos_dp_read_byte_from_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          &data);
+        exynos_dp_write_byte_to_dpcd(
+            dp, DP_TRAINING_PATTERN_SET,
              (u8)(data & ~DP_LINK_SCRAMBLING_DISABLE));
      } else {
          exynos_dp_disable_scrambling(dp);

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            &data);
-        exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            (u8)(data | DP_LINK_SCRAMBLING_DISABLE));
+        exynos_dp_read_byte_from_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          &data);
+        exynos_dp_write_byte_to_dpcd(
+                dp, DP_TRAINING_PATTERN_SET,
+                (u8)(data | DP_LINK_SCRAMBLING_DISABLE));
      }
  }

@@ -915,7 +911,7 @@ static void exynos_dp_commit(struct 
exynos_drm_display *display)
      }

      ret = exynos_dp_set_link_train(dp, dp->video_info->lane_count,
-                    dp->video_info->link_rate);
+                       dp->video_info->link_rate);
      if (ret) {
          dev_err(dp->dev, "unable to do link train\n");
          return;
@@ -1004,7 +1000,7 @@ static struct drm_connector_helper_funcs 
exynos_dp_connector_helper_funcs = {

  /* returns the number of bridges attached */
  static int exynos_drm_attach_lcd_bridge(struct exynos_dp_device *dp,
-        struct drm_encoder *encoder)
+                    struct drm_encoder *encoder)
  {
      int ret;

@@ -1020,7 +1016,7 @@ static int exynos_drm_attach_lcd_bridge(struct 
exynos_dp_device *dp,
  }

  static int exynos_dp_create_connector(struct exynos_drm_display *display,
-                struct drm_encoder *encoder)
+                      struct drm_encoder *encoder)
  {
      struct exynos_dp_device *dp = display_to_dp(display);
      struct drm_connector *connector = &dp->connector;
@@ -1038,7 +1034,8 @@ static int exynos_dp_create_connector(struct 
exynos_drm_display *display,
      connector->polled = DRM_CONNECTOR_POLL_HPD;

      ret = drm_connector_init(dp->drm_dev, connector,
-            &exynos_dp_connector_funcs, DRM_MODE_CONNECTOR_eDP);
+                 &exynos_dp_connector_funcs,
+                 DRM_MODE_CONNECTOR_eDP);
      if (ret) {
          DRM_ERROR("Failed to initialize connector with drm\n");
          return ret;
@@ -1148,8 +1145,8 @@ static struct video_info 
*exynos_dp_dt_parse_pdata(struct device *dev)
      struct device_node *dp_node = dev->of_node;
      struct video_info *dp_video_config;

-    dp_video_config = devm_kzalloc(dev,
-                sizeof(*dp_video_config), GFP_KERNEL);
+    dp_video_config = devm_kzalloc(dev, sizeof(*dp_video_config),
+                       GFP_KERNEL);
      if (!dp_video_config)
          return ERR_PTR(-ENOMEM);

@@ -1163,37 +1160,37 @@ static struct video_info 
*exynos_dp_dt_parse_pdata(struct device *dev)
          of_property_read_bool(dp_node, "interlaced");

      if (of_property_read_u32(dp_node, "samsung,color-space",
-                &dp_video_config->color_space)) {
+                 &dp_video_config->color_space)) {
          dev_err(dev, "failed to get color-space\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,dynamic-range",
-                &dp_video_config->dynamic_range)) {
+                 &dp_video_config->dynamic_range)) {
          dev_err(dev, "failed to get dynamic-range\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,ycbcr-coeff",
-                &dp_video_config->ycbcr_coeff)) {
+                 &dp_video_config->ycbcr_coeff)) {
          dev_err(dev, "failed to get ycbcr-coeff\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,color-depth",
-                &dp_video_config->color_depth)) {
+                 &dp_video_config->color_depth)) {
          dev_err(dev, "failed to get color-depth\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,link-rate",
-                &dp_video_config->link_rate)) {
+                 &dp_video_config->link_rate)) {
          dev_err(dev, "failed to get link-rate\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,lane-count",
-                &dp_video_config->lane_count)) {
+                 &dp_video_config->lane_count)) {
          dev_err(dev, "failed to get lane-count\n");
          return ERR_PTR(-EINVAL);
      }
@@ -1206,7 +1203,7 @@ static int exynos_dp_dt_parse_panel(struct 
exynos_dp_device *dp)
      int ret;

      ret = of_get_videomode(dp->dev->of_node, &dp->priv.vm,
-            OF_USE_NATIVE_MODE);
+                   OF_USE_NATIVE_MODE);
      if (ret) {
          DRM_ERROR("failed: of_get_videomode() : %d\n", ret);
          return ret;
@@ -1302,7 +1299,7 @@ static int exynos_dp_bind(struct device *dev, 
struct device *master, void *data)
      exynos_dp_init_dp(dp);

      ret = devm_request_irq(&pdev->dev, dp->irq, exynos_dp_irq_handler,
-            irq_flags, "exynos-dp", dp);
+                   irq_flags, "exynos-dp", dp);
      if (ret) {
          dev_err(&pdev->dev, "failed to request irq\n");
          return ret;
@@ -1315,7 +1312,7 @@ static int exynos_dp_bind(struct device *dev, 
struct device *master, void *data)
  }

  static void exynos_dp_unbind(struct device *dev, struct device *master,
-                void *data)
+                 void *data)
  {
      struct exynos_dp_device *dp = dev_get_drvdata(dev);

@@ -1334,7 +1331,7 @@ static int exynos_dp_probe(struct platform_device 
*pdev)
      struct exynos_dp_device *dp;

      dp = devm_kzalloc(&pdev->dev, sizeof(struct exynos_dp_device),
-                GFP_KERNEL);
+              GFP_KERNEL);
      if (!dp)
          return -ENOMEM;

@@ -1358,8 +1355,9 @@ static int exynos_dp_probe(struct platform_device 
*pdev)
              of_node_put(bridge_node);
              if (!dp->bridge)
                  return -EPROBE_DEFER;
-        } else
+        } else {
              return -EPROBE_DEFER;
+        }
      }

      return component_add(&pdev->dev, &exynos_dp_ops);
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.h 
b/drivers/gpu/drm/exynos/exynos_dp_core.h
index a4e7996..c321ad5 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.h
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.h
@@ -180,8 +180,8 @@ void exynos_dp_config_interrupt(struct 
exynos_dp_device *dp);
  enum pll_status exynos_dp_get_pll_lock_status(struct exynos_dp_device 
*dp);
  void exynos_dp_set_pll_power_down(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_set_analog_power_down(struct exynos_dp_device *dp,
-                enum analog_power_block block,
-                bool enable);
+                     enum analog_power_block block,
+                     bool enable);
  void exynos_dp_init_analog_func(struct exynos_dp_device *dp);
  void exynos_dp_init_hpd(struct exynos_dp_device *dp);
  enum dp_irq_type exynos_dp_get_irq_type(struct exynos_dp_device *dp);
@@ -192,50 +192,50 @@ int exynos_dp_get_plug_in_status(struct 
exynos_dp_device *dp);
  void exynos_dp_enable_sw_function(struct exynos_dp_device *dp);
  int exynos_dp_start_aux_transaction(struct exynos_dp_device *dp);
  int exynos_dp_write_byte_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char data);
+                 unsigned int reg_addr,
+                 unsigned char data);
  int exynos_dp_read_byte_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char *data);
+                  unsigned int reg_addr,
+                  unsigned char *data);
  int exynos_dp_write_bytes_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[]);
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char data[]);
  int exynos_dp_read_bytes_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[]);
+                   unsigned int reg_addr,
+                   unsigned int count,
+                   unsigned char data[]);
  int exynos_dp_select_i2c_device(struct exynos_dp_device *dp,
                  unsigned int device_addr,
                  unsigned int reg_addr);
  int exynos_dp_read_byte_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int *data);
+                 unsigned int device_addr,
+                 unsigned int reg_addr,
+                 unsigned int *data);
  int exynos_dp_read_bytes_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char edid[]);
+                  unsigned int device_addr,
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char edid[]);
  void exynos_dp_set_link_bandwidth(struct exynos_dp_device *dp, u32 
bwtype);
  void exynos_dp_get_link_bandwidth(struct exynos_dp_device *dp, u32 
*bwtype);
  void exynos_dp_set_lane_count(struct exynos_dp_device *dp, u32 count);
  void exynos_dp_get_lane_count(struct exynos_dp_device *dp, u32 *count);
  void exynos_dp_enable_enhanced_mode(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_set_training_pattern(struct exynos_dp_device *dp,
-                 enum pattern_set pattern);
+                    enum pattern_set pattern);
  void exynos_dp_set_lane0_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane1_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane2_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane3_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane0_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane1_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane2_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane3_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  u32 exynos_dp_get_lane0_link_training(struct exynos_dp_device *dp);
  u32 exynos_dp_get_lane1_link_training(struct exynos_dp_device *dp);
  u32 exynos_dp_get_lane2_link_training(struct exynos_dp_device *dp);
@@ -246,9 +246,8 @@ void exynos_dp_init_video(struct exynos_dp_device *dp);
  void exynos_dp_set_video_color_format(struct exynos_dp_device *dp);
  int exynos_dp_is_slave_video_stream_clock_on(struct exynos_dp_device *dp);
  void exynos_dp_set_video_cr_mn(struct exynos_dp_device *dp,
-            enum clock_recovery_m_value_type type,
-            u32 m_value,
-            u32 n_value);
+                   enum clock_recovery_m_value_type type,
+                   u32 m_value, u32 n_value);
  void exynos_dp_set_video_timing_mode(struct exynos_dp_device *dp, u32 
type);
  void exynos_dp_enable_video_master(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_start_video(struct exynos_dp_device *dp);
diff --git a/drivers/gpu/drm/exynos/exynos_dp_reg.c 
b/drivers/gpu/drm/exynos/exynos_dp_reg.c
index c1f87a2..9aa483d 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_reg.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_reg.c
@@ -202,8 +202,8 @@ void exynos_dp_set_pll_power_down(struct 
exynos_dp_device *dp, bool enable)
  }

  void exynos_dp_set_analog_power_down(struct exynos_dp_device *dp,
-                enum analog_power_block block,
-                bool enable)
+                     enum analog_power_block block,
+                     bool enable)
  {
      u32 reg;

@@ -399,8 +399,8 @@ void exynos_dp_init_aux(struct exynos_dp_device *dp)
      exynos_dp_reset_aux(dp);

      /* Disable AUX transaction H/W retry */
-    reg = AUX_BIT_PERIOD_EXPECTED_DELAY(3) | AUX_HW_RETRY_COUNT_SEL(0)|
-        AUX_HW_RETRY_INTERVAL_600_MICROSECONDS;
+    reg = AUX_BIT_PERIOD_EXPECTED_DELAY(3) | AUX_HW_RETRY_COUNT_SEL(0) |
+          AUX_HW_RETRY_INTERVAL_600_MICROSECONDS;
      writel(reg, dp->reg_base + EXYNOS_DP_AUX_HW_RETRY_CTL);

      /* Receive AUX Channel DEFER commands equal to DEFFER_COUNT*64 */
@@ -483,8 +483,8 @@ int exynos_dp_start_aux_transaction(struct 
exynos_dp_device *dp)
  }

  int exynos_dp_write_byte_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char data)
+                 unsigned int reg_addr,
+                 unsigned char data)
  {
      u32 reg;
      int i;
@@ -519,17 +519,16 @@ int exynos_dp_write_byte_to_dpcd(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      return retval;
  }

  int exynos_dp_read_byte_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char *data)
+                  unsigned int reg_addr,
+                  unsigned char *data)
  {
      u32 reg;
      int i;
@@ -560,9 +559,8 @@ int exynos_dp_read_byte_from_dpcd(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      /* Read data buffer */
@@ -573,9 +571,9 @@ int exynos_dp_read_byte_from_dpcd(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_write_bytes_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[])
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char data[])
  {
      u32 reg;
      unsigned int start_offset;
@@ -625,9 +623,9 @@ int exynos_dp_write_bytes_to_dpcd(struct 
exynos_dp_device *dp,
              retval = exynos_dp_start_aux_transaction(dp);
              if (retval == 0)
                  break;
-            else
-                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                    __func__);
+
+            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                __func__);
          }

          start_offset += cur_data_count;
@@ -637,9 +635,9 @@ int exynos_dp_write_bytes_to_dpcd(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_bytes_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[])
+                   unsigned int reg_addr,
+                   unsigned int count,
+                   unsigned char data[])
  {
      u32 reg;
      unsigned int start_offset;
@@ -683,9 +681,9 @@ int exynos_dp_read_bytes_from_dpcd(struct 
exynos_dp_device *dp,
              retval = exynos_dp_start_aux_transaction(dp);
              if (retval == 0)
                  break;
-            else
-                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                    __func__);
+
+            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                __func__);
          }

          for (cur_data_idx = 0; cur_data_idx < cur_data_count;
@@ -736,9 +734,9 @@ int exynos_dp_select_i2c_device(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_byte_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int *data)
+                 unsigned int device_addr,
+                 unsigned int reg_addr,
+                 unsigned int *data)
  {
      u32 reg;
      int i;
@@ -767,9 +765,8 @@ int exynos_dp_read_byte_from_i2c(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      /* Read data */
@@ -780,10 +777,10 @@ int exynos_dp_read_byte_from_i2c(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_bytes_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char edid[])
+                  unsigned int device_addr,
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char edid[])
  {
      u32 reg;
      unsigned int i, j;
@@ -807,8 +804,8 @@ int exynos_dp_read_bytes_from_i2c(struct 
exynos_dp_device *dp,
               * request without sending address
               */
              if (!defer)
-                retval = exynos_dp_select_i2c_device(dp,
-                        device_addr, reg_addr + i);
+                retval = exynos_dp_select_i2c_device(
+                        dp, device_addr, reg_addr + i);
              else
                  defer = 0;

@@ -828,15 +825,14 @@ int exynos_dp_read_bytes_from_i2c(struct 
exynos_dp_device *dp,
                  retval = exynos_dp_start_aux_transaction(dp);
                  if (retval == 0)
                      break;
-                else
-                    dev_dbg(dp->dev,
-                        "%s: Aux Transaction fail!\n",
-                        __func__);
+
+                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                    __func__);
              }
              /* Check if Rx sends defer */
              reg = readl(dp->reg_base + EXYNOS_DP_AUX_RX_COMM);
              if (reg == AUX_RX_COMM_AUX_DEFER ||
-                reg == AUX_RX_COMM_I2C_DEFER) {
+                reg == AUX_RX_COMM_I2C_DEFER) {
                  dev_err(dp->dev, "Defer: %d\n\n", reg);
                  defer = 1;
              }
@@ -901,7 +897,7 @@ void exynos_dp_enable_enhanced_mode(struct 
exynos_dp_device *dp, bool enable)
  }

  void exynos_dp_set_training_pattern(struct exynos_dp_device *dp,
-                 enum pattern_set pattern)
+                    enum pattern_set pattern)
  {
      u32 reg;

@@ -974,7 +970,7 @@ void exynos_dp_set_lane3_pre_emphasis(struct 
exynos_dp_device *dp, u32 level)
  }

  void exynos_dp_set_lane0_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -983,7 +979,7 @@ void exynos_dp_set_lane0_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane1_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -992,7 +988,7 @@ void exynos_dp_set_lane1_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane2_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -1001,7 +997,7 @@ void exynos_dp_set_lane2_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane3_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -1125,9 +1121,9 @@ int 
exynos_dp_is_slave_video_stream_clock_on(struct exynos_dp_device *dp)
  }

  void exynos_dp_set_video_cr_mn(struct exynos_dp_device *dp,
-        enum clock_recovery_m_value_type type,
-        u32 m_value,
-        u32 n_value)
+                   enum clock_recovery_m_value_type type,
+                   u32 m_value,
+                   u32 n_value)
  {
      u32 reg;

@@ -1221,7 +1217,7 @@ void exynos_dp_config_video_slave_mode(struct 
exynos_dp_device *dp)
      u32 reg;

      reg = readl(dp->reg_base + EXYNOS_DP_FUNC_EN_1);
-    reg &= ~(MASTER_VID_FUNC_EN_N|SLAVE_VID_FUNC_EN_N);
+    reg &= ~(MASTER_VID_FUNC_EN_N | SLAVE_VID_FUNC_EN_N);
      reg |= MASTER_VID_FUNC_EN_N;
      writel(reg, dp->reg_base + EXYNOS_DP_FUNC_EN_1);

-- 
2.1.2



WARNING: multiple messages have this Message-ID (diff)
From: ykk@rock-chips.com (Yakir Yang)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 1/8] drm: exynos/dp: fix code style
Date: Thu, 6 Aug 2015 21:58:38 +0800	[thread overview]
Message-ID: <55C3680E.30705@rock-chips.com> (raw)
In-Reply-To: <1438868947-22132-1-git-send-email-ykk@rock-chips.com>


make checkpatch.pl script happy

Signed-off-by: Yakir Yang <ykk@rock-chips.com>
---
  drivers/gpu/drm/exynos/exynos_dp_core.c | 224 
++++++++++++++++----------------
  drivers/gpu/drm/exynos/exynos_dp_core.h |  53 ++++----
  drivers/gpu/drm/exynos/exynos_dp_reg.c  | 100 +++++++-------
  3 files changed, 185 insertions(+), 192 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c 
b/drivers/gpu/drm/exynos/exynos_dp_core.c
index 172b800..a8097a4 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
@@ -114,8 +114,8 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)

      /* Read Extension Flag, Number of 128-byte EDID extension blocks */
      retval = exynos_dp_read_byte_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
-                EDID_EXTENSION_FLAG,
-                &extend_block);
+                          EDID_EXTENSION_FLAG,
+                          &extend_block);
      if (retval)
          return retval;

@@ -123,10 +123,11 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          dev_dbg(dp->dev, "EDID data includes a single extension!\n");

          /* Read EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
-                        EDID_HEADER_PATTERN,
-                        EDID_BLOCK_LENGTH,
-                        &edid[EDID_HEADER_PATTERN]);
+        retval =
+        exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
+                          EDID_HEADER_PATTERN,
+                          EDID_BLOCK_LENGTH,
+                          &edid[EDID_HEADER_PATTERN]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -138,11 +139,11 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          }

          /* Read additional EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp,
-                I2C_EDID_DEVICE_ADDR,
-                EDID_BLOCK_LENGTH,
-                EDID_BLOCK_LENGTH,
-                &edid[EDID_BLOCK_LENGTH]);
+        retval =
+        exynos_dp_read_bytes_from_i2c(dp, I2C_EDID_DEVICE_ADDR,
+                          EDID_BLOCK_LENGTH,
+                          EDID_BLOCK_LENGTH,
+                          &edid[EDID_BLOCK_LENGTH]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -154,24 +155,22 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
          }

          exynos_dp_read_byte_from_dpcd(dp, DP_TEST_REQUEST,
-                    &test_vector);
+                          &test_vector);
          if (test_vector & DP_TEST_LINK_EDID_READ) {
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_EDID_CHECKSUM,
+            exynos_dp_write_byte_to_dpcd(
+                dp, DP_TEST_EDID_CHECKSUM,
                  edid[EDID_BLOCK_LENGTH + EDID_CHECKSUM]);
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_RESPONSE,
+            exynos_dp_write_byte_to_dpcd(
+                dp, DP_TEST_RESPONSE,
                  DP_TEST_EDID_CHECKSUM_WRITE);
          }
      } else {
          dev_info(dp->dev, "EDID data does not include any extensions.\n");

          /* Read EDID data */
-        retval = exynos_dp_read_bytes_from_i2c(dp,
-                I2C_EDID_DEVICE_ADDR,
-                EDID_HEADER_PATTERN,
-                EDID_BLOCK_LENGTH,
-                &edid[EDID_HEADER_PATTERN]);
+        retval = exynos_dp_read_bytes_from_i2c(
+                dp, I2C_EDID_DEVICE_ADDR, EDID_HEADER_PATTERN,
+                EDID_BLOCK_LENGTH, &edid[EDID_HEADER_PATTERN]);
          if (retval != 0) {
              dev_err(dp->dev, "EDID Read failed!\n");
              return -EIO;
@@ -182,16 +181,15 @@ static int exynos_dp_read_edid(struct 
exynos_dp_device *dp)
              return -EIO;
          }

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TEST_REQUEST,
-            &test_vector);
+        exynos_dp_read_byte_from_dpcd(dp, DP_TEST_REQUEST,
+                          &test_vector);
          if (test_vector & DP_TEST_LINK_EDID_READ) {
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_EDID_CHECKSUM,
-                edid[EDID_CHECKSUM]);
-            exynos_dp_write_byte_to_dpcd(dp,
-                DP_TEST_RESPONSE,
-                DP_TEST_EDID_CHECKSUM_WRITE);
+            exynos_dp_write_byte_to_dpcd(
+                    dp, DP_TEST_EDID_CHECKSUM,
+                    edid[EDID_CHECKSUM]);
+            exynos_dp_write_byte_to_dpcd(
+                    dp, DP_TEST_RESPONSE,
+                    DP_TEST_EDID_CHECKSUM_WRITE);
          }
      }

@@ -206,8 +204,7 @@ static int exynos_dp_handle_edid(struct 
exynos_dp_device *dp)
      int retval;

      /* Read DPCD DP_DPCD_REV~RECEIVE_PORT1_CAP_1 */
-    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_DPCD_REV,
-                12, buf);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_DPCD_REV, 12, buf);
      if (retval)
          return retval;

@@ -222,19 +219,21 @@ static int exynos_dp_handle_edid(struct 
exynos_dp_device *dp)
  }

  static void exynos_dp_enable_rx_to_enhanced_mode(struct 
exynos_dp_device *dp,
-                        bool enable)
+                         bool enable)
  {
      u8 data;

      exynos_dp_read_byte_from_dpcd(dp, DP_LANE_COUNT_SET, &data);

      if (enable)
-        exynos_dp_write_byte_to_dpcd(dp, DP_LANE_COUNT_SET,
-            DP_LANE_COUNT_ENHANCED_FRAME_EN |
-            DPCD_LANE_COUNT_SET(data));
+        exynos_dp_write_byte_to_dpcd(
+                    dp, DP_LANE_COUNT_SET,
+                    DP_LANE_COUNT_ENHANCED_FRAME_EN |
+                    DPCD_LANE_COUNT_SET(data));
      else
-        exynos_dp_write_byte_to_dpcd(dp, DP_LANE_COUNT_SET,
-            DPCD_LANE_COUNT_SET(data));
+        exynos_dp_write_byte_to_dpcd(
+                    dp, DP_LANE_COUNT_SET,
+                    DPCD_LANE_COUNT_SET(data));
  }

  static int exynos_dp_is_enhanced_mode_available(struct 
exynos_dp_device *dp)
@@ -261,13 +260,12 @@ static void exynos_dp_training_pattern_dis(struct 
exynos_dp_device *dp)
  {
      exynos_dp_set_training_pattern(dp, DP_NONE);

-    exynos_dp_write_byte_to_dpcd(dp,
-        DP_TRAINING_PATTERN_SET,
-        DP_TRAINING_PATTERN_DISABLE);
+    exynos_dp_write_byte_to_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                     DP_TRAINING_PATTERN_DISABLE);
  }

  static void exynos_dp_set_lane_lane_pre_emphasis(struct 
exynos_dp_device *dp,
-                    int pre_emphasis, int lane)
+                         int pre_emphasis, int lane)
  {
      switch (lane) {
      case 0:
@@ -307,15 +305,14 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
      /* Setup RX configuration */
      buf[0] = dp->link_train.link_rate;
      buf[1] = dp->link_train.lane_count;
-    retval = exynos_dp_write_bytes_to_dpcd(dp, DP_LINK_BW_SET,
-                2, buf);
+    retval = exynos_dp_write_bytes_to_dpcd(dp, DP_LINK_BW_SET, 2, buf);
      if (retval)
          return retval;

      /* Set TX pre-emphasis to minimum */
      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_lane_pre_emphasis(dp,
-            PRE_EMPHASIS_LEVEL_0, lane);
+        exynos_dp_set_lane_lane_pre_emphasis(dp, PRE_EMPHASIS_LEVEL_0,
+                             lane);

      /* Wait for PLL lock */
      pll_tries = 0;
@@ -333,9 +330,9 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
      exynos_dp_set_training_pattern(dp, TRAINING_PTN1);

      /* Set RX training pattern */
-    retval = exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            DP_LINK_SCRAMBLING_DISABLE | DP_TRAINING_PATTERN_1);
+    retval = exynos_dp_write_byte_to_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          DP_LINK_SCRAMBLING_DISABLE |
+                          DP_TRAINING_PATTERN_1);
      if (retval)
          return retval;

@@ -344,7 +341,7 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
                  DP_TRAIN_VOLTAGE_SWING_LEVEL_0;

      retval = exynos_dp_write_bytes_to_dpcd(dp, DP_TRAINING_LANE0_SET,
-            lane_count, buf);
+                           lane_count, buf);

      return retval;
  }
@@ -352,7 +349,7 @@ static int exynos_dp_link_start(struct 
exynos_dp_device *dp)
  static unsigned char exynos_dp_get_lane_status(u8 link_status[2], int 
lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = link_status[lane>>1];
+    u8 link_value = link_status[lane >> 1];

      return (link_value >> shift) & 0xf;
  }
@@ -371,7 +368,7 @@ static int exynos_dp_clock_recovery_ok(u8 
link_status[2], int lane_count)
  }

  static int exynos_dp_channel_eq_ok(u8 link_status[2], u8 link_align,
-                int lane_count)
+                   int lane_count)
  {
      int lane;
      u8 lane_status;
@@ -390,10 +387,10 @@ static int exynos_dp_channel_eq_ok(u8 
link_status[2], u8 link_align,
  }

  static unsigned char exynos_dp_get_adjust_request_voltage(u8 
adjust_request[2],
-                            int lane)
+                              int lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = adjust_request[lane>>1];
+    u8 link_value = adjust_request[lane >> 1];

      return (link_value >> shift) & 0x3;
  }
@@ -403,13 +400,13 @@ static unsigned char 
exynos_dp_get_adjust_request_pre_emphasis(
                      int lane)
  {
      int shift = (lane & 1) * 4;
-    u8 link_value = adjust_request[lane>>1];
+    u8 link_value = adjust_request[lane >> 1];

      return ((link_value >> shift) & 0xc) >> 2;
  }

  static void exynos_dp_set_lane_link_training(struct exynos_dp_device *dp,
-                    u8 training_lane_set, int lane)
+                         u8 training_lane_set, int lane)
  {
      switch (lane) {
      case 0:
@@ -465,7 +462,7 @@ static void exynos_dp_reduce_link_rate(struct 
exynos_dp_device *dp)
  }

  static void exynos_dp_get_adjust_training_lane(struct exynos_dp_device 
*dp,
-                    u8 adjust_request[2])
+                           u8 adjust_request[2])
  {
      int lane, lane_count;
      u8 voltage_swing, pre_emphasis, training_lane;
@@ -498,13 +495,13 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)

      lane_count = dp->link_train.lane_count;

-    retval =  exynos_dp_read_bytes_from_dpcd(dp,
-            DP_LANE0_1_STATUS, 2, link_status);
+    retval =  exynos_dp_read_bytes_from_dpcd(dp, DP_LANE0_1_STATUS,
+                         2, link_status);
      if (retval)
          return retval;

-    retval =  exynos_dp_read_bytes_from_dpcd(dp,
-            DP_ADJUST_REQUEST_LANE0_1, 2, adjust_request);
+    retval =  exynos_dp_read_bytes_from_dpcd(dp, DP_ADJUST_REQUEST_LANE0_1,
+                         2, adjust_request);
      if (retval)
          return retval;

@@ -512,8 +509,8 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
          /* set training pattern 2 for EQ */
          exynos_dp_set_training_pattern(dp, TRAINING_PTN2);

-        retval = exynos_dp_write_byte_to_dpcd(dp,
-                DP_TRAINING_PATTERN_SET,
+        retval = exynos_dp_write_byte_to_dpcd(
+                dp, DP_TRAINING_PATTERN_SET,
                  DP_LINK_SCRAMBLING_DISABLE |
                  DP_TRAINING_PATTERN_2);
          if (retval)
@@ -527,8 +524,9 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
                              dp, lane);
              voltage_swing = exynos_dp_get_adjust_request_voltage(
                              adjust_request, lane);
-            pre_emphasis = exynos_dp_get_adjust_request_pre_emphasis(
-                            adjust_request, lane);
+            pre_emphasis =
+                exynos_dp_get_adjust_request_pre_emphasis(
+                        adjust_request, lane);

              if (DPCD_VOLTAGE_SWING_GET(training_lane) ==
                      voltage_swing &&
@@ -551,11 +549,11 @@ static int exynos_dp_process_clock_recovery(struct 
exynos_dp_device *dp)
      exynos_dp_get_adjust_training_lane(dp, adjust_request);

      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_link_training(dp,
-            dp->link_train.training_lane[lane], lane);
+        exynos_dp_set_lane_link_training(
+                dp, dp->link_train.training_lane[lane], lane);

-    retval = exynos_dp_write_bytes_to_dpcd(dp,
-            DP_TRAINING_LANE0_SET, lane_count,
+    retval = exynos_dp_write_bytes_to_dpcd(
+            dp, DP_TRAINING_LANE0_SET, lane_count,
              dp->link_train.training_lane);
      if (retval)
          return retval;
@@ -573,8 +571,8 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)

      lane_count = dp->link_train.lane_count;

-    retval = exynos_dp_read_bytes_from_dpcd(dp,
-            DP_LANE0_1_STATUS, 2, link_status);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_LANE0_1_STATUS,
+                        2, link_status);
      if (retval)
          return retval;

@@ -583,13 +581,13 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)
          return -EIO;
      }

-    retval = exynos_dp_read_bytes_from_dpcd(dp,
-            DP_ADJUST_REQUEST_LANE0_1, 2, adjust_request);
+    retval = exynos_dp_read_bytes_from_dpcd(dp, DP_ADJUST_REQUEST_LANE0_1,
+                        2, adjust_request);
      if (retval)
          return retval;

-    retval = exynos_dp_read_byte_from_dpcd(dp,
-            DP_LANE_ALIGN_STATUS_UPDATED, &link_align);
+    retval = exynos_dp_read_byte_from_dpcd(
+            dp, DP_LANE_ALIGN_STATUS_UPDATED, &link_align);
      if (retval)
          return retval;

@@ -628,17 +626,18 @@ static int 
exynos_dp_process_equalizer_training(struct exynos_dp_device *dp)
      }

      for (lane = 0; lane < lane_count; lane++)
-        exynos_dp_set_lane_link_training(dp,
-            dp->link_train.training_lane[lane], lane);
+        exynos_dp_set_lane_link_training(
+                dp, dp->link_train.training_lane[lane], lane);

      retval = exynos_dp_write_bytes_to_dpcd(dp, DP_TRAINING_LANE0_SET,
-            lane_count, dp->link_train.training_lane);
+                           lane_count,
+                           dp->link_train.training_lane);

      return retval;
  }

  static void exynos_dp_get_max_rx_bandwidth(struct exynos_dp_device *dp,
-                    u8 *bandwidth)
+                       u8 *bandwidth)
  {
      u8 data;

@@ -651,7 +650,7 @@ static void exynos_dp_get_max_rx_bandwidth(struct 
exynos_dp_device *dp,
  }

  static void exynos_dp_get_max_rx_lane_count(struct exynos_dp_device *dp,
-                    u8 *lane_count)
+                        u8 *lane_count)
  {
      u8 data;

@@ -664,8 +663,8 @@ static void exynos_dp_get_max_rx_lane_count(struct 
exynos_dp_device *dp,
  }

  static void exynos_dp_init_training(struct exynos_dp_device *dp,
-            enum link_lane_count_type max_lane,
-            enum link_rate_type max_rate)
+                    enum link_lane_count_type max_lane,
+                    enum link_rate_type max_rate)
  {
      /*
       * MACRO_RST must be applied after the PLL_LOCK to avoid
@@ -678,7 +677,7 @@ static void exynos_dp_init_training(struct 
exynos_dp_device *dp,
      exynos_dp_get_max_rx_lane_count(dp, &dp->link_train.lane_count);

      if ((dp->link_train.link_rate != LINK_RATE_1_62GBPS) &&
-       (dp->link_train.link_rate != LINK_RATE_2_70GBPS)) {
+        (dp->link_train.link_rate != LINK_RATE_2_70GBPS)) {
          dev_err(dp->dev, "Rx Max Link Rate is abnormal :%x !\n",
              dp->link_train.link_rate);
          dp->link_train.link_rate = LINK_RATE_1_62GBPS;
@@ -738,8 +737,7 @@ static int exynos_dp_sw_link_training(struct 
exynos_dp_device *dp)
  }

  static int exynos_dp_set_link_train(struct exynos_dp_device *dp,
-                u32 count,
-                u32 bwtype)
+                    u32 count, u32 bwtype)
  {
      int i;
      int retval;
@@ -830,21 +828,19 @@ static void exynos_dp_enable_scramble(struct 
exynos_dp_device *dp, bool enable)
      if (enable) {
          exynos_dp_enable_scrambling(dp);

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            &data);
-        exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
+        exynos_dp_read_byte_from_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          &data);
+        exynos_dp_write_byte_to_dpcd(
+            dp, DP_TRAINING_PATTERN_SET,
              (u8)(data & ~DP_LINK_SCRAMBLING_DISABLE));
      } else {
          exynos_dp_disable_scrambling(dp);

-        exynos_dp_read_byte_from_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            &data);
-        exynos_dp_write_byte_to_dpcd(dp,
-            DP_TRAINING_PATTERN_SET,
-            (u8)(data | DP_LINK_SCRAMBLING_DISABLE));
+        exynos_dp_read_byte_from_dpcd(dp, DP_TRAINING_PATTERN_SET,
+                          &data);
+        exynos_dp_write_byte_to_dpcd(
+                dp, DP_TRAINING_PATTERN_SET,
+                (u8)(data | DP_LINK_SCRAMBLING_DISABLE));
      }
  }

@@ -915,7 +911,7 @@ static void exynos_dp_commit(struct 
exynos_drm_display *display)
      }

      ret = exynos_dp_set_link_train(dp, dp->video_info->lane_count,
-                    dp->video_info->link_rate);
+                       dp->video_info->link_rate);
      if (ret) {
          dev_err(dp->dev, "unable to do link train\n");
          return;
@@ -1004,7 +1000,7 @@ static struct drm_connector_helper_funcs 
exynos_dp_connector_helper_funcs = {

  /* returns the number of bridges attached */
  static int exynos_drm_attach_lcd_bridge(struct exynos_dp_device *dp,
-        struct drm_encoder *encoder)
+                    struct drm_encoder *encoder)
  {
      int ret;

@@ -1020,7 +1016,7 @@ static int exynos_drm_attach_lcd_bridge(struct 
exynos_dp_device *dp,
  }

  static int exynos_dp_create_connector(struct exynos_drm_display *display,
-                struct drm_encoder *encoder)
+                      struct drm_encoder *encoder)
  {
      struct exynos_dp_device *dp = display_to_dp(display);
      struct drm_connector *connector = &dp->connector;
@@ -1038,7 +1034,8 @@ static int exynos_dp_create_connector(struct 
exynos_drm_display *display,
      connector->polled = DRM_CONNECTOR_POLL_HPD;

      ret = drm_connector_init(dp->drm_dev, connector,
-            &exynos_dp_connector_funcs, DRM_MODE_CONNECTOR_eDP);
+                 &exynos_dp_connector_funcs,
+                 DRM_MODE_CONNECTOR_eDP);
      if (ret) {
          DRM_ERROR("Failed to initialize connector with drm\n");
          return ret;
@@ -1148,8 +1145,8 @@ static struct video_info 
*exynos_dp_dt_parse_pdata(struct device *dev)
      struct device_node *dp_node = dev->of_node;
      struct video_info *dp_video_config;

-    dp_video_config = devm_kzalloc(dev,
-                sizeof(*dp_video_config), GFP_KERNEL);
+    dp_video_config = devm_kzalloc(dev, sizeof(*dp_video_config),
+                       GFP_KERNEL);
      if (!dp_video_config)
          return ERR_PTR(-ENOMEM);

@@ -1163,37 +1160,37 @@ static struct video_info 
*exynos_dp_dt_parse_pdata(struct device *dev)
          of_property_read_bool(dp_node, "interlaced");

      if (of_property_read_u32(dp_node, "samsung,color-space",
-                &dp_video_config->color_space)) {
+                 &dp_video_config->color_space)) {
          dev_err(dev, "failed to get color-space\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,dynamic-range",
-                &dp_video_config->dynamic_range)) {
+                 &dp_video_config->dynamic_range)) {
          dev_err(dev, "failed to get dynamic-range\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,ycbcr-coeff",
-                &dp_video_config->ycbcr_coeff)) {
+                 &dp_video_config->ycbcr_coeff)) {
          dev_err(dev, "failed to get ycbcr-coeff\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,color-depth",
-                &dp_video_config->color_depth)) {
+                 &dp_video_config->color_depth)) {
          dev_err(dev, "failed to get color-depth\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,link-rate",
-                &dp_video_config->link_rate)) {
+                 &dp_video_config->link_rate)) {
          dev_err(dev, "failed to get link-rate\n");
          return ERR_PTR(-EINVAL);
      }

      if (of_property_read_u32(dp_node, "samsung,lane-count",
-                &dp_video_config->lane_count)) {
+                 &dp_video_config->lane_count)) {
          dev_err(dev, "failed to get lane-count\n");
          return ERR_PTR(-EINVAL);
      }
@@ -1206,7 +1203,7 @@ static int exynos_dp_dt_parse_panel(struct 
exynos_dp_device *dp)
      int ret;

      ret = of_get_videomode(dp->dev->of_node, &dp->priv.vm,
-            OF_USE_NATIVE_MODE);
+                   OF_USE_NATIVE_MODE);
      if (ret) {
          DRM_ERROR("failed: of_get_videomode() : %d\n", ret);
          return ret;
@@ -1302,7 +1299,7 @@ static int exynos_dp_bind(struct device *dev, 
struct device *master, void *data)
      exynos_dp_init_dp(dp);

      ret = devm_request_irq(&pdev->dev, dp->irq, exynos_dp_irq_handler,
-            irq_flags, "exynos-dp", dp);
+                   irq_flags, "exynos-dp", dp);
      if (ret) {
          dev_err(&pdev->dev, "failed to request irq\n");
          return ret;
@@ -1315,7 +1312,7 @@ static int exynos_dp_bind(struct device *dev, 
struct device *master, void *data)
  }

  static void exynos_dp_unbind(struct device *dev, struct device *master,
-                void *data)
+                 void *data)
  {
      struct exynos_dp_device *dp = dev_get_drvdata(dev);

@@ -1334,7 +1331,7 @@ static int exynos_dp_probe(struct platform_device 
*pdev)
      struct exynos_dp_device *dp;

      dp = devm_kzalloc(&pdev->dev, sizeof(struct exynos_dp_device),
-                GFP_KERNEL);
+              GFP_KERNEL);
      if (!dp)
          return -ENOMEM;

@@ -1358,8 +1355,9 @@ static int exynos_dp_probe(struct platform_device 
*pdev)
              of_node_put(bridge_node);
              if (!dp->bridge)
                  return -EPROBE_DEFER;
-        } else
+        } else {
              return -EPROBE_DEFER;
+        }
      }

      return component_add(&pdev->dev, &exynos_dp_ops);
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.h 
b/drivers/gpu/drm/exynos/exynos_dp_core.h
index a4e7996..c321ad5 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.h
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.h
@@ -180,8 +180,8 @@ void exynos_dp_config_interrupt(struct 
exynos_dp_device *dp);
  enum pll_status exynos_dp_get_pll_lock_status(struct exynos_dp_device 
*dp);
  void exynos_dp_set_pll_power_down(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_set_analog_power_down(struct exynos_dp_device *dp,
-                enum analog_power_block block,
-                bool enable);
+                     enum analog_power_block block,
+                     bool enable);
  void exynos_dp_init_analog_func(struct exynos_dp_device *dp);
  void exynos_dp_init_hpd(struct exynos_dp_device *dp);
  enum dp_irq_type exynos_dp_get_irq_type(struct exynos_dp_device *dp);
@@ -192,50 +192,50 @@ int exynos_dp_get_plug_in_status(struct 
exynos_dp_device *dp);
  void exynos_dp_enable_sw_function(struct exynos_dp_device *dp);
  int exynos_dp_start_aux_transaction(struct exynos_dp_device *dp);
  int exynos_dp_write_byte_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char data);
+                 unsigned int reg_addr,
+                 unsigned char data);
  int exynos_dp_read_byte_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char *data);
+                  unsigned int reg_addr,
+                  unsigned char *data);
  int exynos_dp_write_bytes_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[]);
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char data[]);
  int exynos_dp_read_bytes_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[]);
+                   unsigned int reg_addr,
+                   unsigned int count,
+                   unsigned char data[]);
  int exynos_dp_select_i2c_device(struct exynos_dp_device *dp,
                  unsigned int device_addr,
                  unsigned int reg_addr);
  int exynos_dp_read_byte_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int *data);
+                 unsigned int device_addr,
+                 unsigned int reg_addr,
+                 unsigned int *data);
  int exynos_dp_read_bytes_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char edid[]);
+                  unsigned int device_addr,
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char edid[]);
  void exynos_dp_set_link_bandwidth(struct exynos_dp_device *dp, u32 
bwtype);
  void exynos_dp_get_link_bandwidth(struct exynos_dp_device *dp, u32 
*bwtype);
  void exynos_dp_set_lane_count(struct exynos_dp_device *dp, u32 count);
  void exynos_dp_get_lane_count(struct exynos_dp_device *dp, u32 *count);
  void exynos_dp_enable_enhanced_mode(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_set_training_pattern(struct exynos_dp_device *dp,
-                 enum pattern_set pattern);
+                    enum pattern_set pattern);
  void exynos_dp_set_lane0_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane1_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane2_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane3_pre_emphasis(struct exynos_dp_device *dp, u32 
level);
  void exynos_dp_set_lane0_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane1_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane2_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  void exynos_dp_set_lane3_link_training(struct exynos_dp_device *dp,
-                u32 training_lane);
+                       u32 training_lane);
  u32 exynos_dp_get_lane0_link_training(struct exynos_dp_device *dp);
  u32 exynos_dp_get_lane1_link_training(struct exynos_dp_device *dp);
  u32 exynos_dp_get_lane2_link_training(struct exynos_dp_device *dp);
@@ -246,9 +246,8 @@ void exynos_dp_init_video(struct exynos_dp_device *dp);
  void exynos_dp_set_video_color_format(struct exynos_dp_device *dp);
  int exynos_dp_is_slave_video_stream_clock_on(struct exynos_dp_device *dp);
  void exynos_dp_set_video_cr_mn(struct exynos_dp_device *dp,
-            enum clock_recovery_m_value_type type,
-            u32 m_value,
-            u32 n_value);
+                   enum clock_recovery_m_value_type type,
+                   u32 m_value, u32 n_value);
  void exynos_dp_set_video_timing_mode(struct exynos_dp_device *dp, u32 
type);
  void exynos_dp_enable_video_master(struct exynos_dp_device *dp, bool 
enable);
  void exynos_dp_start_video(struct exynos_dp_device *dp);
diff --git a/drivers/gpu/drm/exynos/exynos_dp_reg.c 
b/drivers/gpu/drm/exynos/exynos_dp_reg.c
index c1f87a2..9aa483d 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_reg.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_reg.c
@@ -202,8 +202,8 @@ void exynos_dp_set_pll_power_down(struct 
exynos_dp_device *dp, bool enable)
  }

  void exynos_dp_set_analog_power_down(struct exynos_dp_device *dp,
-                enum analog_power_block block,
-                bool enable)
+                     enum analog_power_block block,
+                     bool enable)
  {
      u32 reg;

@@ -399,8 +399,8 @@ void exynos_dp_init_aux(struct exynos_dp_device *dp)
      exynos_dp_reset_aux(dp);

      /* Disable AUX transaction H/W retry */
-    reg = AUX_BIT_PERIOD_EXPECTED_DELAY(3) | AUX_HW_RETRY_COUNT_SEL(0)|
-        AUX_HW_RETRY_INTERVAL_600_MICROSECONDS;
+    reg = AUX_BIT_PERIOD_EXPECTED_DELAY(3) | AUX_HW_RETRY_COUNT_SEL(0) |
+          AUX_HW_RETRY_INTERVAL_600_MICROSECONDS;
      writel(reg, dp->reg_base + EXYNOS_DP_AUX_HW_RETRY_CTL);

      /* Receive AUX Channel DEFER commands equal to DEFFER_COUNT*64 */
@@ -483,8 +483,8 @@ int exynos_dp_start_aux_transaction(struct 
exynos_dp_device *dp)
  }

  int exynos_dp_write_byte_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char data)
+                 unsigned int reg_addr,
+                 unsigned char data)
  {
      u32 reg;
      int i;
@@ -519,17 +519,16 @@ int exynos_dp_write_byte_to_dpcd(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      return retval;
  }

  int exynos_dp_read_byte_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned char *data)
+                  unsigned int reg_addr,
+                  unsigned char *data)
  {
      u32 reg;
      int i;
@@ -560,9 +559,8 @@ int exynos_dp_read_byte_from_dpcd(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      /* Read data buffer */
@@ -573,9 +571,9 @@ int exynos_dp_read_byte_from_dpcd(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_write_bytes_to_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[])
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char data[])
  {
      u32 reg;
      unsigned int start_offset;
@@ -625,9 +623,9 @@ int exynos_dp_write_bytes_to_dpcd(struct 
exynos_dp_device *dp,
              retval = exynos_dp_start_aux_transaction(dp);
              if (retval == 0)
                  break;
-            else
-                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                    __func__);
+
+            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                __func__);
          }

          start_offset += cur_data_count;
@@ -637,9 +635,9 @@ int exynos_dp_write_bytes_to_dpcd(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_bytes_from_dpcd(struct exynos_dp_device *dp,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char data[])
+                   unsigned int reg_addr,
+                   unsigned int count,
+                   unsigned char data[])
  {
      u32 reg;
      unsigned int start_offset;
@@ -683,9 +681,9 @@ int exynos_dp_read_bytes_from_dpcd(struct 
exynos_dp_device *dp,
              retval = exynos_dp_start_aux_transaction(dp);
              if (retval == 0)
                  break;
-            else
-                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                    __func__);
+
+            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                __func__);
          }

          for (cur_data_idx = 0; cur_data_idx < cur_data_count;
@@ -736,9 +734,9 @@ int exynos_dp_select_i2c_device(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_byte_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int *data)
+                 unsigned int device_addr,
+                 unsigned int reg_addr,
+                 unsigned int *data)
  {
      u32 reg;
      int i;
@@ -767,9 +765,8 @@ int exynos_dp_read_byte_from_i2c(struct 
exynos_dp_device *dp,
          retval = exynos_dp_start_aux_transaction(dp);
          if (retval == 0)
              break;
-        else
-            dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
-                __func__);
+
+        dev_dbg(dp->dev, "%s: Aux Transaction fail!\n", __func__);
      }

      /* Read data */
@@ -780,10 +777,10 @@ int exynos_dp_read_byte_from_i2c(struct 
exynos_dp_device *dp,
  }

  int exynos_dp_read_bytes_from_i2c(struct exynos_dp_device *dp,
-                unsigned int device_addr,
-                unsigned int reg_addr,
-                unsigned int count,
-                unsigned char edid[])
+                  unsigned int device_addr,
+                  unsigned int reg_addr,
+                  unsigned int count,
+                  unsigned char edid[])
  {
      u32 reg;
      unsigned int i, j;
@@ -807,8 +804,8 @@ int exynos_dp_read_bytes_from_i2c(struct 
exynos_dp_device *dp,
               * request without sending address
               */
              if (!defer)
-                retval = exynos_dp_select_i2c_device(dp,
-                        device_addr, reg_addr + i);
+                retval = exynos_dp_select_i2c_device(
+                        dp, device_addr, reg_addr + i);
              else
                  defer = 0;

@@ -828,15 +825,14 @@ int exynos_dp_read_bytes_from_i2c(struct 
exynos_dp_device *dp,
                  retval = exynos_dp_start_aux_transaction(dp);
                  if (retval == 0)
                      break;
-                else
-                    dev_dbg(dp->dev,
-                        "%s: Aux Transaction fail!\n",
-                        __func__);
+
+                dev_dbg(dp->dev, "%s: Aux Transaction fail!\n",
+                    __func__);
              }
              /* Check if Rx sends defer */
              reg = readl(dp->reg_base + EXYNOS_DP_AUX_RX_COMM);
              if (reg == AUX_RX_COMM_AUX_DEFER ||
-                reg == AUX_RX_COMM_I2C_DEFER) {
+                reg == AUX_RX_COMM_I2C_DEFER) {
                  dev_err(dp->dev, "Defer: %d\n\n", reg);
                  defer = 1;
              }
@@ -901,7 +897,7 @@ void exynos_dp_enable_enhanced_mode(struct 
exynos_dp_device *dp, bool enable)
  }

  void exynos_dp_set_training_pattern(struct exynos_dp_device *dp,
-                 enum pattern_set pattern)
+                    enum pattern_set pattern)
  {
      u32 reg;

@@ -974,7 +970,7 @@ void exynos_dp_set_lane3_pre_emphasis(struct 
exynos_dp_device *dp, u32 level)
  }

  void exynos_dp_set_lane0_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -983,7 +979,7 @@ void exynos_dp_set_lane0_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane1_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -992,7 +988,7 @@ void exynos_dp_set_lane1_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane2_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -1001,7 +997,7 @@ void exynos_dp_set_lane2_link_training(struct 
exynos_dp_device *dp,
  }

  void exynos_dp_set_lane3_link_training(struct exynos_dp_device *dp,
-                    u32 training_lane)
+                       u32 training_lane)
  {
      u32 reg;

@@ -1125,9 +1121,9 @@ int 
exynos_dp_is_slave_video_stream_clock_on(struct exynos_dp_device *dp)
  }

  void exynos_dp_set_video_cr_mn(struct exynos_dp_device *dp,
-        enum clock_recovery_m_value_type type,
-        u32 m_value,
-        u32 n_value)
+                   enum clock_recovery_m_value_type type,
+                   u32 m_value,
+                   u32 n_value)
  {
      u32 reg;

@@ -1221,7 +1217,7 @@ void exynos_dp_config_video_slave_mode(struct 
exynos_dp_device *dp)
      u32 reg;

      reg = readl(dp->reg_base + EXYNOS_DP_FUNC_EN_1);
-    reg &= ~(MASTER_VID_FUNC_EN_N|SLAVE_VID_FUNC_EN_N);
+    reg &= ~(MASTER_VID_FUNC_EN_N | SLAVE_VID_FUNC_EN_N);
      reg |= MASTER_VID_FUNC_EN_N;
      writel(reg, dp->reg_base + EXYNOS_DP_FUNC_EN_1);

-- 
2.1.2

  reply	other threads:[~2015-08-06 13:58 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-06 13:49 [RFC PATCH 0/8] Add Analogix Core Display Port Driver Yakir Yang
2015-08-06 13:49 ` Yakir Yang
2015-08-06 13:58 ` Yakir Yang [this message]
2015-08-06 13:58   ` [RFC PATCH 1/8] drm: exynos/dp: fix code style Yakir Yang
2015-08-06 14:04 ` Yakir Yang
2015-08-06 14:04   ` Yakir Yang
2015-08-06 15:05   ` Joe Perches
2015-08-06 15:05     ` Joe Perches
2015-08-06 15:05     ` Joe Perches
2015-08-06 15:20     ` Yakir Yang
2015-08-06 15:20       ` Yakir Yang
2015-08-06 15:20       ` Yakir Yang
2015-08-06 14:07 ` [RFC PATCH 2/8] drm: exynos/dp: convert to drm bridge mode Yakir Yang
2015-08-06 14:07   ` Yakir Yang
2015-08-06 14:19   ` Jingoo Han
2015-08-06 14:19     ` Jingoo Han
2015-08-06 14:19     ` Jingoo Han
2015-08-06 14:29     ` Yakir Yang
2015-08-06 14:29       ` Yakir Yang
2015-08-06 14:29       ` Yakir Yang
2015-08-07 11:25       ` Daniel Vetter
2015-08-07 11:25         ` Daniel Vetter
2015-08-07 13:18         ` Yakir Yang
2015-08-07 13:18           ` Yakir Yang
2015-08-06 14:19 ` [RFC PATCH 3/8] drm: bridge: analogix_dp: split exynos dp driver to bridge dir Yakir Yang
2015-08-06 14:19   ` Yakir Yang
2015-08-06 14:33   ` Jingoo Han
2015-08-06 14:33     ` Jingoo Han
2015-08-06 14:57     ` Yakir Yang
2015-08-06 14:57       ` Yakir Yang
2015-08-06 14:57       ` Yakir Yang
2015-08-06 14:19 ` [RFC PATCH 4/8] drm: rockchip/dp: add rockchip platform dp driver Yakir Yang
2015-08-06 14:19   ` Yakir Yang
2015-08-06 14:19   ` Yakir Yang
2015-08-06 14:19 ` [RFC PATCH 5/8] drm: bridge/analogix_dp: add platform device type support Yakir Yang
2015-08-06 14:19   ` Yakir Yang
2015-08-06 14:19   ` Yakir Yang
2015-08-06 14:22 ` [RFC PATCH 6/8] drm: bridge: analogix_dp: add some rk3288 special registers setting Yakir Yang
2015-08-06 14:22   ` Yakir Yang
2015-08-06 14:22   ` Yakir Yang
2015-08-06 14:24 ` [RFC PATCH 7/8] drm: bridge: analogix_dp: try force hpd after plug in lookup failed Yakir Yang
2015-08-06 14:24   ` Yakir Yang
2015-08-06 14:24   ` Yakir Yang
2015-08-06 14:26 ` [RFC PATCH 8/8] drm: bridge/analogix_dp: expand the delay time for hpd detect Yakir Yang
2015-08-06 14:26   ` Yakir Yang
2015-08-06 14:26   ` Yakir Yang
2015-08-06 14:41 ` [RFC PATCH 0/8] Add Analogix Core Display Port Driver Jingoo Han
2015-08-06 14:41   ` Jingoo Han
2015-08-06 14:41   ` Jingoo Han
2015-08-06 15:00   ` Yakir Yang
2015-08-06 15:00     ` Yakir Yang
2015-08-07  6:11     ` Ajay kumar
2015-08-07  8:17       ` Yakir Yang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55C3680E.30705@rock-chips.com \
    --to=ykk@rock-chips.com \
    --cc=a.hajda@samsung.com \
    --cc=airlied@linux.ie \
    --cc=ajaykumar.rs@samsung.com \
    --cc=andy.yan@rock-chips.com \
    --cc=dianders@google.com \
    --cc=djkurtz@google.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=fabio.estevam@freescale.com \
    --cc=gustavo.padovan@collabora.co.uk \
    --cc=heiko@sntech.de \
    --cc=inki.dae@samsung.com \
    --cc=jingoohan1@gmail.com \
    --cc=jy0922.shim@samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=mark.yao@rock-chips.com \
    --cc=p.zabel@pengutronix.de \
    --cc=rmk+kernel@arm.linux.org.uk \
    --cc=sw0312.kim@samsung.com \
    --cc=tiwai@suse.de \
    --cc=treding@nvidia.com \
    --cc=vpalatin@chromium.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.