Skip to content

Commit

Permalink
gpio: pl061: convert to use 0 for no irq
Browse files Browse the repository at this point in the history
We don't want drivers using NO_IRQ, so remove its use. For now, 0 or
-1 means no irq until platforms are converted to use 0.

Signed-off-by: Rob Herring <rob.herring@calxeda.com>
Acked-by: Grant Likely <grant.likely@secretlab.ca>
Cc: Linus Walleij <linus.ml.walleij@gmail.com>
  • Loading branch information
Rob Herring committed Jan 4, 2012
1 parent dece904 commit f2ab2ba
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions drivers/gpio/gpio-pl061.c
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ struct pl061_gpio {
spinlock_t irq_lock; /* IRQ registers */

void __iomem *base;
unsigned irq_base;
int irq_base;
struct gpio_chip gc;
};

Expand Down Expand Up @@ -119,7 +119,7 @@ static int pl061_to_irq(struct gpio_chip *gc, unsigned offset)
{
struct pl061_gpio *chip = container_of(gc, struct pl061_gpio, gc);

if (chip->irq_base == NO_IRQ)
if (chip->irq_base <= 0)
return -EINVAL;

return chip->irq_base + offset;
Expand Down Expand Up @@ -250,7 +250,7 @@ static int pl061_probe(struct amba_device *dev, const struct amba_id *id)
chip->irq_base = pdata->irq_base;
} else if (dev->dev.of_node) {
chip->gc.base = -1;
chip->irq_base = NO_IRQ;
chip->irq_base = 0;
} else {
ret = -ENODEV;
goto free_mem;
Expand Down Expand Up @@ -290,7 +290,7 @@ static int pl061_probe(struct amba_device *dev, const struct amba_id *id)
* irq_chip support
*/

if (chip->irq_base == NO_IRQ)
if (chip->irq_base <= 0)
return 0;

writeb(0, chip->base + GPIOIE); /* disable irqs */
Expand Down
2 changes: 1 addition & 1 deletion include/linux/amba/pl061.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ struct pl061_platform_data {
unsigned gpio_base;

/* number of the first IRQ.
* If the IRQ functionality in not desired this must be set to NO_IRQ.
* If the IRQ functionality in not desired this must be set to 0.
*/
unsigned irq_base;

Expand Down

0 comments on commit f2ab2ba

Please sign in to comment.