| 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
| ||
|
Date: Tue, 24 Jan 2023 15:03:17 +0100
From: Konrad Dybcio <konrad.dybcio@...aro.org>
To: Jianhua Lu <lujianhua000@...il.com>,
Andy Gross <agross@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Kees Cook <keescook@...omium.org>,
Tony Luck <tony.luck@...el.com>,
"Guilherme G . Piccoli" <gpiccoli@...lia.com>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-hardening@...r.kernel.org,
~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org
Subject: Re: [PATCH 2/2] arm64: dts: qcom: sm8250: Add device tree for Xiaomi
Mi Pad 5 Pro
On 24.01.2023 14:53, Jianhua Lu wrote:
> Add support for Xiaomi Mi Pad 5 Pro, codename is xiaomi-elish.
>
> This commit brings support for:
> * ADSP/CDSP/SLPI/VENUS
> * Backlight
> * Battery fuel gauge
> * Framebuffer
> * PCIe0
> * USB2.0
>
> Note backlight driver (ktz8866) is waitting for upstreaming[1].
> [1] https://lore.kernel.org/linux-leds/20230120155018.15376-1-lujianhua000@gmail.com
>
> Signed-off-by: Jianhua Lu <lujianhua000@...il.com>
> ---
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/sm8250-xiaomi-elish.dts | 594 ++++++++++++++++++
> 2 files changed, 595 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish.dts
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index b0423ca3e79f..769cee2b450f 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -184,6 +184,7 @@ dtb-$(CONFIG_ARCH_QCOM) += sm8250-hdk.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8250-mtp.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8250-sony-xperia-edo-pdx203.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8250-sony-xperia-edo-pdx206.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += sm8250-xiaomi-elish.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8350-hdk.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8350-microsoft-surface-duo2.dtb
> dtb-$(CONFIG_ARCH_QCOM) += sm8350-mtp.dtb
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish.dts
> new file mode 100644
> index 000000000000..99ae6668e516
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish.dts
> @@ -0,0 +1,594 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) 2022, 2023 Jianhua Lu <lujianhua000@...il.com>
> + */
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +#include "sm8250.dtsi"
> +#include "pm8150.dtsi"
> +#include "pm8150b.dtsi"
> +#include "pm8150l.dtsi"
> +#include "pm8009.dtsi"
> +
> +/*
> + * Delete following upstream (sm8250.dtsi) reserved
> + * memory mappings which are different in this device.
> + */
> +/delete-node/ &xbl_aop_mem;
> +/delete-node/ &slpi_mem;
> +/delete-node/ &adsp_mem;
> +/delete-node/ &spss_mem;
> +/delete-node/ &cdsp_secure_heap;
> +
> +/ {
> + model = "Xiaomi Mi Pad 5 Pro";
> + compatible = "xiaomi,elish", "qcom,sm8250";
> + classis-type = "tablet";
> +
> + /* required for bootloader to select correct board */
> + qcom,msm-id = <356 0x20001>; /* SM8250 v2.1 */
Please include include/dt-bindings/arm/qcom,ids.h and use
the QCOM_ID_SM8250 define.
> + qcom,board-id = <0x10008 0>;
> +
> + chosen {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + framebuffer: framebuffer@...00000 {
> + compatible = "simple-framebuffer";
> + reg = <0 0x9c000000 0 0x2300000>;
Please use 0x0 instead of 0, like in other nodes.
> + width = <1600>;
> + height = <2560>;
> + stride = <(1600 * 4)>;
> + format = "a8r8g8b8";
> + };
> + };
> +
> + gpio_keys: gpio-keys {
> + compatible = "gpio-keys";
You may want to add the pin settings for pm8150_gpio6 here.
> +
> + vol-up {
> + label = "Volume Up";
> + gpios = <&pm8150_gpios 6 GPIO_ACTIVE_LOW>;
> + linux,code = <KEY_VOLUMEUP>;
> + debounce-interval = <15>;
> + linux,can-disable;
> + gpio-key,wakeup;
> + };
> + };
> +
> + bl_vddpos_5p5: bl-vddpos-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "bl_vddpos_5p5";
> + regulator-min-microvolt = <5500000>;
> + regulator-max-microvolt = <5500000>;
> + regulator-enable-ramp-delay = <233>;
> + gpio = <&tlmm 130 0>;
> + enable-active-high;
> + regulator-boot-on;
> + };
> +
> + bl_vddneg_5p5: bl-vddneg-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "bl_vddneg_5p5";
> + regulator-min-microvolt = <5500000>;
> + regulator-max-microvolt = <5500000>;
> + regulator-enable-ramp-delay = <233>;
> + gpio = <&tlmm 131 0>;
> + enable-active-high;
> + regulator-boot-on;
> + };
> +
> + vph_pwr: vph-pwr-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "vph_pwr";
> + regulator-min-microvolt = <3700000>;
> + regulator-max-microvolt = <3700000>;
> + };
> +
> + /* S6c is really ebi.lvl but it's there for supply map completeness sake. */
> + vreg_s6c_0p88: smpc6-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "vreg_s6c_0p88";
> + regulator-min-microvolt = <880000>;
> + regulator-max-microvolt = <880000>;
> + regulator-always-on;
> + vin-supply = <&vph_pwr>;
> + };
> +
> + reserved-memory {
> + xbl_aop_mem: memory@...00000 {
> + reg = <0x0 0x80600000 0x0 0x260000>;
> + no-map;
> + };
> +
> + slpi_mem: memory@...00000 {
> + reg = <0x0 0x88c00000 0x0 0x2f00000>;
> + no-map;
> + };
> +
> + adsp_mem: memory@...00000 {
> + reg = <0x0 0x8bb00000 0x0 0x2500000>;
> + no-map;
> + };
> +
> + spss_mem: memory@...00000 {
> + reg = <0x0 0x8e000000 0x0 0x100000>;
> + no-map;
> + };
> +
> + cdsp_secure_heap: memory@...00000 {
> + reg = <0x0 0x8e100000 0x0 0x4600000>;
> + no-map;
> + };
> +
> + cont_splash_mem: memory@...00000 {
> + reg = <0x0 0x9c000000 0x0 0x2300000>;
> + no-map;
> + };
> +
> + ramoops@...00000 {
> + compatible = "ramoops";
> + reg = <0x0 0xb0000000 0x0 0x400000>;
> + record-size = <0x1000>;
> + console-size = <0x200000>;
> + ecc-size = <16>;
> + no-map;
> + };
> + };
> +
> + battery_l: battery-l {
Move these up to keep the nodes sorted alphabetically.
> + compatible = "simple-battery";
> + voltage-min-design-microvolt = <3870000>;
> + energy-full-design-microwatt-hours = <16600000>;
> + charge-full-design-microamp-hours = <4300000>;
> + };
> +
> + battery_r: battery-r {
> + compatible = "simple-battery";
> + voltage-min-design-microvolt = <3870000>;
> + energy-full-design-microwatt-hours = <16600000>;
> + charge-full-design-microamp-hours = <4300000>;
> + };
> +};
> +
[...]
> +
> +&i2c0 {
> + clock-frequency = <400000>;
> + status = "okay";
> +
> + battery_fg@55 {
fuel-gauge@
> + compatible = "ti,bq27z561";
> + reg = <0x55>;
> + monitored-battery = <&battery_r>;
> + };
> +};
> +
> +&i2c11 {
> + clock-frequency = <400000>;
> + status = "okay";
> +
> + backlight: backlight@11 {
> + compatible = "kinetic,ktz8866";
> + reg = <0x11>;
> + vddpos-supply = <&bl_vddpos_5p5>;
> + vddneg-supply = <&bl_vddneg_5p5>;
> + enable-gpios = <&tlmm 139 GPIO_ACTIVE_HIGH>;
> + current-num-sinks = <5>;
> + kinetic,current-ramp-delay-ms = <128>;
> + kinetic,led-enable-ramp-delay-ms = <1>;
> + kinetic,enable-lcd-bias;
> + };
> +};
> +
> +&i2c13 {
> + clock-frequency = <400000>;
> + status = "okay";
> +
> + battery_fg@55 {
fuel-gauge@
> + compatible = "ti,bq27z561";
> + reg = <0x55>;
> + monitored-battery = <&battery_l>;
> + };
> +};
> +
> +&pcie0 {
> + status = "okay";
> +};
> +
> +&pcie0_phy {
> + vdda-phy-supply = <&vreg_l5a_0p88>;
> + vdda-pll-supply = <&vreg_l9a_1p2>;
> + status = "okay";
> +};
> +
> +&pon_pwrkey {
> + status = "okay";
> +};
> +
> +&pon_resin {
> + linux,code = <KEY_VOLUMEDOWN>;
> + status = "okay";
> +};
> +
> +&qupv3_id_0 {
> + status = "okay";
> +};
> +
> +&qupv3_id_1 {
> + status = "okay";
> +};
> +
> +&qupv3_id_2 {
> + status = "okay";
> +};
> +
> +&slpi {
> + firmware-name = "qcom/sm8250/xiaomi/elish/slpi.mbn";
> + status = "okay";
> +};
> +
> +&tlmm {
> + gpio-reserved-ranges = <40 4>;
> +};
> +
> +&usb_1 {
> + /* USB 2.0 only */
> + qcom,select-utmi-as-pipe-clk;
> + status = "okay";
> +};
> +
> +&usb_1_dwc3 {
> + dr_mode = "peripheral";
> + maximum-spped = "high-speed";
> + /* Remove USB3 phy */
> + phys = <&usb_1_hsphy>;
> + phy-names = "usb2-phy";
> +};
> +
> +&usb_1_hsphy {
> + vdda-pll-supply = <&vreg_l5a_0p88>;
> + vdda18-supply = <&vreg_l12a_1p8>;
> + vdda33-supply = <&vreg_l2a_3p1>;
> + status = "okay";
> +};
> +
> +&ufs_mem_hc {
> + vcc-supply = <&vreg_l17a_3p0>;
> + vcc-max-microamp = <800000>;
> + vccq-supply = <&vreg_l6a_1p2>;
> + vccq-max-microamp = <800000>;
> + vccq2-supply = <&vreg_s4a_1p8>;
> + vccq2-max-microamp = <800000>;
I think you may want to add regulator-allow-set-load on
these regulators.
Konrad
> + status = "okay";
> +};
> +
> +&ufs_mem_phy {
> + vdda-phy-supply = <&vreg_l5a_0p88>;
> + vdda-pll-supply = <&vreg_l9a_1p2>;
> + status = "okay";
> +};
> +
> +&venus {
> + firmware-name = "qcom/sm8250/elish/venus.mbn";
> + status = "okay";
> +};
Powered by blists - more mailing lists