[U-Boot] [PATCH v4 01/10] imx: Homogenize and fix fuse register definitions

Benoît Thébaudeau benoit.thebaudeau at advansee.com
Wed Apr 24 16:44:17 CEST 2013


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 <benoit.thebaudeau at advansee.com>

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


More information about the U-Boot mailing list