Re: [PATCH v2 3/7] regulator: qcom-pm8008: Add PM8010 mode support
From: Guru Das Srinagesh
Date: Mon Oct 19 2026 - 03:42:43 EST
On Wed, Oct 14, 2026 at 04:02:00PM +0530, Jishnu Prakash wrote:
> From: Dhruvin Rajpura <drajpura@xxxxxxxxxxxxxxxx>
>
> Implement set_mode/get_mode support for PM8010 LDOs using the mode
> control and status registers to allow clients to vote for LDO modes.
> Mode voting is not supported in the PM8008 LDOs.
>
> Signed-off-by: Dhruvin Rajpura <drajpura@xxxxxxxxxxxxxxxx>
> Tested-by: Oleg Keri <okerixx@xxxxxxxxx> # Lenovo Yoga Slim 7x Gen 11 (Glymur)
> Signed-off-by: Jishnu Prakash <jishnu.prakash@xxxxxxxxxxxxxxxx>
> ---
> drivers/regulator/qcom-pm8008-regulator.c | 75 ++++++++++++++++++++++++++++++-
> 1 file changed, 74 insertions(+), 1 deletion(-)
...
Patches 2 and 3 both say "Add PM8010 mode support" in their subject lines and edit
the same file. Could they either be squashed into one commit or have their subject
lines amended appropriately to better reflect the changes they are making?
Also, patch 3 uses the wrong subject prefix "qcom-pm8008" when git log on the
regulator driver reveals the convention is to use "pm8008".