[PATCH] W1: Move w1 bus master code into 'w1/masters' and move w1 slave code into 'w1/slaves'

Signed-off-by: Ben Gardner <bgardner@wabtec.com>
Signed-off-by: Evgeniy Polyakov <johnpol@2ka.mipt.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

5 files changed, 718 insertions, 0 deletions

diff --git a/drivers/w1/slaves/Kconfig b/drivers/w1/slaves/Kconfig
new file mode 100644
index 000000000000..f9d4c91fc533
--- /dev/null
+++ b/drivers/w1/slaves/Kconfig
@@ -0,0 +1,38 @@
+#
+# 1-wire slaves configuration
+#
+
+menu "1-wire Slaves"
+        depends on W1
+
+config W1_SLAVE_THERM
+        tristate "Thermal family implementation"
+        depends on W1
+        help
+          Say Y here if you want to connect 1-wire thermal sensors to you
+          wire.
+
+config W1_SLAVE_SMEM
+        tristate "Simple 64bit memory family implementation"
+        depends on W1
+        help
+          Say Y here if you want to connect 1-wire
+          simple 64bit memory rom(ds2401/ds2411/ds1990*) to you wire.
+
+config W1_SLAVE_DS2433
+        tristate "4kb EEPROM family support (DS2433)"
+        depends on W1
+        help
+          Say Y here if you want to use a 1-wire
+          4kb EEPROM family device (DS2433).
+
+config W1_SLAVE_DS2433_CRC
+        bool "Protect DS2433 data with a CRC16"
+        depends on W1_DS2433
+        select CRC16
+        help
+          Say Y here to protect DS2433 data with a CRC16.
+          Each block has 30 bytes of data and a two byte CRC16.
+          Full block writes are only allowed if the CRC is valid.
+
+endmenu
diff --git a/drivers/w1/slaves/Makefile b/drivers/w1/slaves/Makefile
new file mode 100644
index 000000000000..70e21e2d70c3
--- /dev/null
+++ b/drivers/w1/slaves/Makefile
@@ -0,0 +1,12 @@
+#
+# Makefile for the Dallas's 1-wire slaves.
+#
+
+ifeq ($(CONFIG_W1_SLAVE_DS2433_CRC), y)
+EXTRA_CFLAGS += -DCONFIG_W1_F23_CRC
+endif
+
+obj-$(CONFIG_W1_SLAVE_THERM)    += w1_therm.o
+obj-$(CONFIG_W1_SLAVE_SMEM)     += w1_smem.o
+obj-$(CONFIG_W1_SLAVE_DS2433)   += w1_ds2433.o
+
diff --git a/drivers/w1/slaves/w1_ds2433.c b/drivers/w1/slaves/w1_ds2433.c
new file mode 100644
index 000000000000..fb118be789ea
--- /dev/null
+++ b/drivers/w1/slaves/w1_ds2433.c
@@ -0,0 +1,329 @@
+/*
+ *      w1_ds2433.c - w1 family 23 (DS2433) driver
+ *
+ * Copyright (c) 2005 Ben Gardner <bgardner@wabtec.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>
+#ifdef CONFIG_W1_F23_CRC
+#include <linux/crc16.h>
+
+#define CRC16_INIT              0
+#define CRC16_VALID             0xb001
+
+#endif
+
+#include "../w1.h"
+#include "../w1_io.h"
+#include "../w1_int.h"
+#include "../w1_family.h"
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Ben Gardner <bgardner@wabtec.com>");
+MODULE_DESCRIPTION("w1 family 23 driver for DS2433, 4kb EEPROM");
+
+#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_F23_TIME             300
+
+#define W1_F23_READ_EEPROM      0xF0
+#define W1_F23_WRITE_SCRATCH    0x0F
+#define W1_F23_READ_SCRATCH     0xAA
+#define W1_F23_COPY_SCRATCH     0x55
+
+struct w1_f23_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_f23_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;
+}
+
+#ifdef CONFIG_W1_F23_CRC
+static int w1_f23_refresh_block(struct w1_slave *sl, struct w1_f23_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_F23_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;
+}
+#endif  /* CONFIG_W1_F23_CRC */
+
+static ssize_t w1_f23_read_bin(struct kobject *kobj, char *buf, loff_t off,
+                               size_t count)
+{
+        struct w1_slave *sl = kobj_to_w1_slave(kobj);
+#ifdef CONFIG_W1_F23_CRC
+        struct w1_f23_data *data = sl->family_data;
+        int i, min_page, max_page;
+#else
+        u8 wrbuf[3];
+#endif
+
+        if ((count = w1_f23_fix_count(off, count, W1_EEPROM_SIZE)) == 0)
+                return 0;
+
+        atomic_inc(&sl->refcnt);
+        if (down_interruptible(&sl->master->mutex)) {
+                count = 0;
+                goto out_dec;
+        }
+
+#ifdef CONFIG_W1_F23_CRC
+
+        min_page = (off >> W1_PAGE_BITS);
+        max_page = (off + count - 1) >> W1_PAGE_BITS;
+        for (i = min_page; i <= max_page; i++) {
+                if (w1_f23_refresh_block(sl, data, i)) {
+                        count = -EIO;
+                        goto out_up;
+                }
+        }
+        memcpy(buf, &data->memory[off], count);
+
+#else   /* CONFIG_W1_F23_CRC */
+
+        /* read directly from the EEPROM */
+        if (w1_reset_select_slave(sl)) {
+                count = -EIO;
+                goto out_up;
+        }
+
+        wrbuf[0] = W1_F23_READ_EEPROM;
+        wrbuf[1] = off & 0xff;
+        wrbuf[2] = off >> 8;
+        w1_write_block(sl->master, wrbuf, 3);
+        w1_read_block(sl->master, buf, count);
+
+#endif  /* CONFIG_W1_F23_CRC */
+
+out_up:
+        up(&sl->master->mutex);
+out_dec:
+        atomic_dec(&sl->refcnt);
+
+        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_f23_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;
+
+        /* Write the data to the scratchpad */
+        if (w1_reset_select_slave(sl))
+                return -1;
+
+        wrbuf[0] = W1_F23_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_F23_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_F23_COPY_SCRATCH;
+        wrbuf[3] = es;
+        w1_write_block(sl->master, wrbuf, 4);
+
+        /* Sleep for 5 ms to wait for the write to complete */
+        msleep(5);
+
+        /* Reset the bus to wake up the EEPROM (this may not be needed) */
+        w1_reset_bus(sl->master);
+
+        return 0;
+}
+
+static ssize_t w1_f23_write_bin(struct kobject *kobj, char *buf, loff_t off,
+                                size_t count)
+{
+        struct w1_slave *sl = kobj_to_w1_slave(kobj);
+        int addr, len, idx;
+
+        if ((count = w1_f23_fix_count(off, count, W1_EEPROM_SIZE)) == 0)
+                return 0;
+
+#ifdef CONFIG_W1_F23_CRC
+        /* 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;
+                }
+        }
+#endif  /* CONFIG_W1_F23_CRC */
+
+        atomic_inc(&sl->refcnt);
+        if (down_interruptible(&sl->master->mutex)) {
+                count = 0;
+                goto out_dec;
+        }
+
+        /* 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_f23_write(sl, addr, len, &buf[idx]) < 0) {
+                        count = -EIO;
+                        goto out_up;
+                }
+                idx += len;
+        }
+
+out_up:
+        up(&sl->master->mutex);
+out_dec:
+        atomic_dec(&sl->refcnt);
+
+        return count;
+}
+
+static struct bin_attribute w1_f23_bin_attr = {
+        .attr = {
+                .name = "eeprom",
+                .mode = S_IRUGO | S_IWUSR,
+                .owner = THIS_MODULE,
+        },
+        .size = W1_EEPROM_SIZE,
+        .read = w1_f23_read_bin,
+        .write = w1_f23_write_bin,
+};
+
+static int w1_f23_add_slave(struct w1_slave *sl)
+{
+        int err;
+#ifdef CONFIG_W1_F23_CRC
+        struct w1_f23_data *data;
+
+        data = kmalloc(sizeof(struct w1_f23_data), GFP_KERNEL);
+        if (!data)
+                return -ENOMEM;
+        memset(data, 0, sizeof(struct w1_f23_data));
+        sl->family_data = data;
+
+#endif  /* CONFIG_W1_F23_CRC */
+
+        err = sysfs_create_bin_file(&sl->dev.kobj, &w1_f23_bin_attr);
+
+#ifdef CONFIG_W1_F23_CRC
+        if (err)
+                kfree(data);
+#endif  /* CONFIG_W1_F23_CRC */
+
+        return err;
+}
+
+static void w1_f23_remove_slave(struct w1_slave *sl)
+{
+#ifdef CONFIG_W1_F23_CRC
+        kfree(sl->family_data);
+        sl->family_data = NULL;
+#endif  /* CONFIG_W1_F23_CRC */
+        sysfs_remove_bin_file(&sl->dev.kobj, &w1_f23_bin_attr);
+}
+
+static struct w1_family_ops w1_f23_fops = {
+        .add_slave      = w1_f23_add_slave,
+        .remove_slave   = w1_f23_remove_slave,
+};
+
+static struct w1_family w1_family_23 = {
+        .fid = W1_EEPROM_DS2433,
+        .fops = &w1_f23_fops,
+};
+
+static int __init w1_f23_init(void)
+{
+        return w1_register_family(&w1_family_23);
+}
+
+static void __exit w1_f23_fini(void)
+{
+        w1_unregister_family(&w1_family_23);
+}
+
+module_init(w1_f23_init);
+module_exit(w1_f23_fini);
diff --git a/drivers/w1/slaves/w1_smem.c b/drivers/w1/slaves/w1_smem.c
new file mode 100644
index 000000000000..c6d3be54f94c
--- /dev/null
+++ b/drivers/w1/slaves/w1_smem.c
@@ -0,0 +1,71 @@
+/*
+ *      w1_smem.c
+ *
+ * Copyright (c) 2004 Evgeniy Polyakov <johnpol@2ka.mipt.ru>
+ *
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the smems of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include <asm/types.h>
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/moduleparam.h>
+#include <linux/device.h>
+#include <linux/types.h>
+
+#include "../w1.h"
+#include "../w1_io.h"
+#include "../w1_int.h"
+#include "../w1_family.h"
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Evgeniy Polyakov <johnpol@2ka.mipt.ru>");
+MODULE_DESCRIPTION("Driver for 1-wire Dallas network protocol, 64bit memory family.");
+
+static struct w1_family w1_smem_family_01 = {
+        .fid = W1_FAMILY_SMEM_01,
+};
+
+static struct w1_family w1_smem_family_81 = {
+        .fid = W1_FAMILY_SMEM_81,
+};
+
+static int __init w1_smem_init(void)
+{
+        int err;
+
+        err = w1_register_family(&w1_smem_family_01);
+        if (err)
+                return err;
+
+        err = w1_register_family(&w1_smem_family_81);
+        if (err) {
+                w1_unregister_family(&w1_smem_family_01);
+                return err;
+        }
+
+        return 0;
+}
+
+static void __exit w1_smem_fini(void)
+{
+        w1_unregister_family(&w1_smem_family_01);
+        w1_unregister_family(&w1_smem_family_81);
+}
+
+module_init(w1_smem_init);
+module_exit(w1_smem_fini);
diff --git a/drivers/w1/slaves/w1_therm.c b/drivers/w1/slaves/w1_therm.c
new file mode 100644
index 000000000000..536d16d78de7
--- /dev/null
+++ b/drivers/w1/slaves/w1_therm.c
@@ -0,0 +1,268 @@
+/*
+ *      w1_therm.c
+ *
+ * Copyright (c) 2004 Evgeniy Polyakov <johnpol@2ka.mipt.ru>
+ *
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the therms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include <asm/types.h>
+
+#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 "../w1.h"
+#include "../w1_io.h"
+#include "../w1_int.h"
+#include "../w1_family.h"
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Evgeniy Polyakov <johnpol@2ka.mipt.ru>");
+MODULE_DESCRIPTION("Driver for 1-wire Dallas network protocol, temperature family.");
+
+static u8 bad_roms[][9] = {
+                                {0xaa, 0x00, 0x4b, 0x46, 0xff, 0xff, 0x0c, 0x10, 0x87},
+                                {}
+                        };
+
+static ssize_t w1_therm_read_bin(struct kobject *, char *, loff_t, size_t);
+
+static struct bin_attribute w1_therm_bin_attr = {
+        .attr = {
+                .name = "w1_slave",
+                .mode = S_IRUGO,
+                .owner = THIS_MODULE,
+        },
+        .size = W1_SLAVE_DATA_SIZE,
+        .read = w1_therm_read_bin,
+};
+
+static int w1_therm_add_slave(struct w1_slave *sl)
+{
+        return sysfs_create_bin_file(&sl->dev.kobj, &w1_therm_bin_attr);
+}
+
+static void w1_therm_remove_slave(struct w1_slave *sl)
+{
+        sysfs_remove_bin_file(&sl->dev.kobj, &w1_therm_bin_attr);
+}
+
+static struct w1_family_ops w1_therm_fops = {
+        .add_slave      = w1_therm_add_slave,
+        .remove_slave   = w1_therm_remove_slave,
+};
+
+static struct w1_family w1_therm_family_DS18S20 = {
+        .fid = W1_THERM_DS18S20,
+        .fops = &w1_therm_fops,
+};
+
+static struct w1_family w1_therm_family_DS18B20 = {
+        .fid = W1_THERM_DS18B20,
+        .fops = &w1_therm_fops,
+};
+
+static struct w1_family w1_therm_family_DS1822 = {
+        .fid = W1_THERM_DS1822,
+        .fops = &w1_therm_fops,
+};
+
+struct w1_therm_family_converter
+{
+        u8                      broken;
+        u16                     reserved;
+        struct w1_family        *f;
+        int                     (*convert)(u8 rom[9]);
+};
+
+static inline int w1_DS18B20_convert_temp(u8 rom[9]);
+static inline int w1_DS18S20_convert_temp(u8 rom[9]);
+
+static struct w1_therm_family_converter w1_therm_families[] = {
+        {
+                .f              = &w1_therm_family_DS18S20,
+                .convert        = w1_DS18S20_convert_temp
+        },
+        {
+                .f              = &w1_therm_family_DS1822,
+                .convert        = w1_DS18B20_convert_temp
+        },
+        {
+                .f              = &w1_therm_family_DS18B20,
+                .convert        = w1_DS18B20_convert_temp
+        },
+};
+
+static inline int w1_DS18B20_convert_temp(u8 rom[9])
+{
+        int t = (rom[1] << 8) | rom[0];
+        t /= 16;
+        return t;
+}
+
+static inline int w1_DS18S20_convert_temp(u8 rom[9])
+{
+        int t, h;
+
+        if (!rom[7])
+                return 0;
+
+        if (rom[1] == 0)
+                t = ((s32)rom[0] >> 1)*1000;
+        else
+                t = 1000*(-1*(s32)(0x100-rom[0]) >> 1);
+
+        t -= 250;
+        h = 1000*((s32)rom[7] - (s32)rom[6]);
+        h /= (s32)rom[7];
+        t += h;
+
+        return t;
+}
+
+static inline int w1_convert_temp(u8 rom[9], u8 fid)
+{
+        int i;
+
+        for (i=0; i<sizeof(w1_therm_families)/sizeof(w1_therm_families[0]); ++i)
+                if (w1_therm_families[i].f->fid == fid)
+                        return w1_therm_families[i].convert(rom);
+
+        return 0;
+}
+
+static int w1_therm_check_rom(u8 rom[9])
+{
+        int i;
+
+        for (i=0; i<sizeof(bad_roms)/9; ++i)
+                if (!memcmp(bad_roms[i], rom, 9))
+                        return 1;
+
+        return 0;
+}
+
+static ssize_t w1_therm_read_bin(struct kobject *kobj, char *buf, loff_t off, size_t count)
+{
+        struct w1_slave *sl = kobj_to_w1_slave(kobj);
+        struct w1_master *dev = sl->master;
+        u8 rom[9], crc, verdict;
+        int i, max_trying = 10;
+
+        atomic_inc(&sl->refcnt);
+        smp_mb__after_atomic_inc();
+        if (down_interruptible(&sl->master->mutex)) {
+                count = 0;
+                goto out_dec;
+        }
+
+        if (off > W1_SLAVE_DATA_SIZE) {
+                count = 0;
+                goto out;
+        }
+        if (off + count > W1_SLAVE_DATA_SIZE) {
+                count = 0;
+                goto out;
+        }
+
+        memset(buf, 0, count);
+        memset(rom, 0, sizeof(rom));
+
+        count = 0;
+        verdict = 0;
+        crc = 0;
+
+        while (max_trying--) {
+                if (!w1_reset_select_slave(sl)) {
+                        int count = 0;
+                        unsigned int tm = 750;
+
+                        w1_write_8(dev, W1_CONVERT_TEMP);
+
+                        while (tm) {
+                                tm = msleep_interruptible(tm);
+                                if (signal_pending(current))
+                                        flush_signals(current);
+                        }
+
+                        if (!w1_reset_select_slave(sl)) {
+
+                                w1_write_8(dev, W1_READ_SCRATCHPAD);
+                                if ((count = w1_read_block(dev, rom, 9)) != 9) {
+                                        dev_warn(&dev->dev, "w1_read_block() returned %d instead of 9.\n", count);
+                                }
+
+                                crc = w1_calc_crc8(rom, 8);
+
+                                if (rom[8] == crc && rom[0])
+                                        verdict = 1;
+                        }
+                }
+
+                if (!w1_therm_check_rom(rom))
+                        break;
+        }
+
+        for (i = 0; i < 9; ++i)
+                count += sprintf(buf + count, "%02x ", rom[i]);
+        count += sprintf(buf + count, ": crc=%02x %s\n",
+                           crc, (verdict) ? "YES" : "NO");
+        if (verdict)
+                memcpy(sl->rom, rom, sizeof(sl->rom));
+        else
+                dev_warn(&dev->dev, "18S20 doesn't respond to CONVERT_TEMP.\n");
+
+        for (i = 0; i < 9; ++i)
+                count += sprintf(buf + count, "%02x ", sl->rom[i]);
+
+        count += sprintf(buf + count, "t=%d\n", w1_convert_temp(rom, sl->family->fid));
+out:
+        up(&dev->mutex);
+out_dec:
+        smp_mb__before_atomic_inc();
+        atomic_dec(&sl->refcnt);
+
+        return count;
+}
+
+static int __init w1_therm_init(void)
+{
+        int err, i;
+
+        for (i=0; i<sizeof(w1_therm_families)/sizeof(w1_therm_families[0]); ++i) {
+                err = w1_register_family(w1_therm_families[i].f);
+                if (err)
+                        w1_therm_families[i].broken = 1;
+        }
+
+        return 0;
+}
+
+static void __exit w1_therm_fini(void)
+{
+        int i;
+
+        for (i=0; i<sizeof(w1_therm_families)/sizeof(w1_therm_families[0]); ++i)
+                if (!w1_therm_families[i].broken)
+                        w1_unregister_family(w1_therm_families[i].f);
+}
+
+module_init(w1_therm_init);
+module_exit(w1_therm_fini);