linux-kernel - Re: [PATCH 2/7] ARM: dts: qcom: add common device tree for MSM8x26-based Lumia phones

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]

Date:   Sat, 12 Aug 2023 01:02:17 +0200
From:   Konrad Dybcio <konrad.dybcio@...aro.org>
To:     Rayyan Ansari <rayyan@...ari.sh>, linux-arm-msm@...r.kernel.org
Cc:     ~postmarketos/upstreaming@...ts.sr.ht,
        Dominik Kobinski <dominikkobinski314@...il.com>,
        Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>,
        Jack Matthews <jm5112356@...il.com>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        linux-kernel@...r.kernel.org, Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH 2/7] ARM: dts: qcom: add common device tree for
 MSM8x26-based Lumia phones

On 11.08.2023 23:35, Rayyan Ansari wrote:
> Add a common device tree for Lumias based on the Qualcomm MSM8x26
> family of chipsets.
> 
> Currently supports:
> - Framebuffer
> - Touchscreen
> - Keys
> - Regulators
> - MMC
> - USB
> - UART
> 
> Co-authored-by: Dominik Kobinski <dominikkobinski314@...il.com>
> Co-authored-by: Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>
> Co-authored-by: Jack Matthews <jm5112356@...il.com>
> Signed-off-by: Dominik Kobinski <dominikkobinski314@...il.com>
> Signed-off-by: Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>
> Signed-off-by: Jack Matthews <jm5112356@...il.com>
> Signed-off-by: Rayyan Ansari <rayyan@...ari.sh>
> ---
Would be really nice if you could squash it with patch 3 so that
this code is immediately compilable.

[...]

> +	gpio_keys: gpio-keys {
> +		compatible = "gpio-keys";
> +
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&gpio_keys_active>;
property-n
property-names

[...]

> +	reserved-memory {
> +		ranges;
That's already there in soc dtsi

> +
> +		smem_region: smem@...0000 {
Swap the subnodes to keep the nodes ordered by unit address
> +			reg = <0xfa00000 0x100000>;
> +			no-map;
> +		};
> +
> +		display_reserved: framebuffer@...0000 {
> +			reg = <0x3200000 0x800000>;
> +			no-map;
> +		};
> +	};
> +};
> +
> +&blsp1_i2c5 {
> +	status = "okay";
> +
> +	touchscreen@4b {
> +		compatible = "syna,rmi4-i2c";
> +		reg = <0x4b>;
> +
> +		interrupts-extended = <&tlmm 17 IRQ_TYPE_EDGE_FALLING>;
> +		vdd-supply = <&pm8226_l15>;
> +		vio-supply = <&pm8226_l6>;
> +
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&touchscreen_on>;
ditto

> +
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		rmi4-f01@1 {
> +			reg = <0x01>;
> +			syna,nosleep-mode = <1>;
> +		};
> +
> +		rmi4-f11@11 {
> +			reg = <0x11>;
> +			syna,sensor-type = <1>;
> +		};
> +	};
> +};
[...]

> +
> +&tlmm {
> +	gpio_keys_active: gpio-keys-active {
Run `make CHECK_DTBS=1 qcom/msm8226-something-lumia` and fix the warnings
before Krzysztof gets angry :D

Konrad