2022-02-22 04:30:23

by Adam Ward

[permalink] [raw]
Subject: [PATCH 0/3] regulator: da9121: update DA914x support

This series applies corrections and updates to DA914x support in the DA9121 driver

- correct errors in current range
- update maximum voltage to spec update
- remove unwanted settings


Adam Ward (3):
regulator: da9121: Fix DA914x current values
regulator: da9121: Fix DA914x voltage value
regulator: da9121: Remove surplus DA9141 parameters

drivers/regulator/da9121-regulator.c | 16 ++++++----------
1 file changed, 6 insertions(+), 10 deletions(-)

--
2.25.1


2022-02-22 05:05:09

by Adam Ward

[permalink] [raw]
Subject: [PATCH 2/3] regulator: da9121: Fix DA914x voltage value

Update DA9141/2 max voltage to match spec change

Signed-off-by: Adam Ward <[email protected]>
---
drivers/regulator/da9121-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/da9121-regulator.c b/drivers/regulator/da9121-regulator.c
index 39d77726970c..f16649dec17e 100644
--- a/drivers/regulator/da9121-regulator.c
+++ b/drivers/regulator/da9121-regulator.c
@@ -561,7 +561,7 @@ static const struct regulator_desc da9217_reg = {
};

#define DA914X_MIN_MV 500
-#define DA914X_MAX_MV 1000
+#define DA914X_MAX_MV 1300
#define DA914X_STEP_MV 10
#define DA914X_MIN_SEL (DA914X_MIN_MV / DA914X_STEP_MV)
#define DA914X_N_VOLTAGES (((DA914X_MAX_MV - DA914X_MIN_MV) / DA914X_STEP_MV) \
--
2.25.1

2022-02-22 05:09:13

by Adam Ward

[permalink] [raw]
Subject: [PATCH 1/3] regulator: da9121: Fix DA914x current values

Update DA9141/2 ranges to correct errors

Signed-off-by: Adam Ward <[email protected]>
---
drivers/regulator/da9121-regulator.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/regulator/da9121-regulator.c b/drivers/regulator/da9121-regulator.c
index 6f21223a488e..39d77726970c 100644
--- a/drivers/regulator/da9121-regulator.c
+++ b/drivers/regulator/da9121-regulator.c
@@ -87,16 +87,16 @@ static struct da9121_range da9121_3A_1phase_current = {
};

static struct da9121_range da914x_40A_4phase_current = {
- .val_min = 14000000,
- .val_max = 80000000,
- .val_stp = 2000000,
+ .val_min = 26000000,
+ .val_max = 78000000,
+ .val_stp = 4000000,
.reg_min = 1,
.reg_max = 14,
};

static struct da9121_range da914x_20A_2phase_current = {
- .val_min = 7000000,
- .val_max = 40000000,
+ .val_min = 13000000,
+ .val_max = 39000000,
.val_stp = 2000000,
.reg_min = 1,
.reg_max = 14,
--
2.25.1

2022-02-22 05:35:16

by Adam Ward

[permalink] [raw]
Subject: [PATCH 3/3] regulator: da9121: Remove surplus DA9141 parameters

Remove ramp_delay/enable_time values - subject to OTP, incorrect

Signed-off-by: Adam Ward <[email protected]>
---
drivers/regulator/da9121-regulator.c | 4 ----
1 file changed, 4 deletions(-)

diff --git a/drivers/regulator/da9121-regulator.c b/drivers/regulator/da9121-regulator.c
index f16649dec17e..eb9df485bd8a 100644
--- a/drivers/regulator/da9121-regulator.c
+++ b/drivers/regulator/da9121-regulator.c
@@ -585,10 +585,6 @@ static const struct regulator_desc da9141_reg = {
.vsel_mask = DA9121_MASK_BUCK_BUCKx_5_CHx_A_VOUT,
.enable_reg = DA9121_REG_BUCK_BUCK1_0,
.enable_mask = DA9121_MASK_BUCK_BUCKx_0_CHx_EN,
- /* Default value of BUCK_BUCK1_0.CH1_SRC_DVC_UP */
- .ramp_delay = 20000,
- /* tBUCK_EN */
- .enable_time = 20,
};

static const struct regulator_desc da9142_reg = {
--
2.25.1

2022-02-22 16:33:29

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 0/3] regulator: da9121: update DA914x support

On Tue, 22 Feb 2022 00:27:41 +0000, Adam Ward wrote:
> This series applies corrections and updates to DA914x support in the DA9121 driver
>
> - correct errors in current range
> - update maximum voltage to spec update
> - remove unwanted settings
>
>
> [...]

Applied to

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-linus

Thanks!

[1/3] regulator: da9121: Fix DA914x current values
commit: f0fdfc04fd974cea23351b830fcac0822ea19a51
[2/3] regulator: da9121: Fix DA914x voltage value
commit: c8c57fbc1c5067b913077e948c7d957af6834ba3
[3/3] regulator: da9121: Remove surplus DA9141 parameters
commit: 9c7cf33c53ce833b58de9e5c192b4736dbd09cb1

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