Fix regression in set_freq() that wiped out proper r_div setting

etherkit · Mar 24, 2017 · f08d3e1 · f08d3e1
1 parent a5d75a0
commit f08d3e1
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -663,6 +663,10 @@ This library does not currently support the spread spectrum function of the Si53
 
 Changelog
 ---------
+* v2.0.3
+
+    * Fix regression in _set_freq()_ that wiped out proper R div setting, causing errors in setting low frequency outputs
+
 * v2.0.2
 
     * Increase maximum frequency in _set_freq()_ to 225 MHz

diff --git a/library.properties b/library.properties
@@ -1,5 +1,5 @@
 name=Etherkit Si5351
-version=2.0.2
+version=2.0.3
 author=Jason Milldrum <milldrum@gmail.com>
 maintainer=Jason Milldrum <milldrum@gmail.com>
 sentence=A full-featured library for the Si5351 series of clock generator ICs from Silicon Labs

diff --git a/src/si5351.cpp b/src/si5351.cpp
@@ -282,6 +282,7 @@ uint8_t Si5351::set_freq(uint64_t freq, enum si5351_clock clk)
 		}
 		else
 		{
+			//uint64_t temp_freq = freq;
 			clk_freq[(uint8_t)clk] = freq;
 
 			// Enable the output
@@ -302,7 +303,6 @@ uint8_t Si5351::set_freq(uint64_t freq, enum si5351_clock clk)
 
 			// Set multisynth registers
 			set_ms(clk, ms_reg, int_mode, r_div, div_by_4);
-			ms_div(clk, 0, 0);
 
 			// Reset the PLL
 			pll_reset(pll_assignment[clk]);