Hi Tomasz:
On 2013-9-18 16:36, Tomasz Nowicki wrote:
arch/arm64/kernel/perf_event.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+)
diff --git a/arch/arm64/kernel/perf_event.c b/arch/arm64/kernel/perf_event.c index 9ba33c4..99b00d8 100644 --- a/arch/arm64/kernel/perf_event.c +++ b/arch/arm64/kernel/perf_event.c @@ -28,6 +28,9 @@ #include <linux/platform_device.h> #include <linux/spinlock.h> #include <linux/uaccess.h> +#ifdef CONFIG_ACPI +#include <linux/acpi.h> +#endif #include <asm/cputype.h> #include <asm/irq.h> @@ -1221,6 +1224,14 @@ static struct of_device_id armpmu_of_device_ids[] = { {}, }; +#ifdef CONFIG_ACPI +static const struct acpi_device_id armpmu_acpi_match[] = {
- { "LINA0007", },
- {},
+}; +MODULE_DEVICE_TABLE(acpi, armpmu_acpi_match);
perf_event for ARM64 is not a kernel module, so I think +MODULE_DEVICE_TABLE(acpi, armpmu_acpi_match); is not needed here.
I think you had met the compile warning, something like: warning: data definition has no type or storage class [enabled by default] warning: type defaults to ‘int’ in declaration of ‘MODULE_DEVICE_TABLE’ [-Wimplicit-int] warning: parameter names (without types) in function declaration [enabled by default]
Thanks Hanjun