Re: [PATCH v2 2/2] iio: adc: add driver for Texas Instruments TLA2528 adc

From: Uwe Kleine-König

Date: Thu Nov 05 2026 - 10:54:23 EST


On Fri, Oct 30, 2026 at 05:47:04PM +0100, Maxime Chevallier wrote:
> +#include <linux/i2c.h>
> +#include <linux/mod_devicetable.h>

Please don't include <linux/mod_devicetable.h>. <linux/i2c.h> already
provides i2c_device_id and of_device_id. <linux/mod_devicetable.h>
provides many more *_device_id and thus introduces a big list of
unnecessary dependencies.

I recommend to just drop <linux/mod_devicetable.h> here. If you insist
on full iwyu, there is <linux/device-id/i2c.h> and
<linux/device-id/of.h>.

Best regards
Uwe

Attachment: signature.asc
Description: PGP signature