From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Just return the thing directly from the switch statement.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 22 +++++--------------
 1 file changed, 6 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c 
b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
index e5fb66a5dd02..60b3277a857b 100644
--- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
+++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c
@@ -708,8 +708,6 @@ struct hsw_wrpll_rnp {
 
 static unsigned hsw_wrpll_get_budget_for_freq(int clock)
 {
-       unsigned budget;
-
        switch (clock) {
        case 25175000:
        case 25200000:
@@ -742,21 +740,18 @@ static unsigned hsw_wrpll_get_budget_for_freq(int clock)
        case 222750000:
        case 296703000:
        case 297000000:
-               budget = 0;
-               break;
+               return 0;
        case 233500000:
        case 245250000:
        case 247750000:
        case 253250000:
        case 298000000:
-               budget = 1500;
-               break;
+               return 1500;
        case 169128000:
        case 169500000:
        case 179500000:
        case 202000000:
-               budget = 2000;
-               break;
+               return 2000;
        case 256250000:
        case 262500000:
        case 270000000:
@@ -766,18 +761,13 @@ static unsigned hsw_wrpll_get_budget_for_freq(int clock)
        case 281250000:
        case 286000000:
        case 291750000:
-               budget = 4000;
-               break;
+               return 4000;
        case 267250000:
        case 268500000:
-               budget = 5000;
-               break;
+               return 5000;
        default:
-               budget = 1000;
-               break;
+               return 1000;
        }
-
-       return budget;
 }
 
 static void hsw_wrpll_update_rnp(u64 freq2k, unsigned int budget,
-- 
2.35.1

Reply via email to