Messages in this thread | | | From | Fabio Estevam <> | Date | Mon, 12 Jun 2017 11:30:41 -0300 | Subject | Re: [PATCH 2/2] ARM: dts: imx6ul: Add imx6ul-tempmon |
| |
On Mon, Jun 12, 2017 at 10:37 AM, Lothar Waßmann <LW@karo-electronics.de> wrote: > Hi, > > On Mon, 12 Jun 2017 10:22:45 -0300 Fabio Estevam wrote: >> On Mon, Jun 12, 2017 at 8:47 AM, Leonard Crestez >> <leonard.crestez@nxp.com> wrote: >> >> > However it seems that this might be accidental, it just happens that >> > the OCOTP clock starts as enabled and is only disabled later in the >> >> Most likely because U-Boot enabled the OCOTP clock as it reads the >> speed grading fuse. >> > No, the imx6q_opp_check_speed_grading() runs on an i.MX6Q which, as > mentioned in my other mail <20170612124029.1643469d@karo-electronics.de>, > does not require any clock to be enabled for accessing the OCOTP regs.
[Sorry I removed the list in my previous response by mistake]
What I was trying to say is that the OCOTP clock is enabled by U-Boot on mx6ul evk board.
Please check: http://git.denx.de/?p=u-boot.git;a=blob;f=board/freescale/mx6ul_14x14_evk/mx6ul_14x14_evk.c;h=a5746fe08688d5752bf1d69cff465a355404d6da;hb=HEAD#l833
This would explain why Leonard did not see the kernel hang on his test.
So I agree with you that the thermal driver needs the IMX6UL_CLK_OCOTP too.
| |