Skip to content

Commit

Permalink
gpiolib: unduplicate chip guard in set_config path
Browse files Browse the repository at this point in the history
We don't need to guard the GPIO chip until its first dereference in
gpio_do_set_config().

First: change the prototype of gpio_do_set_config() to take the GPIO
line descriptor as argument, then move the gpio_chip protection into it
and drop it in two places where it's done too early.

This has the added benefit of making gpio_go_set_config() safe to use
from outside of this compilation unit without taking the gdev SRCU read
lock and will come in handy when we'll want to make it available to the
character device code.

Reviewed-by: Kent Gibson <warthog618@gmail.com>
Link: https://lore.kernel.org/r/20241018-gpio-notify-in-kernel-events-v5-2-c79135e58a1c@linaro.org
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
  • Loading branch information
Bartosz Golaszewski committed Oct 22, 2024
1 parent 49182c8 commit dd26ffa
Showing 1 changed file with 9 additions and 14 deletions.
23 changes: 9 additions & 14 deletions drivers/gpio/gpiolib.c
Original file line number Diff line number Diff line change
Expand Up @@ -2562,13 +2562,16 @@ EXPORT_SYMBOL_GPL(gpiochip_free_own_desc);
* rely on gpio_request() having been called beforehand.
*/

static int gpio_do_set_config(struct gpio_chip *gc, unsigned int offset,
unsigned long config)
static int gpio_do_set_config(struct gpio_desc *desc, unsigned long config)
{
if (!gc->set_config)
CLASS(gpio_chip_guard, guard)(desc);
if (!guard.gc)
return -ENODEV;

if (!guard.gc->set_config)
return -ENOTSUPP;

return gc->set_config(gc, offset, config);
return guard.gc->set_config(guard.gc, gpio_chip_hwgpio(desc), config);
}

static int gpio_set_config_with_argument(struct gpio_desc *desc,
Expand All @@ -2577,12 +2580,8 @@ static int gpio_set_config_with_argument(struct gpio_desc *desc,
{
unsigned long config;

CLASS(gpio_chip_guard, guard)(desc);
if (!guard.gc)
return -ENODEV;

config = pinconf_to_config_packed(mode, argument);
return gpio_do_set_config(guard.gc, gpio_chip_hwgpio(desc), config);
return gpio_do_set_config(desc, config);
}

static int gpio_set_config_with_argument_optional(struct gpio_desc *desc,
Expand Down Expand Up @@ -2944,11 +2943,7 @@ int gpiod_set_config(struct gpio_desc *desc, unsigned long config)
{
VALIDATE_DESC(desc);

CLASS(gpio_chip_guard, guard)(desc);
if (!guard.gc)
return -ENODEV;

return gpio_do_set_config(guard.gc, gpio_chip_hwgpio(desc), config);
return gpio_do_set_config(desc, config);
}
EXPORT_SYMBOL_GPL(gpiod_set_config);

Expand Down

0 comments on commit dd26ffa

Please sign in to comment.