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: <de111b27-9126-4c03-a7bb-8cce9ea2780e@oss.qualcomm.com>
Date: Fri, 27 Jun 2025 21:48:27 +0200
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: Pengyu Luo <mitltlatltl@...il.com>,
        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>, Eric Biggers <ebiggers@...gle.com>,
        Len Brown <len.brown@...el.com>, Benno Lossin <lossin@...nel.org>,
        Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
        Ingo Molnar <mingo@...nel.org>, Miguel Ojeda <ojeda@...nel.org>,
        Joel Granados <joel.granados@...nel.org>,
        Al Viro <viro@...iv.linux.org.uk>, Alice Ryhl <aliceryhl@...gle.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, Hong Zhu <vanyang@...il.nju.edu.cn>
Subject: Re: [PATCH 3/4] arm64: dts: qcom: sc8280xp: Add initial support for
 Ntmer TW220

On 6/17/25 11:29 AM, Pengyu Luo wrote:
> The Ntmer TW220 is a WOS tablet based on the Qualcomm SC8280XP platform,
> also known as the Robo&Kala 2-in-1 Laptop. Thanks to Hong for providing
> the unlocked device and early development work. This patch adds an
> initial device tree to enable basic functionality.
> 
> Currently supported components include:
> - Bluetooth & Wi-Fi (board file regeneration required)
> - Battery charging (up to 15V/3A fixed PDO) and reporting via pmic-glink
> - Flash LEDs (front and rear)
> - Hall sensor (lid detection)
> - Keyboard (via Bluetooth or USB)
> - NVMe SSD
> - Power and volume keys
> - Simple-framebuffer
> - Sound (playback and capture; top-left DMIC only, top-right works only
>   on Windows)
> - Touchscreen and stylus (requires GPI DMA support [1] and stylus support [2])
> - USB Type-C ports
> 
> The following components are currently non-functional:
> - Cameras (GalaxyCore GC5035; only sensor ID is detectable, no frames in libcamera;
>   partial driver can be found on LKML archives)
> - DSI display (blank screen with `dsi_err_worker: status=4`; primary DSI register
>   dump included below)
> - Stylus wireless charger (CPS4035)
> - UCSI over GLINK
> 
> [1]: https://lore.kernel.org/linux-arm-msm/20250617090032.1487382-3-mitltlatltl@gmail.com
> [2]: https://lore.kernel.org/linux-input/20250605054855.403487-2-mitltlatltl@gmail.com
> 
> Note: This series does **not** include any confidential material. Those
> who wish to run Linux on this device should contact Ntmer, as the
> bootloader is locked via secure boot.
> 
> Co-developed-by: Hong Zhu <vanyang@...il.nju.edu.cn>
> Signed-off-by: Hong Zhu <vanyang@...il.nju.edu.cn>
> Signed-off-by: Pengyu Luo <mitltlatltl@...il.com>
> 
> dsi_ctrl, reg = <0 0x0ae94000 0 0x400>;
> 0xae94000 20050001 000001f3 0000000b dddd1011

This is not something we want in the commit log

[...]

> +	gpio-leds {
> +		compatible = "gpio-leds";
> +
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&camf_indicator_en>, <&camr_indicator_en>;

property-n
property-names

in that order, across the file, please

[...]

> +		wsa-dai-link {
> +			link-name = "WSA Playback";
> +
> +			cpu {
> +				sound-dai = <&q6apmbedai WSA_CODEC_DMA_RX_0>;
> +			};
> +
> +			codec {

'co'dec < 'cp'u

[...]

> +/*
> + * cci0_i2c1
> + * sda: gpio115, scl: gpio116
> + *
> + * CAMI ov9234 @36 @48
> + *
> + * power on sequence
> + * gpio7 out low
> + * l3q 1p8
> + * l6q 2p8
> + * l2q 1p2
> + * gpio7 out high
> + * msleep 5
> + * cam_cc_mclk4_clk 2.4MHz (gpio6)
> + */

It would be useful to enable these buses and set up what we can,
otherwise this is quite a lot of text for comments..

[...]

> +&spi22 {
> +	status = "okay";
> +	pinctrl-0 = <&spi22_default>;
> +	pinctrl-names = "default";

status should be the last property (before subnodes), preferably
with a newline before it, so:

pinctrl-0 = <&spi22_default>;
pinctrl-names = "default";

status = "okay";


> +
> +	touchscreen@0 {
> +		/*
> +		 * The ACPI device ID is GXTS7986, its exact suffix is unknown.
> +		 * The Windows driver suggests it is a GTBerlinB variant and
> +		 * communicates via HID over SPI, which aligns with the Linux
> +		 * driver `drivers/hid/hid-goodix-spi.c`.
> +		 *
> +		 * However, the HID descriptor read from the device appears
> +		 * garbled, preventing proper probe with the HID driver. In
> +		 * contrast, the driver at
> +		 * `drivers/input/touchscreen/goodix_berlin_spi.c` shares many
> +		 * similarities and functions correctly with this hardware.
> +		 *
> +		 * Therefore, we choose to use the goodix_berlin_spi driver
> +		 * instead.

Is this something you could work out with the aforementioned drivers'
maintainers?

[...]

> +&pcie4_port0 {
> +	wifi@0 {
> +		compatible = "pci17cb,1103";
> +		reg = <0x10000 0x0 0x0 0x0 0x0>;
> +
> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn_0p8>;
> +		vddaon-supply = <&vreg_pmu_aon_0p8>;
> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p8>;
> +		vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
> +		vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +
> +		/*
> +		 * bus=pci,vendor=17cb,device=1103,subsystem-vendor=17cb,
> +		 * subsystem-device=0108,qmi-chip-id=18,qmi-board-id=255
> +		 *
> +		 * Regenerate board file, x13s one works well

Please post on the ath11k mailing list and propose and ask for
that variant to be included

Konrad

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ