Re: [PATCH v4 3/3] arm64: dts: qcom: Add support for X1-based Dell XPS 13 9345

[Aleksandrs Vinarskis]

On Tue, 8 Oct 2024 at 23:09, Stefan Schmidt <stefan.schmidt@xxxxxxxxxx> wrote:
>
> Hello Aleksandrs,
>
> [Again in plain text]
>
> On Thu, 3 Oct 2024 at 23:13, Aleksandrs Vinarskis
> <alex.vinarskis@xxxxxxxxx> wrote:
> >
> > Initial support for Dell XPS 9345 13" 2024 (Codenamed 'Tributo') based
> > on X1E80100.
> >
> > Working:
> > * Touchpad
> > * Keyboard (only post suspend&resume, i2c-hid patch required [1])
> > * Touchscreen
> > * eDP (low-res IPS, OLED) with brightness control
> > * NVME
> > * USB Type-C ports in USB2/USB3 (one orientation)
> > * WiFi
> > * GPU/aDSP/cDSP firmware loading (requires binaries from Windows)
> > * Lid switch
> > * Sleep/suspend, nothing visibly broken on resume
> >
> > Not working:
> > * Speakers (WIP, pin guessing, x4 WSA8845)
> > * Microphones (WIP, pin guessing, dual array)
> > * Fingerprint Reader (WIP, USB MP with ptn3222)
> > * USB as DP/USB3 (WIP, PS8830 based)
> > * Camera (Likely OV01A10)
> > * EC over i2c
> >
> > Should be working, but cannot be tested due to lack of hw:
> > * higher res IPS panel
> >
> > [1] https://lore.kernel.org/all/20240925100303.9112-1-alex.vinarskis@xxxxxxxxx/
> >
> > Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@xxxxxxxxx>
> > ---
> >  arch/arm64/boot/dts/qcom/Makefile             |   1 +
> >  .../dts/qcom/x1e80100-dell-xps13-9345.dts     | 863 ++++++++++++++++++
> >  2 files changed, 864 insertions(+)
> >  create mode 100644 arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts
>
> Tested-by: Stefan Schmidt <stefan.schmidt@xxxxxxxxxx>

Hi,

Thanks for taking your time to test this.
For the record, which variant did you test it on (low res
IPS/OLED/high res IPS)?

Regards,
Alex

>
> regards
> Stefan Schmidt