* [PATCH v2] regulator: axp20x: AXP717: set ramp_delay
@ 2024-12-08 12:43 Philippe Simons
2024-12-08 13:32 ` Chen-Yu Tsai
2024-12-10 13:00 ` Mark Brown
0 siblings, 2 replies; 3+ messages in thread
From: Philippe Simons @ 2024-12-08 12:43 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Chen-Yu Tsai,
open list:VOLTAGE AND CURRENT REGULATOR FRAMEWORK
Cc: Philippe Simons, Hironori KIKUCHI, Chris Morgan
AXP717 datasheet says that regulator ramp delay is 15.625 us/step,
which is 10mV in our case.
Add a AXP_DESC_RANGES_DELAY macro and update AXP_DESC_RANGES macro to
expand to AXP_DESC_RANGES_DELAY with ramp_delay = 0
For DCDC4, steps is 100mv
Add a AXP_DESC_DELAY macro and update AXP_DESC macro to
expand to AXP_DESC_DELAY with ramp_delay = 0
This patch fix crashes when using CPU DVFS.
Signed-off-by: Philippe Simons <simons.philippe@gmail.com>
Tested-by: Hironori KIKUCHI <kikuchan98@gmail.com>
Tested-by: Chris Morgan <macromorgan@hotmail.com>
---
Changes in v2:
- set ramp_delay for AXP717 DCDC4
- Link to v1: https://lore.kernel.org/all/20241206123751.981977-1-simons.philippe@gmail.com/
---
drivers/regulator/axp20x-regulator.c | 36 ++++++++++++++++++----------
1 file changed, 24 insertions(+), 12 deletions(-)
diff --git a/drivers/regulator/axp20x-regulator.c b/drivers/regulator/axp20x-regulator.c
index a8e91d9d0..945d2917b 100644
--- a/drivers/regulator/axp20x-regulator.c
+++ b/drivers/regulator/axp20x-regulator.c
@@ -371,8 +371,8 @@
.ops = &axp20x_ops, \
}
-#define AXP_DESC(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
- _vmask, _ereg, _emask) \
+#define AXP_DESC_DELAY(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
+ _vmask, _ereg, _emask, _ramp_delay) \
[_family##_##_id] = { \
.name = (_match), \
.supply_name = (_supply), \
@@ -388,9 +388,15 @@
.vsel_mask = (_vmask), \
.enable_reg = (_ereg), \
.enable_mask = (_emask), \
+ .ramp_delay = (_ramp_delay), \
.ops = &axp20x_ops, \
}
+#define AXP_DESC(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
+ _vmask, _ereg, _emask) \
+ AXP_DESC_DELAY(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
+ _vmask, _ereg, _emask, 0)
+
#define AXP_DESC_SW(_family, _id, _match, _supply, _ereg, _emask) \
[_family##_##_id] = { \
.name = (_match), \
@@ -419,8 +425,8 @@
.ops = &axp20x_ops_fixed \
}
-#define AXP_DESC_RANGES(_family, _id, _match, _supply, _ranges, _n_voltages, \
- _vreg, _vmask, _ereg, _emask) \
+#define AXP_DESC_RANGES_DELAY(_family, _id, _match, _supply, _ranges, _n_voltages, \
+ _vreg, _vmask, _ereg, _emask, _ramp_delay) \
[_family##_##_id] = { \
.name = (_match), \
.supply_name = (_supply), \
@@ -436,9 +442,15 @@
.enable_mask = (_emask), \
.linear_ranges = (_ranges), \
.n_linear_ranges = ARRAY_SIZE(_ranges), \
+ .ramp_delay = (_ramp_delay), \
.ops = &axp20x_ops_range, \
}
+#define AXP_DESC_RANGES(_family, _id, _match, _supply, _ranges, _n_voltages, \
+ _vreg, _vmask, _ereg, _emask) \
+ AXP_DESC_RANGES_DELAY(_family, _id, _match, _supply, _ranges, \
+ _n_voltages, _vreg, _vmask, _ereg, _emask, 0)
+
static const int axp209_dcdc2_ldo3_slew_rates[] = {
1600,
800,
@@ -781,21 +793,21 @@ static const struct linear_range axp717_dcdc3_ranges[] = {
};
static const struct regulator_desc axp717_regulators[] = {
- AXP_DESC_RANGES(AXP717, DCDC1, "dcdc1", "vin1",
+ AXP_DESC_RANGES_DELAY(AXP717, DCDC1, "dcdc1", "vin1",
axp717_dcdc1_ranges, AXP717_DCDC1_NUM_VOLTAGES,
AXP717_DCDC1_CONTROL, AXP717_DCDC_V_OUT_MASK,
- AXP717_DCDC_OUTPUT_CONTROL, BIT(0)),
- AXP_DESC_RANGES(AXP717, DCDC2, "dcdc2", "vin2",
+ AXP717_DCDC_OUTPUT_CONTROL, BIT(0), 640),
+ AXP_DESC_RANGES_DELAY(AXP717, DCDC2, "dcdc2", "vin2",
axp717_dcdc2_ranges, AXP717_DCDC2_NUM_VOLTAGES,
AXP717_DCDC2_CONTROL, AXP717_DCDC_V_OUT_MASK,
- AXP717_DCDC_OUTPUT_CONTROL, BIT(1)),
- AXP_DESC_RANGES(AXP717, DCDC3, "dcdc3", "vin3",
+ AXP717_DCDC_OUTPUT_CONTROL, BIT(1), 640),
+ AXP_DESC_RANGES_DELAY(AXP717, DCDC3, "dcdc3", "vin3",
axp717_dcdc3_ranges, AXP717_DCDC3_NUM_VOLTAGES,
AXP717_DCDC3_CONTROL, AXP717_DCDC_V_OUT_MASK,
- AXP717_DCDC_OUTPUT_CONTROL, BIT(2)),
- AXP_DESC(AXP717, DCDC4, "dcdc4", "vin4", 1000, 3700, 100,
+ AXP717_DCDC_OUTPUT_CONTROL, BIT(2), 640),
+ AXP_DESC_DELAY(AXP717, DCDC4, "dcdc4", "vin4", 1000, 3700, 100,
AXP717_DCDC4_CONTROL, AXP717_DCDC_V_OUT_MASK,
- AXP717_DCDC_OUTPUT_CONTROL, BIT(3)),
+ AXP717_DCDC_OUTPUT_CONTROL, BIT(3), 6400),
AXP_DESC(AXP717, ALDO1, "aldo1", "aldoin", 500, 3500, 100,
AXP717_ALDO1_CONTROL, AXP717_LDO_V_OUT_MASK,
AXP717_LDO0_OUTPUT_CONTROL, BIT(0)),
--
2.47.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] regulator: axp20x: AXP717: set ramp_delay
2024-12-08 12:43 [PATCH v2] regulator: axp20x: AXP717: set ramp_delay Philippe Simons
@ 2024-12-08 13:32 ` Chen-Yu Tsai
2024-12-10 13:00 ` Mark Brown
1 sibling, 0 replies; 3+ messages in thread
From: Chen-Yu Tsai @ 2024-12-08 13:32 UTC (permalink / raw)
To: Philippe Simons
Cc: Liam Girdwood, Mark Brown,
open list:VOLTAGE AND CURRENT REGULATOR FRAMEWORK,
Hironori KIKUCHI, Chris Morgan
On Sun, Dec 8, 2024 at 8:43 PM Philippe Simons
<simons.philippe@gmail.com> wrote:
>
> AXP717 datasheet says that regulator ramp delay is 15.625 us/step,
> which is 10mV in our case.
>
> Add a AXP_DESC_RANGES_DELAY macro and update AXP_DESC_RANGES macro to
> expand to AXP_DESC_RANGES_DELAY with ramp_delay = 0
>
> For DCDC4, steps is 100mv
>
> Add a AXP_DESC_DELAY macro and update AXP_DESC macro to
> expand to AXP_DESC_DELAY with ramp_delay = 0
>
> This patch fix crashes when using CPU DVFS.
>
> Signed-off-by: Philippe Simons <simons.philippe@gmail.com>
> Tested-by: Hironori KIKUCHI <kikuchan98@gmail.com>
> Tested-by: Chris Morgan <macromorgan@hotmail.com>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
The following tags should also be added:
Fixes: d2ac3df75c3a ("regulator: axp20x: add support for the AXP717")
Cc: <stable@vger.kernel.org> # v6.10+
You probably don't need to send a new version just to add them. The scripts
used by maintainers should be able to pick them up together.
> ---
> Changes in v2:
> - set ramp_delay for AXP717 DCDC4
> - Link to v1: https://lore.kernel.org/all/20241206123751.981977-1-simons.philippe@gmail.com/
> ---
> drivers/regulator/axp20x-regulator.c | 36 ++++++++++++++++++----------
> 1 file changed, 24 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/regulator/axp20x-regulator.c b/drivers/regulator/axp20x-regulator.c
> index a8e91d9d0..945d2917b 100644
> --- a/drivers/regulator/axp20x-regulator.c
> +++ b/drivers/regulator/axp20x-regulator.c
> @@ -371,8 +371,8 @@
> .ops = &axp20x_ops, \
> }
>
> -#define AXP_DESC(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
> - _vmask, _ereg, _emask) \
> +#define AXP_DESC_DELAY(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
> + _vmask, _ereg, _emask, _ramp_delay) \
> [_family##_##_id] = { \
> .name = (_match), \
> .supply_name = (_supply), \
> @@ -388,9 +388,15 @@
> .vsel_mask = (_vmask), \
> .enable_reg = (_ereg), \
> .enable_mask = (_emask), \
> + .ramp_delay = (_ramp_delay), \
> .ops = &axp20x_ops, \
> }
>
> +#define AXP_DESC(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
> + _vmask, _ereg, _emask) \
> + AXP_DESC_DELAY(_family, _id, _match, _supply, _min, _max, _step, _vreg, \
> + _vmask, _ereg, _emask, 0)
> +
> #define AXP_DESC_SW(_family, _id, _match, _supply, _ereg, _emask) \
> [_family##_##_id] = { \
> .name = (_match), \
> @@ -419,8 +425,8 @@
> .ops = &axp20x_ops_fixed \
> }
>
> -#define AXP_DESC_RANGES(_family, _id, _match, _supply, _ranges, _n_voltages, \
> - _vreg, _vmask, _ereg, _emask) \
> +#define AXP_DESC_RANGES_DELAY(_family, _id, _match, _supply, _ranges, _n_voltages, \
> + _vreg, _vmask, _ereg, _emask, _ramp_delay) \
> [_family##_##_id] = { \
> .name = (_match), \
> .supply_name = (_supply), \
> @@ -436,9 +442,15 @@
> .enable_mask = (_emask), \
> .linear_ranges = (_ranges), \
> .n_linear_ranges = ARRAY_SIZE(_ranges), \
> + .ramp_delay = (_ramp_delay), \
> .ops = &axp20x_ops_range, \
> }
>
> +#define AXP_DESC_RANGES(_family, _id, _match, _supply, _ranges, _n_voltages, \
> + _vreg, _vmask, _ereg, _emask) \
> + AXP_DESC_RANGES_DELAY(_family, _id, _match, _supply, _ranges, \
> + _n_voltages, _vreg, _vmask, _ereg, _emask, 0)
> +
> static const int axp209_dcdc2_ldo3_slew_rates[] = {
> 1600,
> 800,
> @@ -781,21 +793,21 @@ static const struct linear_range axp717_dcdc3_ranges[] = {
> };
>
> static const struct regulator_desc axp717_regulators[] = {
> - AXP_DESC_RANGES(AXP717, DCDC1, "dcdc1", "vin1",
> + AXP_DESC_RANGES_DELAY(AXP717, DCDC1, "dcdc1", "vin1",
> axp717_dcdc1_ranges, AXP717_DCDC1_NUM_VOLTAGES,
> AXP717_DCDC1_CONTROL, AXP717_DCDC_V_OUT_MASK,
> - AXP717_DCDC_OUTPUT_CONTROL, BIT(0)),
> - AXP_DESC_RANGES(AXP717, DCDC2, "dcdc2", "vin2",
> + AXP717_DCDC_OUTPUT_CONTROL, BIT(0), 640),
> + AXP_DESC_RANGES_DELAY(AXP717, DCDC2, "dcdc2", "vin2",
> axp717_dcdc2_ranges, AXP717_DCDC2_NUM_VOLTAGES,
> AXP717_DCDC2_CONTROL, AXP717_DCDC_V_OUT_MASK,
> - AXP717_DCDC_OUTPUT_CONTROL, BIT(1)),
> - AXP_DESC_RANGES(AXP717, DCDC3, "dcdc3", "vin3",
> + AXP717_DCDC_OUTPUT_CONTROL, BIT(1), 640),
> + AXP_DESC_RANGES_DELAY(AXP717, DCDC3, "dcdc3", "vin3",
> axp717_dcdc3_ranges, AXP717_DCDC3_NUM_VOLTAGES,
> AXP717_DCDC3_CONTROL, AXP717_DCDC_V_OUT_MASK,
> - AXP717_DCDC_OUTPUT_CONTROL, BIT(2)),
> - AXP_DESC(AXP717, DCDC4, "dcdc4", "vin4", 1000, 3700, 100,
> + AXP717_DCDC_OUTPUT_CONTROL, BIT(2), 640),
> + AXP_DESC_DELAY(AXP717, DCDC4, "dcdc4", "vin4", 1000, 3700, 100,
> AXP717_DCDC4_CONTROL, AXP717_DCDC_V_OUT_MASK,
> - AXP717_DCDC_OUTPUT_CONTROL, BIT(3)),
> + AXP717_DCDC_OUTPUT_CONTROL, BIT(3), 6400),
> AXP_DESC(AXP717, ALDO1, "aldo1", "aldoin", 500, 3500, 100,
> AXP717_ALDO1_CONTROL, AXP717_LDO_V_OUT_MASK,
> AXP717_LDO0_OUTPUT_CONTROL, BIT(0)),
> --
> 2.47.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] regulator: axp20x: AXP717: set ramp_delay
2024-12-08 12:43 [PATCH v2] regulator: axp20x: AXP717: set ramp_delay Philippe Simons
2024-12-08 13:32 ` Chen-Yu Tsai
@ 2024-12-10 13:00 ` Mark Brown
1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2024-12-10 13:00 UTC (permalink / raw)
To: Liam Girdwood, Chen-Yu Tsai, linux-kernel, Philippe Simons
Cc: Hironori KIKUCHI, Chris Morgan
On Sun, 08 Dec 2024 13:43:08 +0100, Philippe Simons wrote:
> AXP717 datasheet says that regulator ramp delay is 15.625 us/step,
> which is 10mV in our case.
>
> Add a AXP_DESC_RANGES_DELAY macro and update AXP_DESC_RANGES macro to
> expand to AXP_DESC_RANGES_DELAY with ramp_delay = 0
>
> For DCDC4, steps is 100mv
>
> [...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
Thanks!
[1/1] regulator: axp20x: AXP717: set ramp_delay
commit: f07ae52f5cf6a5584fdf7c8c652f027d90bc8b74
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-12-10 13:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-08 12:43 [PATCH v2] regulator: axp20x: AXP717: set ramp_delay Philippe Simons
2024-12-08 13:32 ` Chen-Yu Tsai
2024-12-10 13:00 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox