[PATCH] devfreq: exynos-bus: workaround dev_pm_opp_set_rate() errors on Exynos5422/5800 SoCs

2019-10-08 Thread k . konieczny
Commit 4294a779bd8d ("PM / devfreq: exynos-bus: Convert to use dev_pm_opp_set_rate()") introduced errors: exynos-bus: new bus device registered: soc:bus_wcore ( 84000 KHz ~ 40 KHz) exynos-bus: new bus device registered: soc:bus_noc ( 67000 KHz ~ 10 KHz) exynos-bus: new bus device registered

[PATCH v5 2/4] devfreq: exynos-bus: convert to use dev_pm_opp_set_rate()

2019-08-07 Thread k . konieczny
Reuse opp core code for setting bus clock and voltage. As a side effect this allow usage of coupled regulators feature (required for boards using Exynos5422/5800 SoCs) because dev_pm_opp_set_rate() uses regulator_set_voltage_triplet() for setting regulator voltage while the old code used regulator_

[PATCH v5 0/4] add coupled regulators for Exynos5422/5800

2019-08-07 Thread k . konieczny
Hi, The main purpose of this patch series is to add coupled regulators for Exynos5422/5800 to keep constrain on voltage difference between vdd_arm and vdd_int to be at most 300mV. In exynos-bus instead of using regulator_set_voltage_tol() with default voltage tolerance it should be used regulator_

[PATCH v5 1/4] devfreq: exynos-bus: correct clock enable sequence

2019-08-07 Thread k . konieczny
Regulators should be enabled before clocks to avoid h/w hang. This require change in exynos_bus_probe() to move exynos_bus_parse_of() after exynos_bus_parent_parse_of() and change in error handling. Similar change is needed in exynos_bus_exit() where clock should be disabled before regulators. Sig

[PATCH v5 3/4] ARM: dts: exynos: add initial data for coupled regulators for Exynos5422/5800

2019-08-07 Thread k . konieczny
From: Marek Szyprowski 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 [k.konieczny: add missing patch description] Signed-off-by: Kamil Koniec

[PATCH v5 4/4] dt-bindings: devfreq: exynos-bus: remove unused property

2019-08-07 Thread k . konieczny
Remove unused DT property "exynos,voltage-tolerance". Signed-off-by: Kamil Konieczny Acked-by: Chanwoo Choi --- Documentation/devicetree/bindings/devfreq/exynos-bus.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/Documentation/devicetree/bindings/devfreq/exynos-bus.txt b/Documentation

[PATCH v4 4/5] dt-bindings: devfreq: exynos-bus: remove unused property

2019-07-25 Thread k . konieczny
Remove unused DT property "exynos,voltage-tolerance". Signed-off-by: Kamil Konieczny Acked-by: Chanwoo Choi --- Documentation/devicetree/bindings/devfreq/exynos-bus.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/Documentation/devicetree/bindings/devfreq/exynos-bus.txt b/Documentation

[PATCH v4 2/5] devfreq: exynos-bus: convert to use dev_pm_opp_set_rate()

2019-07-25 Thread k . konieczny
Reuse opp core code for setting bus clock and voltage. As a side effect this allow usage of coupled regulators feature (required for boards using Exynos5422/5800 SoCs) because dev_pm_opp_set_rate() uses regulator_set_voltage_triplet() for setting regulator voltage while the old code used regulator_

[PATCH v4 1/5] devfreq: exynos-bus: correct clock enable sequence

2019-07-25 Thread k . konieczny
Regulators should be enabled before clocks to avoid h/w hang. This require change in exynos_bus_probe() to move exynos_bus_parse_of() after exynos_bus_parent_parse_of() and change in error handling. Similar change is needed in exynos_bus_exit() where clock should be disabled before regulators. Sig

[PATCH v4 5/5] devfreq: exynos-bus: remove exynos_bus_passive_target()

2019-07-25 Thread k . konieczny
Both functions exynos_bus_passive_target() and exynos_bus_target() have the same code, so remove exynos_bus_passive_target(). In exynos_bus_probe() replace it with exynos_bus_target. Suggested-by: Chanwoo Choi Signed-off-by: Kamil Konieczny --- This patch is new to this series. --- drivers/dev

[PATCH v4 3/5] ARM: dts: exynos: add initial data for coupled regulators for Exynos5422/5800

2019-07-25 Thread k . konieczny
From: Marek Szyprowski 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 [k.konieczny: add missing patch description] Signed-off-by: Kamil Koniec

[PATCH v4 0/5] add coupled regulators for Exynos5422/5800

2019-07-25 Thread k . konieczny
Hi, The main purpose of this patch series is to add coupled regulators for Exynos5422/5800 to keep constrain on voltage difference between vdd_arm and vdd_int to be at most 300mV. In exynos-bus instead of using regulator_set_voltage_tol() with default voltage tolerance it should be used regulator_

[PATCH v3 4/5] ARM: dts: exynos: add initial data for coupled regulators for Exynos5422/5800

2019-07-19 Thread k . konieczny
From: Marek Szyprowski 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 [k.konieczny: add missing patch description] Signed-off-by: Kamil Koniec

[PATCH v3 2/5] opp: core: add regulators enable and disable

2019-07-19 Thread k . konieczny
Add enable regulators to dev_pm_opp_set_regulators() and disable regulators to dev_pm_opp_put_regulators(). Even if bootloader leaves regulators enabled, they should be enabled in kernel in order to increase the reference count. Signed-off-by: Kamil Konieczny --- Changes in v3: - corrected error

[PATCH v3 5/5] dt-bindings: devfreq: exynos-bus: remove unused property

2019-07-19 Thread k . konieczny
Remove unused DT property "exynos,voltage-tolerance". Signed-off-by: Kamil Konieczny Acked-by: Chanwoo Choi --- Documentation/devicetree/bindings/devfreq/exynos-bus.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/Documentation/devicetree/bindings/devfreq/exynos-bus.txt b/Documentation

[PATCH v3 3/5] devfreq: exynos-bus: convert to use dev_pm_opp_set_rate()

2019-07-19 Thread k . konieczny
Reuse opp core code for setting bus clock and voltage. As a side effect this allow useage of coupled regulators feature (required for boards using Exynos5422/5800 SoCs) because dev_pm_opp_set_rate() uses regulator_set_voltage_triplet() for setting regulator voltage while the old code used regulator

[PATCH v3 1/5] devfreq: exynos-bus: correct clock enable sequence

2019-07-19 Thread k . konieczny
Regulators should be enabled before clocks to avoid h/w hang. This require change in exynos_bus_probe() to move exynos_bus_parse_of() after exynos_bus_parent_parse_of() and change in enabling sequence of regulator and clock in exynos_bus_parse_of(). Similar change is needed in exynos_bus_exit() whe

[PATCH v3 0/5] add coupled regulators for Exynos5422/5800

2019-07-19 Thread k . konieczny
Hi, The main purpose of this patch series is to add coupled regulators for Exynos5422/5800 to keep constrain on voltage difference between vdd_arm and vdd_int to be at most 300mV. In exynos-bus instead of using regulator_set_voltage_tol() with default voltage tolerance it should be used regulator_

[PATCH 3/3] ARM: dts: exynos: add initial data for coupled regulators for Exynos5422/5800

2019-07-08 Thread k . konieczny
From: Marek Szyprowski 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 Signed-off-by: Kamil Konieczny --- arch/arm/boot/dts/exynos5420.dtsi

[PATCH 2/3] devfreq: exynos-bus: convert to use dev_pm_opp_set_rate()

2019-07-08 Thread k . konieczny
From: Kamil Konieczny Reuse opp core code for setting bus clock and voltage. As a side effect this allow useage of coupled regulators feature (required for boards using Exynos5422/5800 SoCs) because dev_pm_opp_set_rate() uses regulator_set_voltage_triplet() for setting regulator voltage while the

[PATCH 1/3] opp: core: add regulators enable and disable

2019-07-08 Thread k . konieczny
From: Kamil Konieczny Add enable regulators to dev_pm_opp_set_regulators() and disable regulators to dev_pm_opp_put_regulators(). This prepares for converting exynos-bus devfreq driver to use dev_pm_opp_set_rate(). Signed-off-by: Kamil Konieczny --- drivers/opp/core.c | 13 + 1 fil

[PATCH 0/3] add coupled regulators for Exynos5422/5800

2019-07-08 Thread k . konieczny
From: Kamil Konieczny Hi, The main purpose of this patch series is to add coupled regulators for Exynos5422/5800 to keep constrain on voltage difference between vdd_arm and vdd_int to be at most 300mV. In exynos-bus instead of using regulator_set_voltage_tol() with default voltage tolerance it s