From nobody Thu Apr 2 01:54:50 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AC60A36309B; Wed, 1 Apr 2026 20:32:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775075534; cv=none; b=QRvmVOiOafdomp4HDzKZBJg/PrfO2a9/1nenJaQHF4vxqDeZ9yPr9h1lONehN2g8CwBq9Sq6pFFFGCo8+jvmaFQO7zf68518su74vnh7TOzuDhpVSdiOFjvG3NNXfyf8KKdfJzxOSlZK/x6k2MEdCKTdhAyROPe8mM2esQ8c7G8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775075534; c=relaxed/simple; bh=v0uZOaJ54lAg5lMftorfBK/jklJNEhA7r9GlBZCAK/U=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=AwMiJc6/oM8oMzZg9wwpuMZiHtkMys42ocii92wjgHZkKQenf7/HhzsxBGzJfVkUYyz0VdOS9aEkS0Q8eHPYKEDWe+a/bpU1ozN0Tws+82MgSJ4DAbdF9b7dM6bSViNoirNuaMjITNtdyw3ozjWbh5aPsVgFXGi7ZkAjInmqPZA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=YIjzKQYA; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="YIjzKQYA" Received: by smtp.kernel.org (Postfix) with ESMTPS id 635A9C2BCAF; Wed, 1 Apr 2026 20:32:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1775075534; bh=v0uZOaJ54lAg5lMftorfBK/jklJNEhA7r9GlBZCAK/U=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=YIjzKQYAjui92npHHduZZMQlEbVrsoFuTN0zJIhKHKQe1QRxxPVGywwB/g4Qop/vP Zwsf/4EWJ9bVjTgFWsdkACRtvOIrxLtemTtdBvoBAUCoalxFLMRW6MgMWoH1rKT7C3 XB0+hGAqFRAbk5y9BwJm61f7+07y1oX+qC/nV59MI4DyR9fGSm/edH4s5uukAItZc2 NYVy5+ZvhLrD5m4zHcNc5XQ+ng3l00phYFWzrw5GrluRzyYQeSnHzqzGcliVARFTWK yBbpoRh04ZGaZrgEuchN2iWyZqJz9MCqG7GfgfezGAlqpXxo6CUpG2gwYe6Jun08q0 3luoFyTMbBlIA== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5599FD35154; Wed, 1 Apr 2026 20:32:14 +0000 (UTC) From: Rudraksha Gupta via B4 Relay Date: Wed, 01 Apr 2026 13:32:13 -0700 Subject: [PATCH v3 1/4] ARM: dts: qcom: msm8960: expressatt: Sort node references and includes Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260401-expressatt_fuel_guage-v3-1-9674cfc0b5a2@gmail.com> References: <20260401-expressatt_fuel_guage-v3-0-9674cfc0b5a2@gmail.com> In-Reply-To: <20260401-expressatt_fuel_guage-v3-0-9674cfc0b5a2@gmail.com> To: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Rudraksha Gupta X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1775075533; l=12047; i=guptarud@gmail.com; s=20240916; h=from:subject:message-id; bh=5nbF8uv4w0udRlmJ01uecytHbgghHrqiWUob940/otI=; b=YhGPyy4bFANR9YiWCi7ajd6ylcGqhquHLt8h5MIpQT2uYEMBd9sQSPmEqCsde4b6jy8VGH9mw Oq+n13F9En1ASe7soO1zsvnnNRA5pooxbY5li9bZQJR2bkICmQ4S97r X-Developer-Key: i=guptarud@gmail.com; a=ed25519; pk=ETrudRugWAtOpr0OhRiheQ1lXM4Kk4KGFnBySlKDi2I= X-Endpoint-Received: by B4 Relay for guptarud@gmail.com/20240916 with auth_id=211 X-Original-From: Rudraksha Gupta Reply-To: guptarud@gmail.com From: Rudraksha Gupta Reorganize the DTS file for consistency with other msm8960 board files. Assisted-by: Claude:claude-opus-4.6 Signed-off-by: Rudraksha Gupta --- .../dts/qcom/qcom-msm8960-samsung-expressatt.dts | 408 +++++++++++------= ---- 1 file changed, 207 insertions(+), 201 deletions(-) diff --git a/arch/arm/boot/dts/qcom/qcom-msm8960-samsung-expressatt.dts b/a= rch/arm/boot/dts/qcom/qcom-msm8960-samsung-expressatt.dts index 35514fd53e3d..0e6959f7af6f 100644 --- a/arch/arm/boot/dts/qcom/qcom-msm8960-samsung-expressatt.dts +++ b/arch/arm/boot/dts/qcom/qcom-msm8960-samsung-expressatt.dts @@ -1,13 +1,13 @@ // SPDX-License-Identifier: GPL-2.0 +#include +#include #include #include +#include #include =20 #include "qcom-msm8960.dtsi" #include "pm8921.dtsi" -#include -#include -#include =20 / { model =3D "Samsung Galaxy Express SGH-I437"; @@ -27,8 +27,8 @@ chosen { gpio-keys { compatible =3D "gpio-keys"; =20 - pinctrl-names =3D "default"; pinctrl-0 =3D <&gpio_keys_pin_a>; + pinctrl-names =3D "default"; =20 key-home { label =3D "Home"; @@ -54,58 +54,25 @@ key-volume-down { }; }; =20 - touchkey_enable: touchkey-enable { - compatible =3D "regulator-fixed"; - regulator-name =3D "touchkey_enable"; - gpio =3D <&tlmm 51 GPIO_ACTIVE_HIGH>; - enable-active-high; - regulator-boot-on; - }; - - vreg_flash: regulator-flash { - compatible =3D "regulator-fixed"; - regulator-name =3D "VREG_FLASH_3P3"; - regulator-min-microvolt =3D <3300000>; - regulator-max-microvolt =3D <3300000>; - gpio =3D <&pm8921_mpps 4 GPIO_ACTIVE_HIGH>; - enable-active-high; - pinctrl-0 =3D <&flash_led_unlock>; - pinctrl-names =3D "default"; - }; - - led-controller { - compatible =3D "richtek,rt8515"; - enf-gpios =3D <&tlmm 3 GPIO_ACTIVE_HIGH>; - vin-supply =3D <&vreg_flash>; - richtek,rfs-ohms =3D <16000>; - pinctrl-0 =3D <&cam_flash_en>; - pinctrl-names =3D "default"; - - led { - function =3D LED_FUNCTION_FLASH; - color =3D ; - flash-max-timeout-us =3D <250000>; - }; - }; - i2c-gpio-touchkey { compatible =3D "i2c-gpio"; #address-cells =3D <1>; #size-cells =3D <0>; sda-gpios =3D <&tlmm 71 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; scl-gpios =3D <&tlmm 72 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; - pinctrl-names =3D "default"; pinctrl-0 =3D <&touchkey_i2c_pins>; - status =3D "okay"; + pinctrl-names =3D "default"; i2c-gpio,delay-us =3D <2>; =20 + status =3D "okay"; + touchkey@20 { compatible =3D "coreriver,tc360-touchkey"; reg =3D <0x20>; =20 interrupts-extended =3D <&tlmm 52 IRQ_TYPE_EDGE_FALLING>; - pinctrl-names =3D "default"; pinctrl-0 =3D <&touchkey_irq_pin>; + pinctrl-names =3D "default"; =20 vddio-supply =3D <&touchkey_enable>; vdd-supply =3D <&pm8921_l29>; @@ -114,6 +81,52 @@ touchkey@20 { linux,keycodes =3D ; }; }; + + led-controller { + compatible =3D "richtek,rt8515"; + enf-gpios =3D <&tlmm 3 GPIO_ACTIVE_HIGH>; + vin-supply =3D <&vreg_flash>; + richtek,rfs-ohms =3D <16000>; + pinctrl-0 =3D <&cam_flash_en>; + pinctrl-names =3D "default"; + + led { + function =3D LED_FUNCTION_FLASH; + color =3D ; + flash-max-timeout-us =3D <250000>; + }; + }; + + vreg_flash: regulator-flash { + compatible =3D "regulator-fixed"; + regulator-name =3D "VREG_FLASH_3P3"; + regulator-min-microvolt =3D <3300000>; + regulator-max-microvolt =3D <3300000>; + gpio =3D <&pm8921_mpps 4 GPIO_ACTIVE_HIGH>; + enable-active-high; + pinctrl-0 =3D <&flash_led_unlock>; + pinctrl-names =3D "default"; + }; + + touchkey_enable: touchkey-enable { + compatible =3D "regulator-fixed"; + regulator-name =3D "touchkey_enable"; + gpio =3D <&tlmm 51 GPIO_ACTIVE_HIGH>; + enable-active-high; + regulator-boot-on; + }; +}; + +&gsbi1 { + qcom,mode =3D ; + pinctrl-0 =3D <&spi1_default>; + pinctrl-names =3D "default"; + + status =3D "okay"; +}; + +&gsbi1_spi { + status =3D "okay"; }; =20 &gsbi2 { @@ -141,51 +154,16 @@ light-sensor@39 { }; }; =20 -&gsbi5 { - qcom,mode =3D ; - status =3D "okay"; -}; - -&gsbi5_serial { - status =3D "okay"; -}; - -&sdcc1 { - vmmc-supply =3D <&pm8921_l5>; - status =3D "okay"; -}; - -&sdcc3 { - vmmc-supply =3D <&pm8921_l6>; - vqmmc-supply =3D <&pm8921_l7>; - - pinctrl-0 =3D <&sdcc3_default_state>; - pinctrl-1 =3D <&sdcc3_sleep_state>; - pinctrl-names =3D "default", "sleep"; - - status =3D "okay"; -}; - -&gsbi1 { - qcom,mode =3D ; - pinctrl-0 =3D <&spi1_default>; - pinctrl-names =3D "default"; - status =3D "okay"; -}; - -&gsbi1_spi { - status =3D "okay"; -}; - &gsbi3 { qcom,mode =3D ; + status =3D "okay"; }; =20 &gsbi3_i2c { status =3D "okay"; =20 - // Atmel mXT224S touchscreen + /* Atmel mXT224S touchscreen */ touchscreen@4a { compatible =3D "atmel,maxtouch"; reg =3D <0x4a>; @@ -193,93 +171,69 @@ touchscreen@4a { interrupts =3D <11 IRQ_TYPE_EDGE_FALLING>; vdda-supply =3D <&pm8921_lvs6>; vdd-supply =3D <&pm8921_l17>; - pinctrl-names =3D "default"; pinctrl-0 =3D <&touchscreen>; + pinctrl-names =3D "default"; }; }; =20 -&tlmm { - cam_flash_en: cam-flash-en-state { - pins =3D "gpio3"; - function =3D "gpio"; - drive-strength =3D <16>; - bias-pull-down; - }; +&gsbi5 { + qcom,mode =3D ; =20 - spi1_default: spi1-default-state { - mosi-pins { - pins =3D "gpio6"; - function =3D "gsbi1"; - drive-strength =3D <12>; - bias-disable; - }; + status =3D "okay"; +}; =20 - miso-pins { - pins =3D "gpio7"; - function =3D "gsbi1"; - drive-strength =3D <12>; - bias-disable; - }; +&gsbi5_serial { + status =3D "okay"; +}; =20 - cs-pins { - pins =3D "gpio8"; - function =3D "gsbi1"; - drive-strength =3D <12>; - bias-disable; - output-low; - }; +&gsbi7 { + qcom,mode =3D ; =20 - clk-pins { - pins =3D "gpio9"; - function =3D "gsbi1"; - drive-strength =3D <12>; - bias-disable; - }; - }; + status =3D "okay"; +}; =20 - gpio_keys_pin_a: gpio-keys-active-state { - pins =3D "gpio40", "gpio50", "gpio81"; - function =3D "gpio"; - drive-strength =3D <8>; - bias-disable; - }; +&gsbi7_i2c { + status =3D "okay"; =20 - touchscreen: touchscreen-int-state { - pins =3D "gpio11"; - function =3D "gpio"; - output-enable; - bias-disable; - drive-strength =3D <2>; + nfc@2b { + compatible =3D "nxp,pn544-i2c"; + reg =3D <0x2b>; + interrupts-extended =3D <&tlmm 106 IRQ_TYPE_EDGE_RISING>; + enable-gpios =3D <&pm8921_gpio 21 GPIO_ACTIVE_HIGH>; + firmware-gpios =3D <&tlmm 92 GPIO_ACTIVE_HIGH>; + + pinctrl-0 =3D <&nfc_default &nfc_enable>; + pinctrl-names =3D "default"; }; +}; =20 - nfc_default: nfc-default-state { - irq-pins { - pins =3D "gpio106"; - function =3D "gpio"; - drive-strength =3D <2>; - bias-pull-down; - }; +&gsbi12 { + qcom,mode =3D ; =20 - firmware-pins { - pins =3D "gpio92"; - function =3D "gpio"; - drive-strength =3D <2>; - bias-disable; - }; - }; + status =3D "okay"; +}; =20 - touchkey_i2c_pins: touchkey-i2c-state { - pins =3D "gpio71", "gpio72"; - function =3D "gpio"; - drive-strength =3D <2>; - bias-disable; +&gsbi12_i2c { + status =3D "okay"; + + accelerometer@18 { + compatible =3D "bosch,bma254"; + reg =3D <0x18>; + vdd-supply =3D <&pm8921_l9>; + vddio-supply =3D <&pm8921_lvs4>; + + mount-matrix =3D "-1", "0", "0", + "0", "-1", "0", + "0", "0", "1"; }; =20 - touchkey_irq_pin: touchkey-irq-state { - pins =3D "gpio52"; - function =3D "gpio"; - drive-strength =3D <2>; - bias-disable; + magnetometer@2e { + compatible =3D "yamaha,yas532"; + reg =3D <0x2e>; + vdd-supply =3D <&pm8921_l9>; + iovdd-supply =3D <&pm8921_lvs4>; + + /* TODO: Figure out Mount Matrix */ }; }; =20 @@ -288,18 +242,27 @@ &pm8921 { }; =20 &pm8921_gpio { + nfc_enable: nfc-enable-state { + pins =3D "gpio21"; + function =3D "normal"; + bias-disable; + qcom,drive-strength =3D ; + power-source =3D ; + }; + prox_sensor_int: prox-sensor-int-state { pins =3D "gpio6"; function =3D "normal"; input-enable; bias-disable; }; +}; =20 - nfc_enable: nfc-enable-state { - pins =3D "gpio21"; - function =3D "normal"; - bias-disable; - qcom,drive-strength =3D ; +&pm8921_mpps { + flash_led_unlock: flash-led-unlock-state { + pins =3D "mpp4"; + function =3D "digital"; + output-low; power-source =3D ; }; }; @@ -546,72 +509,115 @@ pm8921_ncp: ncp { }; }; =20 -&usb_hs1_phy { - v3p3-supply =3D <&pm8921_l3>; - v1p8-supply =3D <&pm8921_l4>; -}; +&sdcc1 { + vmmc-supply =3D <&pm8921_l5>; =20 -&usb1 { - dr_mode =3D "otg"; status =3D "okay"; }; =20 -&gsbi7 { - qcom,mode =3D ; +&sdcc3 { + vmmc-supply =3D <&pm8921_l6>; + vqmmc-supply =3D <&pm8921_l7>; + + pinctrl-0 =3D <&sdcc3_default_state>; + pinctrl-1 =3D <&sdcc3_sleep_state>; + pinctrl-names =3D "default", "sleep"; =20 status =3D "okay"; }; =20 -&gsbi7_i2c { - status =3D "okay"; +&tlmm { + cam_flash_en: cam-flash-en-state { + pins =3D "gpio3"; + function =3D "gpio"; + drive-strength =3D <16>; + bias-pull-down; + }; =20 - nfc@2b { - compatible =3D "nxp,pn544-i2c"; - reg =3D <0x2b>; - interrupts-extended =3D <&tlmm 106 IRQ_TYPE_EDGE_RISING>; - enable-gpios =3D <&pm8921_gpio 21 GPIO_ACTIVE_HIGH>; - firmware-gpios =3D <&tlmm 92 GPIO_ACTIVE_HIGH>; + spi1_default: spi1-default-state { + mosi-pins { + pins =3D "gpio6"; + function =3D "gsbi1"; + drive-strength =3D <12>; + bias-disable; + }; =20 - pinctrl-0 =3D <&nfc_default &nfc_enable>; - pinctrl-names =3D "default"; - }; -}; + miso-pins { + pins =3D "gpio7"; + function =3D "gsbi1"; + drive-strength =3D <12>; + bias-disable; + }; =20 -&gsbi12 { - qcom,mode =3D ; + cs-pins { + pins =3D "gpio8"; + function =3D "gsbi1"; + drive-strength =3D <12>; + bias-disable; + output-low; + }; =20 - status =3D "okay"; -}; + clk-pins { + pins =3D "gpio9"; + function =3D "gsbi1"; + drive-strength =3D <12>; + bias-disable; + }; + }; =20 -&gsbi12_i2c { - status =3D "okay"; + touchscreen: touchscreen-int-state { + pins =3D "gpio11"; + function =3D "gpio"; + output-enable; + bias-disable; + drive-strength =3D <2>; + }; =20 - accelerometer@18 { - compatible =3D "bosch,bma254"; - reg =3D <0x18>; - vdd-supply =3D <&pm8921_l9>; - vddio-supply =3D <&pm8921_lvs4>; + gpio_keys_pin_a: gpio-keys-active-state { + pins =3D "gpio40", "gpio50", "gpio81"; + function =3D "gpio"; + drive-strength =3D <8>; + bias-disable; + }; =20 - mount-matrix =3D "-1", "0", "0", - "0", "-1", "0", - "0", "0", "1"; + touchkey_irq_pin: touchkey-irq-state { + pins =3D "gpio52"; + function =3D "gpio"; + drive-strength =3D <2>; + bias-disable; }; =20 - magnetometer@2e { - compatible =3D "yamaha,yas532"; - reg =3D <0x2e>; - vdd-supply =3D <&pm8921_l9>; - iovdd-supply =3D <&pm8921_lvs4>; + touchkey_i2c_pins: touchkey-i2c-state { + pins =3D "gpio71", "gpio72"; + function =3D "gpio"; + drive-strength =3D <2>; + bias-disable; + }; =20 - /* TODO: Figure out Mount Matrix */ + nfc_default: nfc-default-state { + firmware-pins { + pins =3D "gpio92"; + function =3D "gpio"; + drive-strength =3D <2>; + bias-disable; + }; + + irq-pins { + pins =3D "gpio106"; + function =3D "gpio"; + drive-strength =3D <2>; + bias-pull-down; + }; }; }; =20 -&pm8921_mpps { - flash_led_unlock: flash-led-unlock-state { - pins =3D "mpp4"; - function =3D "digital"; - output-low; - power-source =3D ; - }; +&usb_hs1_phy { + v3p3-supply =3D <&pm8921_l3>; + v1p8-supply =3D <&pm8921_l4>; +}; + +&usb1 { + dr_mode =3D "otg"; + + status =3D "okay"; }; --=20 2.53.0