Anson Huang Best Regards!
> -----Original Message----- > From: Stephen Boyd [mailto:[email protected]] > Sent: Thursday, April 19, 2018 11:18 AM > To: Anson Huang <[email protected]>; Shawn Guo > <[email protected]> > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; dl-linux-imx > <[email protected]>; [email protected]; Fabio Estevam > <[email protected]>; S.j. Wang <[email protected]>; > [email protected] > Subject: Re: [PATCH V2 2/2] ARM: dts: imx6sx-sabreauto: add external 24MHz > clock source > > Quoting Shawn Guo (2018-04-17 07:22:05) > > On Mon, Mar 19, 2018 at 10:30:45AM +0800, Anson Huang wrote: > > > On i.MX6SX SabreAuto board, there is external 24MHz clock source for > > > analog clock2, add this clock source to clock tree. > > > > > > Signed-off-by: Anson Huang <[email protected]> > > > --- > > > changes since V1: > > > remove unnecessary clocks container. > > > > I understand this is suggested by Fabio, but I'm afraid that it's not > > going to work with imx_obtain_fixed_clock() call, which is coded to > > look for clocks under /clocks node. > > > > Should patch #1 be dropped from clk tree? If so, I think we should use V1 patch to keep clocks container? Anson.
