i.MX7 clock support broken

[A.S. Dong]

> -----Original Message-----
> From: Sascha Hauer [mailto:s.hauer at pengutronix.de]
> Sent: Friday, February 10, 2017 2:50 PM
> To: Andy Duan
> Cc: A.S. Dong; linux-arm-kernel at lists.infradead.org; Shawn Guo; Fabio
> Estevam; kernel at pengutronix.de; Frank Li
> Subject: Re: i.MX7 clock support broken
> 
> On Fri, Feb 10, 2017 at 03:37:38AM +0000, Andy Duan wrote:
> > From: A.S. Dong Sent: Thursday, February 09, 2017 12:47 AM
> > >To: Sascha Hauer <s.hauer at pengutronix.de>; linux-arm-
> > >kernel at lists.infradead.org
> > >Cc: Shawn Guo <shawnguo at kernel.org>; Fabio Estevam
> > ><fabio.estevam at nxp.com>; kernel at pengutronix.de; Andy Duan
> > ><fugang.duan at nxp.com>; Frank Li <frank.li at nxp.com>
> > >Subject: RE: i.MX7 clock support broken
> > >
> > >Hi Sascha,
> > >
> > >> -----Original Message-----
> > >> From: Sascha Hauer [mailto:s.hauer at pengutronix.de]
> > >> Sent: Tuesday, February 07, 2017 10:22 PM
> > >> To: linux-arm-kernel at lists.infradead.org
> > >> Cc: Shawn Guo; A.S. Dong; Fabio Estevam; kernel at pengutronix.de
> > >> Subject: i.MX7 clock support broken
> > >>
> > >> Hi All,
> > >>
> > >> I recently adopted the i.MX7 clock tree from Linux to barebox and
> > >> wondered why ethernet doesn't work.
> > >
> > >I just reproduced the issue on MX7D SDB board with latest upstream
> > >kernel That kernel NULL point crash during boot on fec_enet_open->..-
> > >>phy_attach_direct.
> > >
> >
> > The issue was caused by one patch submitted at 2017 Jan.31
> 
> The issue I was talking about is that the clock driver does not control
> the real ethernet clocks, but instead some non existing clocks. This issue
> does not cause any problems in the kernel at all, because when the
> ethernet clocks have been enabled in the bootloader then everything runs
> smoothly in the kernel. Nevertheless the kernel driver is buggy of course,
> the actual clocks are never turned off when not needed.
> 

The initial clk driver was developed based on the info provided by our IC
Guys directly in early stage while RM is still not ready.
It's possible the RM is not correct.

I already sent an email to our IC guys yesterday to confirm it, but still
No response...
Will update you the result once there's a conclusion.

Anyway, thanks a lot for pointing out this issue.

> Sascha
> 
> --
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Regards
Dong Aisheng