<div dir="ltr">Hi Zev<div><br></div><div>Thank you for your quick response!</div><div><br></div><div>Ali got it to work - it was much easier than I had thought. Thanks for all your great work!</div><div><br></div><div>Oskar.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 4, 2023 at 5:10 PM Zev Weiss <<a href="mailto:zev@bewilderbeest.net" target="_blank">zev@bewilderbeest.net</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, May 04, 2023 at 01:00:47PM PDT, Oskar Senft wrote:<br>
>Hi Zev<br>
><br>
>In<br>
><a href="https://github.com/openbmc/entity-manager/commit/e22143df37faa0b0f5e2918d2f505b9f64e74b0f" rel="noreferrer" target="_blank">https://github.com/openbmc/entity-manager/commit/e22143df37faa0b0f5e2918d2f505b9f64e74b0f</a><br>
>you "removed devices now managed by hwmontempsensor".<br>
><br>
>I'm trying to figure out how to add SBTSI support for the TYAN S8036 board<br>
>(AMD Milan). Do I just add the device to the DTS and then reference it in<br>
>EntityManager board configuration via its bus and address?<br>
><br>
>Is there still a way to dynamically get Entity Manager (or now<br>
>dbus-sensors) to bind the driver or does it HAVE to happen in the DTS now?<br>
><br>
>Thanks!<br>
><br>
>Oskar.<br>
<br>
Hi Oskar,<br>
<br>
Assuming you use a corresponding, recent enough version of dbus-sensors <br>
(i.e. including commit a1456c4aba, though hopefully also with at least <br>
commit 7627c860fa, which was a follow-up bugfix), it *should* work the <br>
same way it did before, with the device described in your E-M config.<br>
<br>
It shouldn't be in your DTS, because then it'd be statically defined and <br>
hwmontempsensor wouldn't be able to remove it when the host is powered <br>
off (which I assume you'd want).<br>
<br>
In terms of userspace/kernel separation, it's overall pretty similar to <br>
how it had been previously, just with the management of device lifetimes <br>
(instantiation & removal) moved to hwmontempsensor instead of <br>
entity-manager.<br>
<br>
Hopefully that's clear?  Let me know if you're having trouble getting <br>
things working though.<br>
<br>
<br>
Zev<br>
<br>
</blockquote></div>