summaryrefslogtreecommitdiff
path: root/drivers/w1
diff options
context:
space:
mode:
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-04-30 06:12:08 +0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-04-30 06:12:08 +0400
commitc0a6720977f89c50473366af965ae28cd8fce923 (patch)
treef7d0d9fcc56bb03fd87e7e1e83525c906c06bd08 /drivers/w1
parentf4cce69611ee941bac0729c6069795f106905ef9 (diff)
downloadlinux-c0a6720977f89c50473366af965ae28cd8fce923.tar.xz
Revert "w1: Add 1-wire slave device driver for DS28E04-100"
This reverts commit f19420c1acb0b573c88a12deb2d42035e22d4a17. It contained lots of errors and warnings and shouldn't have ever been applied, that was my fault, sorry. Cc: Markus Franke <markus.franke@s2002.tu-chemnitz.de> Cc: Evgeniy Polyakov <zbr@ioremap.net>, Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/w1')
-rw-r--r--drivers/w1/slaves/Kconfig7
-rw-r--r--drivers/w1/slaves/Makefile1
-rw-r--r--drivers/w1/slaves/w1_ds28e04.c464
-rw-r--r--drivers/w1/w1_family.h1
4 files changed, 0 insertions, 473 deletions
diff --git a/drivers/w1/slaves/Kconfig b/drivers/w1/slaves/Kconfig
index 605ee36c96f8..eb9e376d6244 100644
--- a/drivers/w1/slaves/Kconfig
+++ b/drivers/w1/slaves/Kconfig
@@ -94,13 +94,6 @@ config W1_SLAVE_DS2781
If you are unsure, say N.
-config W1_SLAVE_DS28E04
- tristate "4096-Bit Addressable 1-Wire EEPROM with PIO (DS28E04-100)"
- depends on W1
- help
- Say Y here if you want to use a 1-wire
- 4kb EEPROM with PIO family device (DS28E04).
-
config W1_SLAVE_BQ27000
tristate "BQ27000 slave support"
depends on W1
diff --git a/drivers/w1/slaves/Makefile b/drivers/w1/slaves/Makefile
index 05188f6aab5a..c4f1859fb520 100644
--- a/drivers/w1/slaves/Makefile
+++ b/drivers/w1/slaves/Makefile
@@ -12,4 +12,3 @@ obj-$(CONFIG_W1_SLAVE_DS2760) += w1_ds2760.o
obj-$(CONFIG_W1_SLAVE_DS2780) += w1_ds2780.o
obj-$(CONFIG_W1_SLAVE_DS2781) += w1_ds2781.o
obj-$(CONFIG_W1_SLAVE_BQ27000) += w1_bq27000.o
-obj-$(CONFIG_W1_SLAVE_DS28E04) += w1_ds28e04.o
diff --git a/drivers/w1/slaves/w1_ds28e04.c b/drivers/w1/slaves/w1_ds28e04.c
deleted file mode 100644
index 4aa1aa90480d..000000000000
--- a/drivers/w1/slaves/w1_ds28e04.c
+++ /dev/null
@@ -1,464 +0,0 @@
-/*
- * w1_ds28e04.c - w1 family 1C (DS28E04) driver
- *
- * Copyright (c) 2012 Markus Franke <franke.m@sebakmt.com>
- *
- * This source code is licensed under the GNU General Public License,
- * Version 2. See the file COPYING for more details.
- */
-
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/moduleparam.h>
-#include <linux/device.h>
-#include <linux/types.h>
-#include <linux/delay.h>
-#include <linux/slab.h>
-#include <linux/crc16.h>
-#include <linux/uaccess.h>
-
-#define CRC16_INIT 0
-#define CRC16_VALID 0xb001
-
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
-
-MODULE_LICENSE("GPL");
-MODULE_AUTHOR("Markus Franke <franke.m@sebakmt.com>, <franm@hrz.tu-chemnitz.de>");
-MODULE_DESCRIPTION("w1 family 1C driver for DS28E04, 4kb EEPROM and PIO");
-
-/* Allow the strong pullup to be disabled, but default to enabled.
- * If it was disabled a parasite powered device might not get the required
- * current to copy the data from the scratchpad to EEPROM. If it is enabled parasite powered
- * devices have a better chance of getting the current required.
- */
-static int w1_strong_pullup = 1;
-module_param_named(strong_pullup, w1_strong_pullup, int, 0);
-
-/* enable/disable CRC checking on DS28E04-100 memory accesses */
-static char w1_enable_crccheck = 1;
-
-#define W1_EEPROM_SIZE 512
-#define W1_PAGE_COUNT 16
-#define W1_PAGE_SIZE 32
-#define W1_PAGE_BITS 5
-#define W1_PAGE_MASK 0x1F
-
-#define W1_F1C_READ_EEPROM 0xF0
-#define W1_F1C_WRITE_SCRATCH 0x0F
-#define W1_F1C_READ_SCRATCH 0xAA
-#define W1_F1C_COPY_SCRATCH 0x55
-#define W1_F1C_ACCESS_WRITE 0x5A
-
-#define W1_1C_REG_LOGIC_STATE 0x220
-
-struct w1_f1C_data {
- u8 memory[W1_EEPROM_SIZE];
- u32 validcrc;
-};
-
-/**
- * Check the file size bounds and adjusts count as needed.
- * This would not be needed if the file size didn't reset to 0 after a write.
- */
-static inline size_t w1_f1C_fix_count(loff_t off, size_t count, size_t size)
-{
- if (off > size)
- return 0;
-
- if ((off + count) > size)
- return (size - off);
-
- return count;
-}
-
-static int w1_f1C_refresh_block(struct w1_slave *sl, struct w1_f1C_data *data,
- int block)
-{
- u8 wrbuf[3];
- int off = block * W1_PAGE_SIZE;
-
- if (data->validcrc & (1 << block))
- return 0;
-
- if (w1_reset_select_slave(sl)) {
- data->validcrc = 0;
- return -EIO;
- }
-
- wrbuf[0] = W1_F1C_READ_EEPROM;
- wrbuf[1] = off & 0xff;
- wrbuf[2] = off >> 8;
- w1_write_block(sl->master, wrbuf, 3);
- w1_read_block(sl->master, &data->memory[off], W1_PAGE_SIZE);
-
- /* cache the block if the CRC is valid */
- if (crc16(CRC16_INIT, &data->memory[off], W1_PAGE_SIZE) == CRC16_VALID)
- data->validcrc |= (1 << block);
-
- return 0;
-}
-
-static int w1_f1C_read(struct w1_slave *sl, int addr, int len, char *data)
-{
- u8 wrbuf[3];
-
- /* read directly from the EEPROM */
- if (w1_reset_select_slave(sl))
- return -EIO;
-
- wrbuf[0] = W1_F1C_READ_EEPROM;
- wrbuf[1] = addr & 0xff;
- wrbuf[2] = addr >> 8;
-
- w1_write_block(sl->master, wrbuf, sizeof(wrbuf));
- return w1_read_block(sl->master, data, len);
-}
-
-static ssize_t w1_f1C_read_bin(struct file *filp, struct kobject *kobj,
- struct bin_attribute *bin_attr,
- char *buf, loff_t off, size_t count)
-{
- struct w1_slave *sl = kobj_to_w1_slave(kobj);
- struct w1_f1C_data *data = sl->family_data;
- int i, min_page, max_page;
-
- if ((count = w1_f1C_fix_count(off, count, W1_EEPROM_SIZE)) == 0)
- return 0;
-
- mutex_lock(&sl->master->mutex);
-
- if(w1_enable_crccheck) {
- min_page = (off >> W1_PAGE_BITS);
- max_page = (off + count - 1) >> W1_PAGE_BITS;
- for (i = min_page; i <= max_page; i++) {
- if (w1_f1C_refresh_block(sl, data, i)) {
- count = -EIO;
- goto out_up;
- }
- }
- memcpy(buf, &data->memory[off], count);
- }
- else {
- count = w1_f1C_read(sl, off, count, buf);
- }
-
-out_up:
- mutex_unlock(&sl->master->mutex);
-
- return count;
-}
-
-/**
- * Writes to the scratchpad and reads it back for verification.
- * Then copies the scratchpad to EEPROM.
- * The data must be on one page.
- * The master must be locked.
- *
- * @param sl The slave structure
- * @param addr Address for the write
- * @param len length must be <= (W1_PAGE_SIZE - (addr & W1_PAGE_MASK))
- * @param data The data to write
- * @return 0=Success -1=failure
- */
-static int w1_f1C_write(struct w1_slave *sl, int addr, int len, const u8 *data)
-{
- u8 wrbuf[4];
- u8 rdbuf[W1_PAGE_SIZE + 3];
- u8 es = (addr + len - 1) & 0x1f;
- unsigned int tm = 10;
- int i;
- struct w1_f1C_data *f1C = sl->family_data;
-
- /* Write the data to the scratchpad */
- if (w1_reset_select_slave(sl))
- return -1;
-
- wrbuf[0] = W1_F1C_WRITE_SCRATCH;
- wrbuf[1] = addr & 0xff;
- wrbuf[2] = addr >> 8;
-
- w1_write_block(sl->master, wrbuf, 3);
- w1_write_block(sl->master, data, len);
-
- /* Read the scratchpad and verify */
- if (w1_reset_select_slave(sl))
- return -1;
-
- w1_write_8(sl->master, W1_F1C_READ_SCRATCH);
- w1_read_block(sl->master, rdbuf, len + 3);
-
- /* Compare what was read against the data written */
- if ((rdbuf[0] != wrbuf[1]) || (rdbuf[1] != wrbuf[2]) ||
- (rdbuf[2] != es) || (memcmp(data, &rdbuf[3], len) != 0))
- return -1;
-
- /* Copy the scratchpad to EEPROM */
- if (w1_reset_select_slave(sl))
- return -1;
-
- wrbuf[0] = W1_F1C_COPY_SCRATCH;
- wrbuf[3] = es;
-
- for(i = 0; i < sizeof(wrbuf); ++i) {
- /* issue 10ms strong pullup (or delay) on the last byte for writing the data from the scratchpad to EEPROM */
- if(w1_strong_pullup && i == sizeof(wrbuf)-1)
- w1_next_pullup(sl->master, tm);
-
- w1_write_8(sl->master, wrbuf[i]);
- }
-
- if(!w1_strong_pullup)
- msleep(tm);
-
- if(w1_enable_crccheck) {
- /* invalidate cached data */
- f1C->validcrc &= ~(1 << (addr >> W1_PAGE_BITS));
- }
-
- /* Reset the bus to wake up the EEPROM (this may not be needed) */
- w1_reset_bus(sl->master);
-
- return 0;
-}
-
-static ssize_t w1_f1C_write_bin(struct file *filp, struct kobject *kobj,
- struct bin_attribute *bin_attr,
- char *buf, loff_t off, size_t count)
-
-{
- struct w1_slave *sl = kobj_to_w1_slave(kobj);
- int addr, len, idx;
-
- if ((count = w1_f1C_fix_count(off, count, W1_EEPROM_SIZE)) == 0)
- return 0;
-
- if(w1_enable_crccheck) {
- /* can only write full blocks in cached mode */
- if ((off & W1_PAGE_MASK) || (count & W1_PAGE_MASK)) {
- dev_err(&sl->dev, "invalid offset/count off=%d cnt=%zd\n",
- (int)off, count);
- return -EINVAL;
- }
-
- /* make sure the block CRCs are valid */
- for (idx = 0; idx < count; idx += W1_PAGE_SIZE) {
- if (crc16(CRC16_INIT, &buf[idx], W1_PAGE_SIZE) != CRC16_VALID) {
- dev_err(&sl->dev, "bad CRC at offset %d\n", (int)off);
- return -EINVAL;
- }
- }
- }
-
- mutex_lock(&sl->master->mutex);
-
- /* Can only write data to one page at a time */
- idx = 0;
- while (idx < count) {
- addr = off + idx;
- len = W1_PAGE_SIZE - (addr & W1_PAGE_MASK);
- if (len > (count - idx))
- len = count - idx;
-
- if (w1_f1C_write(sl, addr, len, &buf[idx]) < 0) {
- count = -EIO;
- goto out_up;
- }
- idx += len;
- }
-
-out_up:
- mutex_unlock(&sl->master->mutex);
-
- return count;
-}
-
-static ssize_t w1_f1C_read_pio(struct file *filp, struct kobject *kobj,
- struct bin_attribute *bin_attr,
- char *buf, loff_t off, size_t count)
-
-{
- struct w1_slave *sl = kobj_to_w1_slave(kobj);
- int ret;
-
- /* check arguments */
- if(off != 0 || count != 1 || buf == NULL)
- return -EINVAL;
-
- mutex_lock(&sl->master->mutex);
- ret = w1_f1C_read(sl, W1_1C_REG_LOGIC_STATE, count, buf);
- mutex_unlock(&sl->master->mutex);
-
- return ret;
-}
-
-static ssize_t w1_f1C_write_pio(struct file *filp, struct kobject *kobj,
- struct bin_attribute *bin_attr,
- char *buf, loff_t off, size_t count)
-
-{
- struct w1_slave *sl = kobj_to_w1_slave(kobj);
- u8 wrbuf[3];
- u8 ack;
-
- /* check arguments */
- if(off != 0 || count != 1 || buf == NULL)
- return -EINVAL;
-
- mutex_lock(&sl->master->mutex);
-
- /* Write the PIO data */
- if (w1_reset_select_slave(sl)) {
- mutex_unlock(&sl->master->mutex);
- return -1;
- }
-
- /* set bit 7..2 to value '1' */
- *buf = *buf | 0xFC;
-
- wrbuf[0] = W1_F1C_ACCESS_WRITE;
- wrbuf[1] = *buf;
- wrbuf[2] = ~(*buf);
- w1_write_block(sl->master, wrbuf, 3);
-
- w1_read_block(sl->master, &ack, sizeof(ack));
-
- mutex_unlock(&sl->master->mutex);
-
- /* check for acknowledgement */
- if(ack != 0xAA) return -EIO;
-
- return count;
-}
-
-static ssize_t w1_f1C_show_crccheck(struct device *dev, struct device_attribute *attr,
- char *buf)
-{
- if(put_user(w1_enable_crccheck + 0x30, buf))
- return -EFAULT;
-
- return sizeof(w1_enable_crccheck);
-}
-
-static ssize_t w1_f1C_store_crccheck(struct device *dev, struct device_attribute *attr,
- const char *buf, size_t count)
-{
- char val;
-
- if(count != 1 || !buf) return -EINVAL;
-
- if(get_user(val, buf))
- return -EFAULT;
-
- /* convert to decimal */
- val = val - 0x30;
- if(val != 0 && val != 1) return -EINVAL;
-
- /* set the new value */
- w1_enable_crccheck = val;
-
- return sizeof(w1_enable_crccheck);
-}
-
-#define NB_SYSFS_BIN_FILES 2
-static struct bin_attribute w1_f1C_bin_attr[NB_SYSFS_BIN_FILES] = {
- {
- .attr = {
- .name = "eeprom",
- .mode = S_IRUGO | S_IWUSR,
- },
- .size = W1_EEPROM_SIZE,
- .read = w1_f1C_read_bin,
- .write = w1_f1C_write_bin,
- },
- {
- .attr = {
- .name = "pio",
- .mode = S_IRUGO | S_IWUSR,
- },
- .size = 1,
- .read = w1_f1C_read_pio,
- .write = w1_f1C_write_pio,
- }
-};
-
-static DEVICE_ATTR(crccheck, S_IWUSR | S_IRUGO, w1_f1C_show_crccheck, w1_f1C_store_crccheck);
-
-static int w1_f1C_add_slave(struct w1_slave *sl)
-{
- int err = 0;
- int i;
- struct w1_f1C_data *data = NULL;
-
- if(w1_enable_crccheck) {
- data = kzalloc(sizeof(struct w1_f1C_data), GFP_KERNEL);
- if (!data)
- return -ENOMEM;
- sl->family_data = data;
- }
-
- /* create binary sysfs attributes */
- for (i = 0; i < NB_SYSFS_BIN_FILES && !err; ++i)
- err = sysfs_create_bin_file(&sl->dev.kobj, &(w1_f1C_bin_attr[i]));
-
- if(err)
- goto out;
-
- /* create device attributes */
- err = device_create_file(&sl->dev, &dev_attr_crccheck);
-
- if(err) {
- /* remove binary sysfs attributes */
- for (i = 0; i < NB_SYSFS_BIN_FILES; ++i)
- sysfs_remove_bin_file(&sl->dev.kobj, &(w1_f1C_bin_attr[i]));
- }
-
-out:
- if(err) {
- if(w1_enable_crccheck)
- kfree(data);
- }
-
- return err;
-}
-
-static void w1_f1C_remove_slave(struct w1_slave *sl)
-{
- int i;
-
- if(w1_enable_crccheck) {
- kfree(sl->family_data);
- sl->family_data = NULL;
- }
-
- /* remove device attributes */
- device_remove_file(&sl->dev, &dev_attr_crccheck);
-
- /* remove binary sysfs attributes */
- for (i = 0; i < NB_SYSFS_BIN_FILES; ++i)
- sysfs_remove_bin_file(&sl->dev.kobj, &(w1_f1C_bin_attr[i]));
-}
-
-static struct w1_family_ops w1_f1C_fops = {
- .add_slave = w1_f1C_add_slave,
- .remove_slave = w1_f1C_remove_slave,
-};
-
-static struct w1_family w1_family_1C = {
- .fid = W1_FAMILY_DS28E04,
- .fops = &w1_f1C_fops,
-};
-
-static int __init w1_f1C_init(void)
-{
- return w1_register_family(&w1_family_1C);
-}
-
-static void __exit w1_f1C_fini(void)
-{
- w1_unregister_family(&w1_family_1C);
-}
-
-module_init(w1_f1C_init);
-module_exit(w1_f1C_fini);
diff --git a/drivers/w1/w1_family.h b/drivers/w1/w1_family.h
index b00ada44a89b..874aeb05011b 100644
--- a/drivers/w1/w1_family.h
+++ b/drivers/w1/w1_family.h
@@ -30,7 +30,6 @@
#define W1_FAMILY_SMEM_01 0x01
#define W1_FAMILY_SMEM_81 0x81
#define W1_THERM_DS18S20 0x10
-#define W1_FAMILY_DS28E04 0x1C
#define W1_COUNTER_DS2423 0x1D
#define W1_THERM_DS1822 0x22
#define W1_EEPROM_DS2433 0x23