On Fri, Oct 03, 2014 at 02:37:25PM -0700, Ben Zhang wrote:

> +#ifdef CONFIG_ACPI
> +		.acpi_match_table = ACPI_PTR(rt5677_acpi_id),
> +#endif

Why do you need this ifdef - surely the whole point of ACPI_PTR() is to
avoid needing it?