Re: [PATCH 6/7] Input: edt-ft5x06 - add SPI transport and FT8722 support

From: Uwe Kleine-König

Date: Mon Oct 19 2026 - 03:00:50 EST


Hello,

On Sat, Oct 17, 2026 at 04:07:54PM +0200, Luka Panio wrote:
> +#include <linux/mod_devicetable.h>
> [...]
> +#include <linux/spi/spi.h>

<linux/spi/spi.h> already provides both of_device_id and spi_device_id.
So please drop <linux/mod_devicetable.h> which pulls in a lot of
unneeded dependencies.

> [...]
> +static const struct spi_device_id edt_ft5x06_spi_ids[] = {
> + { "ft8722-ts", (kernel_ulong_t)&edt_ft8722_data },

Please use a named initializer here.

> + { }
> +};
> +MODULE_DEVICE_TABLE(spi, edt_ft5x06_spi_ids);

Best regards
Uwe

Attachment: signature.asc
Description: PGP signature