mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	gpio: mlxbf3: Support add_pin_ranges()
Support add_pin_ranges() so that pinctrl_gpio_request() can be called.
The GPIO value is not modified when the user runs the "gpioset" tool.
This is because when gpiochip_generic_request is invoked by the gpio-mlxbf3
driver, "pin_ranges" is empty so it skips "pinctrl_gpio_request()".
pinctrl_gpio_request() is essential in the code flow because it changes the
mux value so that software has control over modifying the GPIO value.
Adding add_pin_ranges() creates a dependency on the pinctrl-mlxbf3.c driver.
Fixes: cd33f216d2 ("gpio: mlxbf3: Add gpio driver support")
Signed-off-by: Asmaa Mnebhi <asmaa@nvidia.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
			
			
This commit is contained in:
		
							parent
							
								
									9e0fa5d85a
								
							
						
					
					
						commit
						38a700efc5
					
				
					 1 changed files with 24 additions and 0 deletions
				
			
		| 
						 | 
				
			
			@ -19,6 +19,8 @@
 | 
			
		|||
 * gpio[1]: HOST_GPIO32->HOST_GPIO55
 | 
			
		||||
 */
 | 
			
		||||
#define MLXBF3_GPIO_MAX_PINS_PER_BLOCK 32
 | 
			
		||||
#define MLXBF3_GPIO_MAX_PINS_BLOCK0    32
 | 
			
		||||
#define MLXBF3_GPIO_MAX_PINS_BLOCK1    24
 | 
			
		||||
 | 
			
		||||
/*
 | 
			
		||||
 * fw_gpio[x] block registers and their offset
 | 
			
		||||
| 
						 | 
				
			
			@ -158,6 +160,26 @@ static const struct irq_chip gpio_mlxbf3_irqchip = {
 | 
			
		|||
	GPIOCHIP_IRQ_RESOURCE_HELPERS,
 | 
			
		||||
};
 | 
			
		||||
 | 
			
		||||
static int mlxbf3_gpio_add_pin_ranges(struct gpio_chip *chip)
 | 
			
		||||
{
 | 
			
		||||
	unsigned int id;
 | 
			
		||||
 | 
			
		||||
	switch(chip->ngpio) {
 | 
			
		||||
	case MLXBF3_GPIO_MAX_PINS_BLOCK0:
 | 
			
		||||
		id = 0;
 | 
			
		||||
		break;
 | 
			
		||||
	case MLXBF3_GPIO_MAX_PINS_BLOCK1:
 | 
			
		||||
		id = 1;
 | 
			
		||||
		break;
 | 
			
		||||
	default:
 | 
			
		||||
		return -EINVAL;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	return gpiochip_add_pin_range(chip, "MLNXBF34:00",
 | 
			
		||||
			chip->base, id * MLXBF3_GPIO_MAX_PINS_PER_BLOCK,
 | 
			
		||||
			chip->ngpio);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static int mlxbf3_gpio_probe(struct platform_device *pdev)
 | 
			
		||||
{
 | 
			
		||||
	struct device *dev = &pdev->dev;
 | 
			
		||||
| 
						 | 
				
			
			@ -197,6 +219,7 @@ static int mlxbf3_gpio_probe(struct platform_device *pdev)
 | 
			
		|||
	gc->request = gpiochip_generic_request;
 | 
			
		||||
	gc->free = gpiochip_generic_free;
 | 
			
		||||
	gc->owner = THIS_MODULE;
 | 
			
		||||
	gc->add_pin_ranges = mlxbf3_gpio_add_pin_ranges;
 | 
			
		||||
 | 
			
		||||
	irq = platform_get_irq(pdev, 0);
 | 
			
		||||
	if (irq >= 0) {
 | 
			
		||||
| 
						 | 
				
			
			@ -243,6 +266,7 @@ static struct platform_driver mlxbf3_gpio_driver = {
 | 
			
		|||
};
 | 
			
		||||
module_platform_driver(mlxbf3_gpio_driver);
 | 
			
		||||
 | 
			
		||||
MODULE_SOFTDEP("pre: pinctrl-mlxbf3");
 | 
			
		||||
MODULE_DESCRIPTION("NVIDIA BlueField-3 GPIO Driver");
 | 
			
		||||
MODULE_AUTHOR("Asmaa Mnebhi <asmaa@nvidia.com>");
 | 
			
		||||
MODULE_LICENSE("Dual BSD/GPL");
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue