linux-kernel - Re: [PATCH 0/3] Add x1e Dell Inpsiron 14p

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAF6AEGtiMLQ6xWrmbkmdkpb7gsdbkXaFw7V84nXDDWtLA=4f9A@mail.gmail.com>
Date: Thu, 24 Apr 2025 08:42:04 -0700
From: Rob Clark <robdclark@...il.com>
To: "Bryan O'Donoghue" <bryan.odonoghue@...aro.org>
Cc: Bjorn Andersson <andersson@...nel.org>, Konrad Dybcio <konradybcio@...nel.org>, 
	Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Rob Clark <robdclark@...omium.org>
Subject: Re: [PATCH 0/3] Add x1e Dell Inpsiron 14p

On Wed, Apr 23, 2025 at 4:53 PM Bryan O'Donoghue
<bryan.odonoghue@...aro.org> wrote:
>
> Add in a dtsi for the Dell Inspiron 14p.
>
> I'm currently using this as a daily driver including sending this series
> from. Its reasonably stable on 6.15-rcX.
>
> The first two patches can be applied without dependency, the final patch
> relies on patches staged for -next in the media tree to be applied,
> presently.
>
> https://gitlab.freedesktop.org/linux-media/media-committers/-/commit/2ab7f87a7f4bf392e3836a2600f115a1baa1415c
> https://lore.kernel.org/linux-media/20250407-b4-media-comitters-next-25-03-13-ov02e10-v4-0-211e3e6fae90@linaro.org/
>
> Working for me included in this series:
>
> - USB
>   Both Type-C USB ports
>   Type-A port
>   Fingerprint reader
> - WiFi
> - Bluetooth
> - RGB Camera
> - Toucpad, keyboard
> - Display
>
> Working for me but not included:
> - Audio jack
> - Iris
>
> Not working:
> - Speaker output
>   Still WiP haven't touched this in a while
>
> - Battery
>   Dell has its own Embedded Controller likely from the x86 platform reused
>   on Qcom which we need to reverse engineer or get some information on to
>   make faster progress on.
>
> - cDSP
>   Haven't tried to bring this up.
>
> - EVA
>   No driver haven't tried it.
>
> - Bugs
>   Occasionally when resuming I get a fencing error with hyperlock and
>   freedreno, TBH it looks like something in the GPU or SMMU according to
>   Rob Clark: https://pastebin.com/AWjCyaap

Did you have the same issues on v6.14 or earlier?

The smmu pm usage count underflow is concerning, the smmu pm should be
tied to the GPU device via device_link_add().  I've never seen
something similar, but I'm still on v6.14.

Could you try reverting commit 0a679336dc17 ("iommu/arm-smmu: Set rpm
auto_suspend once during probe")

BR,
-R

>
>   Ath12k has been splatting for me up to 6.14 when removing a SSID.
>   I switched on ath12k debug when going to 6.15 and predictably couldn't
>   reproduce the bug, either because the timings have changed as a result
>   of Heisenbugging or because a fix has been slipped into ath12k.
>
>   Other than those two I'm pretty happy with this system.
>
>   The DTS is based on Aleksandrs Vinarskis XPS, Lenovo T14s and Qcom CRD.
>
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> ---
> Bryan O'Donoghue (3):
>       dt-bindings: arm: qcom: Add Dell Inspiron 14 Plus 7441
>       arm64: dts: qcom: Add support for X1E80100 Dell Inspirion 14 Plus 7441
>       arm64: dts: qcom: x1e80100-dell-inspiron14-7441: Switch on CAMSS RGB sensor
>
>  Documentation/devicetree/bindings/arm/qcom.yaml    |    1 +
>  arch/arm64/boot/dts/qcom/Makefile                  |    1 +
>  .../qcom/x1e80100-dell-inspirion-14-plus-7441.dts  | 1490 ++++++++++++++++++++
>  3 files changed, 1492 insertions(+)
> ---
> base-commit: f7570505263aff2b63142f0d68fa607cd60eb060
> change-id: 20250417-qcom-linux-arm64-for-6-16-dell-inspiron14p-ed68cd65ebad
>
> Best regards,
> --
> Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>
>