From: Nava kishore Manne <nava.manne@xxxxxxxxxx> Date: Wed, 19 Apr 2017 17:41:20 +0530 This patch fixes the below warning --> Prefer 'unsigned int' to bare use of 'unsigned'. --> Missing a blank line after declarations. Signed-off-by: Nava kishore Manne <navam@xxxxxxxxxx> Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx> Signed-off-by: Alexander Hedges <ahedges@xxxxxxx> (cherry picked from commit 3da7287a068080a6e100ddfef449ab94bebd429d) --- drivers/gpio/gpio-xilinx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-xilinx.c b/drivers/gpio/gpio-xilinx.c index ee1a931223f4..30623f8205d5 100644 --- a/drivers/gpio/gpio-xilinx.c +++ b/drivers/gpio/gpio-xilinx.c @@ -375,7 +375,7 @@ static struct irq_chip xgpio_irqchip = { * Return: * irq number otherwise -EINVAL */ -static int xgpio_to_irq(struct gpio_chip *gc, unsigned offset) +static int xgpio_to_irq(struct gpio_chip *gc, unsigned int offset) { struct of_mm_gpio_chip *mm_gc = to_of_mm_gpio_chip(gc); struct xgpio_instance *chip = container_of(mm_gc, struct xgpio_instance, @@ -432,6 +432,7 @@ static int xgpio_irq_setup(struct device_node *np, struct xgpio_instance *chip) struct resource res; int ret = of_irq_to_resource(np, 0, &res); + if (!ret) { pr_info("GPIO IRQ not connected\n"); return 0; @@ -454,6 +455,7 @@ static int xgpio_irq_setup(struct device_node *np, struct xgpio_instance *chip) */ for (pin_num = 0; pin_num < chip->mmchip.gc.ngpio; pin_num++) { u32 gpio_irq = irq_find_mapping(chip->irq_domain, pin_num); + irq_set_lockdep_class(gpio_irq, &gpio_lock_class, &gpio_request_class); pr_debug("IRQ Base: %d, Pin %d = IRQ %d\n", chip->irq_base, pin_num, gpio_irq); -- 2.17.1 -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html