bmp581: code-style improvements

Remove extra-parentheses when operator precedence is implicit.
Eliminate implicit integer to boolean conversions.

Signed-off-by: Luis Ubieda <[email protected]>

Author: ubieda

drivers/sensor/bosch/bmp581/bmp581.c

@@ -177,7 +177,7 @@ static int power_up_check(const struct device *dev)
 
 	if (rslt == BMP5_OK) {
 		/* Check if nvm_rdy status = 1 and nvm_err status = 0 to proceed */
-		if ((nvm_status & BMP5_INT_NVM_RDY) && (!(nvm_status & BMP5_INT_NVM_ERR))) {
+		if ((nvm_status & BMP5_INT_NVM_RDY) != 0 && (nvm_status & BMP5_INT_NVM_ERR) == 0) {
 			rslt = BMP5_OK;
 		} else {
 			rslt = -EFAULT;
@@ -375,7 +375,7 @@ static int soft_reset(const struct device *dev)
 		k_usleep(BMP5_DELAY_US_SOFT_RESET);
 		ret = get_interrupt_status(&int_status, dev);
 		if (ret == BMP5_OK) {
-			if (int_status & BMP5_INT_ASSERTED_POR_SOFTRESET_COMPLETE) {
+			if ((int_status & BMP5_INT_ASSERTED_POR_SOFTRESET_COMPLETE) != 0) {
 				ret = BMP5_OK;
 			} else {
 				ret = -EFAULT;
@@ -635,7 +635,7 @@ static void bmp581_submit_one_shot(const struct device *dev, struct rtio_iodev_s
 	const struct bmp581_config *conf = dev->config;
 
 	err = rtio_sqe_rx_buf(iodev_sqe, min_buf_len, min_buf_len, &buf, &buf_len);
-	CHECKIF((err < 0) || (buf_len < min_buf_len) || !buf) {
+	CHECKIF(err < 0 || buf_len < min_buf_len || !buf) {
 		LOG_ERR("Failed to get a read buffer of size %u bytes", min_buf_len);
 		rtio_iodev_sqe_err(iodev_sqe, err);
 		return;

drivers/sensor/bosch/bmp581/bmp581_decoder.c

@@ -191,7 +191,7 @@ static int bmp581_decoder_decode(const uint8_t *buffer,
 	do {
 		err = bmp581_convert_raw_to_q31_value(&edata->header, &chan_spec,
 						      edata->frame, fit, out);
-	} while ((err == 0) && (*fit < max_count));
+	} while (err == 0 && *fit < max_count);
 
 	if (*fit == fit_0 || err != 0) {
 		return err;
@@ -205,10 +205,10 @@ static bool bmp581_decoder_has_trigger(const uint8_t *buffer, enum sensor_trigge
 {
 	const struct bmp581_encoded_data *edata = (const struct bmp581_encoded_data *)buffer;
 
-	if (((trigger == SENSOR_TRIG_DATA_READY) &&
-	     (edata->header.events & BMP581_EVENT_DRDY)) ||
-	    ((trigger == SENSOR_TRIG_FIFO_WATERMARK) &&
-	     (edata->header.events & BMP581_EVENT_FIFO_WM))) {
+	if ((trigger == SENSOR_TRIG_DATA_READY &&
+	     edata->header.events & BMP581_EVENT_DRDY) ||
+	    (trigger == SENSOR_TRIG_FIFO_WATERMARK &&
+	     edata->header.events & BMP581_EVENT_FIFO_WM)) {
 		return true;
 	}
 

drivers/sensor/bosch/bmp581/bmp581_stream.c

@@ -289,14 +289,14 @@ void bmp581_stream_submit(const struct device *dev,
 	data->stream.iodev_sqe = iodev_sqe;
 
 	if (atomic_cas(&data->stream.state, BMP581_STREAM_OFF, BMP581_STREAM_ON) ||
-	    (data->stream.enabled_mask != enabled_mask)) {
+	    data->stream.enabled_mask != enabled_mask) {
 		struct rtio_sqe *int_src_sqe;
 		uint8_t val = 0;
 
 		(void)atomic_set(&data->stream.state, BMP581_STREAM_ON);
 		data->stream.enabled_mask = enabled_mask;
 
-		if (enabled_mask & BMP581_EVENT_FIFO_WM) {
+		if ((enabled_mask & BMP581_EVENT_FIFO_WM) != 0) {
 			err = bmp581_stream_prep_fifo_wm_async(dev);
 			if (err < 0) {
 				rtio_iodev_sqe_err(iodev_sqe, err);