diff mbox

[linux,dev-4.10] hwmon: (occ): Return power readings in micro-Watts

Message ID 1502744797-27565-1-git-send-email-eajames@linux.vnet.ibm.com
State Accepted, archived
Headers show

Commit Message

Eddie James Aug. 14, 2017, 9:06 p.m. UTC
From: "Edward A. James" <eajames@us.ibm.com>

hwmon expects -6 exponent.

Signed-off-by: Edward A. James <eajames@us.ibm.com>
---
 drivers/hwmon/occ/common.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Andrew Jeffery Aug. 23, 2017, 7:22 a.m. UTC | #1
On Mon, 2017-08-14 at 16:06 -0500, Eddie James wrote:
> > From: "Edward A. James" <eajames@us.ibm.com>
> 
> hwmon expects -6 exponent.
> 
> Signed-off-by: Edward A. James <eajames@us.ibm.com>

Thanks, applied to dev-4.10.

Andrew

> ---
>  drivers/hwmon/occ/common.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/hwmon/occ/common.c b/drivers/hwmon/occ/common.c
> index 3f63467..34002fb 100644
> --- a/drivers/hwmon/occ/common.c
> +++ b/drivers/hwmon/occ/common.c
> @@ -470,7 +470,7 @@ static ssize_t occ_show_power_1(struct device *dev,
> >  		val = get_unaligned_be32(&power->accumulator);
> >  		break;
> >  	case 3:
> > -		val = get_unaligned_be16(&power->value);
> > +		val = get_unaligned_be16(&power->value) * 1000000;
> >  		break;
> >  	}
>  
> @@ -504,7 +504,7 @@ static ssize_t occ_show_power_2(struct device *dev,
> >  		val = get_unaligned_be64(&power->accumulator);
> >  		break;
> >  	case 3:
> > -		val = get_unaligned_be16(&power->value);
> > +		val = get_unaligned_be16(&power->value) * 1000000;
> >  		break;
> >  	case 4:
> >  		val = power->function_id;
> @@ -541,7 +541,7 @@ static ssize_t occ_show_power_a0(struct device *dev,
> >  		val = get_unaligned_be16(&power->system.update_time);
> >  		break;
> >  	case 2:
> > -		val = get_unaligned_be16(&power->system.value);
> > +		val = get_unaligned_be16(&power->system.value) * 1000000;
> >  		break;
> >  	case 3:
> >  		val = get_unaligned_be32(&power->system.update_tag);
> @@ -553,7 +553,7 @@ static ssize_t occ_show_power_a0(struct device *dev,
> >  		val = get_unaligned_be16(&power->proc.update_time);
> >  		break;
> >  	case 6:
> > -		val = get_unaligned_be16(&power->proc.value);
> > +		val = get_unaligned_be16(&power->proc.value) * 1000000;
> >  		break;
> >  	case 7:
> >  		val = get_unaligned_be32(&power->proc.update_tag);
> @@ -562,7 +562,7 @@ static ssize_t occ_show_power_a0(struct device *dev,
> >  		val = get_unaligned_be64(&power->proc.accumulator);
> >  		break;
> >  	case 9:
> > -		val = get_unaligned_be16(&power->vdd.value);
> > +		val = get_unaligned_be16(&power->vdd.value) * 1000000;
> >  		break;
> >  	case 10:
> >  		val = get_unaligned_be32(&power->vdd.update_tag);
> @@ -571,7 +571,7 @@ static ssize_t occ_show_power_a0(struct device *dev,
> >  		val = get_unaligned_be64(&power->vdd.accumulator);
> >  		break;
> >  	case 12:
> > -		val = get_unaligned_be16(&power->vdn.value);
> > +		val = get_unaligned_be16(&power->vdn.value) * 1000000;
> >  		break;
> >  	case 13:
> >  		val = get_unaligned_be32(&power->vdn.update_tag);
diff mbox

Patch

diff --git a/drivers/hwmon/occ/common.c b/drivers/hwmon/occ/common.c
index 3f63467..34002fb 100644
--- a/drivers/hwmon/occ/common.c
+++ b/drivers/hwmon/occ/common.c
@@ -470,7 +470,7 @@  static ssize_t occ_show_power_1(struct device *dev,
 		val = get_unaligned_be32(&power->accumulator);
 		break;
 	case 3:
-		val = get_unaligned_be16(&power->value);
+		val = get_unaligned_be16(&power->value) * 1000000;
 		break;
 	}
 
@@ -504,7 +504,7 @@  static ssize_t occ_show_power_2(struct device *dev,
 		val = get_unaligned_be64(&power->accumulator);
 		break;
 	case 3:
-		val = get_unaligned_be16(&power->value);
+		val = get_unaligned_be16(&power->value) * 1000000;
 		break;
 	case 4:
 		val = power->function_id;
@@ -541,7 +541,7 @@  static ssize_t occ_show_power_a0(struct device *dev,
 		val = get_unaligned_be16(&power->system.update_time);
 		break;
 	case 2:
-		val = get_unaligned_be16(&power->system.value);
+		val = get_unaligned_be16(&power->system.value) * 1000000;
 		break;
 	case 3:
 		val = get_unaligned_be32(&power->system.update_tag);
@@ -553,7 +553,7 @@  static ssize_t occ_show_power_a0(struct device *dev,
 		val = get_unaligned_be16(&power->proc.update_time);
 		break;
 	case 6:
-		val = get_unaligned_be16(&power->proc.value);
+		val = get_unaligned_be16(&power->proc.value) * 1000000;
 		break;
 	case 7:
 		val = get_unaligned_be32(&power->proc.update_tag);
@@ -562,7 +562,7 @@  static ssize_t occ_show_power_a0(struct device *dev,
 		val = get_unaligned_be64(&power->proc.accumulator);
 		break;
 	case 9:
-		val = get_unaligned_be16(&power->vdd.value);
+		val = get_unaligned_be16(&power->vdd.value) * 1000000;
 		break;
 	case 10:
 		val = get_unaligned_be32(&power->vdd.update_tag);
@@ -571,7 +571,7 @@  static ssize_t occ_show_power_a0(struct device *dev,
 		val = get_unaligned_be64(&power->vdd.accumulator);
 		break;
 	case 12:
-		val = get_unaligned_be16(&power->vdn.value);
+		val = get_unaligned_be16(&power->vdn.value) * 1000000;
 		break;
 	case 13:
 		val = get_unaligned_be32(&power->vdn.update_tag);