Skip to content

Commit

Permalink
eeprom: at24: add support for the write-protect pin
Browse files Browse the repository at this point in the history
AT24 EEPROMs have a write-protect pin, which - when pulled high -
inhibits writes to the upper quadrant of memory (although it has been
observed that on some chips it disables writing to the entire memory
range).

On some boards, this pin is connected to a GPIO and pulled high by
default, which forces the user to manually change its state before
writing. On linux this means that we either need to hog the line all
the time, or set the GPIO value before writing from outside of the
at24 driver.

Make the driver check if the write-protect GPIO was defined in the
device tree and pull it low whenever writing to the EEPROM.

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
  • Loading branch information
Bartosz Golaszewski committed Jan 1, 2018
1 parent 3f3d8ef commit 6ce261e
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions drivers/misc/eeprom/at24.c
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
#include <linux/regmap.h>
#include <linux/platform_data/at24.h>
#include <linux/pm_runtime.h>
#include <linux/gpio/consumer.h>

/*
* I2C EEPROMs from most vendors are inexpensive and mostly interchangeable.
Expand Down Expand Up @@ -77,6 +78,8 @@ struct at24_data {
struct nvmem_config nvmem_config;
struct nvmem_device *nvmem;

struct gpio_desc *wp_gpio;

/*
* Some chips tie up multiple I2C addresses; dummy devices reserve
* them for us, and we'll use them with SMBus calls.
Expand Down Expand Up @@ -427,12 +430,14 @@ static int at24_write(void *priv, unsigned int off, void *val, size_t count)
* from this host, but not from other I2C masters.
*/
mutex_lock(&at24->lock);
gpiod_set_value_cansleep(at24->wp_gpio, 0);

while (count) {
int status;

status = at24_regmap_write(at24, buf, off, count);
if (status < 0) {
gpiod_set_value_cansleep(at24->wp_gpio, 1);
mutex_unlock(&at24->lock);
pm_runtime_put(dev);
return status;
Expand All @@ -442,6 +447,7 @@ static int at24_write(void *priv, unsigned int off, void *val, size_t count)
count -= status;
}

gpiod_set_value_cansleep(at24->wp_gpio, 1);
mutex_unlock(&at24->lock);

pm_runtime_put(dev);
Expand Down Expand Up @@ -573,6 +579,11 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
at24->num_addresses = num_addresses;
at24->offset_adj = at24_get_offset_adj(chip.flags, chip.byte_len);

at24->wp_gpio = devm_gpiod_get_optional(&client->dev,
"wp", GPIOD_OUT_HIGH);
if (IS_ERR(at24->wp_gpio))
return PTR_ERR(at24->wp_gpio);

at24->client[0].client = client;
at24->client[0].regmap = devm_regmap_init_i2c(client, &regmap_config);
if (IS_ERR(at24->client[0].regmap))
Expand Down

0 comments on commit 6ce261e

Please sign in to comment.