Feedback on Current OpenBMC and Proposing Some Improvements ---- "Improvements" Ideas

Brad Bishop bradleyb at fuzziesquirrel.com
Thu Jul 9 22:42:46 AEST 2020


On Wed, Jul 08, 2020 at 11:15:03PM -0700, Ed Tanous wrote:
>On Wed, Jul 8, 2020 at 2:46 PM Alex Qiu <xqiu at google.com> wrote:

>> We've talked with the maintainer of hwmon, and he doesn't think adding
>> these to hwmon interface is a good idea, as hwmon is supposed to be a
>> monitoring interface. 

Except most "monitoring devices" are configurable and/or have a dual 
nature - e.g. voltage regulators.

>> Although we haven't yet met the need to set the
>> voltage other than debugging.

We have a need to tweak things in the production firmware.

>Excellent.  I hadn't realized you'd done that.  You're right,
>userspace is probably the right spot for this then.

I dunno.  It would be nice to have an in-kernel solution that allows 
VRMs to be both monitored and controlled.  Mixing i2c-dev with the 
actual vrm driver makes us do strange things like unbinding drivers 
while the vrms are configured, or rewriting the vrm driver monitor 
function in usersapce and missing out on the benefit of a well 
maintained kernel driver.

This problem comes up over and over again.  What would be great is if 
someone with good relationships and standing in the kernel community 
could work with the kernel maintainers to build their understanding of 
how we are using the kernel and figure out a good solution we could 
implement other than just: "use i2c-dev".


More information about the openbmc mailing list