forked from mirrors/linux
		
	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
	
	 Asmaa Mnebhi
						Asmaa Mnebhi