On Tuesday 28 October 2014 14:17:39 Ezequiel Garcia wrote: > On 10/20/2014 11:44 AM, Arnd Bergmann wrote: > > 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. > > > > Are you sure MTD is the right place? Recently an eFuse driver was merged > in drivers/soc/tegra/fuse: > > http://lxr.free-electrons.com/source/drivers/soc/tegra/fuse/fuse-tegra.c > > Isn't this a similar device? Hmm, I missed that one. I came up with MTD just because a grep for OTP showed most hits in there. I have to admit that I didn't check whether those are actually for the same kind of device or for something else that goes by the same name. Having it in MTD doesn't sound too obscure though. There are a few other references to efuse or otp in the kernel, which means that at some point we probably want to have a common subsystem and user interface for this, either in MTD or standalone. Between drivers/misc/fsl_ocotp.c, drivers/soc/tegra/fuse and drivers/mfd/ab3100-otp.c, is there any commonality that we could base an abstract API on? 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
