<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 26, 2020 at 11:11 PM Andrei Kartashev <<a href="mailto:a.kartashev@yadro.com">a.kartashev@yadro.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">Since there is a plan to move to EM for inventory, I believe it is<br>
really good idea to also have support for FruDevice in phosphor-host-<br>
ipmid. Then we can have a common way on how to handle it.<br>
Same for SDR BTW.<br></blockquote><div><br></div><div>+1.  This was attempted a long time ago, but nobody was able to come up with a design that kept the "old" way working for those that needed it, and at the time there were some missing features.  Given where entity manager has gotten, it's probably time to start that discussion up again.  Do you think you could put together a patch that does what you describe?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
But there is other thing: there is catastrophically not enough<br>
documentation for EntityManager/dbus-sensors. Looks like common way<br>
just to adjust existing config and hope that it still will work.<br>
<sorry, was all the day trying to get adcsensors work yesterday> <br></blockquote><div><br></div><div>That being the case, would you mind taking a look at the docs changes I just put up.  It's trying to improve the EM documentation a bit, although I realize it doesn't get all the way to where it needs to be.</div><div><a href="https://gerrit.openbmc-project.xyz/c/openbmc/entity-manager/+/36110">https://gerrit.openbmc-project.xyz/c/openbmc/entity-manager/+/36110</a><br></div><div><br></div><div>Also, it'd be great if you can come up with some concrete examples of what else we can improve in this regard.  Unfortunately the "copy an existing config and modify" approach was the best way we found to make platform ports easy.  A lot of systems tend to look pretty similar, based on similar reference platforms, so usually there's something to use as a starting point.  Building a config from scratch using first principals and documentation is kind of daunting, and became a non-starter for most people, given that the config files tend to be large.</div><div><br></div><div>What were the biggest roadblocks you hit trying to get ADCSensor working?</div><div> </div></div></div>