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

Troy Kisky
2015-05-15T20:43:32Z

On 5/15/2015 1:38 PM, Tim Harvey wrote:
> On Fri, May 15, 2015 at 12:54 PM, Troy Kisky
> <troy.kisky@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!

Right, it is a 1GHz part.

>
> 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).

Will do.

>
> Thanks!
>
> Tim
>