* [PATCH 1/6] arm64: dts: qcom: msm8992-lg-bullhead: Place LG Bullhead generic code into a DTSI file
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-13 23:33 ` [PATCH 2/6] arm64: dts: qcom: msm8992-lg-bullhead: Add support for LG Bullhead rev 1.0 Petr Vorel
` (6 subsequent siblings)
7 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Jean THOMAS, Konrad Dybcio, Alexey Minnekhanov, Andy Gross,
Bjorn Andersson, Ulf Hansson, Douglas Anderson, Taniya Das,
linux-mmc, Petr Vorel
From: Jean THOMAS <virgule@jeanthomas.me>
This patch puts the generic code common across all hardware revisions
into a DTSI file.
It also prefixes the DTS filename with the vendor name, to follow the
naming convention used by other DTS files.
Reviewed-by: Petr Vorel <petr.vorel@gmail.com>
Signed-off-by: Jean THOMAS <virgule@jeanthomas.me>
[ pvorel: rebased on current qcom/for-next ]
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 2 +-
.../boot/dts/qcom/msm8992-lg-bullhead-rev-101.dts | 14 ++++++++++++++
...llhead-rev-101.dts => msm8992-lg-bullhead.dtsi} | 2 --
3 files changed, 15 insertions(+), 3 deletions(-)
create mode 100644 arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dts
rename arch/arm64/boot/dts/qcom/{msm8992-bullhead-rev-101.dts => msm8992-lg-bullhead.dtsi} (98%)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index f7232052d286..ee8e81f1c449 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -18,7 +18,7 @@ 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-bullhead-rev-101.dtb
+dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-101.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-msft-lumia-octagon-talkman.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-xiaomi-libra.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8994-angler-rev-101.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dts
new file mode 100644
index 000000000000..e6a5ebd30e2f
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dts
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/* Copyright (c) Jean Thomas <virgule@jeanthomas.me>
+ */
+
+/dts-v1/;
+
+#include "msm8992-lg-bullhead.dtsi"
+
+/ {
+ model = "LG Nexus 5X rev 1.01";
+
+ /* required for bootloader to select correct board */
+ qcom,board-id = <0xb64 0>;
+};
diff --git a/arch/arm64/boot/dts/qcom/msm8992-bullhead-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
similarity index 98%
rename from arch/arm64/boot/dts/qcom/msm8992-bullhead-rev-101.dts
rename to arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
index 4da6c44bf532..3b0cc85d6674 100644
--- a/arch/arm64/boot/dts/qcom/msm8992-bullhead-rev-101.dts
+++ b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
@@ -18,9 +18,7 @@ / {
compatible = "lg,bullhead", "qcom,msm8992";
chassis-type = "handset";
- /* required for bootloader to select correct board */
qcom,msm-id = <251 0>, <252 0>;
- qcom,board-id = <0xb64 0>;
qcom,pmic-id = <0x10009 0x1000A 0x0 0x0>;
/* Bullhead firmware doesn't support PSCI */
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/6] arm64: dts: qcom: msm8992-lg-bullhead: Add support for LG Bullhead rev 1.0
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
2022-01-13 23:33 ` [PATCH 1/6] arm64: dts: qcom: msm8992-lg-bullhead: Place LG Bullhead generic code into a DTSI file Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-13 23:33 ` [PATCH 3/6] arm64: dts: qcom: msm8994: Provide missing "xo_board" and "sleep_clk" to GCC Petr Vorel
` (5 subsequent siblings)
7 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Jean THOMAS, Konrad Dybcio, Alexey Minnekhanov, Andy Gross,
Bjorn Andersson, Ulf Hansson, Douglas Anderson, Taniya Das,
linux-mmc, Petr Vorel
From: Jean THOMAS <virgule@jeanthomas.me>
This commit implements a DTS file for LG Bullhead (Nexus 5X) rev 1.0
with its matching "qcom,board-id" property.
Reviewed-by: Petr Vorel <petr.vorel@gmail.com>
Signed-off-by: Jean THOMAS <virgule@jeanthomas.me>
[ pvorel: rebased on current qcom/for-next ]
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 1 +
.../boot/dts/qcom/msm8992-lg-bullhead-rev-10.dts | 14 ++++++++++++++
2 files changed, 15 insertions(+)
create mode 100644 arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dts
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index ee8e81f1c449..0f983400d60c 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -18,6 +18,7 @@ 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
dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-101.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-msft-lumia-octagon-talkman.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-xiaomi-libra.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dts b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dts
new file mode 100644
index 000000000000..7e6bce4af441
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dts
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/* Copyright (c) Jean Thomas <virgule@jeanthomas.me>
+ */
+
+/dts-v1/;
+
+#include "msm8992-lg-bullhead.dtsi"
+
+/ {
+ model = "LG Nexus 5X rev 1.0";
+
+ /* required for bootloader to select correct board */
+ qcom,board-id = <0xa64 0>;
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/6] arm64: dts: qcom: msm8994: Provide missing "xo_board" and "sleep_clk" to GCC
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
2022-01-13 23:33 ` [PATCH 1/6] arm64: dts: qcom: msm8992-lg-bullhead: Place LG Bullhead generic code into a DTSI file Petr Vorel
2022-01-13 23:33 ` [PATCH 2/6] arm64: dts: qcom: msm8992-lg-bullhead: Add support for LG Bullhead rev 1.0 Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-13 23:33 ` [PATCH 4/6] arm64: dts: qcom: msm8994-huawei-angler: Add vendor name huawei Petr Vorel
` (4 subsequent siblings)
7 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Petr Vorel, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Bjorn Andersson, Ulf Hansson, Douglas Anderson,
Taniya Das, linux-mmc
This is needed due changes in commit 0519d1d0bf33 ("clk: qcom:
gcc-msm8994: Modernize the driver"), which removed struct
clk_fixed_factor. Preparation for next commit for enabling SD/eMMC.
Inspired by 2c2f64ae36d9.
This is required for both msm8994-huawei-angler (sdhc1 will be enabled
in next commit) and msm8992-lg-bullhead (where actually fixes sdhc1
- tested on bullhead rev 1.01).
Fixes: 0519d1d0bf33 ("clk: qcom: gcc-msm8994: Modernize the driver")
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
arch/arm64/boot/dts/qcom/msm8994.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8994.dtsi b/arch/arm64/boot/dts/qcom/msm8994.dtsi
index 5a9a5ed0565f..215f56daa26c 100644
--- a/arch/arm64/boot/dts/qcom/msm8994.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8994.dtsi
@@ -713,6 +713,9 @@ gcc: clock-controller@fc400000 {
#reset-cells = <1>;
#power-domain-cells = <1>;
reg = <0xfc400000 0x2000>;
+
+ clock-names = "xo", "sleep_clk";
+ clocks = <&xo_board>, <&sleep_clk>;
};
rpm_msg_ram: sram@fc428000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/6] arm64: dts: qcom: msm8994-huawei-angler: Add vendor name huawei
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
` (2 preceding siblings ...)
2022-01-13 23:33 ` [PATCH 3/6] arm64: dts: qcom: msm8994: Provide missing "xo_board" and "sleep_clk" to GCC Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-13 23:33 ` [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions Petr Vorel
` (3 subsequent siblings)
7 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Petr Vorel, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Bjorn Andersson, Ulf Hansson, Douglas Anderson,
Taniya Das, linux-mmc
to follow the naming convention used by other DTS files.
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 2 +-
...994-angler-rev-101.dts => msm8994-huawei-angler-rev-101.dts} | 0
2 files changed, 1 insertion(+), 1 deletion(-)
rename arch/arm64/boot/dts/qcom/{msm8994-angler-rev-101.dts => msm8994-huawei-angler-rev-101.dts} (100%)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 0f983400d60c..523fe9f63368 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -22,7 +22,7 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-10.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-lg-bullhead-rev-101.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-msft-lumia-octagon-talkman.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8992-xiaomi-libra.dtb
-dtb-$(CONFIG_ARCH_QCOM) += msm8994-angler-rev-101.dtb
+dtb-$(CONFIG_ARCH_QCOM) += msm8994-huawei-angler-rev-101.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8994-msft-lumia-octagon-cityman.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8994-sony-xperia-kitakami-ivy.dtb
dtb-$(CONFIG_ARCH_QCOM) += msm8994-sony-xperia-kitakami-karin.dtb
diff --git a/arch/arm64/boot/dts/qcom/msm8994-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
similarity index 100%
rename from arch/arm64/boot/dts/qcom/msm8994-angler-rev-101.dts
rename to arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
` (3 preceding siblings ...)
2022-01-13 23:33 ` [PATCH 4/6] arm64: dts: qcom: msm8994-huawei-angler: Add vendor name huawei Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-13 23:40 ` Petr Vorel
2022-01-31 22:43 ` Bjorn Andersson
2022-01-13 23:33 ` [PATCH 6/6] arm64: dts: qcom: msm8992-lg-bullhead: Disable sdhc2 Petr Vorel
` (2 subsequent siblings)
7 siblings, 2 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Petr Vorel, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Bjorn Andersson, Ulf Hansson, Douglas Anderson,
Taniya Das, linux-mmc
Although downstream supports HS400, there are overclocking warnings when
using mmc-hs400-1_8v:
mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
mmc0: new HS400 MMC card at address 0001
Using HS200 (i.e. mmc-hs200-1_8v or mmc-ddr-1_8v) would reduce them:
mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
mmc0: new HS200 MMC card at address 0001
But as the problem is probably elsewhere (bullhead behaves the same),
keep mmc-hs400-1_8v.
Angler does not have SD card, thus explicitly disable sdhc2.
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
.../dts/qcom/msm8994-huawei-angler-rev-101.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
index 0e3dd48f0dbf..5ce3dc169bb4 100644
--- a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
+++ b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
@@ -7,6 +7,7 @@
/dts-v1/;
#include "msm8994.dtsi"
+#include <dt-bindings/gpio/gpio.h>
/* Angler's firmware does not report where the memory is allocated */
/delete-node/ &cont_splash_mem;
@@ -41,3 +42,18 @@ serial@f991e000 {
&tlmm {
gpio-reserved-ranges = <85 4>;
};
+
+/*
+ * Although downstream supports also HS400 there are fewer overclocking
+ * warnings when used DDR, also LK bootloader reports DDR mode.
+ */
+&sdhc1 {
+ status = "okay";
+
+ mmc-hs400-1_8v;
+};
+
+/* Angler does not have SD card */
+&sdhc2 {
+ status = "disabled";
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-13 23:33 ` [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions Petr Vorel
@ 2022-01-13 23:40 ` Petr Vorel
2022-01-14 18:27 ` Konrad Dybcio
2022-01-31 22:43 ` Bjorn Andersson
1 sibling, 1 reply; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:40 UTC (permalink / raw)
To: linux-arm-msm
Cc: Konrad Dybcio, Jean Thomas, Alexey Minnekhanov, Andy Gross,
Bjorn Andersson, Ulf Hansson, Douglas Anderson, Taniya Das,
linux-mmc
Hi all,
> Although downstream supports HS400, there are overclocking warnings when
> using mmc-hs400-1_8v:
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> mmc0: new HS400 MMC card at address 0001
> Using HS200 (i.e. mmc-hs200-1_8v or mmc-ddr-1_8v) would reduce them:
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: new HS200 MMC card at address 0001
> But as the problem is probably elsewhere (bullhead behaves the same),
> keep mmc-hs400-1_8v.
> Angler does not have SD card, thus explicitly disable sdhc2.
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
> .../dts/qcom/msm8994-huawei-angler-rev-101.dts | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
> diff --git a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> index 0e3dd48f0dbf..5ce3dc169bb4 100644
> --- a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> @@ -7,6 +7,7 @@
> /dts-v1/;
> #include "msm8994.dtsi"
> +#include <dt-bindings/gpio/gpio.h>
> /* Angler's firmware does not report where the memory is allocated */
> /delete-node/ &cont_splash_mem;
> @@ -41,3 +42,18 @@ serial@f991e000 {
> &tlmm {
> gpio-reserved-ranges = <85 4>;
> };
> +
> +/*
> + * Although downstream supports also HS400 there are fewer overclocking
> + * warnings when used DDR, also LK bootloader reports DDR mode.
> + */
This is a left over from experimenting with mmc-ddr-1_8v.
But maybe it can stay (unless I manage solve the problem).
Kind regards,
Petr
> +&sdhc1 {
> + status = "okay";
> +
> + mmc-hs400-1_8v;
> +};
> +
> +/* Angler does not have SD card */
> +&sdhc2 {
> + status = "disabled";
> +};
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-13 23:40 ` Petr Vorel
@ 2022-01-14 18:27 ` Konrad Dybcio
2022-01-14 21:16 ` Petr Vorel
0 siblings, 1 reply; 15+ messages in thread
From: Konrad Dybcio @ 2022-01-14 18:27 UTC (permalink / raw)
To: Petr Vorel, linux-arm-msm
Cc: Jean Thomas, Alexey Minnekhanov, Andy Gross, Bjorn Andersson,
Ulf Hansson, Douglas Anderson, Taniya Das, linux-mmc
Hi, please check if [1] can solve your clock issues, as it did for me.
Konrad
[1] https://patchwork.kernel.org/project/linux-arm-msm/patch/20211230023101.1122588-1-konrad.dybcio@somainline.org/
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-14 18:27 ` Konrad Dybcio
@ 2022-01-14 21:16 ` Petr Vorel
2022-01-14 21:19 ` Konrad Dybcio
0 siblings, 1 reply; 15+ messages in thread
From: Petr Vorel @ 2022-01-14 21:16 UTC (permalink / raw)
To: Konrad Dybcio
Cc: linux-arm-msm, Jean Thomas, Alexey Minnekhanov, Andy Gross,
Bjorn Andersson, Ulf Hansson, Douglas Anderson, Taniya Das,
linux-mmc
Hi Konrad, all,
> Hi, please check if [1] can solve your clock issues, as it did for me.
Thanks for info. I still need my patch "arm64: dts: qcom: msm8994: Provide
missing "xo_board" and "sleep_clk" to GCC" [2] regardless I use "clk: qcom:
gcc-msm8994: Remove NoC clocks" [1] or not.
Kind regards,
Petr
> Konrad
> [1] https://patchwork.kernel.org/project/linux-arm-msm/patch/20211230023101.1122588-1-konrad.dybcio@somainline.org/
[2] https://lore.kernel.org/linux-arm-msm/20220113233358.17972-4-petr.vorel@gmail.com/T/#u
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-14 21:16 ` Petr Vorel
@ 2022-01-14 21:19 ` Konrad Dybcio
0 siblings, 0 replies; 15+ messages in thread
From: Konrad Dybcio @ 2022-01-14 21:19 UTC (permalink / raw)
To: Petr Vorel
Cc: linux-arm-msm, Jean Thomas, Alexey Minnekhanov, Andy Gross,
Bjorn Andersson, Ulf Hansson, Douglas Anderson, Taniya Das,
linux-mmc
> Hi Konrad, all,
>
>> Hi, please check if [1] can solve your clock issues, as it did for me.
> Thanks for info. I still need my patch "arm64: dts: qcom: msm8994: Provide
> missing "xo_board" and "sleep_clk" to GCC" [2] regardless I use "clk: qcom:
> gcc-msm8994: Remove NoC clocks" [1] or not.
Yeah, that slipped my watchlist as well. I was supposed to send this (or have
sent in the past but then forgot to resubmit it? not sure tbf) before gcc changes
landed, but I forgot to. Providing the clocks is required.
Konrad
>
> Kind regards,
> Petr
>
>> Konrad
>
>> [1] https://patchwork.kernel.org/project/linux-arm-msm/patch/20211230023101.1122588-1-konrad.dybcio@somainline.org/
> [2] https://lore.kernel.org/linux-arm-msm/20220113233358.17972-4-petr.vorel@gmail.com/T/#u
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-13 23:33 ` [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions Petr Vorel
2022-01-13 23:40 ` Petr Vorel
@ 2022-01-31 22:43 ` Bjorn Andersson
2022-01-31 22:46 ` Petr Vorel
1 sibling, 1 reply; 15+ messages in thread
From: Bjorn Andersson @ 2022-01-31 22:43 UTC (permalink / raw)
To: Petr Vorel
Cc: linux-arm-msm, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Ulf Hansson, Douglas Anderson, Taniya Das, linux-mmc
On Thu 13 Jan 17:33 CST 2022, Petr Vorel wrote:
> Although downstream supports HS400, there are overclocking warnings when
> using mmc-hs400-1_8v:
>
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> mmc0: new HS400 MMC card at address 0001
>
> Using HS200 (i.e. mmc-hs200-1_8v or mmc-ddr-1_8v) would reduce them:
>
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> mmc0: new HS200 MMC card at address 0001
>
> But as the problem is probably elsewhere (bullhead behaves the same),
> keep mmc-hs400-1_8v.
>
> Angler does not have SD card, thus explicitly disable sdhc2.
>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
> .../dts/qcom/msm8994-huawei-angler-rev-101.dts | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> index 0e3dd48f0dbf..5ce3dc169bb4 100644
> --- a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> @@ -7,6 +7,7 @@
> /dts-v1/;
>
> #include "msm8994.dtsi"
> +#include <dt-bindings/gpio/gpio.h>
>
> /* Angler's firmware does not report where the memory is allocated */
> /delete-node/ &cont_splash_mem;
> @@ -41,3 +42,18 @@ serial@f991e000 {
> &tlmm {
> gpio-reserved-ranges = <85 4>;
> };
> +
> +/*
> + * Although downstream supports also HS400 there are fewer overclocking
> + * warnings when used DDR, also LK bootloader reports DDR mode.
> + */
> +&sdhc1 {
> + status = "okay";
> +
> + mmc-hs400-1_8v;
> +};
> +
> +/* Angler does not have SD card */
> +&sdhc2 {
But isn't &sdhc2 already disabled from msm8992.dtsi and msm8994.dtsi?
Regards,
Bjorn
> + status = "disabled";
> +};
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
2022-01-31 22:43 ` Bjorn Andersson
@ 2022-01-31 22:46 ` Petr Vorel
0 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-31 22:46 UTC (permalink / raw)
To: Bjorn Andersson
Cc: linux-arm-msm, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Ulf Hansson, Douglas Anderson, Taniya Das, linux-mmc
Hi Bjorn,
> On Thu 13 Jan 17:33 CST 2022, Petr Vorel wrote:
> > Although downstream supports HS400, there are overclocking warnings when
> > using mmc-hs400-1_8v:
> > mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> > mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> > mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> > mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> > mmc0: Card appears overclocked; req 400000000 Hz, actual 768000000 Hz
> > mmc0: new HS400 MMC card at address 0001
> > Using HS200 (i.e. mmc-hs200-1_8v or mmc-ddr-1_8v) would reduce them:
> > mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> > mmc0: Card appears overclocked; req 200000000 Hz, actual 384000000 Hz
> > mmc0: new HS200 MMC card at address 0001
> > But as the problem is probably elsewhere (bullhead behaves the same),
> > keep mmc-hs400-1_8v.
> > Angler does not have SD card, thus explicitly disable sdhc2.
> > Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> > ---
> > .../dts/qcom/msm8994-huawei-angler-rev-101.dts | 16 ++++++++++++++++
> > 1 file changed, 16 insertions(+)
> > diff --git a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > index 0e3dd48f0dbf..5ce3dc169bb4 100644
> > --- a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > +++ b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > @@ -7,6 +7,7 @@
> > /dts-v1/;
> > #include "msm8994.dtsi"
> > +#include <dt-bindings/gpio/gpio.h>
> > /* Angler's firmware does not report where the memory is allocated */
> > /delete-node/ &cont_splash_mem;
> > @@ -41,3 +42,18 @@ serial@f991e000 {
> > &tlmm {
> > gpio-reserved-ranges = <85 4>;
> > };
> > +
> > +/*
> > + * Although downstream supports also HS400 there are fewer overclocking
> > + * warnings when used DDR, also LK bootloader reports DDR mode.
> > + */
> > +&sdhc1 {
> > + status = "okay";
> > +
> > + mmc-hs400-1_8v;
> > +};
> > +
> > +/* Angler does not have SD card */
> > +&sdhc2 {
> But isn't &sdhc2 already disabled from msm8992.dtsi and msm8994.dtsi?
Yes it's disabled, thus this is not needed.
I'll send v2 of this, where I remove this.
Kind regards,
Petr
> Regards,
> Bjorn
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 6/6] arm64: dts: qcom: msm8992-lg-bullhead: Disable sdhc2
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
` (4 preceding siblings ...)
2022-01-13 23:33 ` [PATCH 5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions Petr Vorel
@ 2022-01-13 23:33 ` Petr Vorel
2022-01-31 21:00 ` [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes patchwork-bot+linux-arm-msm
2022-02-01 5:19 ` (subset) " Bjorn Andersson
7 siblings, 0 replies; 15+ messages in thread
From: Petr Vorel @ 2022-01-13 23:33 UTC (permalink / raw)
To: linux-arm-msm
Cc: Petr Vorel, Konrad Dybcio, Jean Thomas, Alexey Minnekhanov,
Andy Gross, Bjorn Andersson, Ulf Hansson, Douglas Anderson,
Taniya Das, linux-mmc
Bullhead does not have SD card, thus explicitly disable sdhc2.
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
index 3b0cc85d6674..b6633be30f68 100644
--- a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
@@ -300,3 +300,8 @@ &sdhc1 {
mmc-hs400-1_8v;
};
+
+/* Bullhead does not have SD card */
+&sdhc2 {
+ status = "disabled";
+};
--
2.34.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
` (5 preceding siblings ...)
2022-01-13 23:33 ` [PATCH 6/6] arm64: dts: qcom: msm8992-lg-bullhead: Disable sdhc2 Petr Vorel
@ 2022-01-31 21:00 ` patchwork-bot+linux-arm-msm
2022-02-01 5:19 ` (subset) " Bjorn Andersson
7 siblings, 0 replies; 15+ messages in thread
From: patchwork-bot+linux-arm-msm @ 2022-01-31 21:00 UTC (permalink / raw)
To: Petr Vorel; +Cc: linux-arm-msm
Hello:
This series was applied to qcom/linux.git (for-next)
by Bjorn Andersson <bjorn.andersson@linaro.org>:
On Fri, 14 Jan 2022 00:33:52 +0100 you wrote:
> Hi,
>
> Besides fixing sdhc1 for msm8994-huawei-angler and msm8992-lg-bullhead
> I also rebased and reposted Jean's patches (Makefile didn't apply on
> current for-next) and also rename angler's DTS.
>
> sdhc1 (i.e. mmc0) is working on both, but there are overclocking
> warnings. I suspected issue similar to using clk_rcg2_ops instead of
> clk_rcg2_floor_ops (i.e. 3f905469c8ce ("clk: qcom: gcc: Use floor ops
> for SDCC clocks") or 148ddaa89d4a ("clk: qcom: gcc-sc7180: Use floor ops
> for the correct sdcc1 clk"), but it looks to be correct for gcc-msm8994.c.
>
> [...]
Here is the summary with links:
- [1/6] arm64: dts: qcom: msm8992-lg-bullhead: Place LG Bullhead generic code into a DTSI file
https://git.kernel.org/qcom/c/3f99518c6f65
- [2/6] arm64: dts: qcom: msm8992-lg-bullhead: Add support for LG Bullhead rev 1.0
https://git.kernel.org/qcom/c/cd4bd4704ec8
- [3/6] arm64: dts: qcom: msm8994: Provide missing "xo_board" and "sleep_clk" to GCC
(no matching commit)
- [4/6] arm64: dts: qcom: msm8994-huawei-angler: Add vendor name huawei
(no matching commit)
- [5/6] arm64: dts: qcom: msm8994-huawei-angler: Add sdhc{1,2} definitions
(no matching commit)
- [6/6] arm64: dts: qcom: msm8992-lg-bullhead: Disable sdhc2
(no matching commit)
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: (subset) [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes
2022-01-13 23:33 [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes Petr Vorel
` (6 preceding siblings ...)
2022-01-31 21:00 ` [PATCH 0/6] msm8994-huawei-angler and msm8992-lg-bullhead sdhc1 fixes patchwork-bot+linux-arm-msm
@ 2022-02-01 5:19 ` Bjorn Andersson
7 siblings, 0 replies; 15+ messages in thread
From: Bjorn Andersson @ 2022-02-01 5:19 UTC (permalink / raw)
To: linux-arm-msm, Petr Vorel
Cc: Konrad Dybcio, Taniya Das, Ulf Hansson, Douglas Anderson,
Andy Gross, linux-mmc, Jean Thomas, Alexey Minnekhanov
On Fri, 14 Jan 2022 00:33:52 +0100, Petr Vorel wrote:
> Besides fixing sdhc1 for msm8994-huawei-angler and msm8992-lg-bullhead
> I also rebased and reposted Jean's patches (Makefile didn't apply on
> current for-next) and also rename angler's DTS.
>
> sdhc1 (i.e. mmc0) is working on both, but there are overclocking
> warnings. I suspected issue similar to using clk_rcg2_ops instead of
> clk_rcg2_floor_ops (i.e. 3f905469c8ce ("clk: qcom: gcc: Use floor ops
> for SDCC clocks") or 148ddaa89d4a ("clk: qcom: gcc-sc7180: Use floor ops
> for the correct sdcc1 clk"), but it looks to be correct for gcc-msm8994.c.
>
> [...]
Applied, thanks!
[3/6] arm64: dts: qcom: msm8994: Provide missing "xo_board" and "sleep_clk" to GCC
commit: 4dd1ad6192748523878463a285346db408b34a02
[4/6] arm64: dts: qcom: msm8994-huawei-angler: Add vendor name huawei
commit: 8af90d6daa36a7180a2cd6aad874136aade27412
Best regards,
--
Bjorn Andersson <bjorn.andersson@linaro.org>
^ permalink raw reply [flat|nested] 15+ messages in thread