[PATCH] drm/amd/powerplay: Fix missing break in switch

Gustavo A. R. Silva gustavo at embeddedor.com
Mon Jan 28 18:23:16 UTC 2019



On 1/28/19 10:19 AM, Alex Deucher wrote:
> On Fri, Jan 25, 2019 at 5:31 PM Gustavo A. R. Silva
> <gustavo at embeddedor.com> wrote:
>>
>> Add missing break statement in order to prevent the code from falling
>> through to the default case.
>>
>> The resoning for this is that pclk_vol_table is an automatic variable.
>> So, it makes no sense to update it just before falling through to the
>> default case and return -EINVAL.
>>
>> This bug was found thanks to the ongoing efforts to enabling
>> -Wimplicit-fallthrough.
>>
>> Fixes: cd70f3d6e3fa ("drm/amd/powerplay: PP/DAL interface changes for dynamic clock switch")
>> Cc: stable at vger.kernel.org
>> Signed-off-by: Gustavo A. R. Silva <gustavo at embeddedor.com>
> 
> Applied.  Thanks!
> 

Great. :)

Thanks, Alex.
--
Gustavo


More information about the amd-gfx mailing list