summaryrefslogtreecommitdiff
path: root/sys
diff options
context:
space:
mode:
authorAnton Lindqvist <anton@cvs.openbsd.org>2022-08-11 07:32:58 +0000
committerAnton Lindqvist <anton@cvs.openbsd.org>2022-08-11 07:32:58 +0000
commit746a206d5143afa95707b1bc3168dee5eb4e25f2 (patch)
tree3ab0851238c3bc5194b3abc15dc9a24ffd86f790 /sys
parent9038d26f3d0a4c07e8de087ca235ffee757d2096 (diff)
Remove prefix from nested battery struct fields.
Diffstat (limited to 'sys')
-rw-r--r--sys/dev/usb/uhidpp.c64
1 files changed, 32 insertions, 32 deletions
diff --git a/sys/dev/usb/uhidpp.c b/sys/dev/usb/uhidpp.c
index 174b59361b6..60918662d2e 100644
--- a/sys/dev/usb/uhidpp.c
+++ b/sys/dev/usb/uhidpp.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: uhidpp.c,v 1.26 2022/02/05 07:31:40 anton Exp $ */
+/* $OpenBSD: uhidpp.c,v 1.27 2022/08/11 07:32:57 anton Exp $ */
/*
* Copyright (c) 2021 Anton Lindqvist <anton@openbsd.org>
@@ -199,12 +199,12 @@ struct uhidpp_device {
#define UHIDPP_DEVICE_FEATURE_BATTERY 0x02
struct {
- struct ksensor b_sens[UHIDPP_NSENSORS];
- uint8_t b_feature_idx;
- uint8_t b_level;
- uint8_t b_status;
- uint8_t b_nlevels;
- uint8_t b_rechargeable;
+ struct ksensor sens[UHIDPP_NSENSORS];
+ uint8_t feature_idx;
+ uint8_t level;
+ uint8_t status;
+ uint8_t nlevels;
+ uint8_t rechargeable;
} d_battery;
};
@@ -431,14 +431,14 @@ uhidpp_detach(struct device *self, int flags)
for (i = 0; i < UHIDPP_NDEVICES; i++) {
struct uhidpp_device *dev = &sc->sc_devices[i];
- struct ksensor *sens = dev->d_battery.b_sens;
+ struct ksensor *sens = dev->d_battery.sens;
if (!dev->d_connected)
continue;
sensor_detach(&sc->sc_sensdev, &sens[0]);
sensor_detach(&sc->sc_sensdev, &sens[1]);
- if (dev->d_battery.b_rechargeable)
+ if (dev->d_battery.rechargeable)
sensor_detach(&sc->sc_sensdev, &sens[2]);
}
@@ -593,7 +593,7 @@ uhidpp_device_connect(struct uhidpp_softc *sc, struct uhidpp_device *dev)
error = hidpp20_root_get_feature(sc, dev->d_id,
HIDPP20_FEAT_BATTERY_ID,
- &dev->d_battery.b_feature_idx, &feature_type);
+ &dev->d_battery.feature_idx, &feature_type);
if (error) {
DPRINTF("%s: battery feature index failure: "
"device_id=%d, error=%d\n",
@@ -602,8 +602,8 @@ uhidpp_device_connect(struct uhidpp_softc *sc, struct uhidpp_device *dev)
}
error = hidpp20_battery_get_capability(sc,
- dev->d_id, dev->d_battery.b_feature_idx,
- &dev->d_battery.b_nlevels, &dev->d_battery.b_rechargeable);
+ dev->d_id, dev->d_battery.feature_idx,
+ &dev->d_battery.nlevels, &dev->d_battery.rechargeable);
if (error) {
DPRINTF("%s: battery capability failure: device_id=%d, "
"error=%d\n", __func__, dev->d_id, error);
@@ -615,20 +615,20 @@ uhidpp_device_connect(struct uhidpp_softc *sc, struct uhidpp_device *dev)
dev->d_connected = 1;
- sens = &dev->d_battery.b_sens[0];
+ sens = &dev->d_battery.sens[0];
strlcpy(sens->desc, "battery level", sizeof(sens->desc));
sens->type = SENSOR_PERCENT;
sens->flags = SENSOR_FUNKNOWN;
sensor_attach(&sc->sc_sensdev, sens);
- sens = &dev->d_battery.b_sens[1];
+ sens = &dev->d_battery.sens[1];
strlcpy(sens->desc, "number of battery levels", sizeof(sens->desc));
sens->type = SENSOR_INTEGER;
- sens->value = dev->d_battery.b_nlevels;
+ sens->value = dev->d_battery.nlevels;
sensor_attach(&sc->sc_sensdev, sens);
- if (dev->d_battery.b_rechargeable) {
- sens = &dev->d_battery.b_sens[2];
+ if (dev->d_battery.rechargeable) {
+ sens = &dev->d_battery.sens[2];
strlcpy(sens->desc, "charger", sizeof(sens->desc));
sens->type = SENSOR_INDICATOR;
sens->value = 0;
@@ -667,8 +667,8 @@ uhidpp_device_refresh(struct uhidpp_softc *sc, struct uhidpp_device *dev)
int charging, error;
error = hidpp20_battery_get_level_status(sc, dev->d_id,
- dev->d_battery.b_feature_idx,
- &dev->d_battery.b_level, &dev->d_battery.b_status);
+ dev->d_battery.feature_idx,
+ &dev->d_battery.level, &dev->d_battery.status);
if (error) {
DPRINTF("%s: battery status failure: device_id=%d, "
"error=%d\n",
@@ -677,11 +677,11 @@ uhidpp_device_refresh(struct uhidpp_softc *sc, struct uhidpp_device *dev)
}
charging = hidpp20_battery_status_is_charging(
- dev->d_battery.b_status);
+ dev->d_battery.status);
- dev->d_battery.b_sens[0].value = dev->d_battery.b_level * 1000;
- dev->d_battery.b_sens[0].flags &= ~SENSOR_FUNKNOWN;
- if (dev->d_battery.b_nlevels < 10) {
+ dev->d_battery.sens[0].value = dev->d_battery.level * 1000;
+ dev->d_battery.sens[0].flags &= ~SENSOR_FUNKNOWN;
+ if (dev->d_battery.nlevels < 10) {
/*
* According to the HID++ 2.0 specification, less than
* 10 levels should be mapped to the following 4 levels:
@@ -696,24 +696,24 @@ uhidpp_device_refresh(struct uhidpp_softc *sc, struct uhidpp_device *dev)
* which the level cannot be trusted.
*/
if (charging)
- dev->d_battery.b_sens[0].status = SENSOR_S_UNKNOWN;
- else if (dev->d_battery.b_level <= 10)
- dev->d_battery.b_sens[0].status = SENSOR_S_CRIT;
- else if (dev->d_battery.b_level <= 30)
- dev->d_battery.b_sens[0].status = SENSOR_S_WARN;
+ dev->d_battery.sens[0].status = SENSOR_S_UNKNOWN;
+ else if (dev->d_battery.level <= 10)
+ dev->d_battery.sens[0].status = SENSOR_S_CRIT;
+ else if (dev->d_battery.level <= 30)
+ dev->d_battery.sens[0].status = SENSOR_S_WARN;
else
- dev->d_battery.b_sens[0].status = SENSOR_S_OK;
+ dev->d_battery.sens[0].status = SENSOR_S_OK;
} else {
/*
* XXX the device supports battery mileage. The current
* level must be checked against resp.fap.params[3]
* given by hidpp20_battery_get_capability().
*/
- dev->d_battery.b_sens[0].status = SENSOR_S_UNKNOWN;
+ dev->d_battery.sens[0].status = SENSOR_S_UNKNOWN;
}
- if (dev->d_battery.b_rechargeable)
- dev->d_battery.b_sens[2].value = charging;
+ if (dev->d_battery.rechargeable)
+ dev->d_battery.sens[2].value = charging;
}
}