Commit f636d4f6 authored by Matti Vaittinen's avatar Matti Vaittinen Committed by Bartosz Golaszewski
Browse files

gpio: Add a valid_mask getter 



The valid_mask member of the struct gpio_chip is unconditionally written
by the GPIO core at driver registration. It shouldn't be directly
populated by drivers. This can be prevented by moving it from the struct
gpio_chip to struct gpio_device, which is internal to the GPIO core.

As a preparatory step, provide a getter function which can be used by
those drivers which need the valid_mask information.

Signed-off-by: default avatarMatti Vaittinen <mazziesaccount@gmail.com>
Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Link: https://lore.kernel.org/r/026f9d78502eca883bfe3faeb684e23d5d6c5e84.1741180097.git.mazziesaccount@gmail.com


Signed-off-by: default avatarBartosz Golaszewski <bartosz.golaszewski@linaro.org>
parent a5016248

drivers/gpio/gpiolib.c

+16 −0
Original line number Diff line number Diff line
@@ -723,6 +723,22 @@ static int gpiochip_add_pin_ranges(struct gpio_chip *gc) 
	return 0;

}



/**

 * gpiochip_query_valid_mask - return the GPIO validity information

 * @gc:	gpio chip which validity information is queried

 *

 * Returns: bitmap representing valid GPIOs or NULL if all GPIOs are valid

 *

 * Some GPIO chips may support configurations where some of the pins aren't

 * available. These chips can have valid_mask set to represent the valid

 * GPIOs. This function can be used to retrieve this information.

 */

const unsigned long *gpiochip_query_valid_mask(const struct gpio_chip *gc)

{

	return gc->valid_mask;

}

EXPORT_SYMBOL_GPL(gpiochip_query_valid_mask);



bool gpiochip_line_is_valid(const struct gpio_chip *gc,

				unsigned int offset)

{

include/linux/gpio/driver.h

+1 −0
Original line number Diff line number Diff line
@@ -720,6 +720,7 @@ bool gpiochip_line_is_open_source(struct gpio_chip *gc, unsigned int offset); 
/* Sleep persistence inquiry for drivers */

bool gpiochip_line_is_persistent(struct gpio_chip *gc, unsigned int offset);

bool gpiochip_line_is_valid(const struct gpio_chip *gc, unsigned int offset);

const unsigned long *gpiochip_query_valid_mask(const struct gpio_chip *gc);



/* get driver data */

void *gpiochip_get_data(struct gpio_chip *gc);