[Kernel-janitors] gcc compilation warning fix

[Kernel-janitors] gcc compilation warning fix

[Alexander Nyberg]

[-- Attachment #1: Type: text/plain, Size: 728 bytes --]

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)



[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

Re: [Kernel-janitors] gcc compilation warning fix

[maximilian attems]

[-- Attachment #1: Type: text/plain, Size: 199 bytes --]

On Sun, 25 Jul 2004, Alexander Nyberg wrote:

> The actual prototype is:
> typedef	void (*acpi_gpe_handler) (void *context);
> 
> (noone was even using the return value)
 
already patched in 2.6.8!


[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors