* [PATCH linux dev-5.4 0/2] hwmon: (pmbus/ibm-cffps) Add another CCIN to version detection and fix LEDs
@ 2020-04-01 22:03 Eddie James
2020-04-01 22:03 ` [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection Eddie James
2020-04-01 22:03 ` [PATCH linux dev-5.4 2/2] hwmon: (pmbus/ibm-cffps) Fix LED behavior again Eddie James
0 siblings, 2 replies; 6+ messages in thread
From: Eddie James @ 2020-04-01 22:03 UTC (permalink / raw)
To: openbmc; +Cc: joel, bjwyman, Eddie James
These commits bring the driver into the same state as -next.
Eddie James (2):
hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection
hwmon: (pmbus/ibm-cffps) Fix LED behavior again
drivers/hwmon/pmbus/ibm-cffps.c | 20 ++++++++++++++------
1 file changed, 14 insertions(+), 6 deletions(-)
--
2.24.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection
2020-04-01 22:03 [PATCH linux dev-5.4 0/2] hwmon: (pmbus/ibm-cffps) Add another CCIN to version detection and fix LEDs Eddie James
@ 2020-04-01 22:03 ` Eddie James
2020-04-01 22:13 ` Guenter Roeck
2020-04-01 22:03 ` [PATCH linux dev-5.4 2/2] hwmon: (pmbus/ibm-cffps) Fix LED behavior again Eddie James
1 sibling, 1 reply; 6+ messages in thread
From: Eddie James @ 2020-04-01 22:03 UTC (permalink / raw)
To: openbmc; +Cc: joel, bjwyman, Eddie James, Guenter Roeck
There is an additional CCIN for the IBM CFFPS that may be classifed as
either version one or version two, based upon the rest of the bits of
the CCIN. Add support for it in the version detection.
Signed-off-by: Eddie James <eajames@linux.ibm.com>
Link: https://lore.kernel.org/r/1583948590-17220-1-git-send-email-eajames@linux.ibm.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
drivers/hwmon/pmbus/ibm-cffps.c | 14 +++++++++++++-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/drivers/hwmon/pmbus/ibm-cffps.c b/drivers/hwmon/pmbus/ibm-cffps.c
index 27593b723d74..7d3720665ad4 100644
--- a/drivers/hwmon/pmbus/ibm-cffps.c
+++ b/drivers/hwmon/pmbus/ibm-cffps.c
@@ -33,9 +33,12 @@
#define CFFPS_INPUT_HISTORY_CMD 0xD6
#define CFFPS_INPUT_HISTORY_SIZE 100
+#define CFFPS_CCIN_REVISION GENMASK(7, 0)
+#define CFFPS_CCIN_REVISION_LEGACY 0xde
#define CFFPS_CCIN_VERSION GENMASK(15, 8)
#define CFFPS_CCIN_VERSION_1 0x2b
#define CFFPS_CCIN_VERSION_2 0x2e
+#define CFFPS_CCIN_VERSION_3 0x51
/* STATUS_MFR_SPECIFIC bits */
#define CFFPS_MFR_FAN_FAULT BIT(0)
@@ -489,11 +492,14 @@ static int ibm_cffps_probe(struct i2c_client *client,
vs = (enum versions)id->driver_data;
if (vs == cffps_unknown) {
+ u16 ccin_revision = 0;
u16 ccin_version = CFFPS_CCIN_VERSION_1;
int ccin = i2c_smbus_read_word_swapped(client, CFFPS_CCIN_CMD);
- if (ccin > 0)
+ if (ccin > 0) {
+ ccin_revision = FIELD_GET(CFFPS_CCIN_REVISION, ccin);
ccin_version = FIELD_GET(CFFPS_CCIN_VERSION, ccin);
+ }
switch (ccin_version) {
default:
@@ -503,6 +509,12 @@ static int ibm_cffps_probe(struct i2c_client *client,
case CFFPS_CCIN_VERSION_2:
vs = cffps2;
break;
+ case CFFPS_CCIN_VERSION_3:
+ if (ccin_revision == CFFPS_CCIN_REVISION_LEGACY)
+ vs = cffps1;
+ else
+ vs = cffps2;
+ break;
}
/* Set the client name to include the version number. */
--
2.24.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH linux dev-5.4 2/2] hwmon: (pmbus/ibm-cffps) Fix LED behavior again
2020-04-01 22:03 [PATCH linux dev-5.4 0/2] hwmon: (pmbus/ibm-cffps) Add another CCIN to version detection and fix LEDs Eddie James
2020-04-01 22:03 ` [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection Eddie James
@ 2020-04-01 22:03 ` Eddie James
1 sibling, 0 replies; 6+ messages in thread
From: Eddie James @ 2020-04-01 22:03 UTC (permalink / raw)
To: openbmc; +Cc: joel, bjwyman, Eddie James
Get the driver into the same state as -next. The driver should not
relinquish control of the LEDs to the PSU firmware.
Signed-off-by: Eddie James <eajames@linux.ibm.com>
---
drivers/hwmon/pmbus/ibm-cffps.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/hwmon/pmbus/ibm-cffps.c b/drivers/hwmon/pmbus/ibm-cffps.c
index 7d3720665ad4..7cf4cc5824ab 100644
--- a/drivers/hwmon/pmbus/ibm-cffps.c
+++ b/drivers/hwmon/pmbus/ibm-cffps.c
@@ -50,13 +50,9 @@
#define CFFPS_MFR_VAUX_FAULT BIT(6)
#define CFFPS_MFR_CURRENT_SHARE_WARNING BIT(7)
-/*
- * LED off state actually relinquishes LED control to PSU firmware, so it can
- * turn on the LED for faults.
- */
-#define CFFPS_LED_OFF 0
#define CFFPS_LED_BLINK BIT(0)
#define CFFPS_LED_ON BIT(1)
+#define CFFPS_LED_OFF BIT(2)
#define CFFPS_BLINK_RATE_MS 250
enum {
--
2.24.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection
2020-04-01 22:03 ` [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection Eddie James
@ 2020-04-01 22:13 ` Guenter Roeck
2020-04-01 22:20 ` Joel Stanley
0 siblings, 1 reply; 6+ messages in thread
From: Guenter Roeck @ 2020-04-01 22:13 UTC (permalink / raw)
To: Eddie James, openbmc; +Cc: joel, bjwyman
On 4/1/20 3:03 PM, Eddie James wrote:
> There is an additional CCIN for the IBM CFFPS that may be classifed as
> either version one or version two, based upon the rest of the bits of
> the CCIN. Add support for it in the version detection.
>
> Signed-off-by: Eddie James <eajames@linux.ibm.com>
> Link: https://lore.kernel.org/r/1583948590-17220-1-git-send-email-eajames@linux.ibm.com
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Those e-mails are confusing. Can you drop me from cc:, please ?
Guenter
> ---
> drivers/hwmon/pmbus/ibm-cffps.c | 14 +++++++++++++-
> 1 file changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/pmbus/ibm-cffps.c b/drivers/hwmon/pmbus/ibm-cffps.c
> index 27593b723d74..7d3720665ad4 100644
> --- a/drivers/hwmon/pmbus/ibm-cffps.c
> +++ b/drivers/hwmon/pmbus/ibm-cffps.c
> @@ -33,9 +33,12 @@
> #define CFFPS_INPUT_HISTORY_CMD 0xD6
> #define CFFPS_INPUT_HISTORY_SIZE 100
>
> +#define CFFPS_CCIN_REVISION GENMASK(7, 0)
> +#define CFFPS_CCIN_REVISION_LEGACY 0xde
> #define CFFPS_CCIN_VERSION GENMASK(15, 8)
> #define CFFPS_CCIN_VERSION_1 0x2b
> #define CFFPS_CCIN_VERSION_2 0x2e
> +#define CFFPS_CCIN_VERSION_3 0x51
>
> /* STATUS_MFR_SPECIFIC bits */
> #define CFFPS_MFR_FAN_FAULT BIT(0)
> @@ -489,11 +492,14 @@ static int ibm_cffps_probe(struct i2c_client *client,
> vs = (enum versions)id->driver_data;
>
> if (vs == cffps_unknown) {
> + u16 ccin_revision = 0;
> u16 ccin_version = CFFPS_CCIN_VERSION_1;
> int ccin = i2c_smbus_read_word_swapped(client, CFFPS_CCIN_CMD);
>
> - if (ccin > 0)
> + if (ccin > 0) {
> + ccin_revision = FIELD_GET(CFFPS_CCIN_REVISION, ccin);
> ccin_version = FIELD_GET(CFFPS_CCIN_VERSION, ccin);
> + }
>
> switch (ccin_version) {
> default:
> @@ -503,6 +509,12 @@ static int ibm_cffps_probe(struct i2c_client *client,
> case CFFPS_CCIN_VERSION_2:
> vs = cffps2;
> break;
> + case CFFPS_CCIN_VERSION_3:
> + if (ccin_revision == CFFPS_CCIN_REVISION_LEGACY)
> + vs = cffps1;
> + else
> + vs = cffps2;
> + break;
> }
>
> /* Set the client name to include the version number. */
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection
2020-04-01 22:13 ` Guenter Roeck
@ 2020-04-01 22:20 ` Joel Stanley
2020-04-02 14:31 ` Eddie James
0 siblings, 1 reply; 6+ messages in thread
From: Joel Stanley @ 2020-04-01 22:20 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Eddie James, OpenBMC Maillist, Brandon Wyman
On Wed, 1 Apr 2020 at 22:13, Guenter Roeck <linux@roeck-us.net> wrote:
>
> On 4/1/20 3:03 PM, Eddie James wrote:
> > There is an additional CCIN for the IBM CFFPS that may be classifed as
> > either version one or version two, based upon the rest of the bits of
> > the CCIN. Add support for it in the version detection.
> >
> > Signed-off-by: Eddie James <eajames@linux.ibm.com>
> > Link: https://lore.kernel.org/r/1583948590-17220-1-git-send-email-eajames@linux.ibm.com
> > Signed-off-by: Guenter Roeck <linux@roeck-us.net>
>
> Those e-mails are confusing. Can you drop me from cc:, please ?
I think Eddie forgot to --suppress-cc=body when he git send-email'd
Sorry for the noise Guenter.
Cheers,
Joel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection
2020-04-01 22:20 ` Joel Stanley
@ 2020-04-02 14:31 ` Eddie James
0 siblings, 0 replies; 6+ messages in thread
From: Eddie James @ 2020-04-02 14:31 UTC (permalink / raw)
To: Joel Stanley, Guenter Roeck; +Cc: OpenBMC Maillist, Brandon Wyman
On 4/1/20 5:20 PM, Joel Stanley wrote:
> On Wed, 1 Apr 2020 at 22:13, Guenter Roeck <linux@roeck-us.net> wrote:
>> On 4/1/20 3:03 PM, Eddie James wrote:
>>> There is an additional CCIN for the IBM CFFPS that may be classifed as
>>> either version one or version two, based upon the rest of the bits of
>>> the CCIN. Add support for it in the version detection.
>>>
>>> Signed-off-by: Eddie James <eajames@linux.ibm.com>
>>> Link: https://lore.kernel.org/r/1583948590-17220-1-git-send-email-eajames@linux.ibm.com
>>> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
>> Those e-mails are confusing. Can you drop me from cc:, please ?
> I think Eddie forgot to --suppress-cc=body when he git send-email'd
>
> Sorry for the noise Guenter.
Indeed, my apologies.
Thanks,
Eddie
>
> Cheers,
>
> Joel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-04-02 14:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-01 22:03 [PATCH linux dev-5.4 0/2] hwmon: (pmbus/ibm-cffps) Add another CCIN to version detection and fix LEDs Eddie James
2020-04-01 22:03 ` [PATCH linux dev-5.4 1/2] hwmon: (pmbus/ibm-cffps) Add another PSU CCIN to version detection Eddie James
2020-04-01 22:13 ` Guenter Roeck
2020-04-01 22:20 ` Joel Stanley
2020-04-02 14:31 ` Eddie James
2020-04-01 22:03 ` [PATCH linux dev-5.4 2/2] hwmon: (pmbus/ibm-cffps) Fix LED behavior again Eddie James
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.