use another lock for hid operations

lib/plugins.py

@@ -307,6 +307,7 @@ class DeviceMgr(ThreadJob, PrintError):
         self.recognised_hardware = set()
         # For synchronization
         self.lock = threading.RLock()
+        self.hid_lock = threading.RLock()
         self.config = config
 
     def thread_jobs(self):
@@ -490,7 +491,8 @@ class DeviceMgr(ThreadJob, PrintError):
         # wallet libraries are not affected.
         import hid
         self.print_error("scanning devices...")
-        hid_list = hid.enumerate(0, 0)
+        with self.hid_lock:
+            hid_list = hid.enumerate(0, 0)
         # First see what's connected that we know about
         devices = []
         for d in hid_list:

plugins/trezor/plugin.py

@@ -136,7 +136,8 @@ class TrezorCompatiblePlugin(HW_PluginBase):
         assert self.main_thread != threading.current_thread()
         devmgr = self.device_manager()
         handler = keystore.handler
-        client = devmgr.client_for_keystore(self, handler, keystore, force_pair)
+        with devmgr.hid_lock:
+            client = devmgr.client_for_keystore(self, handler, keystore, force_pair)
         # returns the client for a given keystore. can use xpub
         if client:
             client.used()