From 75d075cc6765c09e9e71c8d144f218e217480005 Mon Sep 17 00:00:00 2001 From: Yaroslav Chvanov Date: Fri, 28 Jan 2022 04:07:04 +0300 Subject: [PATCH] sinowealth: make some utility functions take unsigned integers To avoid implicit conversions. --- src/driver-sinowealth.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/driver-sinowealth.c b/src/driver-sinowealth.c index 4a465d6e..118266a5 100644 --- a/src/driver-sinowealth.c +++ b/src/driver-sinowealth.c @@ -195,14 +195,14 @@ sinowealth_raw_to_report_rate(uint8_t raw) return 0; } -static int -sinowealth_raw_to_dpi(int raw) +static unsigned int +sinowealth_raw_to_dpi(unsigned int raw) { return (raw + 1) * 100; } -static int -sinowealth_dpi_to_raw(int dpi) +static unsigned int +sinowealth_dpi_to_raw(unsigned int dpi) { assert(dpi >= SINOWEALTH_DPI_MIN && dpi <= SINOWEALTH_DPI_MAX); return dpi / 100 - 1; @@ -241,7 +241,7 @@ sinowealth_rgb_mode_to_brightness(struct rgb_mode mode) /* Convert 0-255 to 0-4. */ static uint8_t -sinowealth_brightness_to_rgb_mode(int brightness) +sinowealth_brightness_to_rgb_mode(unsigned int brightness) { return (brightness + 1) / 64; } @@ -259,7 +259,7 @@ sinowealth_rgb_mode_to_duration(struct rgb_mode mode) } static uint8_t -sinowealth_duration_to_rgb_mode(int duration_ms) +sinowealth_duration_to_rgb_mode(unsigned int duration_ms) { uint8_t mode = 0; if (duration_ms <= 500) {