Re: [PATCH v3 3/6] iio: adc: xilinx-xadc: Split driver into core and platform files
From: Uwe Kleine-König
Date: Thu Jul 30 2026 - 15:52:47 EST
Hello,
On Wed, Jul 29, 2026 at 06:24:52PM +0530, Dileep Kumar Nagavarapu wrote:
> @@ -19,21 +19,13 @@
> #include <linux/err.h>
> #include <linux/interrupt.h>
> #include <linux/io.h>
> -#include <linux/kernel.h>
> +#include <linux/mod_devicetable.h>
I wonder why so many people recently add <linux/mod_devicetable.h> in
their patches. Is there some tool that suggests that?
Anyhow: Please don't. <linux/platform_device.h> is already enough to get
of_device_id (which is the only *_device_id this driver is using unless
I missed something).
linux/mod_devicetable.h will hopefully go away soon, so each patch
adding an #include for it is annoying.
Best regards
Uwe
Attachment:
signature.asc
Description: PGP signature