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
