Re: [PATCH 4/4] Documentation: ABI: Add IBM power supply sysfs documentation

From: Guenter Roeck
Date: Thu Aug 10 2017 - 10:56:26 EST


On Wed, Aug 02, 2017 at 04:17:13PM -0500, Eddie James wrote:
> From: "Edward A. James" <eajames@xxxxxxxxxx>
>
> Signed-off-by: Edward A. James <eajames@xxxxxxxxxx>
> ---
> Documentation/ABI/testing/sysfs-driver-ibmps | 49 ++++++++++++++++++++++++++++
> 1 file changed, 49 insertions(+)
> create mode 100644 Documentation/ABI/testing/sysfs-driver-ibmps
>
> diff --git a/Documentation/ABI/testing/sysfs-driver-ibmps b/Documentation/ABI/testing/sysfs-driver-ibmps
> new file mode 100644
> index 0000000..1645685
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-driver-ibmps
> @@ -0,0 +1,49 @@
> +What: /sys/bus/i2c/devices/<dev>/clear_faults

Please, no. If we do add driver-specific attributes, those should be added to
the hwmon device, not to the i2c device.

Guenter

> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Clears pmbus faults for this power supply.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_word
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_WORD register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_vout
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_VOUT register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_iout
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_IOUT register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_input
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_INPUT register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_temp
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_TEMPERATURE register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_cml
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_CML register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_other
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_OTHER register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_mfr
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_MFR_SPECIFIC register.
> +
> +What: /sys/bus/i2c/devices/<dev>/status_fan
> +KernelVersion: 4.13
> +Contact: eajames@xxxxxxxxxx
> +Description: Displays the STATUS_FAN register.
> --
> 1.8.3.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html