<div dir="ltr"><div>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...</div><div><br></div>I may have my eyes in the same socket right now, but I think fiji_setup_dpm_led_config() always returns 0.<div><br></div><div>> +<span class="Apple-tab-span" style="white-space:pre"> </span>if (mask)<br></div><div><div>> +<span class="Apple-tab-span" style="white-space:pre">         </span>smum_send_msg_to_smc_with_parameter(hwmgr->smumgr,</div><div>> +<span class="Apple-tab-span" style="white-space:pre">                                          </span>    PPSMC_MSG_LedConfig,</div><div>> +<span class="Apple-tab-span" style="white-space:pre">                                             </span>    mask);</div><div>> +<span class="Apple-tab-span" style="white-space:pre">   </span>return 0;</div><div>> +}</div></div><div><br></div><div>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.<br></div><div><br></div><div>Thus,</div><div><pre style="white-space:pre-wrap">><i> +        result = fiji_setup_dpm_led_config(hwmgr);
</i>><i> +       PP_ASSERT_WITH_CODE(0 == result,
</i>><i> +                           "Failed to setup dpm led config", return result);
</i></pre>will always lead to "result" being set to 0... Am I missing something?</div><div><br></div><div>Alexandre Demers</div></div>