From: mpi <mpi@openbsd.org>
Date: Mon, 27 Apr 2015 07:41:41 +0000 (+0000)
Subject: with more unit convertion when new sensors will appear.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=136270bed7df7898bd4d59fa949bd955f50bf634;p=openbsd

with more unit convertion when new sensors will appear.

From David Higgs.
---

diff --git a/sys/dev/usb/upd.c b/sys/dev/usb/upd.c
index 170ed1a6691..fabaf410c96 100644
--- a/sys/dev/usb/upd.c
+++ b/sys/dev/usb/upd.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: upd.c,v 1.15 2015/04/27 07:37:19 mpi Exp $ */
+/*	$OpenBSD: upd.c,v 1.16 2015/04/27 07:41:41 mpi Exp $ */
 
 /*
  * Copyright (c) 2014 Andre de Oliveira <andre@openbsd.org>
@@ -101,6 +101,8 @@ int  upd_detach(struct device *, int);
 
 void upd_refresh(void *);
 void upd_update_sensors(struct upd_softc *, uint8_t *, unsigned int, int);
+void upd_update_sensor_value(struct upd_softc *, struct upd_sensor *,
+    uint8_t *, int);
 void upd_intr(struct uhidev *, void *, uint);
 struct upd_usage_entry *upd_lookup_usage_entry(const struct hid_item *);
 struct upd_sensor *upd_lookup_sensor(struct upd_softc *, int, int);
@@ -324,8 +326,7 @@ upd_update_sensors(struct upd_softc *sc, uint8_t *buf, unsigned int len,
     int repid)
 {
 	struct upd_sensor	*sensor;
-	ulong			hdata, batpres;
-	ulong 			adjust;
+	ulong			batpres;
 	int			i;
 
 	sensor = upd_lookup_sensor(sc, HUP_BATTERY, HUB_BATTERY_PRESENT);
@@ -348,27 +349,35 @@ upd_update_sensors(struct upd_softc *sc, uint8_t *buf, unsigned int len,
 			}
 		}
 
-		switch (HID_GET_USAGE(sensor->hitem.usage)) {
-		case HUB_REL_STATEOF_CHARGE:
-		case HUB_ABS_STATEOF_CHARGE:
-		case HUB_REM_CAPACITY:
-		case HUB_FULLCHARGE_CAPACITY:
-			adjust = 1000; /* scale adjust */
-			break;
-		default:
-			adjust = 1; /* no scale adjust */
-			break;
-		}
-
-		hdata = hid_get_data(buf, len, &sensor->hitem.loc);
-
-		sensor->ksensor.value = hdata * adjust;
-		sensor->ksensor.status = SENSOR_S_OK;
-		sensor->ksensor.flags &= ~SENSOR_FINVALID;
-		DPRINTF(("%s: hidget data: %lu\n", DEVNAME(sc), hdata));
+		upd_update_sensor_value(sc, sensor, buf, len);
 	}
 }
 
+void
+upd_update_sensor_value(struct upd_softc *sc, struct upd_sensor *sensor,
+    uint8_t *buf, int len)
+{
+	int64_t	hdata, adjust;
+
+	switch (HID_GET_USAGE(sensor->hitem.usage)) {
+	case HUB_REL_STATEOF_CHARGE:
+	case HUB_ABS_STATEOF_CHARGE:
+	case HUB_REM_CAPACITY:
+	case HUB_FULLCHARGE_CAPACITY:
+		adjust = 1000; /* scale adjust */
+		break;
+	default:
+		adjust = 1; /* no scale adjust */
+		break;
+	}
+
+	hdata = hid_get_data(buf, len, &sensor->hitem.loc);
+	sensor->ksensor.value = hdata * adjust;
+	sensor->ksensor.status = SENSOR_S_OK;
+	sensor->ksensor.flags &= ~SENSOR_FINVALID;
+	DPRINTF(("%s: %s hidget data: %lld\n", DEVNAME(sc),
+	    sensor->ksensor.desc, hdata));
+}
 
 void
 upd_intr(struct uhidev *uh, void *p, uint len)