Merge #518

bors[bot] · burrbull · web-flow · commit d0c0aa312304 · 2022-08-26T11:35:42.000Z
518: fix PwmHz::get_period r=therealprof a=burrbull See stm32-rs/stm32f1xx-hal#434 Co-authored-by: Andrey Zgarbul <zgarbul.andrey@gmail.com>
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Changed
 
+- `PwmHz::get_period`: fix computation of return value, prevent division by zero
  - apply #[inline] attribute to bitbanding functions [#517]
  - update `stm32f4` to 0.15.1 [#481]
  - use `stm32_i2s_v12x` version 0.3, reexport it, and implements requirement for it [#490]
diff --git a/src/timer/pwm.rs b/src/timer/pwm.rs
@@ -301,7 +301,7 @@ where
         let arr = TIM::read_auto_reload();
 
         // Length in ms of an internal clock pulse
-        clk / (psc * arr)
+        clk / ((psc + 1) * (arr + 1))
     }
 
     pub fn set_period(&mut self, period: Hertz) {

Original file line number	Diff line number	Diff line change
`@@ -301,7 +301,7 @@ where`
`301`	`301`	`let arr = TIM::read_auto_reload();`
`302`	`302`
`303`	`303`	`// Length in ms of an internal clock pulse`
`304`		`- clk / (psc * arr)`
	`304`	`+ clk / ((psc + 1) * (arr + 1))`
`305`	`305`	`}`
`306`	`306`
`307`	`307`	`pub fn set_period(&mut self, period: Hertz) {`