drivers: sensor: ens210: Fix broken CONFIG_ENS210_CRC_CHECK checks

The CONFIG_* prefix was missing, making the #ifdefs always false. Found with a script (CONFIG_ENS210_CRC_CHECK was unused). Also make ens210_crc7() static. Guessing it's unused outside this file. Signed-off-by: Ulf Magnusson <[email protected]>
k0d · Oct 22, 2019 · 53c6882 · 53c6882
1 parent a31672c
commit 53c6882
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/drivers/sensor/ens210/ens210.c b/drivers/sensor/ens210/ens210.c
@@ -16,8 +16,8 @@
 
 LOG_MODULE_REGISTER(ENS210, CONFIG_SENSOR_LOG_LEVEL);
 
-#ifdef ENS210_CRC_CHECK
-u32_t ens210_crc7(u32_t bitstream)
+#ifdef CONFIG_ENS210_CRC_CHECK
+static u32_t ens210_crc7(u32_t bitstream)
 {
 	u32_t polynomial = (ENS210_CRC7_POLY << (ENS210_CRC7_DATA_WIDTH - 1));
 	u32_t bit = ENS210_CRC7_DATA_MSB << ENS210_CRC7_WIDTH;
@@ -34,17 +34,17 @@ u32_t ens210_crc7(u32_t bitstream)
 
 	return val;
 }
-#endif /*ENS210_CRC_CHECK*/
+#endif /* CONFIG_ENS210_CRC_CHECK */
 
 static int ens210_sample_fetch(struct device *dev, enum sensor_channel chan)
 {
 	struct ens210_data *drv_data = dev->driver_data;
 	struct ens210_value_data data[2];
 	int ret, cnt;
 
-#ifdef ENS210_CRC_CHECK
+#ifdef CONFIG_ENS210_CRC_CHECK
 	u32_t temp_valid, humidity_valid;
-#endif /*ENS210_CRC_CHECK*/
+#endif /* CONFIG_ENS210_CRC_CHECK */
 
 	__ASSERT_NO_MSG(chan == SENSOR_CHAN_ALL);
 
@@ -66,7 +66,7 @@ static int ens210_sample_fetch(struct device *dev, enum sensor_channel chan)
 			continue;
 		}
 
-#ifdef ENS210_CRC_CHECK
+#ifdef CONFIG_ENS210_CRC_CHECK
 		temp_valid =      data[0].val |
 				  (data[0].valid << (sizeof(data[0].val) * 8));
 		humidity_valid =  data[1].val |
@@ -81,7 +81,7 @@ static int ens210_sample_fetch(struct device *dev, enum sensor_channel chan)
 			LOG_WRN("Humidity CRC error");
 			continue;
 		}
-#endif /*ENS210_CRC_CHECK*/
+#endif /* CONFIG_ENS210_CRC_CHECK */
 
 		drv_data->temp = data[0];
 		drv_data->humidity = data[1];