linux-kernel - Re: [PATCH 2/2] arm64: dts: qcom: sm8150: Add support for Xiaomi Redmi K20 Pro

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: <5C7DC3D9-8DBB-409C-8672-6388EE01C320@mainlining.org>
Date: Tue, 28 Oct 2025 21:43:43 +0530
From: Piyush Raj Chouhan <pc1598@...nlining.org>
To: Konrad Dybcio <konrad.dybcio@....qualcomm.com>, linux-arm-msm@...r.kernel.org
CC: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-hardening@...r.kernel.org, andersson@...nel.org,
 konradybcio@...nel.org, robh@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, tony.luck@...el.com, gpiccoli@...lia.com
Subject: Re: [PATCH 2/2] arm64: dts: qcom: sm8150: Add support for Xiaomi Redmi K20 Pro

Hi, konrad 


>> +
>> +&i2c19 {
>> +	/* goodix,gt9886 @5d  */
>
>I see there's driver support for the GT98*97* - is there a chance you can
>reuse some of it?
>

I am currently testing touch on gtx8 driver which currently is in lkml, once its merged we could add support for it.

https://lore.kernel.org/linux-input/20250918-gtx8-v1-0-cba879c84775@mainlining.org

[...]
>> +	connector {
>> +		compatible = "usb-c-connector";
>> +		power-role = "source";
>> +		data-role = "dual";
>> +		self-powered;
>> +
>> +		source-pdos = <PDO_FIXED(5000, 3000, PDO_FIXED_DUAL_ROLE |
>> +					 PDO_FIXED_USB_COMM | PDO_FIXED_DATA_SWAP)>;
>
>Have you verified this is in line with your specific device's
>downstream kernel?

Yes i have verified it.

[...]
>> +&pm8150l_lpg {
>> +
>> +	status = "okay";
>
>ditto> +	led@1 {
>> +		reg = <1>;
>> +		color = <LED_COLOR_ID_WHITE>;
>> +		function = LED_FUNCTION_STATUS;
>> +
>> +		status = "disabled";
>
>?

Raphael doesn’t have a white LED. I missed cleaning that. I’ll remove it in the next revision.

>
>> +	};
>> +
>> +
>> +	led@2 {
>> +		reg = <2>;
>> +		color = <LED_COLOR_ID_RED>;
>> +		function = LED_FUNCTION_STATUS;
>> +		function-enumerator = <0>;
>> +	};
>> +
>> +	led@3 {
>> +		reg = <3>;
>> +		color = <LED_COLOR_ID_RED>;
>> +		function = LED_FUNCTION_STATUS;
>> +		function-enumerator = <1>;
>> +	};
>
>Are there really two separate red LEDs?

Yes, it has popup camera with an LED on both side, i have verified functionality of both LEDs. 


[...]
>
>> +&usb_1_dwc3 {
>> +	dr_mode = "otg";
>> +	maximum-speed = "high-speed";
>> +
>> +	/* Remove USB3 phy */
>> +	phys = <&usb_1_hsphy>;
>> +	phy-names = "usb2-phy";
>
>Is this a physical limitation, i.e. missing wires?

Yes it is a physical limitation and downstream configured it similarly,

[...]

Thanks for all the rest of improvement suggestions.

Best regards
Piyush Raj Chouhan