public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Alexey Klimov <alexey.klimov@linaro.org>
Cc: Vinod Koul <vkoul@kernel.org>, Jaroslav Kysela <perex@perex.cz>,
	Takashi Iwai <tiwai@suse.com>,
	Srinivas Kandagatla <srini@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>,
	Patrick Lai <plai@qti.qualcomm.com>,
	Annemarie Porter <annemari@quicinc.com>,
	srinivas.kandagatla@oss.qualcomm.com,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-msm@vger.kernel.org,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	kernel@oss.qualcomm.com,
	Ekansh Gupta <ekansh.gupta@oss.qualcomm.com>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.dev>
Subject: Re: [PATCH v3 0/3] Add raw OPUS codec support for compress offload
Date: Fri, 19 Sep 2025 15:00:01 +0200	[thread overview]
Message-ID: <87plbm8uni.wl-tiwai@suse.de> (raw)
In-Reply-To: <20250917-opus_codec_rfc_v1-v3-0-7737ad40132e@linaro.org>

On Wed, 17 Sep 2025 09:32:49 +0200,
Alexey Klimov wrote:
> 
> This series adds support in kernel to recognise raw (or plain) OPUS
> codec playback for compress offloading. At this point this series
> doesn't deal with OPUS packets packed in any kind of containers (OGG or
> others) and focuses on adding missing bits for pure OPUS packets.
> 
> The second patch adds its usage in Qualcomm Audio DSP code. To correctly
> recognise raw OPUS packets by qdsp6, each packets needs to be prepended
> with 4-bytes field that contains length of a raw OPUS packet.
> It is expected to be useful for usecases when OPUS packets are streamed
> over network and they are not encapsulated in a container. Userspace
> application that will use the compress API has to manually add such
> 4-bytes long field to each OPUS packet.
> 
> This is tested on sm8750-mtp. It is expected that next hardware revisions
> will also support raw OPUS codec offloading.
> 
> Dependencies are:
> -- hardware with DSP that supports decoding OPUS packets (>= Qualcomm sm8750);
> -- adsp fastrpc;
> -- explicitly setting format in sm8750 soundcard driver
> https://lore.kernel.org/linux-sound/20250911154340.2798304-1-alexey.klimov@linaro.org/
> -- running adsprpcd tool with support for Audio PD and DSP libraries
> loading support (or its alternative);
> -- tinycompress fcplay tool that will prepare raw opus packets and
> do the required addition of length field;
> -- mfc module in topology compress-playback path, that module is
> expected to parse channel mapping and do the required things to
> map streams/channels to physically-present output channels on backend.
> 
> The userspace tinycompress tool with support for raw OPUS compress
> playback is located here:
> https://github.com/laklimov/tinycompress_opus
> branch: opus_v3_workinprogress
> 
> The userspace tool is not expected that it is ready and still needs
> some work, for instance recognition that it runs on Qualcomm hardware.
> More like working PoC.
> 
> Signed-off-by: Alexey Klimov <alexey.klimov@linaro.org>

Applied now to for-next branch for 6.18.
For further improvements, please submit on top of that.


thanks,

Takashi

      parent reply	other threads:[~2025-09-19 13:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-17  7:32 [PATCH v3 0/3] Add raw OPUS codec support for compress offload Alexey Klimov
2025-09-17  7:32 ` [PATCH v3 1/3] ALSA: compress: add raw opus codec define and opus decoder structs Alexey Klimov
2025-09-18  5:08   ` Vinod Koul
2025-10-15 16:36     ` Alexey Klimov
2025-09-17  7:32 ` [PATCH v3 2/3] ALSA: compress_offload: increase SNDRV_COMPRESS_VERSION minor version by 1 Alexey Klimov
2025-09-18  5:09   ` Vinod Koul
2025-09-17  7:32 ` [PATCH v3 3/3] ASoC: qcom: qdsp6/audioreach: add support for offloading raw opus playback Alexey Klimov
2025-09-17 11:39   ` Mark Brown
2025-09-19 13:00 ` Takashi Iwai [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87plbm8uni.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alexey.klimov@linaro.org \
    --cc=annemari@quicinc.com \
    --cc=broonie@kernel.org \
    --cc=ekansh.gupta@oss.qualcomm.com \
    --cc=kernel@oss.qualcomm.com \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=pierre-louis.bossart@linux.dev \
    --cc=plai@qti.qualcomm.com \
    --cc=srini@kernel.org \
    --cc=srinivas.kandagatla@oss.qualcomm.com \
    --cc=tiwai@suse.com \
    --cc=vkoul@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox