From 28ecf81c6ce7332ef4cdd0a99505fa27c4c71598 Mon Sep 17 00:00:00 2001
From: ion <thomas.harty@physics.ox.ac.uk>
Date: Tue, 12 Jun 2018 13:10:26 +0100
Subject: [PATCH] Sayma: HMC7043 init and detect no longer need results.

---
 artiq/firmware/libboard_artiq/hmc830_7043.rs | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/artiq/firmware/libboard_artiq/hmc830_7043.rs b/artiq/firmware/libboard_artiq/hmc830_7043.rs
index af4aaa279..cb8b787df 100644
--- a/artiq/firmware/libboard_artiq/hmc830_7043.rs
+++ b/artiq/firmware/libboard_artiq/hmc830_7043.rs
@@ -238,7 +238,7 @@ pub mod hmc7043 {
         Ok(())
     }
 
-    pub fn enable() -> Result<(), &'static str> {
+    pub fn enable() {
         info!("enabling hmc7043");
 
         unsafe {
@@ -249,11 +249,9 @@ pub mod hmc7043 {
         write(0x0, 0x1);   // Software reset
         write(0x0, 0x0);   // Normal operation
         write(0x1, 0x48);  // mute all outputs
-
-        Ok(())
     }
 
-    pub fn init() -> Result<(), &'static str> {
+    pub fn init() {
         spi_setup();
         info!("loading configuration...");
 
@@ -301,8 +299,6 @@ pub mod hmc7043 {
         write(0x1, 0x40);  // Unmute, high-performace/low-noise mode
 
         info!("  ...done");
-
-        Ok(())
     }
 
     pub fn cfg_dac_sysref(dacno: u8, phase: u16) {
@@ -333,7 +329,9 @@ pub fn init() -> Result<(), &'static str> {
     hmc830::set_dividers(1, 24, 0, 2); // 100MHz ref, 1.2GHz out
     hmc830::check_locked()?;
 
-    hmc7043::enable()?;
+    hmc7043::enable();
     hmc7043::detect()?;
-    hmc7043::init()
+    hmc7043::init();
+
+    Ok(())
 }