From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 15 May 2015 16:32:38 +0200 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> Message-ID: <55560386.8000302@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================