~postmarketos/upstreaming

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
5 5

[PATCH v2 0/2] arm64: dts: qcom: msm8916-samsung-j3ltetw: Add initial device tree

Details
Message ID
<20240804065854.42437-1-linmengbo06890@proton.me>
DKIM signature
pass
Download raw message
The dts and dtsi add support for msm8916 variant of Samsung Galaxy J3
SM-J320YZ smartphone released in 2016.

Add a device tree for SM-J320YZ with initial support for:

- GPIO keys
- SDHCI (internal and external storage)
- USB Device Mode
- UART (on USB connector via the SM5703 MUIC)
- WCNSS (WiFi/BT)
- Regulators
- QDSP6 audio
- Speaker/earpiece/headphones/microphones via digital/analog codec in
  MSM8916/PM8916
- WWAN Internet via BAM-DMUX
- Touchscreen
- Accelerometer

There are different variants of J3, with some differences in MUIC, sensor,
NFC and touch key I2C buses.

The common parts are shared in msm8916-samsung-j3-common.dtsi to reduce
duplication.

---
v2: Add accelerometer
V1: Initial device tree
Link: https://lore.kernel.org/r/20240802080701.3643-1-linmengbo06890@proton.me

[PATCH v2 1/2] dt-bindings: qcom: Document samsung,j3ltetw

Details
Message ID
<20240804065854.42437-2-linmengbo06890@proton.me>
In-Reply-To
<20240804065854.42437-1-linmengbo06890@proton.me> (view parent)
DKIM signature
pass
Download raw message
Patch: +1 -0
Document samsung,j3ltetw bindings used in its device tree.

Signed-off-by: "Lin, Meng-Bo" <linmengbo06890@proton.me>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 4ef456cefd6c..c0529486810f 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -235,6 +235,7 @@ properties:
              - samsung,grandprimelte
              - samsung,gt510
              - samsung,gt58
              - samsung,j3ltetw
              - samsung,j5
              - samsung,j5x
              - samsung,rossa
-- 
2.39.2

[PATCH v2 2/2] arm64: dts: qcom: msm8916-samsung-j3ltetw: Add initial device tree

Details
Message ID
<20240804065854.42437-3-linmengbo06890@proton.me>
In-Reply-To
<20240804065854.42437-1-linmengbo06890@proton.me> (view parent)
DKIM signature
pass
Download raw message
Patch: +94 -0
The dts and dtsi add support for msm8916 variant of Samsung Galaxy J3
SM-J320YZ smartphone released in 2016.

Add a device tree for SM-J320YZ with initial support for:

- GPIO keys
- SDHCI (internal and external storage)
- USB Device Mode
- UART (on USB connector via the SM5703 MUIC)
- WCNSS (WiFi/BT)
- Regulators
- QDSP6 audio
- Speaker/earpiece/headphones/microphones via digital/analog codec in
  MSM8916/PM8916
- WWAN Internet via BAM-DMUX
- Touchscreen
- Accelerometer

There are different variants of J3, with some differences in MUIC, sensor,
NFC and touch key I2C buses.

The common parts are shared in msm8916-samsung-j3-common.dtsi to reduce
duplication.

Signed-off-by: "Lin, Meng-Bo" <linmengbo06890@proton.me>
---
 arch/arm64/boot/dts/qcom/Makefile             |  1 +
 .../dts/qcom/msm8916-samsung-j3-common.dtsi   | 62 +++++++++++++++++++
 .../boot/dts/qcom/msm8916-samsung-j3ltetw.dts | 31 ++++++++++
 3 files changed, 94 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/msm8916-samsung-j3-common.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dts

diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index e534442620a1..197ab325c0b9 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -48,6 +48,7 @@ dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-grandmax.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-grandprimelte.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-gt510.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-gt58.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-j3ltetw.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-j5.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-j5x.dtb
dtb-$(CONFIG_ARCH_QCOM)	+= msm8916-samsung-rossa.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j3-common.dtsi b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3-common.dtsi
new file mode 100644
index 000000000000..1d74cccc438a
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3-common.dtsi
@@ -0,0 +1,62 @@
// SPDX-License-Identifier: GPL-2.0-only

#include "msm8916-samsung-j5-common.dtsi"

/ {
	reserved-memory {
		/delete-node/ tz-apps@85500000;

		/* Additional memory used by Samsung firmware modifications */
		tz-apps@85800000 {
			reg = <0x0 0x85800000 0x0 0x800000>;
			no-map;
		};
	};

	reg_vdd_tsp_a: regulator-vdd-tsp-a {
		compatible = "regulator-fixed";
		regulator-name = "vdd_tsp_a";
		regulator-min-microvolt = <3000000>;
		regulator-max-microvolt = <3000000>;

		gpio = <&tlmm 16 GPIO_ACTIVE_HIGH>;
		enable-active-high;

		pinctrl-0 = <&tsp_ldo_en_default>;
		pinctrl-names = "default";
	};
};

&accelerometer {
	vdd-supply = <&pm8916_l5>;
	vddio-supply = <&pm8916_l5>;

	mount-matrix = "0", "-1", "0",
		       "1", "0", "0",
		       "0", "0", "-1";
};

&gpio_hall_sensor {
	status = "disabled";
};

&i2c_muic {
	/* GPIO pins vary depending on model variant */
};

&i2c_sensors {
	/* GPIO pins vary depending on model variant */
};

&touchscreen {
	vdd-supply = <&reg_vdd_tsp_a>;
};

&tlmm {
	tsp_ldo_en_default: tsp-ldo-en-default-state {
		pins = "gpio16";
		function = "gpio";
		drive-strength = <2>;
		bias-disable;
	};
};
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dts
new file mode 100644
index 000000000000..a26d2fd13c94
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dts
@@ -0,0 +1,31 @@
// SPDX-License-Identifier: GPL-2.0-only

/dts-v1/;

#include "msm8916-samsung-j3-common.dtsi"

/ {
	model = "Samsung Galaxy J3 (2016) (SM-J320YZ)";
	compatible = "samsung,j3ltetw", "qcom,msm8916";
	chassis-type = "handset";
};

&i2c_muic {
	sda-gpios = <&tlmm 0 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
	scl-gpios = <&tlmm 1 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
};

&i2c_sensors {
	/* I2C2 */
	sda-gpios = <&tlmm 6 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
	scl-gpios = <&tlmm 7 (GPIO_ACTIVE_HIGH|GPIO_OPEN_DRAIN)>;
};

&muic_i2c_default {
	pins = "gpio0", "gpio1";
};

&sensors_i2c_default {
	/* I2C2 */
	pins = "gpio6", "gpio7";
};
-- 
2.39.2

Re: [PATCH v2 1/2] dt-bindings: qcom: Document samsung,j3ltetw

Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Details
Message ID
<448705b3-be39-4938-a421-2cc36d4c132b@linaro.org>
In-Reply-To
<20240804065854.42437-2-linmengbo06890@proton.me> (view parent)
DKIM signature
pass
Download raw message
On 04/08/2024 08:59, Lin, Meng-Bo wrote:
> Document samsung,j3ltetw bindings used in its device tree.
> 

Please do not work on old kernels... You *must* base on something recent.

<form letter>
Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC (and consider --no-git-fallback argument). It might
happen, that command when run on an older kernel, gives you outdated
entries. Therefore please be sure you base your patches on recent Linux
kernel.

Tools like b4 or scripts/get_maintainer.pl provide you proper list of
people, so fix your workflow. Tools might also fail if you work on some
ancient tree (don't, instead use mainline) or work on fork of kernel
(don't, instead use mainline). Just use b4 and everything should be
fine, although remember about `b4 prep --auto-to-cc` if you added new
patches to the patchset.
</form letter>


> Signed-off-by: "Lin, Meng-Bo" <linmengbo06890@proton.me>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 1 +


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Rob Herring (Arm) <robh@kernel.org>
Details
Message ID
<172286967129.2710260.12547511216605996134.robh@kernel.org>
In-Reply-To
<20240804065854.42437-1-linmengbo06890@proton.me> (view parent)
DKIM signature
pass
Download raw message
On Sun, 04 Aug 2024 06:59:06 +0000, Lin, Meng-Bo wrote:
> The dts and dtsi add support for msm8916 variant of Samsung Galaxy J3
> SM-J320YZ smartphone released in 2016.
> 
> Add a device tree for SM-J320YZ with initial support for:
> 
> - GPIO keys
> - SDHCI (internal and external storage)
> - USB Device Mode
> - UART (on USB connector via the SM5703 MUIC)
> - WCNSS (WiFi/BT)
> - Regulators
> - QDSP6 audio
> - Speaker/earpiece/headphones/microphones via digital/analog codec in
>   MSM8916/PM8916
> - WWAN Internet via BAM-DMUX
> - Touchscreen
> - Accelerometer
> 
> There are different variants of J3, with some differences in MUIC, sensor,
> NFC and touch key I2C buses.
> 
> The common parts are shared in msm8916-samsung-j3-common.dtsi to reduce
> duplication.
> 
> ---
> v2: Add accelerometer
> V1: Initial device tree
> Link: https://lore.kernel.org/r/20240802080701.3643-1-linmengbo06890@proton.me
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y qcom/msm8916-samsung-j3ltetw.dtb' for 20240804065854.42437-1-linmengbo06890@proton.me:

arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dtb: /soc@0/power-manager@b088000: failed to match any schema with compatible: ['qcom,msm8916-acc']
arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dtb: /soc@0/power-manager@b098000: failed to match any schema with compatible: ['qcom,msm8916-acc']
arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dtb: /soc@0/power-manager@b0a8000: failed to match any schema with compatible: ['qcom,msm8916-acc']
arch/arm64/boot/dts/qcom/msm8916-samsung-j3ltetw.dtb: /soc@0/power-manager@b0b8000: failed to match any schema with compatible: ['qcom,msm8916-acc']

Re: (subset) [PATCH v2 0/2] arm64: dts: qcom: msm8916-samsung-j3ltetw: Add initial device tree

Bjorn Andersson <andersson@kernel.org>
Details
Message ID
<172375444825.1011236.8152323041465572346.b4-ty@kernel.org>
In-Reply-To
<20240804065854.42437-1-linmengbo06890@proton.me> (view parent)
DKIM signature
pass
Download raw message
On Sun, 04 Aug 2024 06:59:06 +0000, Lin, Meng-Bo wrote:
> The dts and dtsi add support for msm8916 variant of Samsung Galaxy J3
> SM-J320YZ smartphone released in 2016.
> 
> Add a device tree for SM-J320YZ with initial support for:
> 
> - GPIO keys
> - SDHCI (internal and external storage)
> - USB Device Mode
> - UART (on USB connector via the SM5703 MUIC)
> - WCNSS (WiFi/BT)
> - Regulators
> - QDSP6 audio
> - Speaker/earpiece/headphones/microphones via digital/analog codec in
>   MSM8916/PM8916
> - WWAN Internet via BAM-DMUX
> - Touchscreen
> - Accelerometer
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: qcom: Document samsung,j3ltetw
      commit: c17427c6ca5e0f7c5d1f9bd9c951cb6827c8c228
[2/2] arm64: dts: qcom: msm8916-samsung-j3ltetw: Add initial device tree
      commit: 469fc2e7a995ade8b929c4e2a32c93a231a3844f

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>
Reply to thread Export thread (mbox)