[FFADO-svn] r2510 - trunk/libffado/src/fireworks

[<tr...@ff...>]

Author: mocchi
Date: 2014-05-05 20:46:01 -0700 (Mon, 05 May 2014)
New Revision: 2510

Modified:
   trunk/libffado/src/fireworks/fireworks_device.cpp
Log:
fireworks: Remove useless execution of command for polled values

There are meaningless call of the command.



Modified: trunk/libffado/src/fireworks/fireworks_device.cpp
===================================================================
--- trunk/libffado/src/fireworks/fireworks_device.cpp	2014-05-06 03:30:18 UTC (rev 2509)
+++ trunk/libffado/src/fireworks/fireworks_device.cpp	2014-05-06 03:46:01 UTC (rev 2510)
@@ -182,11 +182,6 @@
     // depends on this
     m_efc_version = m_HwInfo.m_header.version;
 
-    if (!updatePolledValues()) {
-        debugError("Could not update polled values\n");
-        return false;
-    }
-
     m_current_clock = -1;
 
     m_efc_discovery_done = true;
@@ -523,10 +518,10 @@
 bool
 Device::isClockValid(uint32_t id) {
     // always valid
-    if (id==EFC_CMD_HW_CLOCK_INTERNAL) return true;
+    if (id==EFC_CMD_HW_CLOCK_INTERNAL)
+        return true;
 
-    // the polled values are used to detect
-    // whether clocks are valid
+    // the polled values tell thether each clock source is detected or not
     if (!updatePolledValues()) {
         debugError("Could not update polled values\n");
         return false;
@@ -577,13 +572,6 @@
     ClockSource s;
     debugOutput(DEBUG_LEVEL_VERBOSE, "clock id: %u\n", clockid);
 
-    // the polled values are used to detect
-    // whether clocks are valid
-    if (!updatePolledValues()) {
-        debugError("Could not update polled values\n");
-        return s;
-    }
-
     switch (clockid) {
         case EFC_CMD_HW_CLOCK_INTERNAL:
             debugOutput(DEBUG_LEVEL_VERBOSE, "Internal clock\n");