Skip to contentSkip to conversation

RFC: amba: remove NO_IRQ

PATCH Alessandro Rubini 2 participants 3 messages May 24, 2012 Arm primecell bus support

lore
Share & tools
lore
#0 lore raw

I'm using PrimeCell devices under PCIe, x86 platform. We discussed this issue on Mar 1st, and you (Russell) reported that NO_IRQ is expected to go away for ARM, as 0 is the new official "no irq" marker.

The trivial patch below is what I'm currently using to compile my stuff, but I'd like to get an update about NO_IRQ in ARM (which you complained was completely on your shoulders). If needed I can take it over and propose patches.

I'm well aware that the few users within arch/arm are matched by a much bigger number of users in drivers/ . The work is not trivial, but at least I hope to be able to remove NO_IRQ from amba/bus.c by auditing all its users.

thanks
/alessandro

---
 drivers/amba/bus.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
drivers/amba/bus.c +2 -2
index 64ddedd..490a308 100644
@@ -546,10 +546,10 @@ int amba_device_add(struct amba_device *dev, struct resource *parent)
 		goto err_release;
 
 	printk(KERN_INFO "%s %i\n", __func__, __LINE__);
-	if (dev->irq[0] && dev->irq[0] != NO_IRQ)
+	if (dev->irq[0])
 		ret = device_create_file(&dev->dev, &dev_attr_irq0);
 	printk(KERN_INFO "%s %i\n", __func__, __LINE__);
-	if (ret == 0 && dev->irq[1] && dev->irq[1] != NO_IRQ)
+	if (ret == 0 && dev->irq[1])
 		ret = device_create_file(&dev->dev, &dev_attr_irq1);
 	printk(KERN_INFO "%s %i\n", __func__, __LINE__);
 	if (ret == 0)
Signature · 1 line
1.7.7.2
#1 lore raw
Quoted 9 lines · Alessandro Rubini

I'm using PrimeCell devices under PCIe, x86 platform. We discussed this issue on Mar 1st, and you (Russell) reported that NO_IRQ is expected to go away for ARM, as 0 is the new official "no irq" marker.

The trivial patch below is what I'm currently using to compile my stuff, but I'd like to get an update about NO_IRQ in ARM (which you complained was completely on your shoulders). If needed I can take it over and propose patches.

Quoted 4 lines

I'm well aware that the few users within arch/arm are matched by a much bigger number of users in drivers/ . The work is not trivial, but at least I hope to be able to remove NO_IRQ from amba/bus.c by auditing all its users.

See dfb85185bda373a70409c70c2632b02fb0f8f449 in mainline.

#2 lore raw
Quoted 1 line

See dfb85185bda373a70409c70c2632b02fb0f8f449 in mainline.

Thanks!

Then I need to check why my rebase on linux-next didn't drop my own patch.

/alessandro