Received-SPF: pass (google.com: domain of linux-kernel+bounces-174838-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <c97a0563-c4ce-45cd-8141-ee9be1e01899@linux.intel.com>
Date: Thu, 9 May 2024 08:01:04 -0500
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v21 22/39] ALSA: usb-audio: Prevent starting of audio
 stream if in use
To: Wesley Cheng <quic_wcheng@quicinc.com>, srinivas.kandagatla@linaro.org,
 mathias.nyman@intel.com, perex@perex.cz, conor+dt@kernel.org,
 corbet@lwn.net, lgirdwood@gmail.com, andersson@kernel.org,
 krzk+dt@kernel.org, gregkh@linuxfoundation.org, Thinh.Nguyen@synopsys.com,
 broonie@kernel.org, bgoswami@quicinc.com, tiwai@suse.com,
 bagasdotme@gmail.com, robh@kernel.org, konrad.dybcio@linaro.org
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
 linux-sound@vger.kernel.org, linux-usb@vger.kernel.org,
 linux-arm-msm@vger.kernel.org, linux-doc@vger.kernel.org,
 alsa-devel@alsa-project.org
References: <20240507195116.9464-1-quic_wcheng@quicinc.com>
 <20240507195116.9464-23-quic_wcheng@quicinc.com>
 <1e98935e-e35a-49e0-bbbf-ff326d40b581@linux.intel.com>
 <066fe096-a9d3-2498-275d-185f709e9a02@quicinc.com>
Content-Language: en-US
From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
In-Reply-To: <066fe096-a9d3-2498-275d-185f709e9a02@quicinc.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit


On 5/8/24 18:40, Wesley Cheng wrote:
> Hi Pierre,
> 
> On 5/7/2024 2:20 PM, Pierre-Louis Bossart wrote:
>>
>>> If a PCM device is already in use, the check will return an error to
>>> userspace notifying that the stream is currently busy.  This ensures
>>> that
>>> only one path is using the USB substream.
>>
>> What was the point of having a "USB Mixer" then?
> 
> The USB mixer is intended to enable/route the USB offloading path to the
> audio DSP, and is for controlling the ASoC specific entities.  This
> change is needed to resolve any contention between the USB SND PCM
> device (non offload path) and the ASoC USB BE DAI (offload path).

Not following, sorry. Is the "USB Mixer" some sort of hardware entity
related to USB offload or just a pure DAPM processing widget handling
volume and actual mixing between streams?

I was trying to get clarity on whether there can be multiple streams
mixed before going to the USB endpoint. The commit message "only one
path is using the USB substream" is ambiguous, not sure if you are
referring to mutual exclusion between offloaded and non-offloaded paths,
or number of streams when offloaded is supported. Different concepts/levels.