[IA64] minor irq handler cleanups
authorJeff Garzik <jeff@garzik.org>
Fri, 18 Apr 2008 23:22:58 +0000 (19:22 -0400)
committerTony Luck <tony.luck@intel.com>
Tue, 22 Apr 2008 15:56:58 +0000 (08:56 -0700)
- remove unused 'irq' argument from pfm_do_interrupt_handler()

- remove pointless cast to void*

- add KERN_xxx prefix to printk()

- remove braces around singleton C statement

- in tioce_provider.c, start tioce_dma_consistent() and
  tioce_error_intr_handler() function declarations in column 0

This change's main purpose is to prepare for the patchset in
jgarzik/misc-2.6.git#irq-remove, that explores removal of the
never-used 'irq' argument in each interrupt handler.

Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/ia64/kernel/perfmon.c
arch/ia64/sn/kernel/huberror.c
arch/ia64/sn/pci/tioce_provider.c

index d1d24f4598da6ce98be2be0b40b7ffb436b30b45..c8e403752a0c54d5c519b6ada5f3df1b68508bac 100644 (file)
@@ -5511,7 +5511,7 @@ stop_monitoring:
 }
 
 static int
-pfm_do_interrupt_handler(int irq, void *arg, struct pt_regs *regs)
+pfm_do_interrupt_handler(void *arg, struct pt_regs *regs)
 {
        struct task_struct *task;
        pfm_context_t *ctx;
@@ -5591,7 +5591,7 @@ pfm_interrupt_handler(int irq, void *arg)
 
                start_cycles = ia64_get_itc();
 
-               ret = pfm_do_interrupt_handler(irq, arg, regs);
+               ret = pfm_do_interrupt_handler(arg, regs);
 
                total_cycles = ia64_get_itc();
 
index 0101c7924a4d4943dc4c4a4823967e1477dbbf54..08b0d9bb62ec0b4f909dcb23582c30f29dabcb56 100644 (file)
@@ -187,8 +187,8 @@ void hub_error_init(struct hubdev_info *hubdev_info)
 {
 
        if (request_irq(SGI_II_ERROR, hub_eint_handler, IRQF_SHARED,
-                       "SN_hub_error", (void *)hubdev_info)) {
-               printk("hub_error_init: Failed to request_irq for 0x%p\n",
+                       "SN_hub_error", hubdev_info)) {
+               printk(KERN_ERR "hub_error_init: Failed to request_irq for 0x%p\n",
                    hubdev_info);
                return;
        }
index 9b3c11373022d2bd1319836b05eb92b97a06e4d7..94e584527f48c3396e9d831763b390df938c57b4 100644 (file)
@@ -655,7 +655,8 @@ tioce_dma(struct pci_dev *pdev, u64 paddr, size_t byte_count, int dma_flags)
  *
  * Simply call tioce_do_dma_map() to create a map with the barrier bit set
  * in the address.
- */ static u64
+ */
+static u64
 tioce_dma_consistent(struct pci_dev *pdev, u64 paddr, size_t byte_count, int dma_flags)
 {
        return tioce_do_dma_map(pdev, paddr, byte_count, 1, dma_flags);
@@ -668,7 +669,8 @@ tioce_dma_consistent(struct pci_dev *pdev, u64 paddr, size_t byte_count, int dma
  *
  * Handle a CE error interrupt.  Simply a wrapper around a SAL call which
  * defers processing to the SGI prom.
- */ static irqreturn_t
+ */
+static irqreturn_t
 tioce_error_intr_handler(int irq, void *arg)
 {
        struct tioce_common *soft = arg;