From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?= Date: Wed, 24 Apr 2013 16:44:17 +0200 (CEST) Subject: [U-Boot] [PATCH v4 01/10] imx: Homogenize and fix fuse register definitions In-Reply-To: <1366748267-27634-1-git-send-email-benoit.thebaudeau@advansee.com> References: <1366748267-27634-1-git-send-email-benoit.thebaudeau@advansee.com> Message-ID: <1025780228.219881.1366814657481.JavaMail.root@advansee.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 Tuesday, April 23, 2013 10:17:38 PM, Beno?t Th?baudeau wrote: > IIM: > - Homogenize prg_p naming (the reference manuals are not always > self-consistent > for that). > - Add missing SCSx and bank registers. > - Fix the number of banks on i.MX53. > > OCOTP: > - Rename iim to ocotp in order to avoid confusion. > - Rename fuse_data to read_fuse_data, and sticky to sw_sticky, according to > the > reference manual. > - Merge the existing spinoff gp1 fuse definition on i.MX6. > - Fix the number of banks on i.MX6. > > Signed-off-by: Beno?t Th?baudeau FYI, I have build- and runtime-tested this series on i.MX25/35/51/6, and build-tested it on mpc512x. Note that the original OCOTP driver from Freescale has several bugs in the function setting the timing register: - It sets tRELAX to 15.2 ns although it should be > 16.2 ns (for 66-MHz IPG clock). - It uses DEF_RELAX instead of relax to set strobe_prog/read, i.e. it mixes ns with IPG clock cycles. The result is tPGM = 10.6 ?s instead of 10 ?s (but 9 ?s < tPGM < 11 ?s is still satisfied), and tRD = 636 ns > 36 ns. So just by chance, this does not break the hardware behavior. - It unexpectedly clears the WAIT bit-field, which has a non-zero reset value. Best regards, Beno?t