* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
* This automated bisection report was sent to you on the basis *
* that you may be involved with the breaking commit it has *
* found. No manual investigation has been done to verify it, *
* and the root cause of the problem may be somewhere else. *
* *
* If you do send a fix, please include this trailer: *
* Reported-by: "kernelci.org bot" <[email protected]> *
* *
* Hope this helps! *
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
krzysztof/for-next boot bisection: v5.4-rc1-48-g0bc9c79979ea on peach-pi
Summary:
Start: 0bc9c79979ea Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
Details: https://kernelci.org/boot/id/5d9a50af59b5141ce5857c07
Plain log: https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.txt
HTML log: https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.html
Result: 0899a480ac65 ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
Checks:
revert: PASS
verify: PASS
Parameters:
Tree: krzysztof
URL: https://git.kernel.org/pub/scm/linux/kernel/git/krzk/linux.git
Branch: for-next
Target: peach-pi
CPU arch: arm
Lab: lab-collabora
Compiler: gcc-8
Config: multi_v7_defconfig
Test suite: boot
Breaking commit found:
-------------------------------------------------------------------------------
commit 0899a480ac658144b1fa351bb880c2858d43f824
Author: Marek Szyprowski <[email protected]>
Date: Thu Oct 3 12:08:14 2019 +0200
ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
Declare Exynos5422/5800 voltage ranges for opp points for big cpu core and
bus wcore and couple their voltage supllies as vdd_arm and vdd_int should
be in 300mV range.
Signed-off-by: Marek Szyprowski <[email protected]>
[k.konieczny: add missing patch description]
Signed-off-by: Kamil Konieczny <[email protected]>
Reviewed-by: Chanwoo Choi <[email protected]>
Signed-off-by: Krzysztof Kozlowski <[email protected]>
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 2c131ad78c09..d08101b1018c 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -48,62 +48,62 @@
opp-1800000000 {
opp-hz = /bits/ 64 <1800000000>;
- opp-microvolt = <1250000>;
+ opp-microvolt = <1250000 1250000 1500000>;
clock-latency-ns = <140000>;
};
opp-1700000000 {
opp-hz = /bits/ 64 <1700000000>;
- opp-microvolt = <1212500>;
+ opp-microvolt = <1212500 1212500 1500000>;
clock-latency-ns = <140000>;
};
opp-1600000000 {
opp-hz = /bits/ 64 <1600000000>;
- opp-microvolt = <1175000>;
+ opp-microvolt = <1175000 1175000 1500000>;
clock-latency-ns = <140000>;
};
opp-1500000000 {
opp-hz = /bits/ 64 <1500000000>;
- opp-microvolt = <1137500>;
+ opp-microvolt = <1137500 1137500 1500000>;
clock-latency-ns = <140000>;
};
opp-1400000000 {
opp-hz = /bits/ 64 <1400000000>;
- opp-microvolt = <1112500>;
+ opp-microvolt = <1112500 1112500 1500000>;
clock-latency-ns = <140000>;
};
opp-1300000000 {
opp-hz = /bits/ 64 <1300000000>;
- opp-microvolt = <1062500>;
+ opp-microvolt = <1062500 1062500 1500000>;
clock-latency-ns = <140000>;
};
opp-1200000000 {
opp-hz = /bits/ 64 <1200000000>;
- opp-microvolt = <1037500>;
+ opp-microvolt = <1037500 1037500 1500000>;
clock-latency-ns = <140000>;
};
opp-1100000000 {
opp-hz = /bits/ 64 <1100000000>;
- opp-microvolt = <1012500>;
+ opp-microvolt = <1012500 1012500 1500000>;
clock-latency-ns = <140000>;
};
opp-1000000000 {
opp-hz = /bits/ 64 <1000000000>;
- opp-microvolt = < 987500>;
+ opp-microvolt = < 987500 987500 1500000>;
clock-latency-ns = <140000>;
};
opp-900000000 {
opp-hz = /bits/ 64 <900000000>;
- opp-microvolt = < 962500>;
+ opp-microvolt = < 962500 962500 1500000>;
clock-latency-ns = <140000>;
};
opp-800000000 {
opp-hz = /bits/ 64 <800000000>;
- opp-microvolt = < 937500>;
+ opp-microvolt = < 937500 937500 1500000>;
clock-latency-ns = <140000>;
};
opp-700000000 {
opp-hz = /bits/ 64 <700000000>;
- opp-microvolt = < 912500>;
+ opp-microvolt = < 912500 912500 1500000>;
clock-latency-ns = <140000>;
};
};
@@ -1097,23 +1097,23 @@
opp00 {
opp-hz = /bits/ 64 <84000000>;
- opp-microvolt = <925000>;
+ opp-microvolt = <925000 925000 1400000>;
};
opp01 {
opp-hz = /bits/ 64 <111000000>;
- opp-microvolt = <950000>;
+ opp-microvolt = <950000 950000 1400000>;
};
opp02 {
opp-hz = /bits/ 64 <222000000>;
- opp-microvolt = <950000>;
+ opp-microvolt = <950000 950000 1400000>;
};
opp03 {
opp-hz = /bits/ 64 <333000000>;
- opp-microvolt = <950000>;
+ opp-microvolt = <950000 950000 1400000>;
};
opp04 {
opp-hz = /bits/ 64 <400000000>;
- opp-microvolt = <987500>;
+ opp-microvolt = <987500 987500 1400000>;
};
};
diff --git a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
index 829147e320e0..9b8de61b0385 100644
--- a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
+++ b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
@@ -524,6 +524,8 @@
regulator-max-microvolt = <1400000>;
regulator-always-on;
regulator-boot-on;
+ regulator-coupled-with = <&buck2_reg>;
+ regulator-coupled-max-spread = <300000>;
regulator-state-mem {
regulator-off-in-suspend;
@@ -544,6 +546,8 @@
regulator-max-microvolt = <1500000>;
regulator-always-on;
regulator-boot-on;
+ regulator-coupled-with = <&buck3_reg>;
+ regulator-coupled-max-spread = <300000>;
regulator-state-mem {
regulator-off-in-suspend;
diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
index 60ca3d685247..c1e38139ce4f 100644
--- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
+++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
@@ -257,6 +257,8 @@
regulator-always-on;
regulator-boot-on;
regulator-ramp-delay = <12500>;
+ regulator-coupled-with = <&buck3_reg>;
+ regulator-coupled-max-spread = <300000>;
regulator-state-mem {
regulator-off-in-suspend;
};
@@ -269,6 +271,8 @@
regulator-always-on;
regulator-boot-on;
regulator-ramp-delay = <12500>;
+ regulator-coupled-with = <&buck2_reg>;
+ regulator-coupled-max-spread = <300000>;
regulator-state-mem {
regulator-off-in-suspend;
};
diff --git a/arch/arm/boot/dts/exynos5800.dtsi b/arch/arm/boot/dts/exynos5800.dtsi
index de639eecc5c9..27789f5f9798 100644
--- a/arch/arm/boot/dts/exynos5800.dtsi
+++ b/arch/arm/boot/dts/exynos5800.dtsi
@@ -22,61 +22,61 @@
&cluster_a15_opp_table {
opp-1700000000 {
- opp-microvolt = <1250000>;
+ opp-microvolt = <1250000 1250000 1500000>;
};
opp-1600000000 {
- opp-microvolt = <1250000>;
+ opp-microvolt = <1250000 1250000 1500000>;
};
opp-1500000000 {
- opp-microvolt = <1100000>;
+ opp-microvolt = <1100000 1100000 1500000>;
};
opp-1400000000 {
- opp-microvolt = <1100000>;
+ opp-microvolt = <1100000 1100000 1500000>;
};
opp-1300000000 {
- opp-microvolt = <1100000>;
+ opp-microvolt = <1100000 1100000 1500000>;
};
opp-1200000000 {
- opp-microvolt = <1000000>;
+ opp-microvolt = <1000000 1000000 1500000>;
};
opp-1100000000 {
- opp-microvolt = <1000000>;
+ opp-microvolt = <1000000 1000000 1500000>;
};
opp-1000000000 {
- opp-microvolt = <1000000>;
+ opp-microvolt = <1000000 1000000 1500000>;
};
opp-900000000 {
- opp-microvolt = <1000000>;
+ opp-microvolt = <1000000 1000000 1500000>;
};
opp-800000000 {
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
};
opp-700000000 {
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
};
opp-600000000 {
opp-hz = /bits/ 64 <600000000>;
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
clock-latency-ns = <140000>;
};
opp-500000000 {
opp-hz = /bits/ 64 <500000000>;
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
clock-latency-ns = <140000>;
};
opp-400000000 {
opp-hz = /bits/ 64 <400000000>;
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
clock-latency-ns = <140000>;
};
opp-300000000 {
opp-hz = /bits/ 64 <300000000>;
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
clock-latency-ns = <140000>;
};
opp-200000000 {
opp-hz = /bits/ 64 <200000000>;
- opp-microvolt = <900000>;
+ opp-microvolt = <900000 900000 1500000>;
clock-latency-ns = <140000>;
};
};
-------------------------------------------------------------------------------
Git bisection log:
-------------------------------------------------------------------------------
git bisect start
# good: [2924a93b4c2b1934c0ec59d28f46814a83259f11] Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
git bisect good 2924a93b4c2b1934c0ec59d28f46814a83259f11
# bad: [0bc9c79979ea0b607a0751968840483fd296f6ef] Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
git bisect bad 0bc9c79979ea0b607a0751968840483fd296f6ef
# bad: [0899a480ac658144b1fa351bb880c2858d43f824] ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
git bisect bad 0899a480ac658144b1fa351bb880c2858d43f824
# good: [56c126e87e2980d5e2ca5d77b28899f8521af9d7] ARM: dts: exynos: Rename SysRAM node to "sram"
git bisect good 56c126e87e2980d5e2ca5d77b28899f8521af9d7
# first bad commit: [0899a480ac658144b1fa351bb880c2858d43f824] ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
-------------------------------------------------------------------------------
On Mon, 7 Oct 2019 at 02:43, kernelci.org bot <[email protected]> wrote:
>
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> * This automated bisection report was sent to you on the basis *
> * that you may be involved with the breaking commit it has *
> * found. No manual investigation has been done to verify it, *
> * and the root cause of the problem may be somewhere else. *
> * *
> * If you do send a fix, please include this trailer: *
> * Reported-by: "kernelci.org bot" <[email protected]> *
> * *
> * Hope this helps! *
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
>
> krzysztof/for-next boot bisection: v5.4-rc1-48-g0bc9c79979ea on peach-pi
>
> Summary:
> Start: 0bc9c79979ea Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
> Details: https://kernelci.org/boot/id/5d9a50af59b5141ce5857c07
> Plain log: https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.txt
> HTML log: https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.html
> Result: 0899a480ac65 ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
Thanks for the report. Marek Szyprowski reported it on last Friday. I
dropped the patch as of now as it exposes some deadlock in regulator
code.
Best regards,
Krzysztof
>
> Checks:
> revert: PASS
> verify: PASS
>
> Parameters:
> Tree: krzysztof
> URL: https://git.kernel.org/pub/scm/linux/kernel/git/krzk/linux.git
> Branch: for-next
> Target: peach-pi
> CPU arch: arm
> Lab: lab-collabora
> Compiler: gcc-8
> Config: multi_v7_defconfig
> Test suite: boot
>
> Breaking commit found:
>
> -------------------------------------------------------------------------------
> commit 0899a480ac658144b1fa351bb880c2858d43f824
> Author: Marek Szyprowski <[email protected]>
> Date: Thu Oct 3 12:08:14 2019 +0200
>
> ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
>
> Declare Exynos5422/5800 voltage ranges for opp points for big cpu core and
> bus wcore and couple their voltage supllies as vdd_arm and vdd_int should
> be in 300mV range.
>
> Signed-off-by: Marek Szyprowski <[email protected]>
> [k.konieczny: add missing patch description]
> Signed-off-by: Kamil Konieczny <[email protected]>
> Reviewed-by: Chanwoo Choi <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
> index 2c131ad78c09..d08101b1018c 100644
> --- a/arch/arm/boot/dts/exynos5420.dtsi
> +++ b/arch/arm/boot/dts/exynos5420.dtsi
> @@ -48,62 +48,62 @@
>
> opp-1800000000 {
> opp-hz = /bits/ 64 <1800000000>;
> - opp-microvolt = <1250000>;
> + opp-microvolt = <1250000 1250000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1700000000 {
> opp-hz = /bits/ 64 <1700000000>;
> - opp-microvolt = <1212500>;
> + opp-microvolt = <1212500 1212500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1600000000 {
> opp-hz = /bits/ 64 <1600000000>;
> - opp-microvolt = <1175000>;
> + opp-microvolt = <1175000 1175000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1500000000 {
> opp-hz = /bits/ 64 <1500000000>;
> - opp-microvolt = <1137500>;
> + opp-microvolt = <1137500 1137500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1400000000 {
> opp-hz = /bits/ 64 <1400000000>;
> - opp-microvolt = <1112500>;
> + opp-microvolt = <1112500 1112500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1300000000 {
> opp-hz = /bits/ 64 <1300000000>;
> - opp-microvolt = <1062500>;
> + opp-microvolt = <1062500 1062500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1200000000 {
> opp-hz = /bits/ 64 <1200000000>;
> - opp-microvolt = <1037500>;
> + opp-microvolt = <1037500 1037500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1100000000 {
> opp-hz = /bits/ 64 <1100000000>;
> - opp-microvolt = <1012500>;
> + opp-microvolt = <1012500 1012500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-1000000000 {
> opp-hz = /bits/ 64 <1000000000>;
> - opp-microvolt = < 987500>;
> + opp-microvolt = < 987500 987500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-900000000 {
> opp-hz = /bits/ 64 <900000000>;
> - opp-microvolt = < 962500>;
> + opp-microvolt = < 962500 962500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-800000000 {
> opp-hz = /bits/ 64 <800000000>;
> - opp-microvolt = < 937500>;
> + opp-microvolt = < 937500 937500 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-700000000 {
> opp-hz = /bits/ 64 <700000000>;
> - opp-microvolt = < 912500>;
> + opp-microvolt = < 912500 912500 1500000>;
> clock-latency-ns = <140000>;
> };
> };
> @@ -1097,23 +1097,23 @@
>
> opp00 {
> opp-hz = /bits/ 64 <84000000>;
> - opp-microvolt = <925000>;
> + opp-microvolt = <925000 925000 1400000>;
> };
> opp01 {
> opp-hz = /bits/ 64 <111000000>;
> - opp-microvolt = <950000>;
> + opp-microvolt = <950000 950000 1400000>;
> };
> opp02 {
> opp-hz = /bits/ 64 <222000000>;
> - opp-microvolt = <950000>;
> + opp-microvolt = <950000 950000 1400000>;
> };
> opp03 {
> opp-hz = /bits/ 64 <333000000>;
> - opp-microvolt = <950000>;
> + opp-microvolt = <950000 950000 1400000>;
> };
> opp04 {
> opp-hz = /bits/ 64 <400000000>;
> - opp-microvolt = <987500>;
> + opp-microvolt = <987500 987500 1400000>;
> };
> };
>
> diff --git a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> index 829147e320e0..9b8de61b0385 100644
> --- a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> +++ b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi
> @@ -524,6 +524,8 @@
> regulator-max-microvolt = <1400000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-coupled-with = <&buck2_reg>;
> + regulator-coupled-max-spread = <300000>;
>
> regulator-state-mem {
> regulator-off-in-suspend;
> @@ -544,6 +546,8 @@
> regulator-max-microvolt = <1500000>;
> regulator-always-on;
> regulator-boot-on;
> + regulator-coupled-with = <&buck3_reg>;
> + regulator-coupled-max-spread = <300000>;
>
> regulator-state-mem {
> regulator-off-in-suspend;
> diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> index 60ca3d685247..c1e38139ce4f 100644
> --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
> +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> @@ -257,6 +257,8 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay = <12500>;
> + regulator-coupled-with = <&buck3_reg>;
> + regulator-coupled-max-spread = <300000>;
> regulator-state-mem {
> regulator-off-in-suspend;
> };
> @@ -269,6 +271,8 @@
> regulator-always-on;
> regulator-boot-on;
> regulator-ramp-delay = <12500>;
> + regulator-coupled-with = <&buck2_reg>;
> + regulator-coupled-max-spread = <300000>;
> regulator-state-mem {
> regulator-off-in-suspend;
> };
> diff --git a/arch/arm/boot/dts/exynos5800.dtsi b/arch/arm/boot/dts/exynos5800.dtsi
> index de639eecc5c9..27789f5f9798 100644
> --- a/arch/arm/boot/dts/exynos5800.dtsi
> +++ b/arch/arm/boot/dts/exynos5800.dtsi
> @@ -22,61 +22,61 @@
>
> &cluster_a15_opp_table {
> opp-1700000000 {
> - opp-microvolt = <1250000>;
> + opp-microvolt = <1250000 1250000 1500000>;
> };
> opp-1600000000 {
> - opp-microvolt = <1250000>;
> + opp-microvolt = <1250000 1250000 1500000>;
> };
> opp-1500000000 {
> - opp-microvolt = <1100000>;
> + opp-microvolt = <1100000 1100000 1500000>;
> };
> opp-1400000000 {
> - opp-microvolt = <1100000>;
> + opp-microvolt = <1100000 1100000 1500000>;
> };
> opp-1300000000 {
> - opp-microvolt = <1100000>;
> + opp-microvolt = <1100000 1100000 1500000>;
> };
> opp-1200000000 {
> - opp-microvolt = <1000000>;
> + opp-microvolt = <1000000 1000000 1500000>;
> };
> opp-1100000000 {
> - opp-microvolt = <1000000>;
> + opp-microvolt = <1000000 1000000 1500000>;
> };
> opp-1000000000 {
> - opp-microvolt = <1000000>;
> + opp-microvolt = <1000000 1000000 1500000>;
> };
> opp-900000000 {
> - opp-microvolt = <1000000>;
> + opp-microvolt = <1000000 1000000 1500000>;
> };
> opp-800000000 {
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> };
> opp-700000000 {
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> };
> opp-600000000 {
> opp-hz = /bits/ 64 <600000000>;
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-500000000 {
> opp-hz = /bits/ 64 <500000000>;
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-400000000 {
> opp-hz = /bits/ 64 <400000000>;
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-300000000 {
> opp-hz = /bits/ 64 <300000000>;
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> clock-latency-ns = <140000>;
> };
> opp-200000000 {
> opp-hz = /bits/ 64 <200000000>;
> - opp-microvolt = <900000>;
> + opp-microvolt = <900000 900000 1500000>;
> clock-latency-ns = <140000>;
> };
> };
> -------------------------------------------------------------------------------
>
>
> Git bisection log:
>
> -------------------------------------------------------------------------------
> git bisect start
> # good: [2924a93b4c2b1934c0ec59d28f46814a83259f11] Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
> git bisect good 2924a93b4c2b1934c0ec59d28f46814a83259f11
> # bad: [0bc9c79979ea0b607a0751968840483fd296f6ef] Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
> git bisect bad 0bc9c79979ea0b607a0751968840483fd296f6ef
> # bad: [0899a480ac658144b1fa351bb880c2858d43f824] ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
> git bisect bad 0899a480ac658144b1fa351bb880c2858d43f824
> # good: [56c126e87e2980d5e2ca5d77b28899f8521af9d7] ARM: dts: exynos: Rename SysRAM node to "sram"
> git bisect good 56c126e87e2980d5e2ca5d77b28899f8521af9d7
> # first bad commit: [0899a480ac658144b1fa351bb880c2858d43f824] ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
> -------------------------------------------------------------------------------
Hi Krzysztof,
On 07.10.2019 19:09, Krzysztof Kozlowski wrote:
> On Mon, 7 Oct 2019 at 02:43, kernelci.org bot <[email protected]> wrote:
>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
>> * This automated bisection report was sent to you on the basis *
>> * that you may be involved with the breaking commit it has *
>> * found. No manual investigation has been done to verify it, *
>> * and the root cause of the problem may be somewhere else. *
>> * *
>> * If you do send a fix, please include this trailer: *
>> * Reported-by: "kernelci.org bot" <[email protected]> *
>> * *
>> * Hope this helps! *
>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
>>
>> krzysztof/for-next boot bisection: v5.4-rc1-48-g0bc9c79979ea on peach-pi
>>
>> Summary:
>> Start: 0bc9c79979ea Merge branch 'for-v5.5/memory-samsung-dmc-dt' into for-next
>> Details: https://protect2.fireeye.com/url?k=57d30a4c-0ab44e6a-57d28103-0cc47a31384a-5c0d93fa4eb95f2e&u=https://kernelci.org/boot/id/5d9a50af59b5141ce5857c07
>> Plain log: https://protect2.fireeye.com/url?k=7e18d090-237f94b6-7e195bdf-0cc47a31384a-b693d6da384e8a14&u=https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.txt
>> HTML log: https://protect2.fireeye.com/url?k=e9c39b5b-b4a4df7d-e9c21014-0cc47a31384a-5f31e671f969cef8&u=https://storage.kernelci.org//krzysztof/for-next/v5.4-rc1-48-g0bc9c79979ea/arm/multi_v7_defconfig/gcc-8/lab-collabora/boot-exynos5800-peach-pi.html
>> Result: 0899a480ac65 ARM: dts: exynos: Add initial data for coupled regulators for Exynos5422/5800
> Thanks for the report. Marek Szyprowski reported it on last Friday. I
> dropped the patch as of now as it exposes some deadlock in regulator
> code.
Thanks for dropping this patch.
This turned out to be an issue in the regulator core logic, not the
deadlock as I initially suspected. See
https://lkml.org/lkml/2019/10/8/265 for more details.
I will send an updated version of this patch, because it turned out that
the v7 had incorrectly resolved conflicts and in case of Odroids
coupling were added to wrong regulators, what hide the issue in my
initial tests on Odroid XU4.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland