Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.My bad, I will provide expansion for the acronyms in the first usage.
On Thu, Jul 04, 2024 at 11:16:18AM +0000, Akshay Gupta wrote:
AMD's APML interface provides system management functionality accessed byWhat is "APML"? What is "BMC"?
the BMC. Sbrmi driver under hwmon subsystem, which is probed as an i2c
driver and reports power using APML specified protocol.
However, APML interface defines few other protocols to supportWhat is out-of-band here?
full system management functionality out-of-band.
It is not a pci device, will remove the platform device and register hwmon device in misc.
This patchset is an attempt to add all APML core functionality in one placeAre you sure this is a platform device? It better not be a pci one...
and provide hwmon and user interface
1. Move the i2c client probe and sbrmi core functionality from drivers/hwmon
to drivers/misc/
2. Add a platform device, which probes the hwmon/sbrmi and continues to
report power using the symbol exported by the misc/sbrmi-core.
We are planning to add i2c/i3c support and 1byte/2 byte register addressing.
3. Convert i2c to regmap which provides multiple benefitsWhat are those?
over direct smbus APIs.
4. Register a misc device which providesWhy?
a. An ioctl interface through node /dev/sbrmiX
b. Open-sourced and widely used https://github.com/amd/esmi_oob_libraryWill this use the new ioctl api? If it's not present, what will it use
will continue to provide user-space programmable API.
instead?
thanks,
greg k-h