[PATCH] pinctrl: qcom: lpass-lpi: Include value in debugfs output

Bjorn Andersson posted 1 patch 1 week, 2 days ago
drivers/pinctrl/qcom/pinctrl-lpass-lpi.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
[PATCH] pinctrl: qcom: lpass-lpi: Include value in debugfs output
Posted by Bjorn Andersson 1 week, 2 days ago
As with other pinctrl providers, it's useful to be able to inspect the
current GPIO state of the LPASS LPI pins during system integration and
debugging.

Extend the driver's debugfs logic to read the current value and include
this in the printout.

Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
---
 drivers/pinctrl/qcom/pinctrl-lpass-lpi.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/pinctrl/qcom/pinctrl-lpass-lpi.c b/drivers/pinctrl/qcom/pinctrl-lpass-lpi.c
index 5fd4a4eba654..9412fd36a4aa 100644
--- a/drivers/pinctrl/qcom/pinctrl-lpass-lpi.c
+++ b/drivers/pinctrl/qcom/pinctrl-lpass-lpi.c
@@ -421,6 +421,7 @@ static void lpi_gpio_dbg_show_one(struct seq_file *s,
 	struct pinctrl_pin_desc pindesc;
 	unsigned int func;
 	int is_out;
+	int value;
 	int drive;
 	int pull;
 	u32 ctl_reg;
@@ -443,7 +444,10 @@ static void lpi_gpio_dbg_show_one(struct seq_file *s,
 	drive = FIELD_GET(LPI_GPIO_OUT_STRENGTH_MASK, ctl_reg);
 	pull = FIELD_GET(LPI_GPIO_PULL_MASK, ctl_reg);
 
-	seq_printf(s, " %-8s: %-3s %d", pindesc.name, is_out ? "out" : "in", func);
+	value = lpi_gpio_get(chip, offset);
+
+	seq_printf(s, " %-8s: %-3s", pindesc.name, is_out ? "out" : "in");
+	seq_printf(s, " %-4s func%d", str_high_low(value), func);
 	seq_printf(s, " %dmA", lpi_regval_to_drive(drive));
 	seq_printf(s, " %s", pulls[pull]);
 }

---
base-commit: 68142f986ff04b2b70b31db00f719bf690f64a9a
change-id: 20260915-pinctrl-lpass-debug-7aa2a92f0225

Best regards,
--  
Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
Re: [PATCH] pinctrl: qcom: lpass-lpi: Include value in debugfs output
Posted by Bartosz Golaszewski 1 week, 1 day ago
On Tue, 15 Sep 2026 19:00:15 -0500, Bjorn Andersson wrote:
> As with other pinctrl providers, it's useful to be able to inspect the
> current GPIO state of the LPASS LPI pins during system integration and
> debugging.
> 
> Extend the driver's debugfs logic to read the current value and include
> this in the printout.
> 
> [...]

Sashiko's probably right about the return value check but it's debugfs so I
won't worry too much.

[1/1] pinctrl: qcom: lpass-lpi: Include value in debugfs output
      https://git.kernel.org/brgl/c/d29bfb93a19771b822dab6d7adbbbaf188cb1db5

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>