[U-Boot] [PATCH v2 3/4] imx: mx6: add display of CPU temperature grade in print_cpuinfo()

[Tim Harvey]

On Sun, May 17, 2015 at 5:11 PM, Peng Fan <b51431 at freescale.com> wrote:
> Hi Fabio,
>
> On Fri, May 15, 2015 at 10:36:09AM -0300, Fabio Estevam wrote:
>>Hi Tim,
>>
>>On Fri, May 15, 2015 at 10:31 AM, Tim Harvey <tharvey at gateworks.com> wrote:
>>
>>> Yes, that sounds like the best approach. What were your thoughts on
>>> enabling the second CPU: line just to display the temperature grade if
>>> CONFIG_IMX6_THERMAL was not enabled?
>>>
>>> I'm surprised someone on this list hasn't already tested this on IMX6SX.
>>>
>>> Fabio - certainly you must have a reference board with an IMX6SX and
>>> could enable/test that the CPU markings agree with what is
>>> detected/displayed via OTP?
>>
>>Yes, I do have a mx6sx sdb board, but the one I have is from the old
>>silicon and I had issues event with the temperature reading, so I
>>prefer not to rely on my testing.
>>
>>Ye Li / Peng / Nitin,
>>
>>Could some of you please test Tim's series on a mx6sx sdb and let us
>>know what it displays on the CPU lines?
>>
> Tested on mx6sx sdb Revb board. I applied the following patches for this test:
>
> mx6: add OTP bank1 registers
> thermal: imx_thermal: use CPU temperature grade for trip points
> imx: mx6: add display of CPU temperature grade in print_cpuinfo()
> imx: mx6: add get_cpu_temp_grade to obtain cpu temperature grade from OTP
> mx: mx6: display max cpu frequency in print_cpuinfo()
> imx: mx6: add get_cpu_speed_grade_hz func to return MHz speed grade from OTP
>
> The log info: "
> U-Boot 2015.07-rc1-00256-gec154cc (May 18 2015 - 09:00:57)
>
> CPU:   Freescale i.MX6SX rev1.2 996 MHz (running at 792 MHz)
> CPU:    at 39C
> Reset cause: POR
> Board: MX6SX SABRE SDB
> I2C:   ready
> DRAM:  1 GiB
> PMIC:  PFUZE100 ID=0x11
> MMC:   FSL_SDHC: 0, FSL_SDHC: 1, FSL_SDHC: 2
> In:    serial
> Out:   serial
> Err:   serial
> Net:   FEC [PRIME]
> Hit any key to stop autoboot:  0
> "
>>Thanks,
>>
>>Fabio Estevam
>
> Regards,
> Peng.
>
> --

Peng,

I'll send a v3 of the patch that removes disabling of showing the
temperature grade info for imx6sx and please test again with this
version as well as let us know the part markings for your chip so I
can verify the fuses are set properly.

Thanks!

Tim