Re: [RFC 00/15] ALSA/ASoC: USB Audio Offload

[Cezary Rojewski <cezary.rojewski@intel.com>]

On 2025-04-09 2:10 PM, Greg KH wrote:
> On Wed, Apr 09, 2025 at 01:07:15PM +0200, Cezary Rojewski wrote:
>> Note: this series is based on Mark's broonie/for-next. The xHCI
>> dependency is missing so it won't compile.  Goal of this RFC is
>> discussing the direction of sound/usb changes.
>>
>> Note #2: Why exclude xHCI?  Current form of xhci-sideband.c [1] does not
>> fare well with Intel's hardware design for USB Audio Offload feature.
>> The production shape for usb/xHCI subjects is being discussed with
>> Mathias.  Once we're ready, I'll share the rest.
> 
> How is that going to work with the patches that add xhci offload that
> have already been reviewed on the list?
> 
>> Note #3: this series does _NOT_ aim to block QCOM's equivalent series
>> [2].  The team does acknowledge that we came the "table" late.  At the
>> same time, we're prepared to help QCOM switch to the presented sound/usb
>> approach if that would benefit the framework and its users as a whole.
>> Make it part of this very series if need be.
> 
> I don't understand, I'm just about to take the QCOM patches now.  They
> have been on the list and reviewed for years now, why hasn't this work
> happened before now?  Please work to make your changes on top of that
> patch series.
> 
> So what do you expect me to do here?
> 
> confused,

Hi Greg,

I wanted the effort to be transparent, that's it.  Given the size of the 
task, I've decided to get the feedback earlier.  I believe asking 
sound-maintainers now for their take on sound/usb is better than coming 
here much later and shooting everything at once.  If the proposed 
approach to sound/usb is not welcomed by the sound maintainers, 
obviously xhci-sideband additions that will arrive here later will be 
impacted two.  In regard to the sound changes alone, there is no 
intention to blow up the parallel solution.

TLDR:
Work present here will be placed on top of the QCOM patches, no question 
about it.  xhci-sideband.c will receive a number of changes to cope with 
Intel's design but nothing the list haven't seen before.


Kind regards,
Czarek