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.

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

SPDIF: Make it possible to use 352 khz and 384 khz #12783

Merged
merged 2 commits into from
Sep 10, 2017
Merged

Conversation

fritsch
Copy link
Member

@fritsch fritsch commented Sep 9, 2017

I could not believe it first, but there are now SPDIF devices out there that accept 384 khz / 32 bit input. See for example the following device: http://www.ca.marantz.com/ca/Products/Pages/ProductDetails.aspx?CatId=HiFiComponents&SubCatId=0&ProductId=HDAMP1

A user on LibreELEC forum verified with ALSA's speaker-test that it works correctly. In order to support this usecase a minimal change to kodi (only Settings and Language) needs to happen.

Note: ALSA sink currently does not support 352 khz, but pulseaudio does. 384 khz both support.

LE Forum thread: https://forum.libreelec.tv/thread/9823-audio-output-device-sampling-rate/?pageNo=1

@CiNcH83
Copy link

CiNcH83 commented Sep 9, 2017

Specs of the Marantz say that 384kHz/32bit is only supported via USB though. Digital inputs (coax, optical) are limited to 192kHz/24bit which is the max for any S/PDIF transmitter I have seen so far.

@fritsch
Copy link
Member Author

fritsch commented Sep 9, 2017

Better: USB device, that tells it has a SPDIF out and accepts 384 / 32 bit ? Kodi handles it the way ALSA enumerates it and this is SPDIF.

@@ -16383,8 +16383,20 @@ msgctxt "#34128"
msgid "192.0"
msgstr ""

#. SPDIF max sampling rate
#: system/settings/settings.xml
msgctxt "#34130"

This comment was marked as spam.

This comment was marked as spam.

@fritsch
Copy link
Member Author

fritsch commented Sep 9, 2017

@vpeter4: thx - I fixed it up.

@fritsch
Copy link
Member Author

fritsch commented Sep 10, 2017

image
image

Working as it should. Nice.

@lrusak lrusak merged commit 8451ee2 into xbmc:master Sep 10, 2017
@Rechi Rechi added this to the L 18.0-alpha1 milestone Sep 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants