[PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 23, 2017 at 5:50 PM, Alexandre Demers
<alexandre.f.demers at gmail.com> wrote:
> First, sorry for not replying directly as I should normally, but I'm not on
> my usual computer, so I can't. That being said...
>
> I may have my eyes in the same socket right now, but I think
> fiji_setup_dpm_led_config() always returns 0.
>
>> + if (mask)
>> + smum_send_msg_to_smc_with_parameter(hwmgr->smumgr,
>> +    PPSMC_MSG_LedConfig,
>> +    mask);
>> + return 0;
>> +}
>
> Even when "if (mask)" is true, whether smum_send_msg_to_smc_with_parameter()
> succeeds or not, fiji_setup_dpm_led_config() spits a 0 at the end.
>
> Thus,
>
>> + result = fiji_setup_dpm_led_config(hwmgr);
>> + PP_ASSERT_WITH_CODE(0 == result,
>> +    "Failed to setup dpm led config", return result);
>
> will always lead to "result" being set to 0... Am I missing something?
>

Yes, that function can't fail.  I suppose we could just make it a void
function, but I was following the same pattern of all the other
functions in that file.

Alex


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux