[U-Boot] [PATCH v2 1/2] imx: mx6: add get_cpu_speed_grade_hz func to return MHz speed grade from OTP

[Tim Harvey]

On Fri, May 15, 2015 at 12:54 PM, Troy Kisky
<troy.kisky at boundarydevices.com> wrote:
> On 5/15/2015 7:32 AM, Stefano Babic wrote:
>> Hi Christian,
>>
>> On 15/05/2015 16:26, Christian Gmeiner wrote:
>>
>>>> Function does not returns the value of the fuse, else it should returns
>>>> val. The value is parsed for consistency, and if it is not, 0 means a
>>>> failure. IMHO this interpretation is correct - I would agree with you
>>>> only if the return value would be the read value, but it is not.
>>>>
>>>
>>> If all mx6 variants use the same otp register/values, then the
>>> ifs are _NOT_ needed. I hope that later testings/doc updates/etc
>>> proofs it and we can get rid of the superfluous ifs.
>>>
>>
>> Ok - let's see if this can be proofed. It looks like we have to test and
>> ask the hardware for that instead of relying on the documentation.
>>
>> Best regards,
>> Stefano Babic
>>
>
> On our imx6sx board the patch outputs
>
> U-Boot 2015.04-00074-g1bd6819 (May 15 2015 - 12:44:22)
>
> CPU:   Freescale i.MX6SX rev1.0 996 MHz (running at 792 MHz)
> Reset cause: WDOG
> Board: Nitrogen6sx
> ______________
>
>
> Did you want me to test some variation of this?
>
> Thanks
> Troy

Troy,

As far as the frequency grade goes if you know the actual part number
is a 1GHz part then that looks good - thanks for the test!

Could you provide similar results for the thermal grade patch series
as well? (you will need to enable imx6 thermal sensor and remove the
if (!imx6sx) around the print).

Thanks!

Tim