From e4c72d53d612ab9470f366fd594cfaf530590a13 Mon Sep 17 00:00:00 2001 From: Martin Braun Date: Mon, 18 Nov 2019 18:19:08 -0800 Subject: mpm: e31x: Remove superfluous code from e31x_periphs - ref_locked failure would throw a warning, even though this can happen in normal operations, and the return value of the get_ref_locked() function is all the info needed - get_fpga_type() doesn't need to be called from e31x_periphs, because it is not read from the motherboard registers (is read from the EEPROM) --- mpm/python/usrp_mpm/periph_manager/e31x_periphs.py | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) (limited to 'mpm/python/usrp_mpm/periph_manager/e31x_periphs.py') diff --git a/mpm/python/usrp_mpm/periph_manager/e31x_periphs.py b/mpm/python/usrp_mpm/periph_manager/e31x_periphs.py index 33c90ceb7..bb84ebb3e 100644 --- a/mpm/python/usrp_mpm/periph_manager/e31x_periphs.py +++ b/mpm/python/usrp_mpm/periph_manager/e31x_periphs.py @@ -129,14 +129,6 @@ class MboardRegsControl(MboardRegsCommon): self.log.trace("Writing MB_CLOCK_CTRL to 0x{:08X}".format(reg_val)) self.poke32(self.MB_CLOCK_CTRL, reg_val) - def get_fpga_type(self): - """ - Reads the type of the FPGA image currently loaded - Returns a string with the type (SG1, SG3) - """ - #TODO: Add SG1 and SG3? - return "" - def get_refclk_lock(self): """ Check the status of the reference clock in FPGA. @@ -145,11 +137,7 @@ class MboardRegsControl(MboardRegsCommon): with self.regs: reg_val = self.peek32(self.MB_CLOCK_CTRL) locked = (reg_val & mask) > 0 - if not locked: - self.log.warning("Reference Clock reporting unlocked. " - "MB_CLOCK_CTRL reg: 0x{:08X}".format(reg_val)) - else: - self.log.trace("Reference Clock locked!") + self.log.trace("Reference Clock %slocked!", "" if locked else "un") return locked def set_channel_mode(self, channel_mode): -- cgit v1.2.3