Hi Lucas, On Tue, Jan 15, 2019 at 05:53:51PM +0100, Lucas Stach wrote: > Am Dienstag, den 15.01.2019, 17:35 +0100 schrieb Guido Günther: > > Signed-off-by: Guido Günther <agx@xxxxxxxxxxx> > > This is a breaking change in terms of devicetree stability. So either > we rush this into 5.0-rcX, before the current definition spreads around > or we need to drop this change. I'll leave this for Shawn (CCed) to > decide. Note that 232 is currently used for both IMX8MQ_CLK_HDMI_ROOT and IMX8MQ_SYS1_PLL_OUT so we need to fix that one up - or need to come up with a new name and have that duplicate assignment in there forever. So fixing this for 5.0 would be cool. Cheers, -- Guido > > Regards, > Lucas > > > --- > > > > include/dt-bindings/clock/imx8mq-clock.h | 26 ++++++++++++------------ > > 1 file changed, 13 insertions(+), 13 deletions(-) > > > > diff --git a/include/dt-bindings/clock/imx8mq-clock.h b/include/dt-bindings/clock/imx8mq-clock.h > > index b53be41929be..04f7ac345984 100644 > > --- a/include/dt-bindings/clock/imx8mq-clock.h > > +++ b/include/dt-bindings/clock/imx8mq-clock.h > > @@ -350,7 +350,7 @@ > > > #define IMX8MQ_CLK_VPU_G2_ROOT 241 > > > > /* SCCG PLL GATE */ > > > -#define IMX8MQ_SYS1_PLL_OUT 232 > > > +#define IMX8MQ_SYS1_PLL_OUT 242 > > > #define IMX8MQ_SYS2_PLL_OUT 243 > > > #define IMX8MQ_SYS3_PLL_OUT 244 > > > #define IMX8MQ_DRAM_PLL_OUT 245 > > @@ -372,24 +372,24 @@ > > /* txesc clock */ > > #define IMX8MQ_CLK_DSI_IPG_DIV 256 > > > > > -#define IMX8MQ_CLK_TMU_ROOT 265 > > > +#define IMX8MQ_CLK_TMU_ROOT 257 > > > > /* Display root clocks */ > > > -#define IMX8MQ_CLK_DISP_AXI_ROOT 266 > > > -#define IMX8MQ_CLK_DISP_APB_ROOT 267 > > > -#define IMX8MQ_CLK_DISP_RTRM_ROOT 268 > > > +#define IMX8MQ_CLK_DISP_AXI_ROOT 258 > > > +#define IMX8MQ_CLK_DISP_APB_ROOT 259 > > > +#define IMX8MQ_CLK_DISP_RTRM_ROOT 260 > > > > > -#define IMX8MQ_CLK_OCOTP_ROOT 269 > > > +#define IMX8MQ_CLK_OCOTP_ROOT 261 > > > > > -#define IMX8MQ_CLK_DRAM_ALT_ROOT 270 > > > -#define IMX8MQ_CLK_DRAM_CORE 271 > > > +#define IMX8MQ_CLK_DRAM_ALT_ROOT 262 > > > +#define IMX8MQ_CLK_DRAM_CORE 263 > > > > > -#define IMX8MQ_CLK_MU_ROOT 272 > > > -#define IMX8MQ_VIDEO2_PLL_OUT 273 > > > +#define IMX8MQ_CLK_MU_ROOT 264 > > > +#define IMX8MQ_VIDEO2_PLL_OUT 265 > > > > > -#define IMX8MQ_CLK_CLKO2 274 > > > +#define IMX8MQ_CLK_CLKO2 266 > > > > > -#define IMX8MQ_CLK_NAND_USDHC_BUS_RAWNAND_CLK 275 > > > +#define IMX8MQ_CLK_NAND_USDHC_BUS_RAWNAND_CLK 267 > > > > > -#define IMX8MQ_CLK_END 276 > > > +#define IMX8MQ_CLK_END 268 > > #endif /* __DT_BINDINGS_CLOCK_IMX8MQ_H */ >