<div dir="ltr"><div>Ok, sounds good.</div><div><br></div><div>I'll look into moving to the IPMID repo  with a package config instead. </div><div><br></div><div>Maybe we'll get more suggestions in this thread, but  I'll start work in that direction.</div><div><br></div><div>Willy Tu</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 14, 2021 at 12:06 PM Ed Tanous <<a href="mailto:edtanous@google.com" target="_blank">edtanous@google.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">On Thu, Jan 14, 2021 at 11:40 AM Brad Bishop<br>
<<a href="mailto:bradleyb@fuzziesquirrel.com" target="_blank">bradleyb@fuzziesquirrel.com</a>> wrote:<br>
><br>
> On Thu, Jan 14, 2021 at 09:38:05AM -0800, Vernon Mauery wrote:<br>
> >On 14-Jan-2021 08:38 AM, Ed Tanous wrote:<br>
><br>
> >>Would people prefer it to be a package config on IPMID to select between<br>
> >>the two implementations?<br>
> ><br>
> >I don't have a problem with a package config to select sensor<br>
> >implementations.<br>
><br>
> This seems fine to me too.  I looked and there are non-POWER users of<br>
> the fixed sensor id implementation too so openpower-ipmi-oem probably<br>
> doesn't make good sense.<br>
><br>
> >I would say that if IBM is the only company using the sensor<br>
> >implementation that is currently in ipmid, it would be best to move it<br>
> >to the IBM OEM layer. But it is difficult in a project this size who<br>
> >is using what.<br>
><br>
> 'grep yaml-config' of the openbmc tree gives a pretty decent indicator<br>
> of who is using the fixed sensor ID implementation.<br>
><br>
> -brad<br>
<br>
It's been on my list for a while to write a script to go build at<br>
least rudimentary ported entity-manager configs for the existing<br>
hardware in the tree.  I'd asked James to write this a few times in<br>
the past, but as we know, we all get busy.  This is just to say, it's<br>
on my radar to try to try to make this better.<br>
<br>
-Ed<br>
</blockquote></div>