Thread (56 messages) flat view 56 messages, 8 authors, 9d ago

Re: [PATCH v3 11/19] soc: starfive: Add jh7110-hdmi-subsystem driver

From: Michal Wilczynski <m.wilczynski@samsung.com>
Date: 2026-09-14 21:46:10
Also in: dri-devel, linux-arm-kernel, linux-clk, linux-phy, linux-riscv, linux-rockchip, lkml


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:
quoted
+#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 [off-list ref]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help