Re: [PATCH v6 2/3] HID: Add Himax HX83102J SPI touchscreen driver

From: Uwe Kleine-König

Date: Sun Oct 11 2026 - 19:06:36 EST


Hello,

On Sun, Oct 11, 2026 at 08:16:32PM +0200, Michał Kopeć wrote:
> +#include <linux/mod_devicetable.h>

Please don't include <linux/mod_devicetable.h>. This file introduces a
big amount of unneeded dependencies. You can rely on <linux/spi/spi.h>
to provide of_device_id and spi_device_id (which are the ones you
actually use). Alternatively---if you want to go full iwyu---include
linux/device-id/of.h and linux/device-id/spi.h directly.

Best regards
Uwe

Attachment: signature.asc
Description: PGP signature