Re: [PATCH v2 2/2] arm64: dts: qcom: msm8939-longcheer-l9100: Add initial device tree
From: André Apitzsch
Date: Wed Jul 26 2023 - 01:18:25 EST
Am Dienstag, dem 25.07.2023 um 22:05 +0200 schrieb André Apitzsch:
> This dts adds support for BQ Aquaris M5 (Longcheer L9100) released in
> 2015.
>
> Add a device tree with initial support for:
>
> - GPIO keys
> - Hall sensor
> - SDHCI
> - WCNSS (BT/WIFI)
> - Accelerometer/Magnetometer
> - Vibrator
> - Touchscreen
> - Front flash
>
> Signed-off-by: André Apitzsch <git@xxxxxxxxxxx>
> ---
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/msm8939-longcheer-l9100.dts | 336
> +++++++++++++++++++++
> 2 files changed, 337 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile
> b/arch/arm64/boot/dts/qcom/Makefile
> index 23fd31d4bf5a..3200bc92853f 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -39,6 +39,7 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8916-thwc-
> uf896.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8916-thwc-ufi001c.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8916-wingtech-wt88047.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8916-yiming-uz801v3.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += msm8939-longcheer-l9100.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8939-samsung-a7.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8939-sony-xperia-kanuti-
> tulip.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8953-motorola-potter.dtb
> diff --git a/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
> b/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
> new file mode 100644
> index 000000000000..d8e37bdb4916
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/msm8939-longcheer-l9100.dts
> @@ -0,0 +1,336 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +
> +/dts-v1/;
> +
> +#include "msm8939-pm8916.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +#include <dt-bindings/leds/common.h>
> +#include <dt-bindings/pinctrl/qcom,pmic-mpp.h>
> +
> +/ {
> + model = "BQ Aquaris M5 (Longcheer L9100)";
> + compatible = "longcheer,l9100", "qcom,msm8939";
> + chassis-type = "handset";
> +
> + aliases {
> + mmc0 = &sdhc_1; /* eMMC */
> + mmc1 = &sdhc_2; /* SD card */
> + serial0 = &blsp_uart2;
> + };
> +
> + chosen {
> + stdout-path = "serial0";
> + };
> +
> + gpio-hall-sensor {
> + compatible = "gpio-keys";
> +
> + pinctrl-0 = <&gpio_hall_sensor_default>;
> + pinctrl-names = "default";
> +
> + label = "GPIO Hall Effect Sensor";
> +
> + event-hall-sensor {
> + label = "Hall Effect Sensor";
> + gpios = <&tlmm 20 GPIO_ACTIVE_LOW>;
> + linux,input-type = <EV_SW>;
> + linux,code = <SW_LID>;
> + linux,can-disable;
> + };
> + };
> +
> + gpio-keys {
> + compatible = "gpio-keys";
> +
> + pinctrl-0 = <&gpio_keys_default>;
> + pinctrl-names = "default";
> +
> + label = "GPIO Buttons";
> +
> + button-volume-up {
> + label = "Volume Up";
> + gpios = <&tlmm 107 GPIO_ACTIVE_LOW>;
> + linux,code = <KEY_VOLUMEUP>;
> + };
> + };
> +
> + leds {
> + compatible = "gpio-leds";
> +
> + led-0 {
> + gpios = <&tlmm 17 GPIO_ACTIVE_HIGH>;
> + color = <LED_COLOR_ID_WHITE>;
> + default-state = "off";
> + function = LED_FUNCTION_KBD_BACKLIGHT;
> +
> + pinctrl-0 = <&button_backlight_default>;
> + pinctrl-names = "default";
> + };
> + };
> +
> + reg_ts_vdd: regulator-vdd-ts {
> + compatible = "regulator-fixed";
> + regulator-name = "regulator-vdd-ts";
> + regulator-min-microvolt = <2850000>;
> + regulator-max-microvolt = <2850000>;
> +
> + gpio = <&tlmm 78 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> +
> + pinctrl-0 = <&ts_vdd_default>;
> + pinctrl-names = "default";
> + };
> +
> + backlight {
> + compatible = "pwm-backlight";
> + pwms = <&pm8916_pwm 0 100000>;
> + brightness-levels = <0 255>;
> + num-interpolated-steps = <255>;
> + default-brightness-level = <128>;
> + enable-gpios = <&tlmm 98 GPIO_ACTIVE_HIGH>;
> +
> + pinctrl-0 = <&lcd_bl_en_default>;
> + pinctrl-names = "default";
> + };
> +
> + flash-led-controller {
> + compatible = "ocs,ocp8110";
> + flash-gpios = <&tlmm 8 GPIO_ACTIVE_HIGH>;
> + enable-gpios = <&tlmm 49 GPIO_ACTIVE_HIGH>;
> +
> + pinctrl-0 = <&camera_front_flash_default>;
> + pinctrl-names = "default";
> +
> + led {
> + function = LED_FUNCTION_FLASH;
> + color = <LED_COLOR_ID_WHITE>;
> + flash-max-timeout-us = <250000>;
> + };
> + };
> +
> + usb_id: usb-id {
> + compatible = "linux,extcon-usb-gpio";
> + id-gpio = <&tlmm 110 GPIO_ACTIVE_HIGH>;
I just noticed, that I forgot to include the `id-gpios` change
mentioned in the changelog of v2. Will send a v3 later including this
change. But waiting for further feedback first.
André
> + pinctrl-0 = <&usb_id_default &usb_id_switch_default>;
> + pinctrl-names = "default";
> + };
> +
> +};
> +
> +&blsp_i2c3 {
> + status = "okay";
> +
> + magnetometer@d {
> + compatible = "asahi-kasei,ak09911";
> + reg = <0x0d>;
> +
> + vdd-supply = <&pm8916_l17>;
> + vid-supply = <&pm8916_l6>;
> +
> + reset-gpios = <&tlmm 68 GPIO_ACTIVE_HIGH>;
> +
> + pinctrl-0 = <&mag_reset_default>;
> + pinctrl-names = "default";
> + };
> +
> + light-sensor@23 {
> + compatible = "liteon,ltr559";
> + reg = <0x23>;
> +
> + vdd-supply = <&pm8916_l17>;
> + vddio-supply = <&pm8916_l5>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <113 IRQ_TYPE_EDGE_FALLING>;
> +
> + pinctrl-0 = <&light_int_default>;
> + pinctrl-names = "default";
> + };
> +
> + imu@68 {
> + compatible = "bosch,bmi160";
> + reg = <0x68>;
> +
> + vdd-supply = <&pm8916_l17>;
> + vddio-supply = <&pm8916_l6>;
> + };
> +};
> +
> +&blsp_i2c5 {
> + status = "okay";
> +
> + touchscreen@4a {
> + compatible = "atmel,maxtouch";
> + reg = <0x4a>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <13 IRQ_TYPE_LEVEL_LOW>;
> +
> + reset-gpios = <&tlmm 12 GPIO_ACTIVE_LOW>;
> +
> + vdda-supply = <&pm8916_l6>;
> + vdd-supply = <®_ts_vdd>;
> +
> + pinctrl-0 = <&ts_int_reset_default>;
> + pinctrl-names = "default";
> +
> + /* Keys listed from right to left */
> + linux,keycodes = <KEY_APPSELECT KEY_HOMEPAGE
> KEY_BACK>;
> + };
> +};
> +
> +&blsp_uart2 {
> + status = "okay";
> +};
> +
> +&pm8916_mpps {
> + pwm_out: mpp4-state {
> + pins = "mpp4";
> + function = "digital";
> + power-source = <PM8916_MPP_VPH>;
> + output-low;
> + qcom,dtest = <1>;
> + };
> +};
> +
> +&pm8916_pwm {
> + pinctrl-0 = <&pwm_out>;
> + pinctrl-names = "default";
> + status = "okay";
> +};
> +
> +&pm8916_resin {
> + linux,code = <KEY_VOLUMEDOWN>;
> + status = "okay";
> +};
> +
> +&pm8916_rpm_regulators {
> + pm8916_l17: l17 {
> + regulator-min-microvolt = <2850000>;
> + regulator-max-microvolt = <2850000>;
> + };
> +};
> +
> +&pm8916_vib {
> + status = "okay";
> +};
> +
> +&sdhc_1 {
> + status = "okay";
> +};
> +
> +&sdhc_2 {
> + pinctrl-0 = <&sdc2_default &sdc2_cd_default>;
> + pinctrl-1 = <&sdc2_sleep &sdc2_cd_default>;
> + pinctrl-names = "default", "sleep";
> +
> + cd-gpios = <&tlmm 38 GPIO_ACTIVE_LOW>;
> +
> + status = "okay";
> +};
> +
> +&usb {
> + extcon = <&usb_id>, <&usb_id>;
> + status = "okay";
> +};
> +
> +&usb_hs_phy {
> + extcon = <&usb_id>;
> +};
> +
> +&wcnss {
> + status = "okay";
> +};
> +
> +&wcnss_iris {
> + compatible = "qcom,wcn3620";
> +};
> +
> +&tlmm {
> + button_backlight_default: button-backlight-default-state {
> + pins = "gpio17";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + camera_front_flash_default: camera-front-flash-default-state
> {
> + pins = "gpio8", "gpio49";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + gpio_hall_sensor_default: gpio-hall-sensor-default-state {
> + pins = "gpio20";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + gpio_keys_default: gpio-keys-default-state {
> + pins = "gpio107";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-pull-up;
> + };
> +
> + lcd_bl_en_default: lcd-bl-en-default-state {
> + pins = "gpio98";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + light_int_default: light-int-default-state {
> + pins = "gpio113";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + mag_reset_default: mag-reset-default-state {
> + pins = "gpio68";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + sdc2_cd_default: sdc2-cd-default-state {
> + pins = "gpio38";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + ts_int_reset_default: ts-int-reset-default-state {
> + pins = "gpio12", "gpio13";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + ts_vdd_default: ts-vdd-default-state {
> + pins = "gpio78";
> + function = "gpio";
> + drive-strength = <2>;
> + bias-disable;
> + };
> +
> + usb_id_default: usb-id-default-state {
> + pins = "gpio110";
> + function = "gpio";
> + drive-strength = <8>;
> + bias-pull-up;
> + };
> +
> + usb_id_switch_default: usb-id-switch-default-state {
> + pins = "gpio121";
> + function = "gpio";
> + drive-strength = <2>;
> + output-high;
> + };
> +};
>