In the snippets like the following

        if (...)
                return / goto / break / continue ...;
        else
                ...

the 'else' is redundant. Get rid of it.

Reviewed-by: Andi Shyti <andi.sh...@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
Acked-by: Jani Nikula <jani.nik...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dsi_vbt.c | 58 ++++++++++----------
 1 file changed, 28 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dsi_vbt.c 
b/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
index 290a112f1b63..4ed5ede9ec5b 100644
--- a/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
+++ b/drivers/gpu/drm/i915/display/intel_dsi_vbt.c
@@ -142,7 +142,7 @@ static enum port intel_dsi_seq_port_to_port(struct 
intel_dsi *intel_dsi,
        if (seq_port) {
                if (intel_dsi->ports & BIT(PORT_B))
                        return PORT_B;
-               else if (intel_dsi->ports & BIT(PORT_C))
+               if (intel_dsi->ports & BIT(PORT_C))
                        return PORT_C;
        }
 
@@ -670,8 +670,8 @@ static const char *sequence_name(enum mipi_seq seq_id)
 {
        if (seq_id < ARRAY_SIZE(seq_name) && seq_name[seq_id])
                return seq_name[seq_id];
-       else
-               return "(unknown)";
+
+       return "(unknown)";
 }
 
 static void intel_dsi_vbt_exec(struct intel_dsi *intel_dsi,
@@ -865,36 +865,34 @@ bool intel_dsi_vbt_init(struct intel_dsi *intel_dsi, u16 
panel_id)
         * multiply by 100 to preserve remainder
         */
        if (intel_dsi->video_mode == BURST_MODE) {
-               if (mipi_config->target_burst_mode_freq) {
-                       u32 bitrate = intel_dsi_bitrate(intel_dsi);
+               u32 bitrate;
 
-                       /*
-                        * Sometimes the VBT contains a slightly lower clock,
-                        * then the bitrate we have calculated, in this case
-                        * just replace it with the calculated bitrate.
-                        */
-                       if (mipi_config->target_burst_mode_freq < bitrate &&
-                           intel_fuzzy_clock_check(
-                                       mipi_config->target_burst_mode_freq,
-                                       bitrate))
-                               mipi_config->target_burst_mode_freq = bitrate;
-
-                       if (mipi_config->target_burst_mode_freq < bitrate) {
-                               drm_err(&dev_priv->drm,
-                                       "Burst mode freq is less than 
computed\n");
-                               return false;
-                       }
-
-                       burst_mode_ratio = DIV_ROUND_UP(
-                               mipi_config->target_burst_mode_freq * 100,
-                               bitrate);
-
-                       intel_dsi->pclk = DIV_ROUND_UP(intel_dsi->pclk * 
burst_mode_ratio, 100);
-               } else {
-                       drm_err(&dev_priv->drm,
-                               "Burst mode target is not set\n");
+               if (mipi_config->target_burst_mode_freq == 0) {
+                       drm_err(&dev_priv->drm, "Burst mode target is not 
set\n");
                        return false;
                }
+
+               bitrate = intel_dsi_bitrate(intel_dsi);
+
+               /*
+                * Sometimes the VBT contains a slightly lower clock, then
+                * the bitrate we have calculated, in this case just replace it
+                * with the calculated bitrate.
+                */
+               if (mipi_config->target_burst_mode_freq < bitrate &&
+                   intel_fuzzy_clock_check(mipi_config->target_burst_mode_freq,
+                                           bitrate))
+                       mipi_config->target_burst_mode_freq = bitrate;
+
+               if (mipi_config->target_burst_mode_freq < bitrate) {
+                       drm_err(&dev_priv->drm, "Burst mode freq is less than 
computed\n");
+                       return false;
+               }
+
+               burst_mode_ratio =
+                       DIV_ROUND_UP(mipi_config->target_burst_mode_freq * 100, 
bitrate);
+
+               intel_dsi->pclk = DIV_ROUND_UP(intel_dsi->pclk * 
burst_mode_ratio, 100);
        } else
                burst_mode_ratio = 100;
 
-- 
2.40.0.1.gaa8946217a0b

Reply via email to