Messages in this thread | | | From | Linus Walleij <> | Date | Mon, 16 Dec 2019 09:09:52 +0100 | Subject | Re: [PATCH v2 2/4] nvmem: add support for the write-protect pin |
| |
On Tue, Dec 10, 2019 at 4:42 PM Khouloud Touil <ktouil@baylibre.com> 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 memory 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. > > There was a suggestion for introducing the gpiodesc from pdata, but > as pdata is already removed it could be replaced by adding it to > nvmem_config. > > Reference: https://lists.96boards.org/pipermail/dev/2018-August/001056.html > > Signed-off-by: Khouloud Touil <ktouil@baylibre.com>
This is consistent IMO, we just specify that WP is active high as in "when it is high, it actively protects against writing", so: Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours, Linus Walleij
| |