* [PATCH v11 1/5] dt-bindings: arm: qcom: Separate LTE/WIFI SKU for sc7280-evoker
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
@ 2022-11-07 9:43 ` Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 2/5] arm64: dts: qcom: sc7280: Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
` (4 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Sheng-Liang Pan @ 2022-11-07 9:43 UTC (permalink / raw)
To: LKML
Cc: dianders, Sheng-Liang Pan, Krzysztof Kozlowski, Andy Gross,
Bjorn Andersson, Konrad Dybcio, Krzysztof Kozlowski, Rob Herring,
devicetree, linux-arm-msm
evoker will have WIFI/LTE SKU, separate it for each different setting.
Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
---
Changes in v4:
- fix typo in tittle and commit
- recover change for trackpad and touchscreen
Documentation/devicetree/bindings/arm/qcom.yaml | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 207e282602064..04928ca8cd556 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -592,6 +592,11 @@ properties:
- const: google,evoker
- const: qcom,sc7280
+ - description: Google Evoker with LTE (newest rev)
+ items:
+ - const: google,evoker-sku512
+ - const: qcom,sc7280
+
- description: Google Herobrine (newest rev)
items:
- const: google,herobrine
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v11 2/5] arm64: dts: qcom: sc7280: Add LTE SKU for sc7280-evoker family
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 1/5] dt-bindings: arm: qcom: Separate LTE/WIFI SKU for sc7280-evoker Sheng-Liang Pan
@ 2022-11-07 9:43 ` Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 3/5] arm64: dts: qcom: sc7280: Add touchscreen and touchpad support for evoker Sheng-Liang Pan
` (3 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Sheng-Liang Pan @ 2022-11-07 9:43 UTC (permalink / raw)
To: LKML
Cc: dianders, Sheng-Liang Pan, Konrad Dybcio, Andy Gross,
Bjorn Andersson, Krzysztof Kozlowski, Rob Herring, devicetree,
linux-arm-msm
evoker have wifi/lte sku, add different dts for each sku.
Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>
---
Changes in v8:
- updated patch subjects
Changes in v5:
- recover whitespace change
Changes in v4:
- recover change for trackpad and touchscreen
arch/arm64/boot/dts/qcom/Makefile | 3 ++-
.../boot/dts/qcom/sc7280-herobrine-evoker-lte.dts | 14 ++++++++++++++
.../boot/dts/qcom/sc7280-herobrine-evoker.dts | 15 +++++++++++++++
...evoker-r0.dts => sc7280-herobrine-evoker.dtsi} | 7 -------
4 files changed, 31 insertions(+), 8 deletions(-)
create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dts
create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
rename arch/arm64/boot/dts/qcom/{sc7280-herobrine-evoker-r0.dts => sc7280-herobrine-evoker.dtsi} (98%)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index b0558d3389e5a..6f234995284b2 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -106,7 +106,8 @@ dtb-$(CONFIG_ARCH_QCOM) += sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7180-trogdor-r1.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7180-trogdor-r1-lte.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-crd.dtb
-dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-evoker-r0.dtb
+dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-evoker.dtb
+dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-evoker-lte.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-herobrine-r1.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-villager-r0.dtb
dtb-$(CONFIG_ARCH_QCOM) += sc7280-herobrine-villager-r1.dtb
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dts
new file mode 100644
index 0000000000000..3af9224a7492e
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-lte.dts
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Google Evoker board device tree source
+ *
+ * Copyright 2022 Google LLC.
+ */
+
+#include "sc7280-herobrine-evoker.dts"
+#include "sc7280-herobrine-lte-sku.dtsi"
+
+/ {
+ model = "Google Evoker with LTE";
+ compatible = "google,evoker-sku512", "qcom,sc7280";
+};
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
new file mode 100644
index 0000000000000..dcdd4eecfe670
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
@@ -0,0 +1,15 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Google Evoker board device tree source
+ *
+ * Copyright 2022 Google LLC.
+ */
+
+/dts-v1/;
+
+#include "sc7280-herobrine-evoker.dtsi"
+
+/ {
+ model = "Google Evoker";
+ compatible = "google,evoker", "qcom,sc7280";
+};
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-r0.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
similarity index 98%
rename from arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-r0.dts
rename to arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
index 739e81bd6d689..a6015491c6082 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker-r0.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
@@ -5,15 +5,8 @@
* Copyright 2022 Google LLC.
*/
-/dts-v1/;
-
#include "sc7280-herobrine.dtsi"
-/ {
- model = "Google Evoker";
- compatible = "google,evoker", "qcom,sc7280";
-};
-
/*
* ADDITIONS TO FIXED REGULATORS DEFINED IN PARENT DEVICE TREE FILES
*
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v11 3/5] arm64: dts: qcom: sc7280: Add touchscreen and touchpad support for evoker
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 1/5] dt-bindings: arm: qcom: Separate LTE/WIFI SKU for sc7280-evoker Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 2/5] arm64: dts: qcom: sc7280: Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
@ 2022-11-07 9:43 ` Sheng-Liang Pan
2022-11-07 9:43 ` [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi " Sheng-Liang Pan
` (2 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Sheng-Liang Pan @ 2022-11-07 9:43 UTC (permalink / raw)
To: LKML
Cc: dianders, Sheng-Liang Pan, Konrad Dybcio, Andy Gross,
Bjorn Andersson, Krzysztof Kozlowski, Rob Herring, devicetree,
linux-arm-msm
Change touchpad and touchscreen node for evoker
Touchpad: SA461D-1011
Touchscreen: GT7986U
Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>
---
Changes in v8:
- updated patch subjects
Changes in v7:
- add compiatable for gt7986
Changes in v6:
- add removed pinctrl and align touchscreen label with herobrine board
Changes in v5:
- new patch for Touchscreen/trackpad in v5
.../boot/dts/qcom/sc7280-herobrine-evoker.dtsi | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
index a6015491c6082..706dd82a70138 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dtsi
@@ -23,16 +23,15 @@ ap_tp_i2c: &i2c0 {
status = "okay";
clock-frequency = <400000>;
- trackpad: trackpad@2c {
- compatible = "hid-over-i2c";
- reg = <0x2c>;
+ trackpad: trackpad@15 {
+ compatible = "elan,ekth3000";
+ reg = <0x15>;
pinctrl-names = "default";
pinctrl-0 = <&tp_int_odl>;
interrupt-parent = <&tlmm>;
interrupts = <7 IRQ_TYPE_EDGE_FALLING>;
- hid-descr-addr = <0x20>;
vcc-supply = <&pp3300_z1>;
wakeup-source;
@@ -43,9 +42,9 @@ ts_i2c: &i2c13 {
status = "okay";
clock-frequency = <400000>;
- ap_ts: touchscreen@10 {
- compatible = "elan,ekth6915";
- reg = <0x10>;
+ ap_ts: touchscreen@5d {
+ compatible = "goodix,gt7986u", "goodix,gt7375p";
+ reg = <0x5d>;
pinctrl-names = "default";
pinctrl-0 = <&ts_int_conn>, <&ts_rst_conn>;
@@ -54,7 +53,7 @@ ap_ts: touchscreen@10 {
reset-gpios = <&tlmm 54 GPIO_ACTIVE_LOW>;
- vcc33-supply = <&ts_avdd>;
+ vdd-supply = <&ts_avdd>;
};
};
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi for evoker
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
` (2 preceding siblings ...)
2022-11-07 9:43 ` [PATCH v11 3/5] arm64: dts: qcom: sc7280: Add touchscreen and touchpad support for evoker Sheng-Liang Pan
@ 2022-11-07 9:43 ` Sheng-Liang Pan
2022-11-07 10:02 ` Konrad Dybcio
2022-11-07 18:15 ` Doug Anderson
2022-11-07 9:43 ` [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi Sheng-Liang Pan
2022-11-08 1:27 ` [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Bjorn Andersson
5 siblings, 2 replies; 11+ messages in thread
From: Sheng-Liang Pan @ 2022-11-07 9:43 UTC (permalink / raw)
To: LKML
Cc: dianders, Sheng-Liang Pan, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Krzysztof Kozlowski, Rob Herring, devicetree,
linux-arm-msm
add specific 3mic setting as sc7280-herobrine-audio-rt5682-3mic.dtsi,
so we can include sc7280-herobrine-audio-rt5682-3mic.dtsi for evoker
as it uses rt5682 with 3 mics.
Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
---
Changes in v11:
- sort out the "Status" property with sc7280-herobrine-audio-rt5682-3mic.dtsi
Changes in v10:
- add evoker include specific sc7280-herobrine-audio-rt5682-3mic.dtsi setting
.../sc7280-herobrine-audio-rt5682-3mic.dtsi | 195 ++++++++++++++++++
.../boot/dts/qcom/sc7280-herobrine-evoker.dts | 1 +
2 files changed, 196 insertions(+)
create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
new file mode 100644
index 0000000000000..cf34334451d6b
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
@@ -0,0 +1,195 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ *
+ * This file defines the common audio settings for the child boards
+ * using rt5682 codec and having 3 dmics connected to sc7280.
+ *
+ * Copyright 2022 Google LLC.
+ */
+
+/ {
+ /* BOARD-SPECIFIC TOP LEVEL NODES */
+ sound: sound {
+ compatible = "google,sc7280-herobrine";
+ model = "sc7280-rt5682-max98360a-3mic";
+
+ audio-routing = "VA DMIC0", "vdd-micb",
+ "VA DMIC1", "vdd-micb",
+ "VA DMIC2", "vdd-micb",
+ "VA DMIC3", "vdd-micb",
+
+ "Headphone Jack", "HPOL",
+ "Headphone Jack", "HPOR";
+
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ dai-link@0 {
+ link-name = "MAX98360";
+ reg = <0>;
+
+ cpu {
+ sound-dai = <&lpass_cpu MI2S_SECONDARY>;
+ };
+
+ codec {
+ sound-dai = <&max98360a>;
+ };
+ };
+
+ dai-link@1 {
+ link-name = "DisplayPort";
+ reg = <1>;
+
+ cpu {
+ sound-dai = <&lpass_cpu LPASS_DP_RX>;
+ };
+
+ codec {
+ sound-dai = <&mdss_dp>;
+ };
+ };
+
+ dai-link@2 {
+ link-name = "ALC5682";
+ reg = <2>;
+
+ cpu {
+ sound-dai = <&lpass_cpu MI2S_PRIMARY>;
+ };
+
+ codec {
+ sound-dai = <&alc5682 0 /* aif1 */>;
+ };
+ };
+
+ dai-link@4 {
+ link-name = "DMIC";
+ reg = <4>;
+
+ cpu {
+ sound-dai = <&lpass_cpu LPASS_CDC_DMA_VA_TX0>;
+ };
+
+ codec {
+ sound-dai = <&lpass_va_macro 0>;
+ };
+ };
+ };
+};
+
+hp_i2c: &i2c2 {
+ clock-frequency = <400000>;
+ status = "okay";
+
+ alc5682: codec@1a {
+ compatible = "realtek,rt5682s";
+ reg = <0x1a>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&hp_irq>;
+
+ #sound-dai-cells = <1>;
+
+ interrupt-parent = <&tlmm>;
+ interrupts = <101 IRQ_TYPE_EDGE_BOTH>;
+
+ AVDD-supply = <&pp1800_alc5682>;
+ MICVDD-supply = <&pp3300_codec>;
+
+ realtek,dmic1-data-pin = <1>;
+ realtek,dmic1-clk-pin = <2>;
+ realtek,jd-src = <1>;
+ realtek,dmic-clk-rate-hz = <2048000>;
+ };
+};
+
+&lpass_cpu {
+ pinctrl-names = "default";
+ pinctrl-0 = <&mi2s0_data0>, <&mi2s0_data1>, <&mi2s0_mclk>, <&mi2s0_sclk>, <&mi2s0_ws>,
+ <&mi2s1_data0>, <&mi2s1_sclk>, <&mi2s1_ws>;
+
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ status = "okay";
+
+ dai-link@0 {
+ reg = <MI2S_PRIMARY>;
+ qcom,playback-sd-lines = <1>;
+ qcom,capture-sd-lines = <0>;
+ };
+
+ dai-link@1 {
+ reg = <MI2S_SECONDARY>;
+ qcom,playback-sd-lines = <0>;
+ };
+
+ dai-link@5 {
+ reg = <LPASS_DP_RX>;
+ };
+
+ dai-link@25 {
+ reg = <LPASS_CDC_DMA_VA_TX0>;
+ };
+};
+
+&lpass_va_macro {
+ vdd-micb-supply = <&pp1800_l2c>;
+ pinctrl-0 = <&lpass_dmic01_clk>, <&lpass_dmic01_data>, <&lpass_dmic23_clk>,
+ <&lpass_dmic23_data>;
+
+ status = "okay";
+};
+
+/* PINCTRL - ADDITIONS TO NODES IN PARENT DEVICE TREE FILES */
+
+&lpass_dmic01_clk {
+ drive-strength = <8>;
+ bias-disable;
+};
+
+&lpass_dmic01_clk_sleep {
+ drive-strength = <2>;
+};
+
+&lpass_dmic01_data {
+ bias-pull-down;
+};
+
+&lpass_dmic23_clk {
+ drive-strength = <8>;
+ bias-disable;
+};
+
+&lpass_dmic23_clk_sleep {
+ drive-strength = <2>;
+};
+
+&lpass_dmic23_data {
+ bias-pull-down;
+};
+
+&mi2s0_data0 {
+ drive-strength = <6>;
+ bias-disable;
+};
+
+&mi2s0_data1 {
+ drive-strength = <6>;
+ bias-disable;
+};
+
+&mi2s0_mclk {
+ drive-strength = <6>;
+ bias-disable;
+};
+
+&mi2s0_sclk {
+ drive-strength = <6>;
+ bias-disable;
+};
+
+&mi2s0_ws {
+ drive-strength = <6>;
+ bias-disable;
+};
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
index dcdd4eecfe670..51f0401b11ed7 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
@@ -8,6 +8,7 @@
/dts-v1/;
#include "sc7280-herobrine-evoker.dtsi"
+#include "sc7280-herobrine-audio-rt5682-3mic.dtsi"
/ {
model = "Google Evoker";
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi for evoker
2022-11-07 9:43 ` [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi " Sheng-Liang Pan
@ 2022-11-07 10:02 ` Konrad Dybcio
2022-11-07 18:15 ` Doug Anderson
1 sibling, 0 replies; 11+ messages in thread
From: Konrad Dybcio @ 2022-11-07 10:02 UTC (permalink / raw)
To: Sheng-Liang Pan, LKML
Cc: dianders, Andy Gross, Bjorn Andersson, Krzysztof Kozlowski,
Rob Herring, devicetree, linux-arm-msm
On 07/11/2022 10:43, Sheng-Liang Pan wrote:
> add specific 3mic setting as sc7280-herobrine-audio-rt5682-3mic.dtsi,
> so we can include sc7280-herobrine-audio-rt5682-3mic.dtsi for evoker
> as it uses rt5682 with 3 mics.
>
> Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
>
> Changes in v11:
> - sort out the "Status" property with sc7280-herobrine-audio-rt5682-3mic.dtsi
>
> Changes in v10:
> - add evoker include specific sc7280-herobrine-audio-rt5682-3mic.dtsi setting
>
> .../sc7280-herobrine-audio-rt5682-3mic.dtsi | 195 ++++++++++++++++++
> .../boot/dts/qcom/sc7280-herobrine-evoker.dts | 1 +
> 2 files changed, 196 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
> new file mode 100644
> index 0000000000000..cf34334451d6b
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
> @@ -0,0 +1,195 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + *
> + * This file defines the common audio settings for the child boards
> + * using rt5682 codec and having 3 dmics connected to sc7280.
> + *
> + * Copyright 2022 Google LLC.
> + */
> +
> +/ {
> + /* BOARD-SPECIFIC TOP LEVEL NODES */
> + sound: sound {
> + compatible = "google,sc7280-herobrine";
> + model = "sc7280-rt5682-max98360a-3mic";
> +
> + audio-routing = "VA DMIC0", "vdd-micb",
> + "VA DMIC1", "vdd-micb",
> + "VA DMIC2", "vdd-micb",
> + "VA DMIC3", "vdd-micb",
> +
> + "Headphone Jack", "HPOL",
> + "Headphone Jack", "HPOR";
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + dai-link@0 {
> + link-name = "MAX98360";
> + reg = <0>;
> +
> + cpu {
> + sound-dai = <&lpass_cpu MI2S_SECONDARY>;
> + };
> +
> + codec {
> + sound-dai = <&max98360a>;
> + };
> + };
> +
> + dai-link@1 {
> + link-name = "DisplayPort";
> + reg = <1>;
> +
> + cpu {
> + sound-dai = <&lpass_cpu LPASS_DP_RX>;
> + };
> +
> + codec {
> + sound-dai = <&mdss_dp>;
> + };
> + };
> +
> + dai-link@2 {
> + link-name = "ALC5682";
> + reg = <2>;
> +
> + cpu {
> + sound-dai = <&lpass_cpu MI2S_PRIMARY>;
> + };
> +
> + codec {
> + sound-dai = <&alc5682 0 /* aif1 */>;
> + };
> + };
> +
> + dai-link@4 {
> + link-name = "DMIC";
> + reg = <4>;
> +
> + cpu {
> + sound-dai = <&lpass_cpu LPASS_CDC_DMA_VA_TX0>;
> + };
> +
> + codec {
> + sound-dai = <&lpass_va_macro 0>;
> + };
> + };
> + };
> +};
> +
> +hp_i2c: &i2c2 {
> + clock-frequency = <400000>;
> + status = "okay";
> +
> + alc5682: codec@1a {
> + compatible = "realtek,rt5682s";
> + reg = <0x1a>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&hp_irq>;
> +
> + #sound-dai-cells = <1>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <101 IRQ_TYPE_EDGE_BOTH>;
> +
> + AVDD-supply = <&pp1800_alc5682>;
> + MICVDD-supply = <&pp3300_codec>;
> +
> + realtek,dmic1-data-pin = <1>;
> + realtek,dmic1-clk-pin = <2>;
> + realtek,jd-src = <1>;
> + realtek,dmic-clk-rate-hz = <2048000>;
> + };
> +};
> +
> +&lpass_cpu {
> + pinctrl-names = "default";
> + pinctrl-0 = <&mi2s0_data0>, <&mi2s0_data1>, <&mi2s0_mclk>, <&mi2s0_sclk>, <&mi2s0_ws>,
> + <&mi2s1_data0>, <&mi2s1_sclk>, <&mi2s1_ws>;
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + status = "okay";
> +
> + dai-link@0 {
> + reg = <MI2S_PRIMARY>;
> + qcom,playback-sd-lines = <1>;
> + qcom,capture-sd-lines = <0>;
> + };
> +
> + dai-link@1 {
> + reg = <MI2S_SECONDARY>;
> + qcom,playback-sd-lines = <0>;
> + };
> +
> + dai-link@5 {
> + reg = <LPASS_DP_RX>;
> + };
> +
> + dai-link@25 {
> + reg = <LPASS_CDC_DMA_VA_TX0>;
> + };
> +};
> +
> +&lpass_va_macro {
> + vdd-micb-supply = <&pp1800_l2c>;
> + pinctrl-0 = <&lpass_dmic01_clk>, <&lpass_dmic01_data>, <&lpass_dmic23_clk>,
> + <&lpass_dmic23_data>;
> +
> + status = "okay";
> +};
> +
> +/* PINCTRL - ADDITIONS TO NODES IN PARENT DEVICE TREE FILES */
> +
> +&lpass_dmic01_clk {
> + drive-strength = <8>;
> + bias-disable;
> +};
> +
> +&lpass_dmic01_clk_sleep {
> + drive-strength = <2>;
> +};
> +
> +&lpass_dmic01_data {
> + bias-pull-down;
> +};
> +
> +&lpass_dmic23_clk {
> + drive-strength = <8>;
> + bias-disable;
> +};
> +
> +&lpass_dmic23_clk_sleep {
> + drive-strength = <2>;
> +};
> +
> +&lpass_dmic23_data {
> + bias-pull-down;
> +};
> +
> +&mi2s0_data0 {
> + drive-strength = <6>;
> + bias-disable;
> +};
> +
> +&mi2s0_data1 {
> + drive-strength = <6>;
> + bias-disable;
> +};
> +
> +&mi2s0_mclk {
> + drive-strength = <6>;
> + bias-disable;
> +};
> +
> +&mi2s0_sclk {
> + drive-strength = <6>;
> + bias-disable;
> +};
> +
> +&mi2s0_ws {
> + drive-strength = <6>;
> + bias-disable;
> +};
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
> index dcdd4eecfe670..51f0401b11ed7 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-evoker.dts
> @@ -8,6 +8,7 @@
> /dts-v1/;
>
> #include "sc7280-herobrine-evoker.dtsi"
> +#include "sc7280-herobrine-audio-rt5682-3mic.dtsi"
>
> / {
> model = "Google Evoker";
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi for evoker
2022-11-07 9:43 ` [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi " Sheng-Liang Pan
2022-11-07 10:02 ` Konrad Dybcio
@ 2022-11-07 18:15 ` Doug Anderson
1 sibling, 0 replies; 11+ messages in thread
From: Doug Anderson @ 2022-11-07 18:15 UTC (permalink / raw)
To: Sheng-Liang Pan
Cc: LKML, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Krzysztof Kozlowski, Rob Herring, devicetree, linux-arm-msm,
Srinivasa Rao Mandadapu
Hi,
On Mon, Nov 7, 2022 at 1:45 AM Sheng-Liang Pan
<sheng-liang.pan@quanta.corp-partner.google.com> wrote:
>
> +&lpass_va_macro {
> + vdd-micb-supply = <&pp1800_l2c>;
> + pinctrl-0 = <&lpass_dmic01_clk>, <&lpass_dmic01_data>, <&lpass_dmic23_clk>,
> + <&lpass_dmic23_data>;
> +
> + status = "okay";
> +};
> +
> +/* PINCTRL - ADDITIONS TO NODES IN PARENT DEVICE TREE FILES */
> +
> +&lpass_dmic01_clk {
> + drive-strength = <8>;
> + bias-disable;
> +};
> +
> +&lpass_dmic01_clk_sleep {
> + drive-strength = <2>;
> +};
All of these "sleep" pinctrl states don't actually do anything useful
because (currently) nobody refers to them. Off-list I've asked
Srinivasa (now CCed here) to address this and I think it's fine for
you to have them here for now and then we can use or remove them all
at once.
In any case, this patch looks good to me now, thanks!
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
` (3 preceding siblings ...)
2022-11-07 9:43 ` [PATCH v11 4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi " Sheng-Liang Pan
@ 2022-11-07 9:43 ` Sheng-Liang Pan
2022-11-07 10:02 ` Konrad Dybcio
2022-11-07 18:12 ` Doug Anderson
2022-11-08 1:27 ` [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Bjorn Andersson
5 siblings, 2 replies; 11+ messages in thread
From: Sheng-Liang Pan @ 2022-11-07 9:43 UTC (permalink / raw)
To: LKML
Cc: dianders, Sheng-Liang Pan, Andy Gross, Bjorn Andersson,
Konrad Dybcio, Krzysztof Kozlowski, Rob Herring, devicetree,
linux-arm-msm
To keep diffs clean, sort out "Status" to last property.
Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
---
Changes in v11:
- sort out the "Status" property with sc7280-herobrine-audio-rt5682.dtsi
.../dts/qcom/sc7280-herobrine-audio-rt5682.dtsi | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
index 2dbdeeb29ecec..fc7a659dfe4ae 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
@@ -13,14 +13,14 @@ sound: sound {
compatible = "google,sc7280-herobrine";
model = "sc7280-rt5682-max98360a-1mic";
- status = "okay";
- audio-routing =
- "Headphone Jack", "HPOL",
- "Headphone Jack", "HPOR";
+ audio-routing = "Headphone Jack", "HPOL",
+ "Headphone Jack", "HPOR";
#address-cells = <1>;
#size-cells = <0>;
+ status = "okay";
+
dai-link@0 {
link-name = "MAX98360";
reg = <0>;
@@ -50,8 +50,8 @@ codec {
};
hp_i2c: &i2c2 {
- status = "okay";
clock-frequency = <400000>;
+ status = "okay";
alc5682: codec@1a {
compatible = "realtek,rt5682s";
@@ -75,8 +75,6 @@ alc5682: codec@1a {
};
&lpass_cpu {
- status = "okay";
-
pinctrl-names = "default";
pinctrl-0 = <&mi2s0_data0>, <&mi2s0_data1>, <&mi2s0_mclk>, <&mi2s0_sclk>, <&mi2s0_ws>,
<&mi2s1_data0>, <&mi2s1_sclk>, <&mi2s1_ws>;
@@ -84,6 +82,8 @@ &lpass_cpu {
#address-cells = <1>;
#size-cells = <0>;
+ status = "okay";
+
dai-link@0 {
reg = <MI2S_PRIMARY>;
qcom,playback-sd-lines = <1>;
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi
2022-11-07 9:43 ` [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi Sheng-Liang Pan
@ 2022-11-07 10:02 ` Konrad Dybcio
2022-11-07 18:12 ` Doug Anderson
1 sibling, 0 replies; 11+ messages in thread
From: Konrad Dybcio @ 2022-11-07 10:02 UTC (permalink / raw)
To: Sheng-Liang Pan, LKML
Cc: dianders, Andy Gross, Bjorn Andersson, Krzysztof Kozlowski,
Rob Herring, devicetree, linux-arm-msm
On 07/11/2022 10:43, Sheng-Liang Pan wrote:
> To keep diffs clean, sort out "Status" to last property.
>
> Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
>
> Changes in v11:
> - sort out the "Status" property with sc7280-herobrine-audio-rt5682.dtsi
>
> .../dts/qcom/sc7280-herobrine-audio-rt5682.dtsi | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
> index 2dbdeeb29ecec..fc7a659dfe4ae 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682.dtsi
> @@ -13,14 +13,14 @@ sound: sound {
> compatible = "google,sc7280-herobrine";
> model = "sc7280-rt5682-max98360a-1mic";
>
> - status = "okay";
> - audio-routing =
> - "Headphone Jack", "HPOL",
> - "Headphone Jack", "HPOR";
> + audio-routing = "Headphone Jack", "HPOL",
> + "Headphone Jack", "HPOR";
>
> #address-cells = <1>;
> #size-cells = <0>;
>
> + status = "okay";
> +
> dai-link@0 {
> link-name = "MAX98360";
> reg = <0>;
> @@ -50,8 +50,8 @@ codec {
> };
>
> hp_i2c: &i2c2 {
> - status = "okay";
> clock-frequency = <400000>;
> + status = "okay";
>
> alc5682: codec@1a {
> compatible = "realtek,rt5682s";
> @@ -75,8 +75,6 @@ alc5682: codec@1a {
> };
>
> &lpass_cpu {
> - status = "okay";
> -
> pinctrl-names = "default";
> pinctrl-0 = <&mi2s0_data0>, <&mi2s0_data1>, <&mi2s0_mclk>, <&mi2s0_sclk>, <&mi2s0_ws>,
> <&mi2s1_data0>, <&mi2s1_sclk>, <&mi2s1_ws>;
> @@ -84,6 +82,8 @@ &lpass_cpu {
> #address-cells = <1>;
> #size-cells = <0>;
>
> + status = "okay";
> +
> dai-link@0 {
> reg = <MI2S_PRIMARY>;
> qcom,playback-sd-lines = <1>;
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi
2022-11-07 9:43 ` [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi Sheng-Liang Pan
2022-11-07 10:02 ` Konrad Dybcio
@ 2022-11-07 18:12 ` Doug Anderson
1 sibling, 0 replies; 11+ messages in thread
From: Doug Anderson @ 2022-11-07 18:12 UTC (permalink / raw)
To: Sheng-Liang Pan
Cc: LKML, Andy Gross, Bjorn Andersson, Konrad Dybcio,
Krzysztof Kozlowski, Rob Herring, devicetree, linux-arm-msm
Hi
On Mon, Nov 7, 2022 at 1:46 AM Sheng-Liang Pan
<sheng-liang.pan@quanta.corp-partner.google.com> wrote:
>
> To keep diffs clean, sort out "Status" to last property.
>
> Signed-off-by: Sheng-Liang Pan <sheng-liang.pan@quanta.corp-partner.google.com>
> ---
>
> Changes in v11:
> - sort out the "Status" property with sc7280-herobrine-audio-rt5682.dtsi
>
> .../dts/qcom/sc7280-herobrine-audio-rt5682.dtsi | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
Thanks!
I probably would have also removed the 'status = "okay";' from the
top-level sound node in this patch, but that can always be done later.
Certainly your patch is a step forward. :-)
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family
2022-11-07 9:43 [PATCH v11 0/5] Add LTE SKU for sc7280-evoker family Sheng-Liang Pan
` (4 preceding siblings ...)
2022-11-07 9:43 ` [PATCH v11 5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi Sheng-Liang Pan
@ 2022-11-08 1:27 ` Bjorn Andersson
5 siblings, 0 replies; 11+ messages in thread
From: Bjorn Andersson @ 2022-11-08 1:27 UTC (permalink / raw)
To: LKML, sheng-liang.pan
Cc: devicetree, robh+dt, konrad.dybcio, krzysztof.kozlowski+dt,
agross, dianders, linux-arm-msm
On Mon, 7 Nov 2022 17:43:40 +0800, Sheng-Liang Pan wrote:
> This patch add common dtsi and WIFI/LTE dts for evoker.
>
> Changes in v11:
> - sort out the "Status" property with sc7280-herobrine-audio-rt5682.dtsi
> - sort out the "Status" property with sc7280-herobrine-audio-rt5682-3mic.dtsi
>
> Changes in v10:
> - add sc7280-herobrine-audio-rt5682-3mic.dtsi then evoker can include it
>
> [...]
Applied, thanks!
[1/5] dt-bindings: arm: qcom: Separate LTE/WIFI SKU for sc7280-evoker
commit: f1714f85d40e53645139824d0d903c1e050cdf00
[2/5] arm64: dts: qcom: sc7280: Add LTE SKU for sc7280-evoker family
commit: ef4fc701d4021eeb2a614fdffb3231560ee43c18
[3/5] arm64: dts: qcom: sc7280: Add touchscreen and touchpad support for evoker
commit: 928263d17413e406d7bbcd10e585b081f36c4114
[4/5] arm64: dts: qcom: sc7280: add sc7280-herobrine-audio-rt5682-3mic3.dtsi for evoker
commit: 5977c14285ca9ba933f4d048b0d995b046727788
[5/5] arm64: dts: qcom: sc7280: sort out the "Status" to last property with sc7280-herobrine-audio-rt5682.dtsi
commit: 3d11e7e120eee29ef837830bee8442195a2c4552
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 11+ messages in thread