Re: [PATCH] dt-bindings: imx8mq: Number clocks consecutively

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 */
> 



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux