[PATCH v3 11/19] soc: starfive: Add jh7110-hdmi-subsystem driver
Michal Wilczynski
m.wilczynski at samsung.com
Mon Sep 14 14:46:04 PDT 2026
On 9/7/26 10:23, Uwe Kleine-König wrote:
> Hello Michael,
>
> On Fri, Sep 04, 2026 at 03:27:12PM +0200, Michal Wilczynski wrote:
>> +#include <linux/mod_devicetable.h>
>> [...]
>> +#include <linux/platform_device.h>
>
> Please don't add includes for <linux/mod_devicetable.h> as it introduces
> many unneeded build dependencies. Either rely on
> <linux/platform_device.h> providing of_device_id (IMHO preferred), or
> include <linux/device-id/of.h> directly.
>
> The same applies to two more patches in this series.
>
> Best regards
> Uwe
Thank you for your review. This will be fixed for v4.
Best regards,
--
Michal Wilczynski <m.wilczynski at samsung.com>
More information about the linux-arm-kernel
mailing list