Fix SPI read_passive_target timeout

Moving the `with` statement seems to correct the timeout issue.
This commit is contained in:
tgikal 2019-07-01 14:48:24 -04:00 committed by GitHub
parent c44abc207e
commit 3f759879c8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -76,14 +76,14 @@ class PN532_SPI(PN532):
status = bytearray([reverse_bit(_SPI_STATREAD), 0]) status = bytearray([reverse_bit(_SPI_STATREAD), 0])
timestamp = time.monotonic() timestamp = time.monotonic()
while (time.monotonic() - timestamp) < timeout: with self._spi as spi:
with self._spi as spi: while (time.monotonic() - timestamp) < timeout:
#time.sleep(0.02) # required time.sleep(0.02) # required (not needed when tested on rPi 3)
spi.write_readinto(status, status) #pylint: disable=no-member spi.write_readinto(status, status) #pylint: disable=no-member
if reverse_bit(status[1]) == 0x01: # LSB data is read in MSB if reverse_bit(status[1]) == 0x01: # LSB data is read in MSB
return True # Not busy anymore! return True # Not busy anymore!
#else: else: # (not needed when tested on rPi 3)
# time.sleep(0.01) # pause a bit till we ask again time.sleep(0.01) # pause a bit till we ask again
# We timed out! # We timed out!
return False return False
@ -95,12 +95,12 @@ class PN532_SPI(PN532):
frame[0] = reverse_bit(_SPI_DATAREAD) frame[0] = reverse_bit(_SPI_DATAREAD)
with self._spi as spi: with self._spi as spi:
#time.sleep(0.02) # required time.sleep(0.02) # required (not needed when tested on rPi 3)
spi.write_readinto(frame, frame) #pylint: disable=no-member spi.write_readinto(frame, frame) #pylint: disable=no-member
for i, val in enumerate(frame): for i, val in enumerate(frame):
frame[i] = reverse_bit(val) # turn LSB data to MSB frame[i] = reverse_bit(val) # turn LSB data to MSB
#if self.debug: if self.debug:
# print("Reading: ", [hex(i) for i in frame[1:]]) print("Reading: ", [hex(i) for i in frame[1:]])
return frame[1:] return frame[1:]
def _write_data(self, framebytes): def _write_data(self, framebytes):
@ -108,8 +108,8 @@ class PN532_SPI(PN532):
# start by making a frame with data write in front, # start by making a frame with data write in front,
# then rest of bytes, and LSBify it # then rest of bytes, and LSBify it
rev_frame = [reverse_bit(x) for x in bytes([_SPI_DATAWRITE]) + framebytes] rev_frame = [reverse_bit(x) for x in bytes([_SPI_DATAWRITE]) + framebytes]
#if self.debug: if self.debug:
# print("Writing: ", [hex(i) for i in rev_frame]) print("Writing: ", [hex(i) for i in rev_frame])
with self._spi as spi: with self._spi as spi:
#time.sleep(0.02) # required time.sleep(0.02) # required (not needed when tested on rPi 3)
spi.write(bytes(rev_frame)) #pylint: disable=no-member spi.write(bytes(rev_frame)) #pylint: disable=no-member