Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

arm64: dts: qcom: msm8916-lenovo-phab: add device #289

Draft
wants to merge 5 commits into
base: msm8916/6.2-rc5
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions arch/arm64/boot/dts/qcom/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8916-asus-z010d.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-gplus-fl8005a.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-huawei-g7.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-huawei-y635.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-lenovo-phab.dtb
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message: Please drop "not tested" :)

(you can do a reword with interactive rebase fyi)

dtb-$(CONFIG_ARCH_QCOM) += msm8916-lg-c50.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-lg-m216.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-longcheer-l8150.dtb
Expand Down
338 changes: 338 additions & 0 deletions arch/arm64/boot/dts/qcom/msm8916-lenovo-phab.dts
Original file line number Diff line number Diff line change
@@ -0,0 +1,338 @@
// SPDX-License-Identifier: GPL-2.0-only

/dts-v1/;

#include "msm8916-pm8916.dtsi"
#include "msm8916-modem.dtsi"

#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/input/input.h>
#include <dt-bindings/pinctrl/qcom,pmic-mpp.h>
#include <dt-bindings/sound/apq8016-lpass.h>

/ {
model = "Lenovo PHAB";
compatible = "lenovo,phab", "qcom,msm8916";
chassis-type = "tablet";

aliases {
serial0 = &blsp1_uart2;
};

chosen {
stdout-path = "serial0";
};

reserved-memory {
mpss_mem: mpss@86800000 {
reg = <0x0 0x86800000 0x0 0x5500000>;
no-map;
};

gps_mem: gps@8bd00000 {
reg = <0x0 0x8bd00000 0x0 0x200000>;
no-map;
};
};

gpio-keys {
compatible = "gpio-keys";
pinctrl-0 = <&gpio_keys_default>;
pinctrl-names = "default";
label = "GPIO Buttons";

button-volume-up {
label = "Volume Up";
gpios = <&msmgpio 107 GPIO_ACTIVE_LOW>;
linux,code = <KEY_VOLUMEUP>;
debounce-interval = <15>;
};
};

usb_id: usb-id {
compatible = "linux,extcon-usb-gpio";
id-gpios = <&msmgpio 110 GPIO_ACTIVE_HIGH>;
pinctrl-0 = <&usb_id_default>;
pinctrl-names = "default";
};
};

&blsp_i2c1 {
status = "okay";

tas2552: tas2552@40 {
compatible = "ti,tas2552";
reg = <0x40>;
#sound-dai-cells = <0>;
enable-gpio = <&msmgpio 98 GPIO_ACTIVE_HIGH>;
};
};

&blsp_i2c2 {
status = "okay";

accelerometer: accelerometer@68 {
compatible = "invensense,mpu6050";
reg = <0x68>;
interrupt-parent = <&msmgpio>;
interrupts = <115 IRQ_TYPE_EDGE_FALLING>;
pinctrl-0 = <&mpu_default>;
pinctrl-1 = <&mpu_sleep>;
pinctrl-names = "default", "sleep";
vdd-supply = <&pm8916_l17>;
};
};

&blsp_i2c4 {
status = "okay";

charger: charger@57 {
compatible = "summit,smb358";
reg = <0x36>;
interrupt-parent = <&msmgpio>;
interrupts = <62 IRQ_TYPE_LEVEL_LOW>;
pinctrl-0 = <&smb_int_default>;
pinctrl-names = "default";
summit,enable-usb-charging;
summit,enable-mains-charging;
Comment on lines +120 to +121
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if you actually need both here... is "mains" required? and/or is there an example that uses mains that you have took this from? (I'd expect "mains" to mean "mains power" aka 220v or a barrel jack)


usb_vbus: usb-vbus {
regulator-name = "usb_vbus";
regulator-min-microvolt = <5000000>;
regulator-max-microvolt = <5000000>;
regulator-min-microamp = <750000>;
regulator-max-microamp = <750000>;
summit,needs-inok-toggle;
};
};
};

&blsp1_uart2 {
status = "okay";
};

&lpass {
dai@3 {
reg = <MI2S_QUATERNARY>;
qcom,playback-sd-lines = <1>;
};
};

&pm8916_resin {
linux,code = <KEY_VOLUMEDOWN>;
status = "okay";
};

&pm8916_vib {
status = "okay";
};

&pronto {
status = "okay";

iris {
compatible = "qcom,wcn3660b";
};
};
Comment on lines +218 to +224

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A note about new changes since 6.3+: 3244442


&q6afedai {
dai@22 {
reg = <QUATERNARY_MI2S_RX>;
qcom,sd-lines = <1>;
};
};

&sdhc_1 {
pinctrl-0 = <&sdc1_clk_on &sdc1_cmd_on &sdc1_data_on>;
pinctrl-1 = <&sdc1_clk_off &sdc1_cmd_off &sdc1_data_off>;
pinctrl-names = "default", "sleep";
status = "okay";
};

&sdhc_2 {
pinctrl-0 = <&sdc2_clk_on &sdc2_cmd_on &sdc2_data_on &sdc2_cd_on>;
pinctrl-1 = <&sdc2_clk_off &sdc2_cmd_off &sdc2_data_off &sdc2_cd_off>;
pinctrl-names = "default", "sleep";
cd-gpios = <&msmgpio 38 GPIO_ACTIVE_LOW>;
status = "okay";
};

&sound {
model = "msm8916";
k1gen marked this conversation as resolved.
Show resolved Hide resolved
pinctrl-0 = <&cdc_pdm_lines_act &ext_sec_tlmm_lines_act>;
pinctrl-1 = <&cdc_pdm_lines_sus &ext_sec_tlmm_lines_sus>;
pinctrl-names = "default", "sleep";
qcom,audio-routing =
"RX_BIAS", "MCLK",
"SPK_RX_BIAS", "MCLK",
"INT_LDO_H", "MCLK",
"MIC BIAS External", "Handset Mic",
"MIC BIAS Internal2", "Headset Mic",
"MIC BIAS External", "Secondary Mic",
"AMIC1", "MIC BIAS External",
"AMIC2", "MIC BIAS Internal2",
"AMIC3", "MIC BIAS External",
"DMIC1", "MIC BIAS Internal1",
"MIC BIAS Internal1", "Digital Mic1",
"DMIC2", "MIC BIAS Internal1",
"MIC BIAS Internal1", "Digital Mic2";
status = "okay";
};

&sound_dai_quaternary {
status = "okay";

codec {
sound-dai = <&tas2552>;
};
};

&usb {
extcon = <&charger>, <&usb_id>;
vbus-supply = <&usb_vbus>;
status = "okay";
};

&usb_hs_phy {
extcon = <&charger>;
};

&smd_rpm_regulators {
vdd_l1_l2_l3-supply = <&pm8916_s3>;
vdd_l4_l5_l6-supply = <&pm8916_s4>;
vdd_l7-supply = <&pm8916_s4>;

s3 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1300000>;
};

s4 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <2100000>;
};

l1 {
regulator-min-microvolt = <1225000>;
regulator-max-microvolt = <1225000>;
};

l2 {
regulator-min-microvolt = <1200000>;
regulator-max-microvolt = <1200000>;
};

l4 {
regulator-min-microvolt = <2050000>;
regulator-max-microvolt = <2050000>;
};

l5 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};

l6 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};

l7 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
};

l8 {
regulator-min-microvolt = <2850000>;
regulator-max-microvolt = <2900000>;
};

l9 {
regulator-min-microvolt = <3300000>;
regulator-max-microvolt = <3300000>;
};

l10 {
regulator-min-microvolt = <2700000>;
regulator-max-microvolt = <2800000>;
};

l11 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <2950000>;
regulator-allow-set-load;
regulator-system-load = <200000>;
};

l12 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <2950000>;
};

l13 {
regulator-min-microvolt = <3075000>;
regulator-max-microvolt = <3075000>;
};

l14 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
};

l15 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
};

l16 {
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
};

l17 {
regulator-min-microvolt = <2850000>;
regulator-max-microvolt = <2850000>;
};

l18 {
regulator-min-microvolt = <2700000>;
regulator-max-microvolt = <2700000>;
};
};

&msmgpio {
usb_id_default: usb-id-default-state {
k1gen marked this conversation as resolved.
Show resolved Hide resolved
pins = "gpio110";
function = "gpio";
drive-strength = <8>;
bias-pull-up;
};

gpio_keys_default: gpio-keys-default-state {
pins = "gpio107";
function = "gpio";
drive-strength = <2>;
bias-pull-up;
};

smb_int_default: smb-int-default-state {
pins = "gpio62";
function = "gpio";

drive-strength = <2>;
bias-pull-up;
};

mpu_default: mpu-default-state {
k1gen marked this conversation as resolved.
Show resolved Hide resolved
pins = "gpio115";
function = "gpio";
drive-strength = <6>;
bias-pull-up;
};

mpu_sleep: mpu-sleep-state {
pins = "gpio115";
function = "gpio";
drive-strength = <2>;
};
};