On Mon, Apr 3, 2017 at 2:41 PM, Dan Carpenter <dan.carpen...@oracle.com> wrote: > This switch statement is missing breaks. > > Fixes: 2a5071056e6a ("drm/amd/powerplay: add global PowerPlay mutex.") > Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
Applied. thanks! > > diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > index dfd4fe6f0578..260b732be7c3 100644 > --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c > @@ -539,10 +539,13 @@ 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; > _______________________________________________ > dri-devel mailing list > dri-de...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx