Il 19/09/22 18:56, Guillaume Ranquet ha scritto:
Add the DPI1 hdmi path support in mtk dpi driver

Signed-off-by: Guillaume Ranquet <granq...@baylibre.com>

diff --git a/drivers/gpu/drm/mediatek/mtk_dpi.c 
b/drivers/gpu/drm/mediatek/mtk_dpi.c
index 630a4e301ef6..91212b7610e8 100644
--- a/drivers/gpu/drm/mediatek/mtk_dpi.c
+++ b/drivers/gpu/drm/mediatek/mtk_dpi.c
@@ -15,7 +15,10 @@
  #include <linux/of_graph.h>
  #include <linux/pinctrl/consumer.h>
  #include <linux/platform_device.h>
+#include <linux/reset.h>
  #include <linux/types.h>
+#include <linux/regmap.h>
+#include <linux/mfd/syscon.h>
#include <video/videomode.h> @@ -66,10 +69,14 @@ struct mtk_dpi {
        struct drm_bridge *next_bridge;
        struct drm_connector *connector;
        void __iomem *regs;
+       struct reset_control *reset_ctl;
        struct device *dev;
        struct clk *engine_clk;
+       struct clk *dpi_ck_cg;
        struct clk *pixel_clk;
+       struct clk *dpi_sel_clk;
        struct clk *tvd_clk;
+       struct clk *hdmi_cg;


You're adding new clocks and then you're making *all clocks*, including the
already existing ones... optional.

That looks seriously odd.... can you please give a devicetree example for
MT8195 in the next version, perhaps in the cover letter?

Would also make it easier to test this entire big series.

Regards,
Angelo

Reply via email to