From mboxrd@z Thu Jan 1 00:00:00 1970 From: Troy Kisky Date: Fri, 15 May 2015 13:43:32 -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: References: <1431580121-24726-1-git-send-email-tharvey@gateworks.com> <1431580121-24726-2-git-send-email-tharvey@gateworks.com> <5555F299.70307@denx.de> <55560386.8000302@denx.de> <55564EDF.5010207@boundarydevices.com> Message-ID: <55565A74.3080809@boundarydevices.com> List-Id: 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 1:38 PM, Tim Harvey wrote: > On Fri, May 15, 2015 at 12:54 PM, Troy Kisky > 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 >