<div dir="ltr">Harry, I suggest that you participate in the discussion on the review you linked to give suggestions. You're right that it's not implemented - if you have a vested interest in it, your best bet is to find out what approach Jason etc. took and see if you can help fix the problems they found in your own implementation. Based on the inactivity there, I'm guessing Intel deprioritized that command, so you should probably expect to drive development effort (or modification of that patch) yourself.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 12, 2019 at 7:08 AM Harry Sung1 <<a href="mailto:hsung1@lenovo.com">hsung1@lenovo.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="ZH-TW">
<div class="gmail-m_1992553702417816486WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi Team,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">I have a question about Add SEL command. <u></u>
<u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">This standard command seems not be implemented in “phosphor-host-ipmid”.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">(<a href="https://github.com/openbmc/phosphor-host-ipmid/blob/master/storagehandler.cpp#L605" target="_blank">https://github.com/openbmc/phosphor-host-ipmid/blob/master/storagehandler.cpp#L605</a>)<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Though I found a patch set about it, but it doesn’t be merged in master due to some issues.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><a href="https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-host-ipmid/+/12951" target="_blank">https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-host-ipmid/+/12951</a><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Is anyone having any plans to implement it? Or other repository we can refer to?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Thanks,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Harry<u></u><u></u></span></p>
</div>
</div>

</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Emily Shaffer</div></div>