Skip to content

Conversation

@caternuson
Copy link
Contributor

Some fixes for #2. The _BUFFER gets overwritten with response so must be rebuilt each time in loop. This fixes station and same fix was applied to rds_buffer. However, I could never get the contents of rds_buffer to show up anywhere. Same thing testing with Arduino version. The documentation on that is...poor. The reference for the TX_RDS_BUFF command does not match what is shown in the programming example in AN332 Si47XX Programming Guide.

Adafruit CircuitPython 4.0.2 on 2019-06-27; Adafruit ItsyBitsy M4 Express with samd51g19
>>> import board, digitalio, adafruit_si4713
>>> si_reset = digitalio.DigitalInOut(board.D10)
>>> si4713 = adafruit_si4713.SI4713(board.I2C(), reset=si_reset, timeout_s=0.5)
>>> si4713.tx_frequency_khz = 87500
>>> si4713.tx_power = 115
>>> si4713.configure_rds(0xADAF, station=b'shoegaze4ever', rds_buffer=b'boots and pants')
>>> 

rds_test

@ladyada
Copy link
Member

ladyada commented Jul 12, 2019

can merge after travis happy

@caternuson caternuson merged commit 71c5a54 into adafruit:master Jul 12, 2019
@ladyada
Copy link
Member

ladyada commented Jul 12, 2019

epic, thanks!

@caternuson caternuson mentioned this pull request Jul 12, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jul 12, 2019
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI to 1.5.0 from 1.4.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#29 from martymcguire/mm-chunk-socket-sends

Updating https://github.com/adafruit/Adafruit_CircuitPython_SI4713 to 1.1.3 from 1.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_SI4713#7 from caternuson/iss2_rds

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Button to 1.1.2 from 1.1.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Button#8 from makermelissa/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_PyBadger to 0.9.0 from 0.8.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_PyBadger#3 from kattni/business-card-update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants