Re: [PATCH v2] arm64: dts: qcom: Add initial audio support for Hamoa-IOT-EVK

[Quill Qi]

From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Sent: Thursday, August 14, 2025 4:41 PM
To: Quill Qi <leqi@xxxxxxxxxxxxxxxx>; Konrad Dybcio <konradybcio@xxxxxxxxxx>; Bjorn Andersson <andersson@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>
Cc: linux-arm-msm@xxxxxxxxxxxxxxx <linux-arm-msm@xxxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx <devicetree@xxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH v2] arm64: dts: qcom: Add initial audio support for Hamoa-IOT-EVK


>On 28/07/2025 13:08, Krzysztof Kozlowski wrote:
>>On 28/07/2025 09:16, leqi via B4 Relay wrote:
>>>From: leqi <leqi@xxxxxxxxxxxxxxxx>
>>>
>>>This patch adds initial audio codec support for the Hamoa-IOT-EVK board,
>>>including WCD9385 configuration, micbias voltage settings, GPIO reset,
>>>and power supply bindings. It enables basic audio functionality for
>>>further development. Basic test is good in Hamoa-IOT-EVK board.
>>>
>>>Signed-off-by: leqi <leqi@xxxxxxxxxxxxxxxx>
>>>---
>>>Changes in v2:
>>>- Updated author email address to leqi@xxxxxxxxxxxxxxxx.
>>>- Clarified that audio is validated with this change.
>>>- Link to v1: https://lore.kernel.org/all/20250723-initial_audio_support_for_qualcomm_hamoa_iot_evk_board-v1-1-816991701952@xxxxxxxxxxx/
>>>---
>>> arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts | 232 +++++++++++++++++++++++++++++
>>> 1 file changed, 232 insertions(+)
>>>
>>>diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
>>>index 843f39c9d59286a9303a545411b2518d7649a059..91618e22e86c46c698b3639f60bc19314705b391 100644
>>>--- a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
>>>+++ b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
>>>@@ -124,6 +124,94 @@ pmic_glink_ss2_con_sbu_in: endpoint {
>>
>>
>>This was not merged, was it? Same comment as other patch, when you have
>>entire code ready send entire board. Not chunk by chunk.
>>
>>You are not following properly release early, release often.
>
>Why this is still not included in initial submission which happens now?
>Either you work Linux style (release early) or, if you decide to wait
>till everything is ready, you submit board as one patch, not everything
>as 100 different patches.

Hi Krzysztof, this audio change initial test is good with sound output and record fine. But with further test in Ubuntu gui only
2 speakers have sound output. I am still seeing why this issue is happening. I thought to solve the issues before replying
and merging this, thanks.

>Best regards,
>Krzysztof

--
BR
Quill