drivers: sensor: Fix unused function warning

thughes · kartben · commit 31be8696723f · 2025-02-12T02:25:10.000+01:00
Building with clang warns:

drivers/sensor/st/lsm9ds0_mfd/lsm9ds0_mfd.c:42:19: error: unused
function 'lsm9ds0_mfd_accel_set_odr_raw' [-Werror,-Wunused-function]
static inline int lsm9ds0_mfd_accel_set_odr_raw(const struct device *dev,
                  ^

lsm9ds0_mfd_accel_set_odr_raw is only used by code that was guarded by
defined(CONFIG_LSM9DS0_MFD_ACCEL_SAMPLING_RATE_RUNTIME) in addition to
!defined(LSM9DS0_MFD_ACCEL_DISABLED).

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
diff --git a/drivers/sensor/st/lsm9ds0_mfd/lsm9ds0_mfd.c b/drivers/sensor/st/lsm9ds0_mfd/lsm9ds0_mfd.c
@@ -38,7 +38,8 @@ static inline int lsm9ds0_mfd_reboot_memory(const struct device *dev)
 	return 0;
 }
 
-#if !defined(LSM9DS0_MFD_ACCEL_DISABLED)
+#if !defined(LSM9DS0_MFD_ACCEL_DISABLED) && defined(CONFIG_LSM9DS0_MFD_ACCEL_SAMPLING_RATE_RUNTIME)
+
 static inline int lsm9ds0_mfd_accel_set_odr_raw(const struct device *dev,
 						uint8_t odr)
 {
@@ -49,7 +50,6 @@ static inline int lsm9ds0_mfd_accel_set_odr_raw(const struct device *dev,
 				      odr << LSM9DS0_MFD_SHIFT_CTRL_REG1_XM_AODR);
 }
 
-#if defined(CONFIG_LSM9DS0_MFD_ACCEL_SAMPLING_RATE_RUNTIME)
 static const struct {
 	int freq_int;
 	int freq_micro;
@@ -122,7 +122,6 @@ static int lsm9ds0_mfd_accel_set_fs(const struct device *dev, int val)
 	return -ENOTSUP;
 }
 #endif
-#endif
 
 #if !defined(LSM9DS0_MFD_MAGN_DISABLED)
 static inline int lsm9ds0_mfd_magn_set_odr_raw(const struct device *dev,

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,8 @@ static inline int lsm9ds0_mfd_reboot_memory(const struct device *dev)`
`38`	`38`	`return 0;`
`39`	`39`	`}`
`40`	`40`
`41`		`-#if !defined(LSM9DS0_MFD_ACCEL_DISABLED)`
	`41`	`+#if !defined(LSM9DS0_MFD_ACCEL_DISABLED) && defined(CONFIG_LSM9DS0_MFD_ACCEL_SAMPLING_RATE_RUNTIME)`
	`42`	`+`
`42`	`43`	`static inline int lsm9ds0_mfd_accel_set_odr_raw(const struct device *dev,`
`43`	`44`	`uint8_t odr)`
`44`	`45`	`{`
`@@ -49,7 +50,6 @@ static inline int lsm9ds0_mfd_accel_set_odr_raw(const struct device *dev,`
`49`	`50`	`odr << LSM9DS0_MFD_SHIFT_CTRL_REG1_XM_AODR);`
`50`	`51`	`}`
`51`	`52`
`52`		`-#if defined(CONFIG_LSM9DS0_MFD_ACCEL_SAMPLING_RATE_RUNTIME)`
`53`	`53`	`static const struct {`
`54`	`54`	`int freq_int;`
`55`	`55`	`int freq_micro;`
`@@ -122,7 +122,6 @@ static int lsm9ds0_mfd_accel_set_fs(const struct device *dev, int val)`
`122`	`122`	`return -ENOTSUP;`
`123`	`123`	`}`
`124`	`124`	`#endif`
`125`		`-#endif`
`126`	`125`
`127`	`126`	`#if !defined(LSM9DS0_MFD_MAGN_DISABLED)`
`128`	`127`	`static inline int lsm9ds0_mfd_magn_set_odr_raw(const struct device *dev,`