Re: [PATCH V2 2/2] ARM: dts: imx6ul: add GPIO clocks

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

 



On Fri, Jul 06, 2018 at 10:15:34AM -0700, Stephen Boyd wrote:
> Quoting Shawn Guo (2018-06-30 19:19:49)
> > On Fri, Jun 29, 2018 at 11:39:38AM -0700, Stephen Boyd wrote:
> > > Quoting Shawn Guo (2018-06-27 17:52:18)
> > > > On Mon, Jun 25, 2018 at 03:14:39AM +0000, Anson Huang wrote:
> > > > > Gentle Ping...
> > > > 
> > > > I cannot apply this dts patch until the clock patch is landed on
> > > > mainline, because it has a dependency on new clock ID IMX6UL_CLK_GPIO1
> > > > created by clock patch.
> > > > 
> > > 
> > > Does it matter if that clk ID changes across branches? Or is everything
> > > good if it just exists as some number?
> > 
> > We use macro IMX6UL_CLK_GPIO1 instead of hard-coded number in DTS.  So
> > it doesn't matter if IMX6UL_CLK_GPIO1 changes.
> > 
> > > The patch adding the define
> > > conflicts with another patch to fix the numbering scheme to be
> > > incremental instead of changing IMX6UL and IMX6ULL which is annoying.
> > > I'll probably make a topic branch just for the fix and merge it into
> > > clk-fixes and this branch so you can pick it up from there if you like.
> > 
> > It will be good if you can have a topic branch.  Otherwise, we are also
> > fine to wait till the clock patches lands on mainline.
> > 
> 
> Ok. The topic branch is out there as clk-imx-gpio-gates

Thanks.  I applied dts patch now.

Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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