diff options
| author | Huang Rui <ray.huang@amd.com> | 2016-04-06 15:44:15 +0800 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2016-04-19 06:32:35 -0700 |
| commit | 1d28e01628aebab8fe403e7e9d0760f3787763d5 (patch) | |
| tree | 87a7544d026dba96ddb9663ff72df8f86d82e42b | |
| parent | a6e232f78698953e6fd9c2f4c95e16d5f5f9d0c3 (diff) | |
| download | op-kernel-dev-1d28e01628aebab8fe403e7e9d0760f3787763d5.zip op-kernel-dev-1d28e01628aebab8fe403e7e9d0760f3787763d5.tar.gz | |
hwmon: (fam15h_power) Add platform check function
This patch adds a platform check function to make code more readable.
Signed-off-by: Huang Rui <ray.huang@amd.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
| -rw-r--r-- | drivers/hwmon/fam15h_power.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/hwmon/fam15h_power.c b/drivers/hwmon/fam15h_power.c index 7d9d697..eb97a92 100644 --- a/drivers/hwmon/fam15h_power.c +++ b/drivers/hwmon/fam15h_power.c @@ -78,6 +78,11 @@ struct fam15h_power_data { unsigned long power_period; }; +static bool is_carrizo_or_later(void) +{ + return boot_cpu_data.x86 == 0x15 && boot_cpu_data.x86_model >= 0x60; +} + static ssize_t show_power(struct device *dev, struct device_attribute *attr, char *buf) { @@ -94,7 +99,7 @@ static ssize_t show_power(struct device *dev, * On Carrizo and later platforms, TdpRunAvgAccCap bit field * is extended to 4:31 from 4:25. */ - if (boot_cpu_data.x86 == 0x15 && boot_cpu_data.x86_model >= 0x60) { + if (is_carrizo_or_later()) { running_avg_capture = val >> 4; running_avg_capture = sign_extend32(running_avg_capture, 27); } else { @@ -111,7 +116,7 @@ static ssize_t show_power(struct device *dev, * On Carrizo and later platforms, ApmTdpLimit bit field * is extended to 16:31 from 16:28. */ - if (boot_cpu_data.x86 == 0x15 && boot_cpu_data.x86_model >= 0x60) + if (is_carrizo_or_later()) tdp_limit = val >> 16; else tdp_limit = (val >> 16) & 0x1fff; |
