From mboxrd@z Thu Jan 1 00:00:00 1970 From: Troy Kisky Date: Fri, 15 May 2015 15:34:41 -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: <55565A74.3080809@boundarydevices.com> 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> <55565A74.3080809@boundarydevices.com> Message-ID: <55567481.6000709@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:43 PM, Troy Kisky wrote: > 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 >> Ok. I think I gave bad info The part number is PCIMX6X4AVM08AA, which should be -40 to 125 C, 800 MHz and u-boot prints U-Boot 2015.07-rc1-01048-gc4f95cb-dirty (May 15 2015 - 14:46:05) CPU: Freescale i.MX6SX rev1.0 996 MHz (running at 792 MHz) CPU: Extended Commercial temperature grade (-20C to 105C) _______________ U-boot then freezes in "uclass_get_device(UCLASS_THERMAL, 0, &thermal_dev);" Troy