This patchset adds support for hardware monitoring features of PMBus devices. The driver has not yet been tested with actual hardware. It is submitted for early review and to solicit test coverage. A PMBus device emulator is provided for reference, to show the level of testing performed. I don't currently plan to submit this emulator in the final patchset. If there is interest to have it included, and if someone is willing to review the code, I will be happy to clean it up and include it. I will test the driver with real HW for BMR453 and LTC2978 as soon as I get access to it. For other devices, I will add a remark that support is untested, unless I get feedback that a specific device was tested and is working. --- Open issues/questions: - The driver adds new power monitoring attributes to the hwmon sysfs API. Need to get feedback if those are acceptable. - Currently no support for chips with multiple phases (PHASE command). Adding this will require access to a system with a chip supporting multiple phases (e.g., UCD9240/46/48) to determine a feasible implementation. - Need to add support for COEFFICIENTS command. Will require system with real chip for testing, or will not add support at this time. - Need to add fan support. Will require system with real chip for testing, or will not add support at this time. Datasheets of chips with fan control support indicate that chip specific code may be necessary. See UCD9240 or UCD90124 datasheets. --- v2 changes: - Calculate maximum number of attributes / sensors / booleans / labels instead of using defines - Increased maximum number of supported pages to the PMBus maximum of 32. - Replaced most fixed size arrays with dynamically allocated arrays/structures. - Added comments and clarifications - Renamed local defines to start with PB_ or PMBUS_ prefix - Removed pages module parameter. Replaced with code to auto-detect the number of supported pages. - Split probe code into several functions to make it better readable - Replaced _fault attributes with _lcrit_alarm and _crit_alarm attributes - Several other minor changes to address review feedback - PMBus emulator: Fixed fault status detection code. Also use more reasonable power limits (eg 200W instead of 2kW). _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
