Merge pull request #27 from libdriver/dev

libdriver · web-flow · commit c2d8a6e58575 · 2024-08-15T22:52:47.000+08:00
fix: fix sample rate divider bugs
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## 1.0.3 (2024-08-15)
+
+## Bug Fixes
+
+- fix sample rate divider bugs
+
 ## 1.0.2 (2024-08-06)
 
 ## Bug Fixes
diff --git a/example/driver_mpu6050_basic.c b/example/driver_mpu6050_basic.c
@@ -102,7 +102,7 @@ uint8_t mpu6050_basic_init(mpu6050_address_t addr_pin)
     }
     
     /* set the default rate */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_BASIC_DEFAULT_RATE - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_BASIC_DEFAULT_RATE) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/example/driver_mpu6050_dmp.c b/example/driver_mpu6050_dmp.c
@@ -146,7 +146,7 @@ uint8_t mpu6050_dmp_init(mpu6050_address_t addr_pin,
     }
     
     /* set the default rate */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_DMP_DEFAULT_RATE - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_DMP_DEFAULT_RATE) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/example/driver_mpu6050_fifo.c b/example/driver_mpu6050_fifo.c
@@ -121,7 +121,7 @@ uint8_t mpu6050_fifo_init(mpu6050_address_t addr_pin)
     }
     
     /* set the default rate */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_FIFO_DEFAULT_RATE - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_FIFO_DEFAULT_RATE) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/test/driver_mpu6050_dmp_pedometer_test.c b/test/driver_mpu6050_dmp_pedometer_test.c
@@ -178,7 +178,7 @@ uint8_t mpu6050_dmp_pedometer_test(mpu6050_address_t addr, uint32_t times)
     }
     
     /* set 50Hz */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/test/driver_mpu6050_dmp_read_test.c b/test/driver_mpu6050_dmp_read_test.c
@@ -184,7 +184,7 @@ uint8_t mpu6050_dmp_read_test(mpu6050_address_t addr, uint32_t times)
     }
     
     /* set 200Hz */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (200 - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 200) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/test/driver_mpu6050_dmp_tap_orient_motion_test.c b/test/driver_mpu6050_dmp_tap_orient_motion_test.c
@@ -341,7 +341,7 @@ uint8_t mpu6050_dmp_tap_orient_motion_test(mpu6050_address_t addr)
     }
     
     /* set 50Hz */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/test/driver_mpu6050_fifo_test.c b/test/driver_mpu6050_fifo_test.c
@@ -154,7 +154,7 @@ uint8_t mpu6050_fifo_test(mpu6050_address_t addr, uint32_t times)
     }
     
     /* set 50Hz */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");
diff --git a/test/driver_mpu6050_read_test.c b/test/driver_mpu6050_read_test.c
@@ -130,7 +130,7 @@ uint8_t mpu6050_read_test(mpu6050_address_t addr, uint32_t times)
     }
     
     /* set 50Hz */
-    res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));
+    res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);
     if (res != 0)
     {
         mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ uint8_t mpu6050_basic_init(mpu6050_address_t addr_pin)`
`102`	`102`	`}`
`103`	`103`
`104`	`104`	`/* set the default rate */`
`105`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_BASIC_DEFAULT_RATE - 1));`
	`105`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_BASIC_DEFAULT_RATE) - 1);`
`106`	`106`	`if (res != 0)`
`107`	`107`	`{`
`108`	`108`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ uint8_t mpu6050_dmp_init(mpu6050_address_t addr_pin,`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`/* set the default rate */`
`149`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_DMP_DEFAULT_RATE - 1));`
	`149`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_DMP_DEFAULT_RATE) - 1);`
`150`	`150`	`if (res != 0)`
`151`	`151`	`{`
`152`	`152`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ uint8_t mpu6050_fifo_init(mpu6050_address_t addr_pin)`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`/* set the default rate */`
`124`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (MPU6050_FIFO_DEFAULT_RATE - 1));`
	`124`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / MPU6050_FIFO_DEFAULT_RATE) - 1);`
`125`	`125`	`if (res != 0)`
`126`	`126`	`{`
`127`	`127`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ uint8_t mpu6050_dmp_pedometer_test(mpu6050_address_t addr, uint32_t times)`
`178`	`178`	`}`
`179`	`179`
`180`	`180`	`/* set 50Hz */`
`181`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));`
	`181`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);`
`182`	`182`	`if (res != 0)`
`183`	`183`	`{`
`184`	`184`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,7 @@ uint8_t mpu6050_dmp_read_test(mpu6050_address_t addr, uint32_t times)`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`/* set 200Hz */`
`187`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (200 - 1));`
	`187`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 200) - 1);`
`188`	`188`	`if (res != 0)`
`189`	`189`	`{`
`190`	`190`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -341,7 +341,7 @@ uint8_t mpu6050_dmp_tap_orient_motion_test(mpu6050_address_t addr)`
`341`	`341`	`}`
`342`	`342`
`343`	`343`	`/* set 50Hz */`
`344`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));`
	`344`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);`
`345`	`345`	`if (res != 0)`
`346`	`346`	`{`
`347`	`347`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ uint8_t mpu6050_fifo_test(mpu6050_address_t addr, uint32_t times)`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`/* set 50Hz */`
`157`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));`
	`157`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);`
`158`	`158`	`if (res != 0)`
`159`	`159`	`{`
`160`	`160`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ uint8_t mpu6050_read_test(mpu6050_address_t addr, uint32_t times)`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`/* set 50Hz */`
`133`		`- res = mpu6050_set_sample_rate_divider(&gs_handle, 1000 / (50 - 1));`
	`133`	`+ res = mpu6050_set_sample_rate_divider(&gs_handle, (1000 / 50) - 1);`
`134`	`134`	`if (res != 0)`
`135`	`135`	`{`
`136`	`136`	`mpu6050_interface_debug_print("mpu6050: set sample rate divider failed.\n");`