From mboxrd@z Thu Jan  1 00:00:00 1970
From: Troy Kisky <troy.kisky@boundarydevices.com>
Date: Fri, 15 May 2015 12:54:07 -0700
Subject: [U-Boot] [PATCH v2 1/2] imx: mx6: add get_cpu_speed_grade_hz
 func to return MHz speed grade from OTP
In-Reply-To: <55560386.8000302@denx.de>
References: <1431580121-24726-1-git-send-email-tharvey@gateworks.com>
 <1431580121-24726-2-git-send-email-tharvey@gateworks.com>
 <CAH9NwWczchwW2T7aAi_sR6sX2qXuvG+hb5810ETFvruT1VY0zg@mail.gmail.com>
 <5555F299.70307@denx.de>
 <CAH9NwWdgzuf_iW2p6gLsUagMPYUU2UN9PkhACyZ+SbAfp3v9hg@mail.gmail.com>
 <55560386.8000302@denx.de>
Message-ID: <55564EDF.5010207@boundarydevices.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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