IPMI Add SEL command question

[Emily Shaffer]

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.

On Fri, Jul 12, 2019 at 7:08 AM Harry Sung1 <hsung1 at lenovo.com> wrote:

> Hi Team,
>
>
>
> I have a question about Add SEL command.
>
> This standard command seems not be implemented in “phosphor-host-ipmid”.
>
> (
> https://github.com/openbmc/phosphor-host-ipmid/blob/master/storagehandler.cpp#L605
> )
>
>
>
> Though I found a patch set about it, but it doesn’t be merged in master
> due to some issues.
>
> https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-host-ipmid/+/12951
>
>
>
> Is anyone having any plans to implement it? Or other repository we can
> refer to?
>
>
>
> Thanks,
>
> Harry
>


-- 
Emily Shaffer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20190712/78c10e71/attachment.htm>