search

karawin / Ka-Radio32

esp32 version of Ka-Radio (https://github.com/karawin/Ka-Radio) by jp Cocatrix
515 stars 155 forks source link

Built in DAC output doesn't work on Ka-Radio32 V2.1 R3 and V2.2 R0 #252

Closed Ant-12 closed 2 years ago

Ant-12 commented 2 years ago

To solve this problem, these changes in audio_renderer.c must be reverted.

What was the reason of these changes?

P.S. I also got a similar error, trying to flash firmware using "make flash". The solution for me was to change "Bootloader config ---> Bootloader log verbosity" to "Info".

karawin commented 2 years ago

I think it comes from the spdif test. I will change it.

Try the prerelease sys.prerelease

Ant-12 commented 2 years ago

Release 2.2, Revision 1 - still getting noise

Ant-12 commented 2 years ago

Release 2.2R1 and prerelease(2021-02-05) - built in DAC still doesn't work...

There are too many "2.2R1" versions. Maybe you will add a timestamp of build for prereleases?

karawin commented 2 years ago

Release 2.2R1 before today was a mistake. I cannot test dac. I added the missing parts but .... Please help.

Ant-12 commented 2 years ago

You did not revert all changes. There need to correct this line.

And please decrease "Bootloader log verbosity".

P.S. Have you tried to build Ka-radio32 under linux OS? I got better results in decoding of HE-AACv2 streams(64 kbit) using this: Toolchain version: crosstool-ng-1.22.0-97-gc752ad5 Compiler version: 5.2.0 ESP-IDF v3.3.5-dirty

karawin commented 2 years ago

Right, I never thought that it may be the cause. It works now. "And please decrease "Bootloader log verbosity"." Ok

I will reload a good R1 this night.

KaRadio is under linux, but KaRadio32 is under windows. In fact I dont have so much time to change, and a bit lazy ;-)

I will migrate to esp_idf v5 as soon as possible but .... Thanks

karawin commented 2 years ago

Generated in release mode, now fdkaac_decoder: pcm_size 8192, channels: 2, sample rate: 48000, object type: 2, bitrate: 62812

Works well. Updated and ready for OTA.

Ant-12 commented 2 years ago

Now I can confirm that built in DAC works. But your builds still have problems(stuttering) with playing HE-AACv2 streams(64 kbit) through S/PDIF output...