Skip to content
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.

Sign up for GitHub

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

Jump to bottom

radio does not work well with updated kernel #220

Open
hypnotoad opened this issue Dec 27, 2022 · 8 comments
Open

radio does not work well with updated kernel #220

hypnotoad opened this issue Dec 27, 2022 · 8 comments
Assignees
@ski7777
Labels
bug platform support
Milestone
new-kernel

Comments

@hypnotoad
Copy link
Collaborator

The radio app stops working after approx 1 second. There is some spi error. I guess that is related to the minor kernel upgrade: To reproduce:

mpg123 -f 8000 -q --stdout --encoding u8 --rate 22050 --mono http://liveradio.sr.de/sr/sr1/mp3/64/stream.mp3 | /home/ftc/apps/ad5e0ea4-4606-4254-add6-60c452c3db36/txt_snd_cat
@ski7777
Copy link
Member

ski7777 commented Dec 28, 2022

I hate kernel and C things...
I will try to debug it :)

@ski7777 ski7777 added this to the V1.0 milestone Dec 28, 2022
@ski7777
Copy link
Member

ski7777 commented Dec 31, 2022

Just added a package to build the ft_txt_snd_cat in future. I did this to ensure building against matching kernel headers. Sadly still the same behavior

@ski7777
Copy link
Member

ski7777 commented Dec 31, 2022

Do we want to revert the kernel update?
Does anyone know how to write good C code?
@rkunze @ftrobopy

@hypnotoad
Copy link
Collaborator Author

hypnotoad commented Jan 1, 2023 •
edited
Loading

I will try if reverting 1503cc8 fixes the problem. I am a little afraid that this bug also affects the ft-server's sound output abilities.

@hypnotoad
Copy link
Collaborator Author

after reverting 1503cc8, the command above works

@ski7777 ski7777 modified the milestones: V1.0, new-kernel Jan 1, 2023
@hypnotoad hypnotoad changed the title radio does not work well (new kernel?) radio does not work well with updated kernel Jan 1, 2023
@ski7777
Copy link
Member

ski7777 commented May 21, 2023

where did you fix this?

@ski7777 ski7777 reopened this May 21, 2023
@hypnotoad
Copy link
Collaborator Author

hypnotoad commented May 21, 2023 •
edited
Loading

it was fixed by your revert. I have to check to be sure...

@ski7777
Copy link
Member

ski7777 commented May 21, 2023

yes, but we still need to update the kernel. So I want to keep this open until it works with a new kernel

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

@ski7777 ski7777

Labels
bug platform support
Projects
None yet
Milestone
new-kernel
Development

When branches are created from issues, their pull requests are automatically linked.

220-radio-does-not-work-well-new-kernel ftCommunity/ftcommunity-TXT
2 participants
@hypnotoad @ski7777