On Fri, Jun 07, 2024 at 01:28:35AM +0000, Peng Fan wrote: > > Subject: [PATCH 1/7] arm64: dts: imx8qm: add lvds subsystem > > > > Add irqstear, pwm and i2c in lvds subsystem. > > irqsteer > > > > > Signed-off-by: Frank Li <Frank.Li@xxxxxxx> > > --- > > arch/arm64/boot/dts/freescale/imx8qm-ss-lvds.dtsi | 231 > > ++++++++++++++++++++++ > > arch/arm64/boot/dts/freescale/imx8qm.dtsi | 1 + > > 2 files changed, 232 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qm-ss-lvds.dtsi > > b/arch/arm64/boot/dts/freescale/imx8qm-ss-lvds.dtsi > > new file mode 100644 > > index 0000000000000..eb8208cddeaf9 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/freescale/imx8qm-ss-lvds.dtsi > > @@ -0,0 +1,231 @@ > > +// SPDX-License-Identifier: GPL-2.0+ > > GPL-2.0-only and MIT? > > > + > > +/* > > + * Copyright 2024 NXP > > + */ > > + > > +/{ > > + > > + lvds1_ipg_clk: lvds0_ipg_clk: clock-controller-lvds-ipg { > > Two alias name? It is alllowed by dt. link to one fixed 24M clock. of course we can change lvds1_ipg_clk to lvds0_ipg_clk. To avoid confuse, I'd better use two name to align node nams. Frank > > > + compatible = "fixed-clock"; > > + #clock-cells = <0>; > > + clock-frequency = <24000000>; > > + clock-output-names = "lvds0_ipg_clk"; > > + }; > > + > > Regards, > Peng.