On Thursday, February 23, 2017, Alex Deucher <<a href="mailto:alexdeucher@gmail.com">alexdeucher@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Feb 23, 2017 at 5:50 PM, Alexandre Demers<br>
<<a href="javascript:;" onclick="_e(event, 'cvml', 'alexandre.f.demers@gmail.com')">alexandre.f.demers@gmail.com</a>> wrote:<br>
> First, sorry for not replying directly as I should normally, but I'm not on<br>
> my usual computer, so I can't. That being said...<br>
><br>
> I may have my eyes in the same socket right now, but I think<br>
> fiji_setup_dpm_led_config() always returns 0.<br>
><br>
>> + if (mask)<br>
>> + smum_send_msg_to_smc_with_<wbr>parameter(hwmgr->smumgr,<br>
>> +    PPSMC_MSG_LedConfig,<br>
>> +    mask);<br>
>> + return 0;<br>
>> +}<br>
><br>
> Even when "if (mask)" is true, whether smum_send_msg_to_smc_with_<wbr>parameter()<br>
> succeeds or not, fiji_setup_dpm_led_config() spits a 0 at the end.<br>
><br>
> Thus,<br>
><br>
>> + result = fiji_setup_dpm_led_config(<wbr>hwmgr);<br>
>> + PP_ASSERT_WITH_CODE(0 == result,<br>
>> +    "Failed to setup dpm led config", return result);<br>
><br>
> will always lead to "result" being set to 0... Am I missing something?<br>
><br>
<br>
Yes, that function can't fail.  I suppose we could just make it a void<br>
function, but I was following the same pattern of all the other<br>
functions in that file.<br>
<br>
Alex<br>
</blockquote><div><br></div><div>I see. Well, if there is no forseseeable change that would justify to return a different value, I would be tempted to go with the void. But either way, I'll live with it.</div><div><br></div><div>Alexandre Demers</div>