2023-02-13 14:46:48

by Luca Weiss

[permalink] [raw]
Subject: [PATCH v2 0/2] Enable remaining I2C busses on sm7225-fairphone-fp4

The first patch consolidate the two different styles where the status
property is placed in the dts, and the second patch enables all the I2C
busses used on the phone even if no subnode is added yet.

Signed-off-by: Luca Weiss <[email protected]>
---
Changes in v2:
- Enable gpi_dma* nodes as well, mention that in commit message
- Pick up tags
- Link to v1: https://lore.kernel.org/r/[email protected]

---
Luca Weiss (2):
arm64: dts: qcom: sm7225-fairphone-fp4: move status property down
arm64: dts: qcom: sm7225-fairphone-fp4: enable remaining i2c busses

arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 67 +++++++++++++++++------
1 file changed, 50 insertions(+), 17 deletions(-)
---
base-commit: 09e41676e35ab06e4bce8870ea3bf1f191c3cb90
change-id: 20230213-fp4-more-i2c-f90ccce84e46

Best regards,
--
Luca Weiss <[email protected]>



2023-02-13 14:46:51

by Luca Weiss

[permalink] [raw]
Subject: [PATCH v2 1/2] arm64: dts: qcom: sm7225-fairphone-fp4: move status property down

Currently the dts contains a mix of status-as-first-property (old qcom
style) and status-as-last-property (new style).

Move all status properties down to the bottom once and for all so that
the style is consistent between different nodes.

Reviewed-by: Konrad Dybcio <[email protected]>
Signed-off-by: Luca Weiss <[email protected]>
---
arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 34 +++++++++++------------
1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
index f7602e93121a..86114dd2c0c4 100644
--- a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
+++ b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
@@ -110,8 +110,8 @@ active-config0 {
};

&adsp {
- status = "okay";
firmware-name = "qcom/sm7225/fairphone4/adsp.mdt";
+ status = "okay";
};

&apps_rsc {
@@ -355,13 +355,13 @@ &cci1_i2c0 {
};

&cdsp {
- status = "okay";
firmware-name = "qcom/sm7225/fairphone4/cdsp.mdt";
+ status = "okay";
};

&i2c10 {
- status = "okay";
clock-frequency = <400000>;
+ status = "okay";

haptics@5a {
compatible = "awinic,aw8695";
@@ -396,8 +396,8 @@ &ipa {
};

&mpss {
- status = "okay";
firmware-name = "qcom/sm7225/fairphone4/modem.mdt";
+ status = "okay";
};

&pm6150l_flash {
@@ -423,11 +423,11 @@ led-1 {
};

&pm6150l_wled {
- status = "okay";
-
qcom,switching-freq = <800>;
qcom,current-limit-microamp = <20000>;
qcom,num-strings = <2>;
+
+ status = "okay";
};

&pm6350_gpios {
@@ -441,8 +441,8 @@ gpio_keys_pin: gpio-keys-state {
};

&pm6350_resin {
- status = "okay";
linux,code = <KEY_VOLUMEDOWN>;
+ status = "okay";
};

&pm7250b_adc {
@@ -535,21 +535,21 @@ &uart9 {
};

&ufs_mem_hc {
- status = "okay";
-
reset-gpios = <&tlmm 156 GPIO_ACTIVE_LOW>;

vcc-supply = <&vreg_l7e>;
vcc-max-microamp = <800000>;
vccq2-supply = <&vreg_l12a>;
vccq2-max-microamp = <800000>;
+
+ status = "okay";
};

&ufs_mem_phy {
- status = "okay";
-
vdda-phy-supply = <&vreg_l18a>;
vdda-pll-supply = <&vreg_l22a>;
+
+ status = "okay";
};

&usb_1 {
@@ -562,26 +562,26 @@ &usb_1_dwc3 {
};

&usb_1_hsphy {
- status = "okay";
-
vdd-supply = <&vreg_l18a>;
vdda-pll-supply = <&vreg_l2a>;
vdda-phy-dpdm-supply = <&vreg_l3a>;
+
+ status = "okay";
};

&usb_1_qmpphy {
- status = "okay";
-
vdda-phy-supply = <&vreg_l22a>;
vdda-pll-supply = <&vreg_l16a>;
+
+ status = "okay";
};

&wifi {
- status = "okay";
-
vdd-0.8-cx-mx-supply = <&vreg_l4a>;
vdd-1.8-xo-supply = <&vreg_l7a>;
vdd-1.3-rfa-supply = <&vreg_l2e>;
vdd-3.3-ch0-supply = <&vreg_l10e>;
vdd-3.3-ch1-supply = <&vreg_l11e>;
+
+ status = "okay";
};

--
2.39.1


2023-02-13 14:46:54

by Luca Weiss

[permalink] [raw]
Subject: [PATCH v2 2/2] arm64: dts: qcom: sm7225-fairphone-fp4: enable remaining i2c busses

Enable all i2c busses where something is connected on this phone. Add
comments as placeholders for which components are still missing.

Also enable gpi_dma and the other qupv3 for that.

Reviewed-by: Konrad Dybcio <[email protected]>
Signed-off-by: Luca Weiss <[email protected]>
---
arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 33 +++++++++++++++++++++++
1 file changed, 33 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
index 86114dd2c0c4..7ae6aba5d2ec 100644
--- a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
+++ b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
@@ -359,10 +359,39 @@ &cdsp {
status = "okay";
};

+&gpi_dma0 {
+ status = "okay";
+};
+
+&gpi_dma1 {
+ status = "okay";
+};
+
+&i2c0 {
+ clock-frequency = <400000>;
+ status = "okay";
+
+ /* ST21NFCD NFC @ 8 */
+ /* VL53L3 ToF @ 29 */
+ /* AW88264A amplifier @ 34 */
+ /* AW88264A amplifier @ 35 */
+};
+
+&i2c8 {
+ clock-frequency = <400000>;
+ status = "okay";
+
+ /* HX83112A touchscreen @ 48 */
+};
+
&i2c10 {
clock-frequency = <400000>;
status = "okay";

+ /* PM8008 PMIC @ 8 and 9 */
+ /* PX8618 @ 26 */
+ /* SMB1395 PMIC @ 34 */
+
haptics@5a {
compatible = "awinic,aw8695";
reg = <0x5a>;
@@ -495,6 +524,10 @@ adc-chan@644 {
};
};

+&qupv3_id_0 {
+ status = "okay";
+};
+
&qupv3_id_1 {
status = "okay";
};

--
2.39.1


2023-02-13 22:21:30

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] Enable remaining I2C busses on sm7225-fairphone-fp4

On Mon, 13 Feb 2023 15:46:29 +0100, Luca Weiss wrote:
> The first patch consolidate the two different styles where the status
> property is placed in the dts, and the second patch enables all the I2C
> busses used on the phone even if no subnode is added yet.
>
>

Applied, thanks!

[1/2] arm64: dts: qcom: sm7225-fairphone-fp4: move status property down
commit: 65aedb0b48c09595065c81722d9cca53760f7220
[2/2] arm64: dts: qcom: sm7225-fairphone-fp4: enable remaining i2c busses
commit: 574f2ffd45dc786dc443c5e11ea477420ffa5f52

Best regards,
--
Bjorn Andersson <[email protected]>