Merge branch 'maint'
authorJohnathan Corgan <jcorgan@corganenterprises.com>
Thu, 5 Aug 2010 01:54:55 +0000 (18:54 -0700)
committerJohnathan Corgan <jcorgan@corganenterprises.com>
Thu, 5 Aug 2010 01:54:55 +0000 (18:54 -0700)
* maint:
  usrp2: fixes for gcc 4.5.0 compatibility
  formatting change: move break statements
  Fix unitialized variable pointed to by compiler warning

usrp/host/lib/db_wbxng_adf4350_regs.cc
usrp2/host/lib/usrp2.cc

index aa13d50b9d30b7ab7facb0d0770af89eba990914..c9707c3c68040e2f441bdf2ec41426c63251bfc9 100644 (file)
@@ -82,15 +82,17 @@ adf4350_regs::_reg_shift(uint32_t data, uint32_t shift){
 
 uint32_t
 adf4350_regs::compute_register(uint8_t addr){
-       uint32_t data;
+       uint32_t data = 0;
        switch (addr){
                case 0: data = (
                        _reg_shift(d_int, 15)                           |
-                       _reg_shift(d_frac, 3)); break;
+                       _reg_shift(d_frac, 3));
+                       break;
                case 1: data = (
                        _reg_shift(d_prescaler, 27)                     |
                        _reg_shift(s_phase, 15)                         |
-                       _reg_shift(d_mod, 3)); break;
+                       _reg_shift(d_mod, 3));
+                       break;
                case 2: data = (
                        _reg_shift(s_low_noise_and_low_spur_modes, 29)  |
                        _reg_shift(s_muxout, 26)                        |
@@ -104,11 +106,13 @@ adf4350_regs::compute_register(uint8_t addr){
                        _reg_shift(s_pd_polarity, 6)                    |
                        _reg_shift(s_power_down, 5)                     |
                        _reg_shift(s_cp_three_state, 4)                 |
-                       _reg_shift(s_counter_reset, 3)); break;
+                       _reg_shift(s_counter_reset, 3));
+                       break;
                case 3: data = (
                        _reg_shift(s_csr, 18)                           |
                        _reg_shift(s_clk_div_mode, 15)                  |
-                       _reg_shift(s_12_bit_clock_divider_value, 3)); break;
+                       _reg_shift(s_12_bit_clock_divider_value, 3));
+                       break;
                case 4: data = (
                        _reg_shift(s_feedback_select, 23)               |
                        _reg_shift(d_divider_select, 20)                |
@@ -119,9 +123,11 @@ adf4350_regs::compute_register(uint8_t addr){
                        _reg_shift(s_aux_output_enable, 8)              |
                        _reg_shift(s_aux_output_power, 6)               |
                        _reg_shift(s_rf_output_enable, 5)               |
-                       _reg_shift(s_output_power, 3)); break;
+                       _reg_shift(s_output_power, 3));
+                       break;
                case 5: data = (
-                       _reg_shift(s_ld_pin_mode, 22)); break;
+                       _reg_shift(s_ld_pin_mode, 22));
+                       break;
                default: return data;
        }
        /* return the data to write out to spi */
index f0ee564be002e6702c34019b3e030c8a305da442..0842482e40b37229c9ef832107fc250e771aba8e 100644 (file)
@@ -38,9 +38,9 @@ namespace usrp2 {
   struct usrp_table_entry {
     // inteface + normalized mac addr ("eth0:01:23:45:67:89:ab")
     std::string        key;
-    boost::weak_ptr<usrp2::usrp2>  value;
+    boost::weak_ptr<usrp2>  value;
 
-    usrp_table_entry(const std::string &_key, boost::weak_ptr<usrp2::usrp2> _value)
+    usrp_table_entry(const std::string &_key, boost::weak_ptr<usrp2> _value)
       : key(_key), value(_value) {}
   };
 
@@ -70,7 +70,7 @@ namespace usrp2 {
     // We don't have the USRP2 we're looking for
 
     // create a new one and stick it in the table.
-    usrp2::sptr r(new usrp2::usrp2(ifc, pr, rx_bufsize));
+    usrp2::sptr r(new usrp2(ifc, pr, rx_bufsize));
     usrp_table_entry t(key, r);
     s_table.push_back(t);