summaryrefslogtreecommitdiffstats
path: root/drivers/gpio
diff options
context:
space:
mode:
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>2024-07-09 17:12:34 +0200
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>2024-07-10 14:17:31 +0200
commit1d8267bcbb7aa2176365299ad57e067f73b5174d (patch)
tree8d4098101a0a20f78c6ff8bccdb61025d9c8ba9b /drivers/gpio
parentdt-bindings: gpio: convert Atmel GPIO to json-schema (diff)
downloadlinux-1d8267bcbb7aa2176365299ad57e067f73b5174d.tar.xz
linux-1d8267bcbb7aa2176365299ad57e067f73b5174d.zip
gpio: virtuser: actually use the "trimmed" local variable
This variable is set with the intention of being passed to the subsequent strcmp() but something went wrong and it was left unused. Use it now. Fixes: 91581c4b3f29 ("gpio: virtuser: new virtual testing driver for the GPIO API") Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202407092245.BPnW2mr6-lkp@intel.com/ Link: https://lore.kernel.org/r/20240709151235.90523-1-brgl@bgdev.pl Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Diffstat (limited to 'drivers/gpio')
-rw-r--r--drivers/gpio/gpio-virtuser.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpio/gpio-virtuser.c b/drivers/gpio/gpio-virtuser.c
index 2ac09c6a3763..0e0d55da4f01 100644
--- a/drivers/gpio/gpio-virtuser.c
+++ b/drivers/gpio/gpio-virtuser.c
@@ -406,12 +406,12 @@ static ssize_t gpio_virtuser_direction_do_write(struct file *file,
trimmed = strim(buf);
- if (strcmp(buf, "input") == 0) {
+ if (strcmp(trimmed, "input") == 0) {
dir = 1;
- } else if (strcmp(buf, "output-high") == 0) {
+ } else if (strcmp(trimmed, "output-high") == 0) {
dir = 0;
val = 1;
- } else if (strcmp(buf, "output-low") == 0) {
+ } else if (strcmp(trimmed, "output-low") == 0) {
dir = val = 0;
} else {
return -EINVAL;