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