New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

player eventually freezes #4

Closed
hoosierEE opened this Issue Jun 5, 2014 · 5 comments

Comments

Projects
None yet
4 participants
@hoosierEE

hoosierEE commented Jun 5, 2014

The feedBuffer() method relies on DREQ for determining when it is okay to grab more bytes from the SD card, but according to the datasheet this is not sufficient, as DREQ can change for other reasons besides "ready for data".

In my sketch I noticed that repeatedly calling startPlayingFile() will eventually cause the Arduino to freeze, and when it does the DREQ pin remains low.

Doing a softReset() of the VS1053 prevents the freezing, but that is not ideal.

@hoosierEE

This comment has been minimized.

Show comment
Hide comment
@hoosierEE

hoosierEE Jun 11, 2014

My PR did not actually fix the eventual freeze; I just hadn't waited long enough to find that out.
It still freezes after a while (at which point the entire sketch locks up). My current (not ideal) workaround is to manually call feedBuffer() and do a softReset() every once in a while.

hoosierEE commented Jun 11, 2014

My PR did not actually fix the eventual freeze; I just hadn't waited long enough to find that out.
It still freezes after a while (at which point the entire sketch locks up). My current (not ideal) workaround is to manually call feedBuffer() and do a softReset() every once in a while.

@s00500

This comment has been minimized.

Show comment
Hide comment
@s00500

s00500 Nov 23, 2014

Is there any better solution to this yet?
Thanks
Lukas

s00500 commented Nov 23, 2014

Is there any better solution to this yet?
Thanks
Lukas

@PaulStoffregen

This comment has been minimized.

Show comment
Hide comment
@PaulStoffregen

PaulStoffregen Dec 12, 2014

Contributor

Please give this version a try. Let me know if it solve the freezing problem?

https://github.com/PaulStoffregen/Adafruit_VS1053_Library

Also, you might try with Arduino 1.5.8, which has the newer SPI library with transaction support.....

Contributor

PaulStoffregen commented Dec 12, 2014

Please give this version a try. Let me know if it solve the freezing problem?

https://github.com/PaulStoffregen/Adafruit_VS1053_Library

Also, you might try with Arduino 1.5.8, which has the newer SPI library with transaction support.....

joelbschwartz pushed a commit to Sproutel/Adafruit_VS1053_Library that referenced this issue Feb 18, 2017

joelbschwartz pushed a commit to Sproutel/Adafruit_VS1053_Library that referenced this issue Feb 18, 2017

Joel Schwartz
Prevent interrupts during feedBuffer(), similar to a feature of Paul …
…Stoffregen’s fork, possibly related to issue #4

joelbschwartz pushed a commit to Sproutel/Adafruit_VS1053_Library that referenced this issue Feb 18, 2017

Joel Schwartz
Prevent interrupts at the beginning of feedBuffer() where flag is che…
…cked if feedBuffer is already running, similar to a feature of Paul Stoffregen’s fork, possibly related to issue #4

joelbschwartz pushed a commit to Sproutel/Adafruit_VS1053_Library that referenced this issue Feb 18, 2017

Joel Schwartz
Disable interrupts at the beginning of feedBuffer(), where a flag is …
…checked if feedBuffer is already running, similar to a feature of Paul Stoffregen’s fork, possibly related to issue #4

awong-dev added a commit to awong-dev/Adafruit_VS1053_Library that referenced this issue May 26, 2017

Disable interrupts at the beginning of feedBuffer(), where a flag is …
…checked if feedBuffer is already running, similar to a feature of Paul Stoffregen’s fork, possibly related to issue #4

@harshcht harshcht referenced this issue Jun 3, 2017

Open

No sound . #30

@ladyada

This comment has been minimized.

Show comment
Hide comment
@ladyada

ladyada Jun 3, 2017

Member

not sure but we've just pulled some lock-tweaks - please reopen if it still happens!

Member

ladyada commented Jun 3, 2017

not sure but we've just pulled some lock-tweaks - please reopen if it still happens!

@ladyada ladyada closed this Jun 3, 2017

@hoosierEE

This comment has been minimized.

Show comment
Hide comment
@hoosierEE

hoosierEE Jun 5, 2017

Hey, I hate it when OP abandons the thread, but unfortunately I won't be able to confirm the fix in the foreseeable future as I'm busy with other projects. Thanks for working on it though, hopefully the next person to use this will benefit.

hoosierEE commented Jun 5, 2017

Hey, I hate it when OP abandons the thread, but unfortunately I won't be able to confirm the fix in the foreseeable future as I'm busy with other projects. Thanks for working on it though, hopefully the next person to use this will benefit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment