logo       

[Kernel-janitors] gcc compilation warning fix: msg#00279

Subject: [Kernel-janitors] gcc compilation warning fix
The actual prototype is:
typedef void (*acpi_gpe_handler) (void *context);

(noone was even using the return value)

===== drivers/char/ipmi/ipmi_si_intf.c 1.6 vs edited =====
--- 1.6/drivers/char/ipmi/ipmi_si_intf.c        2004-07-13 13:46:29 +00:00
+++ edited/drivers/char/ipmi/ipmi_si_intf.c     2004-07-25 19:21:12 +00:00
@@ -1132,7 +1132,7 @@
 static int acpi_failure = 0;
 
 /* For GPE-type interrupts. */
-u32 ipmi_acpi_gpe(void *context)
+void ipmi_acpi_gpe(void *context)
 {
        struct smi_info *smi_info = context;
        unsigned long   flags;
@@ -1156,7 +1156,6 @@
        smi_event_handler(smi_info, 0);
  out:
        spin_unlock_irqrestore(&(smi_info->si_lock), flags);
-       return 0;
 }
 
 static int acpi_gpe_irq_setup(struct smi_info *info)


_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@xxxxxxxxxxxxxx
http://lists.osdl.org/mailman/listinfo/kernel-janitors
<Prev in Thread] Current Thread [Next in Thread>