On Saturday 18 October 2014 10:32:51 Stefan Wahren wrote: > This patch brings readonly support for the On Chip OTP cells in the i.MX23 > and i.MX28 processor. The driver uses files (one for each cell) in sysfs > as interface. > > Signed-off-by: Stefan Wahren <stefan.wahren@xxxxxxxx> > --- > drivers/misc/Kconfig | 13 ++ > drivers/misc/Makefile | 1 + > drivers/misc/fsl_ocotp.c | 332 ++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 346 insertions(+) > create mode 100644 drivers/misc/fsl_ocotp.c > > diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig > index b841180..7455efa 100644 > --- a/drivers/misc/Kconfig > +++ b/drivers/misc/Kconfig > @@ -515,6 +515,19 @@ config VEXPRESS_SYSCFG > bus. System Configuration interface is one of the possible means > of generating transactions on this bus. > > +config FSL_OCOTP > + tristate "Freescale MXS On-Chip OTP Memory Support" > + depends on ARCH_MXS && SYSFS > + help > + If you say Y here, you will get support for a readonly > + SysFS interface for the One Time Programmable memory pages that > + are stored on the Freescale i.MX23/i.MX28 processor. > + > + To compile this driver as a module, choose M here: the module > + will be called fsl_ocotp. > + > + If unsure, it is safe to say N. > I think this needs to be an MTD driver, not a "misc" driver, and it should use the proper MTD interfaces instead of introducing an incompatible set of interfaces. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html