drm/amd/powerplay: fix pp_dpm_get_current_power_state() (v2)
authorDan Carpenter <dan.carpenter@oracle.com>
Mon, 3 Apr 2017 18:41:47 +0000 (21:41 +0300)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 4 Apr 2017 17:40:33 +0000 (13:40 -0400)
This switch statement is missing breaks.

v2: agd: break in default case as well

Fixes: 2a5071056e6a ("drm/amd/powerplay: add global PowerPlay mutex.")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/amd_powerplay.c

index 6f0ed07bb58735555ad2b0a0008fe900bd914d8d..07dbeb225df75061c48b678a2107b213c8d14af6 100644 (file)
@@ -539,15 +539,19 @@ static enum amd_pm_state_type pp_dpm_get_current_power_state(void *handle)
        switch (state->classification.ui_label) {
        case PP_StateUILabel_Battery:
                pm_type = POWER_STATE_TYPE_BATTERY;
+               break;
        case PP_StateUILabel_Balanced:
                pm_type = POWER_STATE_TYPE_BALANCED;
+               break;
        case PP_StateUILabel_Performance:
                pm_type = POWER_STATE_TYPE_PERFORMANCE;
+               break;
        default:
                if (state->classification.flags & PP_StateClassificationFlag_Boot)
                        pm_type = POWER_STATE_TYPE_INTERNAL_BOOT;
                else
                        pm_type = POWER_STATE_TYPE_DEFAULT;
+               break;
        }
        mutex_unlock(&pp_handle->pp_lock);