Commit 90b39402 authored by Linus Walleij's avatar Linus Walleij

gpio: Add API to explicitly name a consumer

The GPIO (descriptor) API registers a "label" naming what is
currently using the GPIO line. Typically this is taken from
things like the device tree node, so "reset-gpios" will result
in he line being labeled "reset".

The technical effect is pretty much zero: the use is for
debug and introspection, such as "lsgpio" and debugfs files.

However sometimes the user want this cuddly feeling of
listing all GPIO lines and seeing exactly what they are for
and it gives a very fulfilling sense of control. Especially
in the cases when the device tree node doesn't provide a
good name, or anonymous GPIO lines assigned just to
"gpios" in the device tree because the usage is implicit.

For these cases it may be nice to be able to label the
line directly and explicitly.
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent f3a049e7
...@@ -3193,6 +3193,19 @@ int gpiod_cansleep(const struct gpio_desc *desc) ...@@ -3193,6 +3193,19 @@ int gpiod_cansleep(const struct gpio_desc *desc)
} }
EXPORT_SYMBOL_GPL(gpiod_cansleep); EXPORT_SYMBOL_GPL(gpiod_cansleep);
/**
* gpiod_set_consumer_name() - set the consumer name for the descriptor
* @desc: gpio to set the consumer name on
* @name: the new consumer name
*/
void gpiod_set_consumer_name(struct gpio_desc *desc, const char *name)
{
VALIDATE_DESC_VOID(desc);
/* Just overwrite whatever the previous name was */
desc->label = name;
}
EXPORT_SYMBOL_GPL(gpiod_set_consumer_name);
/** /**
* gpiod_to_irq() - return the IRQ corresponding to a GPIO * gpiod_to_irq() - return the IRQ corresponding to a GPIO
* @desc: gpio whose IRQ will be returned (already requested) * @desc: gpio whose IRQ will be returned (already requested)
......
...@@ -145,6 +145,7 @@ int gpiod_is_active_low(const struct gpio_desc *desc); ...@@ -145,6 +145,7 @@ int gpiod_is_active_low(const struct gpio_desc *desc);
int gpiod_cansleep(const struct gpio_desc *desc); int gpiod_cansleep(const struct gpio_desc *desc);
int gpiod_to_irq(const struct gpio_desc *desc); int gpiod_to_irq(const struct gpio_desc *desc);
void gpiod_set_consumer_name(struct gpio_desc *desc, const char *name);
/* Convert between the old gpio_ and new gpiod_ interfaces */ /* Convert between the old gpio_ and new gpiod_ interfaces */
struct gpio_desc *gpio_to_desc(unsigned gpio); struct gpio_desc *gpio_to_desc(unsigned gpio);
...@@ -467,6 +468,12 @@ static inline int gpiod_to_irq(const struct gpio_desc *desc) ...@@ -467,6 +468,12 @@ static inline int gpiod_to_irq(const struct gpio_desc *desc)
return -EINVAL; return -EINVAL;
} }
static inline void gpiod_set_consumer_name(struct gpio_desc *desc, const char *name)
{
/* GPIO can never have been requested */
WARN_ON(1);
}
static inline struct gpio_desc *gpio_to_desc(unsigned gpio) static inline struct gpio_desc *gpio_to_desc(unsigned gpio)
{ {
return ERR_PTR(-EINVAL); return ERR_PTR(-EINVAL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment