Re: [PATCH v2 0/4] at24: move write-protect pin handling to nvmem core

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

 





On 19/12/2019 10:53, Bartosz Golaszewski wrote:
czw., 19 gru 2019 o 11:51 Srinivas Kandagatla
<srinivas.kandagatla@xxxxxxxxxx> napisał(a):



On 10/12/2019 15:41, Khouloud Touil wrote:
The write-protect pin handling looks like a standard property that
could benefit other users if available in the core nvmem framework.

Instead of modifying all the drivers to check this pin, make the
nvmem subsystem check if the write-protect GPIO being passed
through the nvmem_config or defined in the device tree and pull it
low whenever writing to the memory.

This patchset:

- adds support for the write-protect pin split into two parts.
The first patch modifies modifies the relevant binding document,
while the second modifies the nvmem code to pull the write-protect
GPIO low (if present) during write operations.

- removes support for the write-protect pin split into two parts.
The first patch modifies the relevant binding document to remove
the wp-gpio, while the second removes the relevant code in the
at24 driver.

Changes since v1:
-Add an explenation on how the wp-gpios works
-keep reference to the wp-gpios in the at24 binding

Khouloud Touil (4):
    dt-bindings: nvmem: new optional property write-protect-gpios
    nvmem: add support for the write-protect pin
    dt-bindings: at24: remove the optional property write-protect-gpios
    eeprom: at24: remove the write-protect pin support


Thanks Khouloud for this patchset,

I can take this via nvmem tree once we get an ack on dt bindings from DT
maintainers.


Hi Srinivas,

this will conflict with my at24 tree for this release - can you put
those patches (once they're fine) into an immutable branch for me to
merge in?

I can ack nvmem core patch so that you can take it directly via at24 tree if thats okay.


-srini

Bart


--srini
   .../devicetree/bindings/eeprom/at24.yaml      |  6 +-----
   .../devicetree/bindings/nvmem/nvmem.yaml      |  9 +++++++++
   drivers/misc/eeprom/at24.c                    |  9 ---------
   drivers/nvmem/core.c                          | 19 +++++++++++++++++--
   drivers/nvmem/nvmem.h                         |  2 ++
   include/linux/nvmem-provider.h                |  3 +++
   6 files changed, 32 insertions(+), 16 deletions(-)




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux