[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMcHhXqcf5F+02+9XnpXWSvUrkhmfbaAtK9sJtghD4sh9yKA9w@mail.gmail.com>
Date: Wed, 2 Jul 2025 00:04:48 +0200
From: Aleksandrs Vinarskis <alex.vinarskis@...il.com>
To: Val Packett <val@...kett.cool>
Cc: "Bryan O'Donoghue" <bryan.odonoghue@...aro.org>, 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
Subject: Re: [PATCH 0/3] Add x1e Dell Inpsiron 14p
On Tue, 1 Jul 2025 at 23:48, Val Packett <val@...kett.cool> wrote:
>
> On Tue, 2025-07-01 at 22:43 +0200, Aleksandrs Vinarskis wrote:
> >
> > On 4/24/25 01:53, Bryan O'Donoghue 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
> > >
> > > 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>
> >
> > Hi,
> >
> > Just noticed that the device name is misspelled in a few occasions,
> > need
> > to s/inspirion/inspiron/. DT filename is wrong, model name in DT is
> > wrong, one of commit messages is wrong. Firmware paths and compatible
> > in
> > DT are correct.
>
> (BTW, in this submission in particular, firmware paths are not
> consistent with existing models: the XPS uses lowercase 'dell')
>
Good catch
> > Otherwise, is the plan to wait for CAMSS to land, and then land this
> > one, or perhaps the rest of it can go in already? There is also
> > Latitude
> > that was recently submitted which is very similar, perhaps those
> > should
> > be unified (CC: Val), probably easier to do if Inspiron lands first.
> >
>
> I have a unified patchset pretty much ready to go since yesterday.
> With the way I have it, landing any one of the non-unified
> submissions would only increase churn. I'll submit it now.
>
Nice, then my assumption was wrong, ignore it.
Thanks,
Alex
> > Regards,
> > Alex
> >
> > > ---
> > > 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,
Powered by blists - more mailing lists