* [PATCH v3 1/5] dt-bindings: qcom: Document bindings for new msm8916-samsung-j5 devices
2022-09-28 16:50 [RESEND PATCH v3 0/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
@ 2022-09-28 16:54 ` Lin, Meng-Bo
2022-09-28 17:00 ` [PATCH v3 2/5] arm64: dts: qcom: msm8916-samsung-j5: Drop from Makefile Lin, Meng-Bo
` (3 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Lin, Meng-Bo @ 2022-09-28 16:54 UTC (permalink / raw)
To: devicetree
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Julian Ribbeck, Stephan Gerhold,
Nikita Travkin, Josef W Menad, Markuss Broks, linux-arm-msm,
linux-kernel, ~postmarketos/upstreaming
Document the new samsung-j3/j5x device tree bindings used in their
device trees.
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/arm/qcom.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index fb1d00bcc847..b1c69d4d9a9a 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -176,7 +176,9 @@ properties:
- longcheer,l8910
- samsung,a3u-eur
- samsung,a5u-eur
+ - samsung,j3
- samsung,j5
+ - samsung,j5x
- samsung,serranove
- wingtech,wt88047
- const: qcom,msm8916
--
2.30.2
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v3 2/5] arm64: dts: qcom: msm8916-samsung-j5: Drop from Makefile
2022-09-28 16:50 [RESEND PATCH v3 0/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
2022-09-28 16:54 ` [PATCH v3 1/5] dt-bindings: qcom: Document bindings for new msm8916-samsung-j5 devices Lin, Meng-Bo
@ 2022-09-28 17:00 ` Lin, Meng-Bo
2022-09-28 17:14 ` Krzysztof Kozlowski
2022-09-28 17:00 ` [PATCH v3 3/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add initial common device tree Lin, Meng-Bo
` (2 subsequent siblings)
4 siblings, 1 reply; 8+ messages in thread
From: Lin, Meng-Bo @ 2022-09-28 17:00 UTC (permalink / raw)
To: devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
Before moving msm8916-samsung-j5.dts to msm8916-samsung-j5-common.dtsi,
drop it from Makefile temporarily to avoid errors.
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 1d86a33de528..6db7d9d791d5 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -15,7 +15,6 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8916-longcheer-l8910.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-mtp.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-a3u-eur.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-a5u-eur.dtb
-dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-j5.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-serranove.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-wingtech-wt88047.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-10.dtb
--
2.30.2
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH v3 2/5] arm64: dts: qcom: msm8916-samsung-j5: Drop from Makefile
2022-09-28 17:00 ` [PATCH v3 2/5] arm64: dts: qcom: msm8916-samsung-j5: Drop from Makefile Lin, Meng-Bo
@ 2022-09-28 17:14 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-28 17:14 UTC (permalink / raw)
To: Lin, Meng-Bo, devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
On 28/09/2022 19:00, Lin, Meng-Bo wrote:
> Before moving msm8916-samsung-j5.dts to msm8916-samsung-j5-common.dtsi,
> drop it from Makefile temporarily to avoid errors.
>
> Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
This change does not make any sense. Rename of DTS into DTSI must be
together with adding back that DTS.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v3 3/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add initial common device tree
2022-09-28 16:50 [RESEND PATCH v3 0/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
2022-09-28 16:54 ` [PATCH v3 1/5] dt-bindings: qcom: Document bindings for new msm8916-samsung-j5 devices Lin, Meng-Bo
2022-09-28 17:00 ` [PATCH v3 2/5] arm64: dts: qcom: msm8916-samsung-j5: Drop from Makefile Lin, Meng-Bo
@ 2022-09-28 17:00 ` Lin, Meng-Bo
2022-09-28 17:14 ` Krzysztof Kozlowski
2022-09-28 17:00 ` [PATCH v3 5/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add Hall sensor Lin, Meng-Bo
2022-09-28 17:00 ` [PATCH v3 4/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
4 siblings, 1 reply; 8+ messages in thread
From: Lin, Meng-Bo @ 2022-09-28 17:00 UTC (permalink / raw)
To: devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
The smartphones below are using the MSM8916 SoC,
which are released in 2015-2016:
Samsung Galaxy J5 2015 (SM-J500*)
Samsung Galaxy J5 2016 (SM-J510*)
Samsung Galaxy J3 2016
- SM-J3109/SM-J320Y/SM-J320YZ
- SM-J320N0/SM-J320ZN
- SM-J320P/SM-J320R4/SM-J320V/SM-S320VL
Add a common device tree for 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
The three devices (some varints of J3, all other variants of J5 released
in 2015 and J5X released in 2016) are very similar, with some differences
in display and GPIO pins. The common parts are shared in
msm8916-samsung-j5-common.dtsi to reduce duplication.
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
---
...6-samsung-j5.dts => msm8916-samsung-j5-common.dtsi} | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
rename arch/arm64/boot/dts/qcom/{msm8916-samsung-j5.dts => msm8916-samsung-j5-common.dtsi} (94%)
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
similarity index 94%
rename from arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts
rename to arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
index eabeed18cfaa..4f71609bf6f8 100644
--- a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
@@ -1,15 +1,10 @@
// SPDX-License-Identifier: GPL-2.0-only
-/dts-v1/;
-
#include "msm8916-pm8916.dtsi"
+
#include <dt-bindings/gpio/gpio.h>
/ {
- model = "Samsung Galaxy J5 (2015)";
- compatible = "samsung,j5", "qcom,msm8916";
- chassis-type = "handset";
-
aliases {
serial0 = &blsp1_uart2;
};
@@ -92,7 +87,6 @@ &usb {
&usb_hs_phy {
extcon = <&pm8916_usbin>;
- qcom,init-seq = /bits/ 8 <0x1 0x19 0x2 0x0b>;
};
&smd_rpm_regulators {
@@ -199,7 +193,7 @@ l18 {
};
&msmgpio {
- gpio_keys_default: gpio-keys-default {
+ gpio_keys_default: gpio-keys-default-state {
pins = "gpio107", "gpio109";
function = "gpio";
--
2.30.2
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH v3 3/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add initial common device tree
2022-09-28 17:00 ` [PATCH v3 3/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add initial common device tree Lin, Meng-Bo
@ 2022-09-28 17:14 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-28 17:14 UTC (permalink / raw)
To: Lin, Meng-Bo, devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
On 28/09/2022 19:00, Lin, Meng-Bo wrote:
> The smartphones below are using the MSM8916 SoC,
> which are released in 2015-2016:
>
> Samsung Galaxy J5 2015 (SM-J500*)
> Samsung Galaxy J5 2016 (SM-J510*)
> Samsung Galaxy J3 2016
> - SM-J3109/SM-J320Y/SM-J320YZ
> - SM-J320N0/SM-J320ZN
> - SM-J320P/SM-J320R4/SM-J320V/SM-S320VL
>
> Add a common device tree for 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
>
> The three devices (some varints of J3, all other variants of J5 released
> in 2015 and J5X released in 2016) are very similar, with some differences
> in display and GPIO pins. The common parts are shared in
> msm8916-samsung-j5-common.dtsi to reduce duplication.
>
> Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
> ---
> ...6-samsung-j5.dts => msm8916-samsung-j5-common.dtsi} | 10 ++--------
Nope, we loose a board in this commit. Your changes are still not
bisectable - board is gone.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v3 5/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add Hall sensor
2022-09-28 16:50 [RESEND PATCH v3 0/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
` (2 preceding siblings ...)
2022-09-28 17:00 ` [PATCH v3 3/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add initial common device tree Lin, Meng-Bo
@ 2022-09-28 17:00 ` Lin, Meng-Bo
2022-09-28 17:00 ` [PATCH v3 4/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
4 siblings, 0 replies; 8+ messages in thread
From: Lin, Meng-Bo @ 2022-09-28 17:00 UTC (permalink / raw)
To: devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
Samsung Galaxy J5 2015 and 2016 have a Hall sensor on GPIO pin 52.
Add GPIO Hall sensor for them.
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
---
.../boot/dts/qcom/msm8916-samsung-j3.dts | 4 +++
.../dts/qcom/msm8916-samsung-j5-common.dtsi | 26 +++++++++++++++++++
2 files changed, 30 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
index bf8672ebedcd..7fd357b7f728 100644
--- a/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
@@ -19,3 +19,7 @@ tz-apps@85a00000 {
};
};
};
+
+&gpio_hall_sensor {
+ status = "disabled";
+};
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
index 4f71609bf6f8..54190144f823 100644
--- a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5-common.dtsi
@@ -3,6 +3,7 @@
#include "msm8916-pm8916.dtsi"
#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
/ {
aliases {
@@ -21,6 +22,23 @@ tz-apps@85500000 {
};
};
+ gpio_hall_sensor: gpio-hall-sensor {
+ compatible = "gpio-keys";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&gpio_hall_sensor_default>;
+
+ label = "GPIO Hall Effect Sensor";
+
+ event-hall-sensor {
+ label = "Hall Effect Sensor";
+ gpios = <&msmgpio 52 GPIO_ACTIVE_LOW>;
+ linux,input-type = <EV_SW>;
+ linux,code = <SW_LID>;
+ linux,can-disable;
+ };
+ };
+
gpio-keys {
compatible = "gpio-keys";
@@ -193,6 +211,14 @@ l18 {
};
&msmgpio {
+ gpio_hall_sensor_default: gpio-hall-sensor-default-state {
+ pins = "gpio52";
+ function = "gpio";
+
+ drive-strength = <2>;
+ bias-disable;
+ };
+
gpio_keys_default: gpio-keys-default-state {
pins = "gpio107", "gpio109";
function = "gpio";
--
2.30.2
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v3 4/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree
2022-09-28 16:50 [RESEND PATCH v3 0/5] arm64: dts: qcom: msm8916-samsung-j5: Use common device tree Lin, Meng-Bo
` (3 preceding siblings ...)
2022-09-28 17:00 ` [PATCH v3 5/5] arm64: dts: qcom: msm8916-samsung-j5-common: Add Hall sensor Lin, Meng-Bo
@ 2022-09-28 17:00 ` Lin, Meng-Bo
4 siblings, 0 replies; 8+ messages in thread
From: Lin, Meng-Bo @ 2022-09-28 17:00 UTC (permalink / raw)
To: devicetree
Cc: Stephan Gerhold, Nikita Travkin, Julian Ribbeck, Josef W Menad,
Markuss Broks, linux-arm-msm, linux-kernel,
~postmarketos/upstreaming, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Rob Herring, Krzysztof Kozlowski
After moving msm8916-samsung-j5.dts to msm8916-samsung-j5-common.dtsi,
Add new J3, J5 2015/2016 device trees.
[Add j5x device tree]
Co-developed-by: Josef W Menad <JosefWMenad@protonmail.ch>
Signed-off-by: Josef W Menad <JosefWMenad@protonmail.ch>
[Use &pm8916_usbin as USB extcon and add chassis-type for j5x]
Co-developed-by: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
[Use common init device tree and add j3 device tree]
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 3 +++
.../boot/dts/qcom/msm8916-samsung-j3.dts | 21 +++++++++++++++++++
.../boot/dts/qcom/msm8916-samsung-j5.dts | 15 +++++++++++++
.../boot/dts/qcom/msm8916-samsung-j5x.dts | 11 ++++++++++
4 files changed, 50 insertions(+)
create mode 100644 arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
create mode 100644 arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts
create mode 100644 arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dts
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 6db7d9d791d5..357a2529fe74 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -15,6 +15,9 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8916-longcheer-l8910.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-mtp.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-a3u-eur.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-a5u-eur.dtb
+dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-j3.dtb
+dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-j5.dtb
+dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-j5x.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-samsung-serranove.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8916-wingtech-wt88047.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-10.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
new file mode 100644
index 000000000000..bf8672ebedcd
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j3.dts
@@ -0,0 +1,21 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+/dts-v1/;
+
+#include "msm8916-samsung-j5-common.dtsi"
+
+/ {
+ model = "Samsung Galaxy J3 (2016)";
+ compatible = "samsung,j3", "qcom,msm8916";
+ chassis-type = "handset";
+
+ reserved-memory {
+ /delete-node/ tz-apps@85500000;
+
+ /* Additional memory used by Samsung firmware modifications */
+ tz-apps@85a00000 {
+ reg = <0x0 0x85800000 0x0 0x800000>;
+ no-map;
+ };
+ };
+};
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts
new file mode 100644
index 000000000000..0a32d33e9778
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dts
@@ -0,0 +1,15 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+/dts-v1/;
+
+#include "msm8916-samsung-j5-common.dtsi"
+
+/ {
+ model = "Samsung Galaxy J5 (2015)";
+ compatible = "samsung,j5", "qcom,msm8916";
+ chassis-type = "handset";
+};
+
+&usb_hs_phy {
+ qcom,init-seq = /bits/ 8 <0x1 0x19 0x2 0x0b>;
+};
diff --git a/arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dts b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dts
new file mode 100644
index 000000000000..7656ac4508cf
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dts
@@ -0,0 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0-only
+
+/dts-v1/;
+
+#include "msm8916-samsung-j5-common.dtsi"
+
+/ {
+ model = "Samsung Galaxy J5 (2016)";
+ compatible = "samsung,j5x", "qcom,msm8916";
+ chassis-type = "handset";
+};
--
2.30.2
^ permalink raw reply related [flat|nested] 8+ messages in thread