On vr, 2015-07-10 at 18:11 +0300, Robert Dolca wrote:

+static struct i2c_driver silead_ts_driver = {
+	[...]
+	.driver = {
+		[...]
+#ifdef CONFIG_ACPI
+		.acpi_match_table = ACPI_PTR(silead_ts_acpi_match),
+#endif

A nit: for the !CONFIG_ACPI case include/linux/acpi.h contains
    #define ACPI_PTR(_ptr)  (NULL)

So I think the above CONFIG_ACPI guard is superfluous and can be dropped.

+[...]
+	},
+};

Thanks,


Paul Bolle