Re: [PATCH] fix platform_rename_gsi related ia32 build breakage

From: Arnaldo Carvalho de Melo
Date: Wed Nov 10 2004 - 19:33:58 EST

Linus Torvalds wrote:

On Wed, 10 Nov 2004, Arnaldo Carvalho de Melo wrote:

This one compiles and links OK.

Including when CONFIG_ACPI_BOOT is set? Does it see the prototype then? Looks like <asm/acpi.h> should be included, but I assume it gets included some other way?

Quite frankly, I think the whole thing is broken. #ifdef's inside code is _evil_, and "platform_rename_gsi()" doesn't make sense as a name. I'll apply your patch, but quite frankly, I think the ACPI layer is doing crap.

Agreed, I leave this now to the ACPI guys, it may well be that the whole
case is to be ifdefed or something different, who knows. Just please
apply the one in the message I was looking for a brown paper bag (i.e.
the one with the #endif _before_ the break statement).

Ouch, time to make a hole for the nose, I need some air :o)

Len, can you please use a more descriptive name, and have it be always defined (make it an inline function that just becomes a no-op or something).


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at