From 6f3598e330f2f800ed674dac97542fb295260091 Mon Sep 17 00:00:00 2001 From: roy_lee Date: Thu, 1 Nov 2018 15:12:36 +0800 Subject: [PATCH] [platform]as5916-26xb/as7316-26xb: take fan speed value to be unsigned to get correct RPM. Signed-off-by: roy_lee --- .../modules/builds/x86-64-accton-as5916-26xb-psu.c | 4 ++-- .../modules/builds/x86-64-accton-as7316-26xb-psu.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) mode change 100644 => 100755 packages/platforms/accton/x86-64/x86-64-accton-as5916-26xb/modules/builds/x86-64-accton-as5916-26xb-psu.c diff --git a/packages/platforms/accton/x86-64/x86-64-accton-as5916-26xb/modules/builds/x86-64-accton-as5916-26xb-psu.c b/packages/platforms/accton/x86-64/x86-64-accton-as5916-26xb/modules/builds/x86-64-accton-as5916-26xb-psu.c old mode 100644 new mode 100755 index ba71289b..1a8d4d0a --- a/packages/platforms/accton/x86-64/x86-64-accton-as5916-26xb/modules/builds/x86-64-accton-as5916-26xb-psu.c +++ b/packages/platforms/accton/x86-64/x86-64-accton-as5916-26xb/modules/builds/x86-64-accton-as5916-26xb-psu.c @@ -425,8 +425,8 @@ static ssize_t show_psu(struct device *dev, struct device_attribute *da, char *b case PSU1_FAN_INPUT: case PSU2_FAN_INPUT: VALIDATE_PRESENT_RETURN(pid); - value = ((int)data->ipmi_resp[pid].status[PSU_FAN0] | - (int)data->ipmi_resp[pid].status[PSU_FAN1] << 8); + value = (((unsigned char)data->ipmi_resp[pid].status[PSU_FAN0]) | + ((unsigned char)data->ipmi_resp[pid].status[PSU_FAN1]) << 8); break; default: return -EINVAL; diff --git a/packages/platforms/accton/x86-64/x86-64-accton-as7316-26xb/modules/builds/x86-64-accton-as7316-26xb-psu.c b/packages/platforms/accton/x86-64/x86-64-accton-as7316-26xb/modules/builds/x86-64-accton-as7316-26xb-psu.c index 1611df7a..a96a5da7 100644 --- a/packages/platforms/accton/x86-64/x86-64-accton-as7316-26xb/modules/builds/x86-64-accton-as7316-26xb-psu.c +++ b/packages/platforms/accton/x86-64/x86-64-accton-as7316-26xb/modules/builds/x86-64-accton-as7316-26xb-psu.c @@ -425,8 +425,8 @@ static ssize_t show_psu(struct device *dev, struct device_attribute *da, char *b case PSU1_FAN_INPUT: case PSU2_FAN_INPUT: VALIDATE_PRESENT_RETURN(pid); - value = ((int)data->ipmi_resp[pid].status[PSU_FAN0] | - (int)data->ipmi_resp[pid].status[PSU_FAN1] << 8); + value = (((unsigned char)data->ipmi_resp[pid].status[PSU_FAN0]) | + ((unsigned char)data->ipmi_resp[pid].status[PSU_FAN1]) << 8); break; default: return -EINVAL;