drivers: sensor: uint8_t var cannot be greater than 0xff

SENSOR_XBR818_CLOCKRATE / val->val1 is getting assigned
to uint8_t variable which is causing the logically
dead issue, this commit is a fix for it.

(cherry picked from commit 177e7d2)

Original-Signed-off-by: Jilay Pandya <jilay.pandya@outlook.com>
GitOrigin-RevId: 177e7d2
Cr-Build-Id: 8710912179129064817
Cr-Build-Url: https://cr-buildbucket.appspot.com/build/8710912179129064817
Copybot-Job-Name: zephyr-main-copybot-downstream
Change-Id: I4827f80cb857e5b711fff50af8ac929fe71257f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/6683776
Tested-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
Tested-by: ChromeOS Copybot <copybot@chops-service-accounts.iam.gserviceaccount.com>

Author: Jilay Pandya

drivers/sensor/xbr818/xbr818.c

@@ -154,10 +154,14 @@ static int xbr818_attr_set(const struct device *dev, enum sensor_channel chan,
 		if (val->val1 > SENSOR_XBR818_CLOCKRATE || val->val1 <= 0) {
 			return -EINVAL;
 		}
-		tmp[0] = SENSOR_XBR818_CLOCKRATE / val->val1;
-		if (tmp[0] > 0xFF) {
+		const int32_t temp = SENSOR_XBR818_CLOCKRATE / val->val1;
+
+		if (temp > 0xFF) {
 			return -EINVAL;
 		}
+
+		tmp[0] = (uint8_t)temp;
+
 		ret = i2c_reg_write_byte_dt(&config->i2c, XBR818_SAMPLE_RATE_DIVIDER, tmp[0]);
 	} else {
 		ret = xbr818_disable_i2c(dev);