On Tue, 2012-11-20 at 02:04 +0100, Rafael J. Wysocki wrote: > On Monday, November 19, 2012 10:26:43 AM Toshi Kani wrote: > > On Mon, 2012-11-19 at 18:35 +0100, Rafael J. Wysocki wrote: > > > On Monday, November 19, 2012 09:16:52 AM Toshi Kani wrote: > > > > On Mon, 2012-11-19 at 08:06 -0800, Joe Perches wrote: > > > > > On Mon, 2012-11-19 at 08:51 -0700, Toshi Kani wrote: > > > > > > On Mon, 2012-11-19 at 00:56 -0800, Joe Perches wrote: > > > > > > > is acpi_bus.h really the right file for these prototypes? > > > > > > This interface is limited for ACPI, so it should be declared in a header > > > > > > file under include/acpi. Among the files in this directory, acpi_bus.h > > > > > > seems to be a good fit as it declares the interfaces provided by ACPI > > > > > > core. > > > > > > > > > > I'd've put them in acpi.h or maybe created acpi_utils.h > > > > > > > > Hi Joe, > > > > > > > > We cannot use acpi.h since it is an ACPICA header. > > > > > > Which acpi.h are you referring to? > > > > > > I was referring "include/acpi/acpi.h". > > There is another acpi.h in include/linux that can be used for this purpose. I see. I agree that include/linux/acpi.h can be used. I will update the patchset to use this header. Thanks, -Toshi -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html