[PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone

Luca Weiss posted 5 patches 9 months, 1 week ago
There is a newer version of this series
.../devicetree/bindings/sound/qcom,q6afe.yaml      | 13 ++++++++
.../devicetree/bindings/sound/qcom,sm8250.yaml     |  1 +
arch/arm64/boot/dts/qcom/sm6350.dtsi               | 11 +++++++
arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts  | 36 ++++++++++++++++++++++
sound/soc/qcom/sm8250.c                            |  1 +
5 files changed, 62 insertions(+)
[PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Luca Weiss 9 months, 1 week ago
Since the series for the USB sound offloading driver was finally merged,
we can add the sm6350 dts and enable it on Fairphone 4.

A few devicetree binding bits have also been missing in that series, so
there's some extra patches for the basics in this series.

Depends on:
- For qcom,sm8250.yaml & sm8250.c:
  https://lore.kernel.org/linux-arm-msm/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone.com/T/
- For dts:
  https://lore.kernel.org/linux-arm-msm/20250321-sm6350-apr-v1-1-7805ce7b4dcf@fairphone.com/

Devicetree patches go through qcom

Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
Changes in v2:
- Move num-hc-interrupters prop to sm6350.dtsi (Konrad)
- Sort cpu & codec nodes correctly (Konrad)
- Remove usb-soc-be prop as it's no longer needed (Wesley)
- Pick up tags
- Link to v1: https://lore.kernel.org/r/20250425-fp4-usb-audio-offload-v1-0-f90f571636e4@fairphone.com

---
Luca Weiss (5):
      ASoC: dt-bindings: qcom,q6afe: Document q6usb subnode
      ASoC: dt-bindings: qcom,sm8250: Add Fairphone 4 sound card
      ASoC: qcom: sm8250: Add Fairphone 4 soundcard compatible
      arm64: dts: qcom: sm6350: Add q6usbdai node
      arm64: dts: qcom: sm7225-fairphone-fp4: Enable USB audio offload support

 .../devicetree/bindings/sound/qcom,q6afe.yaml      | 13 ++++++++
 .../devicetree/bindings/sound/qcom,sm8250.yaml     |  1 +
 arch/arm64/boot/dts/qcom/sm6350.dtsi               | 11 +++++++
 arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts  | 36 ++++++++++++++++++++++
 sound/soc/qcom/sm8250.c                            |  1 +
 5 files changed, 62 insertions(+)
---
base-commit: 393d0c54cae31317deaa9043320c5fd9454deabc
change-id: 20250425-fp4-usb-audio-offload-e5d7fb55d283
prerequisite-change-id: 20250321-sm6350-apr-15f3e7e4e5ca:v1
prerequisite-patch-id: 69859554c94de52068406dab0d2869ca26072c78
prerequisite-change-id: 20240809-fp5-dp-sound-b3768f3019bd:v3
prerequisite-patch-id: 2d8997a2be02cfddf3a054c79e7eb308a6d06710
prerequisite-patch-id: 1b7c8f00ab50ae71c0221868578b9c1dfe939e4d
prerequisite-patch-id: 4fd0673c4d35c1fefce63ec5785ff5ea67cc3d3a
prerequisite-patch-id: f5ec3a893de19900f62dc691d83986f1104914b0
prerequisite-patch-id: fce7573c39e768f7a09c002064b6159b8e91161a

Best regards,
-- 
Luca Weiss <luca.weiss@fairphone.com>
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Luca Weiss 6 months, 3 weeks ago
Hi all,

On Thu May 1, 2025 at 8:48 AM CEST, Luca Weiss wrote:
> Since the series for the USB sound offloading driver was finally merged,
> we can add the sm6350 dts and enable it on Fairphone 4.
>
> A few devicetree binding bits have also been missing in that series, so
> there's some extra patches for the basics in this series.
>
> Depends on:
> - For qcom,sm8250.yaml & sm8250.c:
>   https://lore.kernel.org/linux-arm-msm/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone.com/T/
> - For dts:
>   https://lore.kernel.org/linux-arm-msm/20250321-sm6350-apr-v1-1-7805ce7b4dcf@fairphone.com/
>
> Devicetree patches go through qcom
>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
> Changes in v2:
> - Move num-hc-interrupters prop to sm6350.dtsi (Konrad)
> - Sort cpu & codec nodes correctly (Konrad)
> - Remove usb-soc-be prop as it's no longer needed (Wesley)
> - Pick up tags
> - Link to v1: https://lore.kernel.org/r/20250425-fp4-usb-audio-offload-v1-0-f90f571636e4@fairphone.com
>
> ---
> Luca Weiss (5):
>       ASoC: dt-bindings: qcom,q6afe: Document q6usb subnode
>       ASoC: dt-bindings: qcom,sm8250: Add Fairphone 4 sound card
>       ASoC: qcom: sm8250: Add Fairphone 4 soundcard compatible
>       arm64: dts: qcom: sm6350: Add q6usbdai node
>       arm64: dts: qcom: sm7225-fairphone-fp4: Enable USB audio offload support

All dependencies for the patches have been applied already, so this
series can land as well!

Devicetree patches go through qcom, the rest through their trees.

Regards
Luca

>
>  .../devicetree/bindings/sound/qcom,q6afe.yaml      | 13 ++++++++
>  .../devicetree/bindings/sound/qcom,sm8250.yaml     |  1 +
>  arch/arm64/boot/dts/qcom/sm6350.dtsi               | 11 +++++++
>  arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts  | 36 ++++++++++++++++++++++
>  sound/soc/qcom/sm8250.c                            |  1 +
>  5 files changed, 62 insertions(+)
> ---
> base-commit: 393d0c54cae31317deaa9043320c5fd9454deabc
> change-id: 20250425-fp4-usb-audio-offload-e5d7fb55d283
> prerequisite-change-id: 20250321-sm6350-apr-15f3e7e4e5ca:v1
> prerequisite-patch-id: 69859554c94de52068406dab0d2869ca26072c78
> prerequisite-change-id: 20240809-fp5-dp-sound-b3768f3019bd:v3
> prerequisite-patch-id: 2d8997a2be02cfddf3a054c79e7eb308a6d06710
> prerequisite-patch-id: 1b7c8f00ab50ae71c0221868578b9c1dfe939e4d
> prerequisite-patch-id: 4fd0673c4d35c1fefce63ec5785ff5ea67cc3d3a
> prerequisite-patch-id: f5ec3a893de19900f62dc691d83986f1104914b0
> prerequisite-patch-id: fce7573c39e768f7a09c002064b6159b8e91161a
>
> Best regards,
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Luca Weiss 6 months, 3 weeks ago
Hi Mark Brown,

On Wed Jul 16, 2025 at 9:19 AM CEST, Luca Weiss wrote:
> Hi all,
>
> On Thu May 1, 2025 at 8:48 AM CEST, Luca Weiss wrote:
>> Since the series for the USB sound offloading driver was finally merged,
>> we can add the sm6350 dts and enable it on Fairphone 4.
>>
>> A few devicetree binding bits have also been missing in that series, so
>> there's some extra patches for the basics in this series.
>>
>> Depends on:
>> - For qcom,sm8250.yaml & sm8250.c:
>>   https://lore.kernel.org/linux-arm-msm/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone.com/T/
>> - For dts:
>>   https://lore.kernel.org/linux-arm-msm/20250321-sm6350-apr-v1-1-7805ce7b4dcf@fairphone.com/
>>
>> Devicetree patches go through qcom
>>
>> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
>> ---
>> Changes in v2:
>> - Move num-hc-interrupters prop to sm6350.dtsi (Konrad)
>> - Sort cpu & codec nodes correctly (Konrad)
>> - Remove usb-soc-be prop as it's no longer needed (Wesley)
>> - Pick up tags
>> - Link to v1: https://lore.kernel.org/r/20250425-fp4-usb-audio-offload-v1-0-f90f571636e4@fairphone.com
>>
>> ---
>> Luca Weiss (5):
>>       ASoC: dt-bindings: qcom,q6afe: Document q6usb subnode
>>       ASoC: dt-bindings: qcom,sm8250: Add Fairphone 4 sound card
>>       ASoC: qcom: sm8250: Add Fairphone 4 soundcard compatible
>>       arm64: dts: qcom: sm6350: Add q6usbdai node
>>       arm64: dts: qcom: sm7225-fairphone-fp4: Enable USB audio offload support
>
> All dependencies for the patches have been applied already, so this
> series can land as well!

Is it still possible to pick up the sound patches (1-3) for 6.17? Dts
has been applied already.

Regards
Luca

>
> Devicetree patches go through qcom, the rest through their trees.
>
> Regards
> Luca
>
>>
>>  .../devicetree/bindings/sound/qcom,q6afe.yaml      | 13 ++++++++
>>  .../devicetree/bindings/sound/qcom,sm8250.yaml     |  1 +
>>  arch/arm64/boot/dts/qcom/sm6350.dtsi               | 11 +++++++
>>  arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts  | 36 ++++++++++++++++++++++
>>  sound/soc/qcom/sm8250.c                            |  1 +
>>  5 files changed, 62 insertions(+)
>> ---
>> base-commit: 393d0c54cae31317deaa9043320c5fd9454deabc
>> change-id: 20250425-fp4-usb-audio-offload-e5d7fb55d283
>> prerequisite-change-id: 20250321-sm6350-apr-15f3e7e4e5ca:v1
>> prerequisite-patch-id: 69859554c94de52068406dab0d2869ca26072c78
>> prerequisite-change-id: 20240809-fp5-dp-sound-b3768f3019bd:v3
>> prerequisite-patch-id: 2d8997a2be02cfddf3a054c79e7eb308a6d06710
>> prerequisite-patch-id: 1b7c8f00ab50ae71c0221868578b9c1dfe939e4d
>> prerequisite-patch-id: 4fd0673c4d35c1fefce63ec5785ff5ea67cc3d3a
>> prerequisite-patch-id: f5ec3a893de19900f62dc691d83986f1104914b0
>> prerequisite-patch-id: fce7573c39e768f7a09c002064b6159b8e91161a
>>
>> Best regards,
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Mark Brown 6 months, 2 weeks ago
On Mon, Jul 21, 2025 at 08:22:06AM +0200, Luca Weiss wrote:
> On Wed Jul 16, 2025 at 9:19 AM CEST, Luca Weiss wrote:

> > All dependencies for the patches have been applied already, so this
> > series can land as well!

> Is it still possible to pick up the sound patches (1-3) for 6.17? Dts
> has been applied already.

As previously discussed they won't apply until after the merge window.

Please don't send content free pings and please allow a reasonable time
for review.  People get busy, go on holiday, attend conferences and so 
on so unless there is some reason for urgency (like critical bug fixes)
please allow at least a couple of weeks for review.  If there have been
review comments then people may be waiting for those to be addressed.

Sending content free pings adds to the mail volume (if they are seen at
all) which is often the problem and since they can't be reviewed
directly if something has gone wrong you'll have to resend the patches
anyway, so sending again is generally a better approach though there are
some other maintainers who like them - if in doubt look at how patches
for the subsystem are normally handled.
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Luca Weiss 6 months, 2 weeks ago
Hi Mark,

On Wed Jul 23, 2025 at 12:57 PM CEST, Mark Brown wrote:
> On Mon, Jul 21, 2025 at 08:22:06AM +0200, Luca Weiss wrote:
>> On Wed Jul 16, 2025 at 9:19 AM CEST, Luca Weiss wrote:
>
>> > All dependencies for the patches have been applied already, so this
>> > series can land as well!
>
>> Is it still possible to pick up the sound patches (1-3) for 6.17? Dts
>> has been applied already.
>
> As previously discussed they won't apply until after the merge window.

Sorry about that, I thought the conflict was for the 6.16 merge window,
not 6.17?

The patches this depends on have been applied by you on 2025-06-09:
https://lore.kernel.org/linux-arm-msm/174950282564.277844.4634804513095204160.b4-ty@kernel.org/

So I'm not aware of another conflict, that's why I was asking in the
first place.

Regards
Luca

>
> Please don't send content free pings and please allow a reasonable time
> for review.  People get busy, go on holiday, attend conferences and so 
> on so unless there is some reason for urgency (like critical bug fixes)
> please allow at least a couple of weeks for review.  If there have been
> review comments then people may be waiting for those to be addressed.
>
> Sending content free pings adds to the mail volume (if they are seen at
> all) which is often the problem and since they can't be reviewed
> directly if something has gone wrong you'll have to resend the patches
> anyway, so sending again is generally a better approach though there are
> some other maintainers who like them - if in doubt look at how patches
> for the subsystem are normally handled.
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Mark Brown 6 months, 2 weeks ago
On Wed, Jul 23, 2025 at 01:09:35PM +0200, Luca Weiss wrote:
> On Wed Jul 23, 2025 at 12:57 PM CEST, Mark Brown wrote:

> > As previously discussed they won't apply until after the merge window.

> Sorry about that, I thought the conflict was for the 6.16 merge window,
> not 6.17?

There is a conflict.  You could check this yourself...

> So I'm not aware of another conflict, that's why I was asking in the
> first place.

You were sending a content free ping:

> > Please don't send content free pings and please allow a reasonable time
> > for review.  People get busy, go on holiday, attend conferences and so 
> > on so unless there is some reason for urgency (like critical bug fixes)
> > please allow at least a couple of weeks for review.  If there have been
> > review comments then people may be waiting for those to be addressed.
> >
> > Sending content free pings adds to the mail volume (if they are seen at
> > all) which is often the problem and since they can't be reviewed
> > directly if something has gone wrong you'll have to resend the patches
> > anyway, so sending again is generally a better approach though there are
> > some other maintainers who like them - if in doubt look at how patches
> > for the subsystem are normally handled.
Re: [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Luca Weiss 6 months, 2 weeks ago
On Wed Jul 23, 2025 at 2:19 PM CEST, Mark Brown wrote:
> On Wed, Jul 23, 2025 at 01:09:35PM +0200, Luca Weiss wrote:
>> On Wed Jul 23, 2025 at 12:57 PM CEST, Mark Brown wrote:
>
>> > As previously discussed they won't apply until after the merge window.
>
>> Sorry about that, I thought the conflict was for the 6.16 merge window,
>> not 6.17?
>
> There is a conflict.  You could check this yourself...

I see now where the conflict is, due to an updated previous patch.

I will send a v3 shortly with that resolved. Then it applies on your
for-next branch cleanly.

Regards
Luca
Re: (subset) [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Mark Brown 6 months, 2 weeks ago
On Thu, 01 May 2025 08:48:46 +0200, Luca Weiss wrote:
> Since the series for the USB sound offloading driver was finally merged,
> we can add the sm6350 dts and enable it on Fairphone 4.
> 
> A few devicetree binding bits have also been missing in that series, so
> there's some extra patches for the basics in this series.
> 
> Depends on:
> - For qcom,sm8250.yaml & sm8250.c:
>   https://lore.kernel.org/linux-arm-msm/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone.com/T/
> - For dts:
>   https://lore.kernel.org/linux-arm-msm/20250321-sm6350-apr-v1-1-7805ce7b4dcf@fairphone.com/
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/5] ASoC: dt-bindings: qcom,q6afe: Document q6usb subnode
      commit: b102c9d89fecd72be83eaab9b384285e2d0dc940
[2/5] ASoC: dt-bindings: qcom,sm8250: Add Fairphone 4 sound card
      commit: d664e75317e19bb79b6d207f7729e35eca504a6a
[3/5] ASoC: qcom: sm8250: Add Fairphone 4 soundcard compatible
      commit: c58c35ef6ae62e36927f506a5afc66610b7261d9

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
Re: (subset) [PATCH v2 0/5] Enable USB audio offloading on Fairphone 4 smartphone
Posted by Bjorn Andersson 6 months, 3 weeks ago
On Thu, 01 May 2025 08:48:46 +0200, Luca Weiss wrote:
> Since the series for the USB sound offloading driver was finally merged,
> we can add the sm6350 dts and enable it on Fairphone 4.
> 
> A few devicetree binding bits have also been missing in that series, so
> there's some extra patches for the basics in this series.
> 
> Depends on:
> - For qcom,sm8250.yaml & sm8250.c:
>   https://lore.kernel.org/linux-arm-msm/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone.com/T/
> - For dts:
>   https://lore.kernel.org/linux-arm-msm/20250321-sm6350-apr-v1-1-7805ce7b4dcf@fairphone.com/
> 
> [...]

Applied, thanks!

[4/5] arm64: dts: qcom: sm6350: Add q6usbdai node
      commit: 30237c2bfda59e3abd7f7e92879e75bc730edfd8
[5/5] arm64: dts: qcom: sm7225-fairphone-fp4: Enable USB audio offload support
      commit: ddcc0ad9a9b8488d3ee0b4ca6fd9b092c462491f

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>