| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <d0fd03c2-141b-47f5-8133-1b09d40d082c@oss.qualcomm.com>
Date: Thu, 24 Jul 2025 10:50:52 +0100
From: Srinivas Kandagatla <srinivas.kandagatla@....qualcomm.com>
To: Stephan Gerhold <stephan.gerhold@...aro.org>
Cc: andersson@...nel.org, konradybcio@...nel.org, robh@...nel.org,
krzk+dt@...nel.org, conor+dt@...nel.org,
cros-qcom-dts-watchers@...omium.org, linux-arm-msm@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 01/23] arm64: dts: qcom: x1e80100: move dsp audio nodes to
dedicated dts
On 7/24/25 8:23 AM, Stephan Gerhold wrote:
> On Wed, Jul 23, 2025 at 11:27:15PM +0100,
> srinivas.kandagatla@....qualcomm.com wrote:
>> From: Srinivas Kandagatla <srinivas.kandagatla@....qualcomm.com>
>>
>> All the device tree nodes for audioreach dsp are duplicated across all
>> the SoC dtsi files, Move this to a dedicated dtsi file so to remove some
>> duplication, make it consistent across all device trees and also make it
>> easy for new SoC's to add audio support.
>>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@....qualcomm.com>
>> ---
>> .../arm64/boot/dts/qcom/audioreach-audio.dtsi | 45 +++++++++++++++++++
>> .../boot/dts/qcom/x1-asus-zenbook-a14.dtsi | 1 +
>> arch/arm64/boot/dts/qcom/x1-crd.dtsi | 1 +
>> arch/arm64/boot/dts/qcom/x1e001de-devkit.dts | 1 +
>> .../qcom/x1e78100-lenovo-thinkpad-t14s.dtsi | 1 +
>> .../dts/qcom/x1e80100-asus-vivobook-s15.dts | 1 +
>> arch/arm64/boot/dts/qcom/x1e80100-audio.dtsi | 6 +++
>> .../dts/qcom/x1e80100-dell-xps13-9345.dts | 1 +
>> .../dts/qcom/x1e80100-hp-omnibook-x14.dts | 1 +
>> .../dts/qcom/x1e80100-lenovo-yoga-slim7x.dts | 1 +
>> .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 1 +
>> arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 1 +
>> arch/arm64/boot/dts/qcom/x1e80100.dtsi | 43 ++----------------
>> 13 files changed, 64 insertions(+), 40 deletions(-)
>> create mode 100644 arch/arm64/boot/dts/qcom/audioreach-audio.dtsi
>> create mode 100644 arch/arm64/boot/dts/qcom/x1e80100-audio.dtsi
>>
>> [...]
>> diff --git a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
>> index c771fd1d8029..1d8a75fba301 100644
>> --- a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
>> @@ -11,6 +11,7 @@
>> #include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
>> #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>
>> +#include "x1e80100-audio.dtsi"
>> #include "x1e80100-pmics.dtsi"
>>
>> / {
>> diff --git a/arch/arm64/boot/dts/qcom/x1-crd.dtsi b/arch/arm64/boot/dts/qcom/x1-crd.dtsi
>> index c9f0d5052670..19b993fdd17f 100644
>> --- a/arch/arm64/boot/dts/qcom/x1-crd.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/x1-crd.dtsi
>> @@ -9,6 +9,7 @@
>> #include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
>> #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>
>> +#include "x1e80100-audio.dtsi"
>> #include "x1e80100-pmics.dtsi"
>>
>> / {
>> diff --git a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts b/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
>> index 2d9627e6c798..6df3ee553bc7 100644
>> --- a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
>> +++ b/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
>> @@ -10,6 +10,7 @@
>>
>> #include "x1e80100.dtsi"
>> #include "x1e80100-pmics.dtsi"
>> +#include "x1e80100-audio.dtsi"
>>
>> / {
>> model = "Qualcomm Technologies, Inc. X1E001DE Snapdragon Devkit for Windows";
>> diff --git a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi b/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
>> index ac1dddf27da3..cf2cf3e520fa 100644
>> --- a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
>> @@ -13,6 +13,7 @@
>> #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>
>> #include "x1e80100.dtsi"
>> +#include "x1e80100-audio.dtsi"
>> #include "x1e80100-pmics.dtsi"
>>
>> / {
>> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts b/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts
>> index 71b2cc6c392f..82b08350da17 100644
>> --- a/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts
>> +++ b/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts
>> @@ -13,6 +13,7 @@
>>
>> #include "x1e80100.dtsi"
>> #include "x1e80100-pmics.dtsi"
>> +#include "x1e80100-audio.dtsi"
>>
>> / {
>> model = "ASUS Vivobook S 15";
>> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-audio.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-audio.dtsi
>> new file mode 100644
>> index 000000000000..e03441bb2a79
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/x1e80100-audio.dtsi
>> @@ -0,0 +1,6 @@
>> +#include "audioreach-audio.dtsi"
>> +
>> +&q6apmdai{
>> + iommus = <&apps_smmu 0x1001 0x80>,
>> + <&apps_smmu 0x1061 0x0>;
>> +};
>
> I think you should inline this into x1e80100.dtsi and drop the extra
> #include from all the device DTs. x1e80100.dtsi already references
> audioreach nodes, so it's not possible to build without including
> x1e80100-audio.dtsi anyway:
thanks Stephan,
Yes, we can not build it without audio.dtsi, as suggested inlining this
in to end of SoC dtsi should help.
--srini
>
> ../arch/arm64/boot/dts/qcom/x1e80100.dtsi:4098.34-4114.5: ERROR (phandle_references): /soc@...odec@...0000: Reference to non-existent node or label "q6prmcc"
> ../arch/arm64/boot/dts/qcom/x1e80100.dtsi:4320.31-4436.5: ERROR (phandle_references): /soc@...inctrl@...0000: Reference to non-existent node or label "q6prmcc"
>
> Thanks,
> Stephan
Powered by blists - more mailing lists