Received-SPF: pass (google.com: domain of linux-kernel+bounces-103790-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <0e9f0f2f-a404-3b76-3c52-9eca7594efa3@quicinc.com>
Date: Thu, 14 Mar 2024 13:57:19 -0700
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.13.1
Subject: Re: [PATCH v18 20/41] ALSA: usb-audio: qcom: Introduce QC USB SND
 offloading support
Content-Language: en-US
To: Albert Wang <albertccwang@google.com>
CC: <srinivas.kandagatla@linaro.org>, <mathias.nyman@intel.com>,
        <perex@perex.cz>, <conor+dt@kernel.org>, <corbet@lwn.net>,
        <lgirdwood@gmail.com>, <andersson@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>, <gregkh@linuxfoundation.org>,
        <Thinh.Nguyen@synopsys.com>, <broonie@kernel.org>,
        <bgoswami@quicinc.com>, <tiwai@suse.com>, <robh+dt@kernel.org>,
        <konrad.dybcio@linaro.org>, <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: <20240228013619.29758-1-quic_wcheng@quicinc.com>
 <20240228013619.29758-21-quic_wcheng@quicinc.com>
 <CANqn-rjTgHgzssxZiuwvTKzOS31wzjS4Y9G-XacZN4a7c82MaA@mail.gmail.com>
 <d97f635f-053b-70a7-5ffe-a1ae273091d1@quicinc.com>
 <CANqn-ring2uf=A-F7VuRwnJ--n=FtFzSddCmR-=nfxCGcFAF2g@mail.gmail.com>
From: Wesley Cheng <quic_wcheng@quicinc.com>
In-Reply-To: <CANqn-ring2uf=A-F7VuRwnJ--n=FtFzSddCmR-=nfxCGcFAF2g@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit

Hi Albert

On 3/14/2024 3:29 AM, Albert Wang wrote:
> On Thu, Mar 14, 2024 at 3:18 AM Wesley Cheng <quic_wcheng@quicinc.com> wrote:
>>
>> Hi Albert,
>>
>> On 3/13/2024 1:03 AM, Albert Wang wrote:
>>> Hi Wesley,
>>>
>>> The suspend function `qc_usb_audio_offload_suspend()` looks to stop
>>> the traffic on the bus, so that the bus can be suspended. That allows
>>> the application processor(AP) to enter suspend. There is a subtle
>>> difference with our feature, which is to allow AP suspend with the
>>> Host and USB controller active to continue the audio offloading. We
>>> call this feature `allow AP suspend in playback`. So, I have some
>>> points to clarify with you:
>>
>> Yes, I'm aware of that feature also.
>>
>>> 1. Will the suspend flow `usb_audio_suspend() -->
>>> platform_ops->suspend_cb() --> qc_usb_audio_offload_suspend()` be
>>> called when offloading is active?
>>
>> It can be.  This is why in our case, we are going to issue the
>> disconnect event to the audio DSP to stop the session if it is currently
>> in one.
>>
>>> 2. As my understanding, the suspend function is to allow AP suspend
>>> when the offloading is IDLE, but it won't allow AP suspend when in
>>> playback or capture. Please correct me if anything is wrong.
>>
>> As mentioned above, it will let apps go into PM suspend after forcing
>> the audio stream to be idle.  We won't block PM suspend entry.
>>
> Right. Your design is to force the audio stream idle, or say, inform
> the audio DSP
> to stop the current offloading session first, then AP can go into PM
> suspend as usual.
> Then I can say the current design did not support the `allow AP
> suspend in playback`
> feature, right?
> 

Correct, this series does not cover this mechanism.

>> Yes, I saw that patch as well.  I'll take a look once this series lands
>> upstream.
> 
> That patch is rejected and archived now. So we need to find another
> approach to do
> that, even based on your framework.
> 

We can discuss that offline and come up with an approach that is 
reviewable by maintainers and the community.

Thanks
Wesley Cheng

> Thanks,
> Albert
> 
> 
>>> 3. We would like to integrate the `allow AP suspend in playback`
>>> feature with your framework to become one upstream offload solution.
>>> Here is the patch:
>>> https://patchwork.kernel.org/project/linux-pm/patch/20240223143833.1509961-1-guanyulin@google.com/
>>> .
>>
>> Yes, I saw that patch as well.  I'll take a look once this series lands
>> upstream.
>>
>> Thanks
>> Wesley Cheng