Hi Marek,
On Mon, May 22, 2023 at 11:57 PM Marek Vasut <marex@xxxxxxx> wrote:
> > + lvds_bridge: bridge@18 {
> > + compatible = "fsl,imx6sx-ldb";
> > + reg = <0x18 0x4>;
> > + clocks = <&clks IMX6SX_CLK_LDB_DI0>;
> > + clock-names = "ldb";
>
> Since there is only once clock, is this clock-names even needed ?
As of today, clock-names is needed because
drivers/gpu/drm/bridge/fsl-ldb.c retrieves the ldb clock like this:
fsl_ldb->clk = devm_clk_get(dev, "ldb")
If you want, I can change it to fsl_ldb->clk = devm_clk_get(dev, NULL)
and also remove clock-names from
fsl,ldb.yaml and from imx8mp.dtsi.
Or this cleanup can also be a follow-up patch. Just let me know what you prefer.
