Skip to contentSkip to conversation

FAILED: Patch "gpiolib: fix debugfs newline separators" failed to apply to v5.10-stable tree

PATCH Sasha Levin 1 participant 1 message Nov 6, 2024

Subsystems (2)
lore
Share & tools
lore

Sasha Levin

author maintainer
#0 lore raw

The patch below does not apply to the v5.10-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.

Thanks,
Sasha

------------------ original commit in Linus's tree ------------------

From 3e8b7238b427e05498034c240451af5f5495afda Mon Sep 17 00:00:00 2001
From: Johan Hovold <[email protected]>
Date: Mon, 28 Oct 2024 13:49:58 +0100
Subject: [PATCH] gpiolib: fix debugfs newline separators

The gpiolib debugfs interface exports a list of all gpio chips in a system and the state of their pins.

The gpio chip sections are supposed to be separated by a newline character, but a long-standing bug prevents the separator from being included when output is generated in multiple sessions, making the output inconsistent and hard to read.

Make sure to only suppress the newline separator at the beginning of the file as intended.

Fixes f9c4a31f6150 ("gpiolib: Use seq_file's iterator interface")
Cc Thierry Reding <[email protected]>
---
 drivers/gpio/gpiolib.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/gpio/gpiolib.c +2 -0
index d5952ab7752c2..e27488a90bc97 100644
@@ -4926,6 +4926,8 @@ static void *gpiolib_seq_start(struct seq_file *s, loff_t *pos)
 		return NULL;
 
 	s->private = priv;
+	if (*pos > 0)
+		priv->newline = true;
 	priv->idx = srcu_read_lock(&gpio_devices_srcu);
 
 	list_for_each_entry_srcu(gdev, &gpio_devices, list,
Signature · 1 line
2.43.0