FeatherHAB/wsprhab Changeset - b5b9635e3cd8

Changeset - b5b9635e3cd8

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

Ethan Zonca - 10 years ago 2016-03-06 21:56:52
ez@ethanzonca.com

Fix comms issues with the SI5351 which actually works now

3 files changed with 18 insertions and 17 deletions:

.settings/language.settings.xml

lib/si5351/si5351.c

src/main.c

0 comments (0 inline, 0 general)

.settings/language.settings.xml

➞

Show inline comments

@@ @@ -2,13 +2,13 @@ @@
 <project>
 	<configuration id="ilg.gnuarmeclipse.managedbuild.cross.toolchain.base.1179702141" name="Default">
 		<extension point="org.eclipse.cdt.core.LanguageSettingsProvider">
 			<provider copy-of="extension" id="org.eclipse.cdt.ui.UserLanguageSettingsProvider"/>
 			<provider-reference id="org.eclipse.cdt.core.ReferencedProjectsLanguageSettingsProvider" ref="shared-provider"/>
 			<provider copy-of="extension" id="org.eclipse.cdt.managedbuilder.core.GCCBuildCommandParser"/>
-			<provider class="org.eclipse.cdt.managedbuilder.language.settings.providers.GCCBuiltinSpecsDetector" console="false" env-hash="1398045149230176998" id="ilg.gnuarmeclipse.managedbuild.cross.GCCBuiltinSpecsDetector" keep-relative-paths="false" name="CDT GCC Built-in Compiler Settings Cross ARM" parameter="${COMMAND} ${FLAGS} ${cross_toolchain_flags} -E -P -v -dD &quot;${INPUTS}&quot;" prefer-non-shared="true">
+			<provider class="org.eclipse.cdt.managedbuilder.language.settings.providers.GCCBuiltinSpecsDetector" console="false" env-hash="1398560194249728998" id="ilg.gnuarmeclipse.managedbuild.cross.GCCBuiltinSpecsDetector" keep-relative-paths="false" name="CDT GCC Built-in Compiler Settings Cross ARM" parameter="${COMMAND} ${FLAGS} ${cross_toolchain_flags} -E -P -v -dD &quot;${INPUTS}&quot;" prefer-non-shared="true">
 				<language-scope id="org.eclipse.cdt.core.gcc"/>
 				<language-scope id="org.eclipse.cdt.core.g++"/>
 			</provider>
 			<provider-reference id="org.eclipse.cdt.managedbuilder.core.MBSLanguageSettingsProvider" ref="shared-provider"/>
 		</extension>
 	</configuration>

lib/si5351/si5351.c

➞

Show inline comments

@@ @@ -331,13 +331,14 @@ uint8_t si5351_set_freq(uint64_t freq, u @@
  */
 uint8_t params[20];
 void si5351_set_pll(uint64_t pll_freq, enum si5351_pll target_pll)
 { struct Si5351RegSet pll_reg;
+{
   struct Si5351RegSet pll_reg;
   si5351_pll_calc(pll_freq, &pll_reg, ref_correction);
   // Derive the register values to write
   // Prepare an array for parameters to be written to
@@ @@ -902,22 +903,16 @@ void si5351_set_clock_fanout(enum si5351 @@
 	si5351_write(SI5351_FANOUT_ENABLE, reg_val);
+}
 /* AAAH REFREAKINGIMPLEMENT WITH COMMON STANDARD I2C THINGS:
 ~/Projects/featherhab-fw/libopencm3/lib/stm32/common/i2c_common_all.c
 */
 uint8_t si5351_write_bulk(uint8_t addr, uint8_t bytes, uint8_t *data)
+{
     HAL_Delay(300);
     uint32_t res = HAL_I2C_Mem_Write(si5351_i2cport, SI5351_BUS_BASE_ADDR, addr, 1, data, bytes, 100);
     led_blink(res);
     HAL_Delay(300);
 /*
 	Wire.beginTransmission(SI5351_BUS_BASE_ADDR);
 	Wire.write(addr);
@@ @@ -928,48 +923,44 @@ uint8_t si5351_write_bulk(uint8_t addr, @@
 	Wire.endTransmission();
 */
+}
 uint8_t si5351_write(uint8_t addr, uint8_t data)
+{
     HAL_Delay(300);
     uint8_t data_arr[1] = {data};
-    uint8_t res = HAL_I2C_Mem_Write(si5351_i2cport, SI5351_BUS_BASE_ADDR, addr, 1, data, 1, 100);
+    volatile uint8_t res = HAL_I2C_Mem_Write(si5351_i2cport, SI5351_BUS_BASE_ADDR, addr, 1, data_arr, 1, 100);
     led_blink(res);
     HAL_Delay(300);
 /*
 	Wire.beginTransmission(SI5351_BUS_BASE_ADDR);
 	Wire.write(addr);
 	Wire.write(data);
 	Wire.endTransmission();
 */
+}
 uint8_t si5351_read(uint8_t addr)
+{
 	uint8_t data_arr[1] = {0};
     HAL_Delay(300);
     uint8_t res = HAL_I2C_Mem_Read(si5351_i2cport, SI5351_BUS_BASE_ADDR, addr, 1, data_arr, 1, 100);
     led_blink(res);
     HAL_Delay(300);
 /*
 	Wire.beginTransmission(SI5351_BUS_BASE_ADDR);
 	Wire.write(addr);
 	Wire.endTransmission();
 	Wire.requestFrom(SI5351_BUS_BASE_ADDR, 1, false);
 	while(Wire.available())
+	{
 		reg_val = Wire.read();
+	}
 */
-	return data_arr[1];
+	return data_arr[0];
+}
 /*********************/
 /* Private functions */
 /*********************/

src/main.c

➞

Show inline comments

@@ @@ -20,13 +20,13 @@ uint8_t tx_buffer[255]; @@
 int main(void)
+{
     HAL_Init();
     sysclk_init();
     gpio_init();
     led_blink(5);
+    //led_blink(5);
     MX_DMA_Init();
     MX_ADC_Init();
     i2c_init();
     HAL_GPIO_WritePin(OSC_NOTEN, 0);
     HAL_GPIO_WritePin(TCXO_EN, 1);
@@ @@ -36,24 +36,34 @@ int main(void) @@
     //jtencode_init();
     //gps_init();
     si5351_init(i2c_get(), SI5351_CRYSTAL_LOAD_8PF, 0);
     si5351_set_correction(0);
     //SI5351_XTAL_FREQ
     si5351_set_pll(SI5351_PLL_FIXED, SI5351_PLLA);
-    si5351_set_pll(SI5351_PLL_FIXED, SI5351_PLLA);
     si5351_set_ms_source(SI5351_CLK0, SI5351_PLLA);
     si5351_set_ms_source(SI5351_CLK1, SI5351_PLLA);
     si5351_set_ms_source(SI5351_CLK2, SI5351_PLLA);
     si5351_set_ms_source(SI5351_CLK3, SI5351_PLLA);
-    si5351_set_freq(100000UL * 100, 0, SI5351_CLK0);
+    si5351_set_freq(1000000UL * 100, SI5351_PLL_FIXED, SI5351_CLK0);
     si5351_drive_strength(SI5351_CLK0, SI5351_DRIVE_2MA); // Set for max power if desired (8ma max)
     si5351_output_enable(SI5351_CLK0, 1); // Disable the clock initially
     //wspr_encode(call, loc, dbm, tx_buffer);
     si5351_pll_reset(SI5351_PLLA);
     HAL_Delay(1000);
     uint32_t led_timer = HAL_GetTick();
     uint32_t last_gps  = HAL_GetTick();
     while (1)
+    {
         if(HAL_GetTick() - led_timer > 100)

0 comments (0 inline, 0 general)