hwmon: (pmbus/ibm-cffps) Fix write bits for LED control
authorBrandon Wyman <bjwyman@gmail.com>
Fri, 6 Aug 2021 22:51:31 +0000 (22:51 +0000)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 17 Aug 2021 21:54:26 +0000 (14:54 -0700)
When doing a PMBus write for the LED control on the IBM Common Form
Factor Power Supplies (ibm-cffps), the DAh command requires that bit 7
be low and bit 6 be high in order to indicate that you are truly
attempting to do a write.

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Link: https://lore.kernel.org/r/20210806225131.1808759-1-bjwyman@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/pmbus/ibm-cffps.c

index 5668d83..df712ce 100644 (file)
@@ -50,9 +50,9 @@
 #define CFFPS_MFR_VAUX_FAULT                   BIT(6)
 #define CFFPS_MFR_CURRENT_SHARE_WARNING                BIT(7)
 
-#define CFFPS_LED_BLINK                                BIT(0)
-#define CFFPS_LED_ON                           BIT(1)
-#define CFFPS_LED_OFF                          BIT(2)
+#define CFFPS_LED_BLINK                                (BIT(0) | BIT(6))
+#define CFFPS_LED_ON                           (BIT(1) | BIT(6))
+#define CFFPS_LED_OFF                          (BIT(2) | BIT(6))
 #define CFFPS_BLINK_RATE_MS                    250
 
 enum {