Re: [PATCH V4 RFC 2/3] nvmem: add driver for ocotp in i.MX23 and i.MX28

On Monday, July 20, 2015 at 08:45:21 AM, Stefan Wahren wrote:

Hi Marek,

Hi!

Am 20.07.2015 um 00:12 schrieb Marek Vasut:

quoted

On Monday, July 20, 2015 at 12:04:58 AM, Stefan Wahren wrote:

quoted

This patch brings read-only support for the On-Chip OTP cells
in the i.MX23 and i.MX28 processor. The driver implements the
new NVMEM provider API.

Signed-off-by: Stefan Wahren <redacted>

How come this driver isn't doing any manipulation with the VDDIO
voltage -- or did I miss that? I seem to remember that the VDDIO
had to be lowered to 2.8V before the fuses could be programmed
and restored to 3.3V afterward. See for example recent mainline
u-boot, drivers/misc/mxs_ocotp.c .

Why should the driver implement VDDIO manipulation when there is only
read-only support for the OTP bits? There is no ambition to implement
write support. Maybe i should clarify that in the comments.

Urm, sorry, I missed that read-only part. I apologize.

Btw i'm on your side that because programming the OTP from userspace is
to dangerous.

:)

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html