matthew/sonic-buildimage
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[device][as7816-64x] Update fan speed reading. (#5102)

Browse Source 
Original converting from register is wrong, it makes the fan speed much higher than it is. Change the way fan speed is calculated from CPLD.

Signed-off-by: roy_lee <roy_lee@accton.com>
This commit is contained in:
Roy Lee 2020-08-14 01:50:31 +08:00 committed by GitHub
parent 54fcdbb380
commit 6f9acee89b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
platform/broadcom/sonic-platform-modules-accton/as7816-64x/modules/x86-64-accton-as7816-64x-fan.c
View File

@ -176,7 +176,6 @@ static struct attribute *as7816_64x_fan_attributes[] = {
#define FAN_DUTY_CYCLE_REG_MASK 0xF #define FAN_DUTY_CYCLE_REG_MASK 0xF
#define FAN_MAX_DUTY_CYCLE 100 #define FAN_MAX_DUTY_CYCLE 100
#define FAN_REG_VAL_TO_SPEED_RPM_STEP 100
static int as7816_64x_fan_read_value(struct i2c_client *client, u8 reg) static int as7816_64x_fan_read_value(struct i2c_client *client, u8 reg)
{ {
@ -220,7 +219,14 @@ static u8 duty_cycle_to_reg_val(u8 duty_cycle)
static u32 reg_val_to_speed_rpm(u8 reg_val) static u32 reg_val_to_speed_rpm(u8 reg_val)
{ {
return (u32)reg_val * FAN_REG_VAL_TO_SPEED_RPM_STEP; if (reg_val == 0 || reg_val == 255) {
return 0;
} else {
u64 f, dv;
dv = 2 * 2 * 40960 * (u64)(255 - reg_val);
f = 60000000000 / dv;
return (u32)f;
}
} }
static u8 reg_val_to_direction(u8 reg_val, enum fan_id id) static u8 reg_val_to_direction(u8 reg_val, enum fan_id id)