tinycompress icon indicating copy to clipboard operation
tinycompress copied to clipboard

(E)AC3, DTS, MPEG-H support in tinycompress and the kernel uapi?

Open dv1 opened this issue 2 years ago • 3 comments
trafficstars

Have there been discussions about adding support for (E)AC3, DTS, and MPEG-H? These days, 3D Audio is a use case where the Compress-Offload API is useful. It therefore would make sense to add support for these formats. I'm talking about direct support, not through IEC958 / IEC61937 payloading. I know that some HW vendors add their own custom changes to tinycompress to support these formats, so why not add these to upstream?

dv1 avatar Nov 22 '22 11:11 dv1

no discussions AFAIK. But patches are welcomed.

dbaluta avatar Nov 22 '22 13:11 dbaluta

But since this would also require changes to the sound/compress_params.h kernel UAPI header, how should this start? Should a tinycompress PR be preceded by a discussion about extra formats in the linux-media mailing list?

dv1 avatar Nov 23 '22 12:11 dv1

Ideally you should first create a patch for alsa devel mailing list introducing new formats followed by a PR here where you update the UAPI header and add the implementation.

it doesn't really matter the order. what it matter is to send the patches and people will guide you.

dbaluta avatar Nov 25 '22 09:11 dbaluta