Re: [PATCH v8 1/9] nvmem: Add a simple NVMEM framework for nvmem providers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 23/07/15 16:26, Stefan Wahren wrote:
Hi Srinivas,

Am 20.07.2015 um 16:43 schrieb Srinivas Kandagatla:
This patch adds just providers part of the framework just to enable easy
review.

Up until now, NVMEM drivers like eeprom were stored in drivers/misc,
where they all had to duplicate pretty much the same code to register
a sysfs file, allow in-kernel users to access the content of the devices
they were driving, etc.

This was also a problem as far as other in-kernel users were involved,
since the solutions used were pretty much different from on driver to
another, there was a rather big abstraction leak.

This introduction of this framework aims at solving this. It also
introduces DT representation for consumer devices to go get the data
they require (MAC Addresses, SoC/Revision ID, part numbers, and so on)
from the nvmems.

Having regmap interface to this framework would give much better
abstraction for nvmems on different buses.

Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
[Maxime Ripard: intial version of eeprom framework]
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
---
  drivers/Kconfig                |   2 +
  drivers/Makefile               |   1 +
  drivers/nvmem/Kconfig          |  13 ++
  drivers/nvmem/Makefile         |   6 +
  drivers/nvmem/core.c           | 384 +++++++++++++++++++++++++++++++++++++++++
  include/linux/nvmem-consumer.h |  23 +++
  include/linux/nvmem-provider.h |  47 +++++
  7 files changed, 476 insertions(+)
  create mode 100644 drivers/nvmem/Kconfig
  create mode 100644 drivers/nvmem/Makefile
  create mode 100644 drivers/nvmem/core.c
  create mode 100644 include/linux/nvmem-consumer.h
  create mode 100644 include/linux/nvmem-provider.h

i've tested this patch with my mxs-ocotp driver [1].

So you can add

Tested-by: Stefan Wahren <stefan.wahren@xxxxxxxx>

Thanks for tested-by, That helps.

--srini




Regards
Stefan

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux