w1: Add subsystem kernel public interface
authorAndrew F. Davis <afd@ti.com>
Mon, 5 Jun 2017 13:52:08 +0000 (08:52 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 9 Jun 2017 09:54:54 +0000 (11:54 +0200)
Like other subsystems we should be able to define slave devices outside
of the w1 directory. To do this we move public facing interface
definitions to include/linux/w1.h and rename the internal definition
file to w1_internal.h.

As w1_family.h and w1_int.h contained almost entirely public
driver interface definitions we simply removed these files and
moved the remaining definitions into w1_internal.h.

With this we can now start to move slave devices out of w1/slaves and
into the subsystem based on the function they implement, again like
other drivers.

Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Sebastian Reichel <sre@kernel.org>
Acked-by: Evgeniy Polyakov <zbr@ioremap.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
36 files changed:
MAINTAINERS
drivers/power/supply/ds2760_battery.c
drivers/power/supply/ds2780_battery.c
drivers/power/supply/ds2781_battery.c
drivers/w1/masters/ds1wm.c
drivers/w1/masters/ds2482.c
drivers/w1/masters/ds2490.c
drivers/w1/masters/matrox_w1.c
drivers/w1/masters/mxc_w1.c
drivers/w1/masters/omap_hdq.c
drivers/w1/masters/w1-gpio.c
drivers/w1/slaves/w1_bq27000.c
drivers/w1/slaves/w1_ds2405.c
drivers/w1/slaves/w1_ds2406.c
drivers/w1/slaves/w1_ds2408.c
drivers/w1/slaves/w1_ds2413.c
drivers/w1/slaves/w1_ds2423.c
drivers/w1/slaves/w1_ds2431.c
drivers/w1/slaves/w1_ds2433.c
drivers/w1/slaves/w1_ds2438.c
drivers/w1/slaves/w1_ds2760.c
drivers/w1/slaves/w1_ds2780.c
drivers/w1/slaves/w1_ds2781.c
drivers/w1/slaves/w1_ds28e04.c
drivers/w1/slaves/w1_smem.c
drivers/w1/slaves/w1_therm.c
drivers/w1/w1.c
drivers/w1/w1_family.c
drivers/w1/w1_family.h [deleted file]
drivers/w1/w1_int.c
drivers/w1/w1_int.h [deleted file]
drivers/w1/w1_internal.h [new file with mode: 0644]
drivers/w1/w1_io.c
drivers/w1/w1_netlink.c
drivers/w1/w1_netlink.h
include/linux/w1.h [moved from drivers/w1/w1.h with 76% similarity]

index 2dc114b9e7f760f9f2c13e533e26909f3f0d540b..2461e78b173f75b275e8321d79e8aeced5449217 100644 (file)
@@ -13790,6 +13790,7 @@ M:      Evgeniy Polyakov <zbr@ioremap.net>
 S:     Maintained
 F:     Documentation/w1/
 F:     drivers/w1/
+F:     include/linux/w1.h
 
 W83791D HARDWARE MONITORING DRIVER
 M:     Marc Hulsman <m.hulsman@tudelft.nl>
index 17225689e3f65c6e3c03746b1e868ae287c0067f..ae180dc929c9360e1b6efb8b0677b802b3c91e83 100644 (file)
@@ -28,7 +28,7 @@
 #include <linux/platform_device.h>
 #include <linux/power_supply.h>
 
-#include "../../w1/w1.h"
+#include <linux/w1.h>
 #include "../../w1/slaves/w1_ds2760.h"
 
 struct ds2760_device_info {
index 1b3b6fa89c28d3cf528b880c7699aa3d2d74417b..8edd4aa5f47510c68cd9e92e32c422ef3e02cc7d 100644 (file)
@@ -21,7 +21,7 @@
 #include <linux/power_supply.h>
 #include <linux/idr.h>
 
-#include "../../w1/w1.h"
+#include <linux/w1.h>
 #include "../../w1/slaves/w1_ds2780.h"
 
 /* Current unit measurement in uA for a 1 milli-ohm sense resistor */
index cc0149131f89b12e6232faa33ecd8f386653a980..4400402f9ec51293b49b1718c05f6423a5333b43 100644 (file)
@@ -19,7 +19,7 @@
 #include <linux/power_supply.h>
 #include <linux/idr.h>
 
-#include "../../w1/w1.h"
+#include <linux/w1.h>
 #include "../../w1/slaves/w1_ds2781.h"
 
 /* Current unit measurement in uA for a 1 milli-ohm sense resistor */
index e0b8a4bc73df948aa45584ac4b2cdea4c13c30c1..fd2e9da27c4bf7ff6b3171d59d3704b3e6470ae7 100644 (file)
@@ -25,8 +25,7 @@
 
 #include <asm/io.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 
 #define DS1WM_CMD      0x00    /* R/W 4 bits command */
index e8730ea3e3aaca5b69633e8a7c38dc2107216852..d49681cd29af924ec9af6797573313a853e2397c 100644 (file)
@@ -20,8 +20,7 @@
 #include <linux/delay.h>
 #include <asm/delay.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 /**
  * Allow the active pullup to be disabled, default is enabled.
index d748e34d4ce55473e68612c49200eb5a0bf0dff0..46ccb2fc4f60da05c71087c913687f09f0767043 100644 (file)
@@ -25,8 +25,7 @@
 #include <linux/usb.h>
 #include <linux/slab.h>
 
-#include "../w1_int.h"
-#include "../w1.h"
+#include <linux/w1.h>
 
 /* USB Standard */
 /* USB Control request vendor type */
index f20d03ecfd1d91f3426a693fe207cdc9d867ef22..d83d7c99d81d3623cbeb0800ee0217d2d9f21c27 100644 (file)
@@ -34,8 +34,7 @@
 #include <linux/pci_ids.h>
 #include <linux/pci.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 /*
  * Matrox G400 DDC registers.
index a4621757a47f5184d96b99fda56c20f11562425d..74f2e6e6202a98eba4882f1f9b62ea0705e6451b 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/module.h>
 #include <linux/platform_device.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 /*
  * MXC W1 Register offsets
index 3302cbd2344ab10511234ce270f87ba6ef460f3a..3612542b604444bbfdf30a2212e45e830eb5e9d3 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/pm_runtime.h>
 #include <linux/of.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 #define        MOD_NAME        "OMAP_HDQ:"
 
index a373ae69d9f6ef8a5925dbd5004cd5e9b8fc5c05..a90728ceec5ad8968489021f9da44e161f399998 100644 (file)
@@ -20,8 +20,7 @@
 #include <linux/of.h>
 #include <linux/delay.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
+#include <linux/w1.h>
 
 static u8 w1_gpio_set_pullup(void *data, int delay)
 {
index d480900a28ab45653758835481d0bda70c8a2193..8046ac45381a7c241f77610621b6a314c204eb0d 100644 (file)
@@ -17,9 +17,9 @@
 #include <linux/mutex.h>
 #include <linux/power/bq27xxx_battery.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_BQ27000      0x01
 
 #define HDQ_CMD_READ   (0)
 #define HDQ_CMD_WRITE  (1<<7)
index d5d54876cb64a2cba90ffb39917ad29852b6278f..42a1e81060ce8af43b043aab479ff7500d8406c5 100644 (file)
@@ -24,8 +24,9 @@
 #include <linux/string.h>
 #include <linux/types.h>
 
-#include "../w1.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS2405       0x05
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Maciej S. Szmigiero <mail@maciej.szmigiero.name>");
index 709bd5e02eed2a3113fae3b82783c10fc562f00b..fac266366ca398bcaaeaeb1208546d89dfbcceae 100644 (file)
@@ -17,9 +17,9 @@
 #include <linux/slab.h>
 #include <linux/crc16.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS2406       0x12
 
 #define W1_F12_FUNC_READ_STATUS                   0xAA
 #define W1_F12_FUNC_WRITE_STATUS          0x55
index e01d2b3997bce2102e52e10bb0b83e9c7fe36d07..b535d5ec35b6a90b6f6c677bbf630c7a55d7ef1c 100644 (file)
@@ -15,9 +15,9 @@
 #include <linux/delay.h>
 #include <linux/slab.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS2408       0x29
 
 #define W1_F29_RETRIES         3
 
index 9cc6f0bc2e95634919a896dbf666619601589218..492e3d010321d18ed2a34dba66a17a82842df8bc 100644 (file)
@@ -16,9 +16,9 @@
 #include <linux/delay.h>
 #include <linux/slab.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS2413       0x3A
 
 #define W1_F3A_RETRIES                     3
 #define W1_F3A_FUNC_PIO_ACCESS_READ        0xF5
index 306240fe496c86c0296a2c0b82eca65c8a5d0d21..050407c53b167804f1ce3cee9a4c18f89cb6dcf8 100644 (file)
@@ -30,9 +30,9 @@
 #include <linux/delay.h>
 #include <linux/crc16.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_COUNTER_DS2423      0x1D
 
 #define CRC16_VALID    0xb001
 #define CRC16_INIT     0
index 20182d4a4b198de2f81064693c108e291d6eb2de..5adecd3face19b72edcd941e0820ca983fdb0379 100644 (file)
@@ -16,9 +16,9 @@
 #include <linux/types.h>
 #include <linux/delay.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_EEPROM_DS2431       0x2D
 
 #define W1_F2D_EEPROM_SIZE             128
 #define W1_F2D_PAGE_COUNT              4
index 86c30aceea3edb3ca1461390bfae60894ae5648e..75ad70cfe8e87fff0e30d126e0744abff6973a9b 100644 (file)
@@ -22,9 +22,9 @@
 
 #endif
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_EEPROM_DS2433       0x23
 
 #define W1_EEPROM_SIZE         512
 #define W1_PAGE_COUNT          16
index 5ededb4965e143ab774d2705b2ad1e6b0d4fa51b..6487fb772a2072295fc842839cbd13de00e39c8d 100644 (file)
@@ -13,8 +13,9 @@
 #include <linux/types.h>
 #include <linux/delay.h>
 
-#include "../w1.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS2438               0x26
 
 #define W1_DS2438_RETRIES              3
 
index f35d1e2ef9bb15bfc159be9261284915120d03d1..26168abfb8b824ef10e2b1824517f11046730327 100644 (file)
 #include <linux/idr.h>
 #include <linux/gfp.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
 #include "w1_ds2760.h"
 
+#define W1_FAMILY_DS2760       0x30
+
 static int w1_ds2760_io(struct device *dev, char *buf, int addr, size_t count,
                        int io)
 {
index 292972212107ce1170f5fe6d4f9cb76e30344b00..a60528131154b5c4b54a3704e1d6b0f9ce5e3397 100644 (file)
 #include <linux/mutex.h>
 #include <linux/idr.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
 #include "w1_ds2780.h"
 
+#define W1_FAMILY_DS2780       0x32
+
 static int w1_ds2780_do_io(struct device *dev, char *buf, int addr,
                        size_t count, int io)
 {
index a5d75067b2307dce1bfe7a709ff945025d8ff3a4..645be6e0b24a4120b8621e216f3bd723b2f3cb81 100644 (file)
 #include <linux/platform_device.h>
 #include <linux/mutex.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
 #include "w1_ds2781.h"
 
+#define W1_FAMILY_DS2781       0x3D
+
 static int w1_ds2781_do_io(struct device *dev, char *buf, int addr,
                        size_t count, int io)
 {
index c62858c05e8f2651b3f4c1d25a5a4c7e060c9c47..ec234b846eb3cbede0ca07564068fee3135d53f2 100644 (file)
@@ -20,9 +20,9 @@
 #define CRC16_INIT             0
 #define CRC16_VALID            0xb001
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_DS28E04      0x1C
 
 /* Allow the strong pullup to be disabled, but default to enabled.
  * If it was disabled a parasite powered device might not get the required
index 99b03bfb99415b774981ffdd5145622abd7672d8..e556b0caff7191245c6627ba36240713fca550b8 100644 (file)
 #include <linux/device.h>
 #include <linux/types.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_FAMILY_SMEM_01      0x01
+#define W1_FAMILY_SMEM_81      0x81
 
 static struct w1_family w1_smem_family_01 = {
        .fid = W1_FAMILY_SMEM_01,
index ccaf29994a42023a74c49e841e68239c9629931b..cb3fc3c6b0d17a4615c2f522bd5fe4d15d4d7919 100644 (file)
 #include <linux/slab.h>
 #include <linux/delay.h>
 
-#include "../w1.h"
-#include "../w1_int.h"
-#include "../w1_family.h"
+#include <linux/w1.h>
+
+#define W1_THERM_DS18S20       0x10
+#define W1_THERM_DS1822                0x22
+#define W1_THERM_DS18B20       0x28
+#define W1_THERM_DS1825                0x3B
+#define W1_THERM_DS28EA00      0x42
 
 /* Allow the strong pullup to be disabled, but default to enabled.
  * If it was disabled a parasite powered device might not get the require
index 8172dee5e23cffd5d031acf786eae1e77d198409..95ea7e6b1d991b681579477653e173fd2412a4b0 100644 (file)
 
 #include <linux/atomic.h>
 
-#include "w1.h"
-#include "w1_int.h"
-#include "w1_family.h"
+#include "w1_internal.h"
 #include "w1_netlink.h"
 
+#define W1_FAMILY_DEFAULT      0
+
 static int w1_timeout = 10;
 module_param_named(timeout, w1_timeout, int, 0);
 MODULE_PARM_DESC(timeout, "time in seconds between automatic slave searches");
index 9759cdaf22f4544541f5f6c545a20c29c2241fff..f14ab0b340b543351b2e41f0e7ea43c689dc7806 100644 (file)
@@ -18,8 +18,7 @@
 #include <linux/delay.h>
 #include <linux/export.h>
 
-#include "w1_family.h"
-#include "w1.h"
+#include "w1_internal.h"
 
 DEFINE_SPINLOCK(w1_flock);
 static LIST_HEAD(w1_families);
diff --git a/drivers/w1/w1_family.h b/drivers/w1/w1_family.h
deleted file mode 100644 (file)
index 869a3ff..0000000
+++ /dev/null
@@ -1,98 +0,0 @@
-/*
- * Copyright (c) 2004 Evgeniy Polyakov <zbr@ioremap.net>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms 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.
- */
-
-#ifndef __W1_FAMILY_H
-#define __W1_FAMILY_H
-
-#include <linux/types.h>
-#include <linux/device.h>
-#include <linux/atomic.h>
-
-#define W1_FAMILY_DEFAULT      0
-#define W1_FAMILY_BQ27000      0x01
-#define W1_FAMILY_SMEM_01      0x01
-#define W1_FAMILY_SMEM_81      0x81
-#define W1_FAMILY_DS2405       0x05
-#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
-#define W1_FAMILY_DS2438       0x26
-#define W1_THERM_DS18B20       0x28
-#define W1_FAMILY_DS2408       0x29
-#define W1_EEPROM_DS2431       0x2D
-#define W1_FAMILY_DS2760       0x30
-#define W1_FAMILY_DS2780       0x32
-#define W1_FAMILY_DS2413       0x3A
-#define W1_FAMILY_DS2406       0x12
-#define W1_THERM_DS1825                0x3B
-#define W1_FAMILY_DS2781       0x3D
-#define W1_THERM_DS28EA00      0x42
-
-#define MAXNAMELEN             32
-
-struct w1_slave;
-
-/**
- * struct w1_family_ops - operations for a family type
- * @add_slave: add_slave
- * @remove_slave: remove_slave
- * @groups: sysfs group
- */
-struct w1_family_ops
-{
-       int  (* add_slave)(struct w1_slave *);
-       void (* remove_slave)(struct w1_slave *);
-       const struct attribute_group **groups;
-};
-
-/**
- * struct w1_family - reference counted family structure.
- * @family_entry:      family linked list
- * @fid:               8 bit family identifier
- * @fops:              operations for this family
- * @refcnt:            reference counter
- */
-struct w1_family
-{
-       struct list_head        family_entry;
-       u8                      fid;
-
-       struct w1_family_ops    *fops;
-
-       atomic_t                refcnt;
-};
-
-extern spinlock_t w1_flock;
-
-void w1_family_put(struct w1_family *);
-void __w1_family_get(struct w1_family *);
-struct w1_family * w1_family_registered(u8);
-void w1_unregister_family(struct w1_family *);
-int w1_register_family(struct w1_family *);
-
-/**
- * module_w1_driver() - Helper macro for registering a 1-Wire families
- * @__w1_family: w1_family struct
- *
- * Helper macro for 1-Wire families which do not do anything special in module
- * init/exit. This eliminates a lot of boilerplate. Each module may only
- * use this macro once, and calling it replaces module_init() and module_exit()
- */
-#define module_w1_family(__w1_family) \
-       module_driver(__w1_family, w1_register_family, \
-                       w1_unregister_family)
-
-#endif /* __W1_FAMILY_H */
index 4439d10709bb5a2c0936ebbd9e7b488da8a4d8e4..1c776178f598d07724b262b3a43b4f90c1513d9c 100644 (file)
@@ -21,9 +21,8 @@
 #include <linux/export.h>
 #include <linux/moduleparam.h>
 
-#include "w1.h"
+#include "w1_internal.h"
 #include "w1_netlink.h"
-#include "w1_int.h"
 
 static int w1_search_count = -1; /* Default is continual scan */
 module_param_named(search_count, w1_search_count, int, 0);
diff --git a/drivers/w1/w1_int.h b/drivers/w1/w1_int.h
deleted file mode 100644 (file)
index 3719891..0000000
+++ /dev/null
@@ -1,27 +0,0 @@
-/*
- * Copyright (c) 2004 Evgeniy Polyakov <zbr@ioremap.net>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms 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.
- */
-
-#ifndef __W1_INT_H
-#define __W1_INT_H
-
-#include <linux/kernel.h>
-#include <linux/device.h>
-
-#include "w1.h"
-
-int w1_add_master_device(struct w1_bus_master *);
-void w1_remove_master_device(struct w1_bus_master *);
-void __w1_remove_master_device(struct w1_master *);
-
-#endif /* __W1_INT_H */
diff --git a/drivers/w1/w1_internal.h b/drivers/w1/w1_internal.h
new file mode 100644 (file)
index 0000000..1623e2f
--- /dev/null
@@ -0,0 +1,87 @@
+/*
+ * Copyright (c) 2004 Evgeniy Polyakov <zbr@ioremap.net>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms 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.
+ */
+
+#ifndef __W1_H
+#define __W1_H
+
+#include <linux/w1.h>
+
+#include <linux/completion.h>
+#include <linux/mutex.h>
+
+#define W1_SLAVE_ACTIVE                0
+#define W1_SLAVE_DETACH                1
+
+/**
+ * struct w1_async_cmd - execute callback from the w1_process kthread
+ * @async_entry: link entry
+ * @cb: callback function, must list_del and destroy this list before
+ * returning
+ *
+ * When inserted into the w1_master async_list, w1_process will execute
+ * the callback.  Embed this into the structure with the command details.
+ */
+struct w1_async_cmd {
+       struct list_head        async_entry;
+       void (*cb)(struct w1_master *dev, struct w1_async_cmd *async_cmd);
+};
+
+int w1_create_master_attributes(struct w1_master *master);
+void w1_destroy_master_attributes(struct w1_master *master);
+void w1_search(struct w1_master *dev, u8 search_type,
+              w1_slave_found_callback cb);
+void w1_search_devices(struct w1_master *dev, u8 search_type,
+                      w1_slave_found_callback cb);
+/* call w1_unref_slave to release the reference counts w1_search_slave added */
+struct w1_slave *w1_search_slave(struct w1_reg_num *id);
+/*
+ * decrements the reference on sl->master and sl, and cleans up if zero
+ * returns the reference count after it has been decremented
+ */
+int w1_unref_slave(struct w1_slave *sl);
+void w1_slave_found(struct w1_master *dev, u64 rn);
+void w1_search_process_cb(struct w1_master *dev, u8 search_type,
+                         w1_slave_found_callback cb);
+struct w1_slave *w1_slave_search_device(struct w1_master *dev,
+                                       struct w1_reg_num *rn);
+struct w1_master *w1_search_master_id(u32 id);
+
+/* Disconnect and reconnect devices in the given family.  Used for finding
+ * unclaimed devices after a family has been registered or releasing devices
+ * after a family has been unregistered.  Set attach to 1 when a new family
+ * has just been registered, to 0 when it has been unregistered.
+ */
+void w1_reconnect_slaves(struct w1_family *f, int attach);
+int w1_attach_slave_device(struct w1_master *dev, struct w1_reg_num *rn);
+/* 0 success, otherwise EBUSY */
+int w1_slave_detach(struct w1_slave *sl);
+
+void __w1_remove_master_device(struct w1_master *dev);
+
+void w1_family_put(struct w1_family *f);
+void __w1_family_get(struct w1_family *f);
+struct w1_family *w1_family_registered(u8 fid);
+
+extern struct device_driver w1_master_driver;
+extern struct device w1_master_device;
+extern int w1_max_slave_count;
+extern int w1_max_slave_ttl;
+extern struct list_head w1_masters;
+extern struct mutex w1_mlock;
+extern spinlock_t w1_flock;
+
+int w1_process_callbacks(struct w1_master *dev);
+int w1_process(void *data);
+
+#endif /* __W1_H */
index 1134e6b1eb023b26ea2f978c046d7796be5f69f6..d191e1f8057993bb9a76ba38a540ff6a6178ce5f 100644 (file)
@@ -18,7 +18,7 @@
 #include <linux/moduleparam.h>
 #include <linux/module.h>
 
-#include "w1.h"
+#include "w1_internal.h"
 
 static int w1_delay_parm = 1;
 module_param_named(delay_coef, w1_delay_parm, int, 0);
index 027950f997d12cb91642ebedf8b18e3242b798c1..f2f099caeb77ec436e3563fa134d22f52bd2fde1 100644 (file)
@@ -17,7 +17,7 @@
 #include <linux/netlink.h>
 #include <linux/connector.h>
 
-#include "w1.h"
+#include "w1_internal.h"
 #include "w1_netlink.h"
 
 #if defined(CONFIG_W1_CON) && (defined(CONFIG_CONNECTOR) || (defined(CONFIG_CONNECTOR_MODULE) && defined(CONFIG_W1_MODULE)))
index b389e5ff5fa503435b9ac5dd21c01dee6e51a4e5..a36661cd1f053f4a8b53a1ddbe43fa65f044b17a 100644 (file)
@@ -18,7 +18,7 @@
 #include <asm/types.h>
 #include <linux/connector.h>
 
-#include "w1.h"
+#include "w1_internal.h"
 
 /**
  * enum w1_cn_msg_flags - bitfield flags for struct cn_msg.flags
similarity index 76%
rename from drivers/w1/w1.h
rename to include/linux/w1.h
index 758a7a6322e983d527c1f26e4262a5f853b877bc..90cbe7e65059f6b604a87c6bf39cd9bbeae7684c 100644 (file)
  * GNU General Public License for more details.
  */
 
-#ifndef __W1_H
-#define __W1_H
+#ifndef __LINUX_W1_H
+#define __LINUX_W1_H
+
+#include <linux/device.h>
 
 /**
  * struct w1_reg_num - broken out slave device id
@@ -22,8 +24,7 @@
  * @id: along with family is the unique device id
  * @crc: checksum of the other bytes
  */
-struct w1_reg_num
-{
+struct w1_reg_num {
 #if defined(__LITTLE_ENDIAN_BITFIELD)
        __u64   family:8,
                id:48,
@@ -39,12 +40,6 @@ struct w1_reg_num
 
 #ifdef __KERNEL__
 
-#include <linux/completion.h>
-#include <linux/device.h>
-#include <linux/mutex.h>
-
-#include "w1_family.h"
-
 #define W1_MAXNAMELEN          32
 
 #define W1_SEARCH              0xF0
@@ -59,9 +54,6 @@ struct w1_reg_num
 #define W1_MATCH_ROM           0x55
 #define W1_RESUME_CMD          0xA5
 
-#define W1_SLAVE_ACTIVE                0
-#define W1_SLAVE_DETACH                1
-
 /**
  * struct w1_slave - holds a single slave device on the bus
  *
@@ -78,8 +70,7 @@ struct w1_reg_num
  * @dev: kernel device identifier
  *
  */
-struct w1_slave
-{
+struct w1_slave {
        struct module           *owner;
        unsigned char           name[W1_MAXNAMELEN];
        struct list_head        w1_slave_entry;
@@ -96,7 +87,6 @@ struct w1_slave
 
 typedef void (*w1_slave_found_callback)(struct w1_master *, u64);
 
-
 /**
  * struct w1_bus_master - operations available on a bus master
  *
@@ -142,8 +132,7 @@ typedef void (*w1_slave_found_callback)(struct w1_master *, u64);
  * reset_bus.
  *
  */
-struct w1_bus_master
-{
+struct w1_bus_master {
        void            *data;
 
        u8              (*read_bit)(void *);
@@ -209,8 +198,7 @@ enum w1_master_flags {
  * @bus_master:                io operations available
  * @seq:               sequence number used for netlink broadcasts
  */
-struct w1_master
-{
+struct w1_master {
        struct list_head        w1_master_entry;
        struct module           *owner;
        unsigned char           name[W1_MAXNAMELEN];
@@ -254,45 +242,51 @@ struct w1_master
        u32                     seq;
 };
 
+int w1_add_master_device(struct w1_bus_master *master);
+void w1_remove_master_device(struct w1_bus_master *master);
+
 /**
- * struct w1_async_cmd - execute callback from the w1_process kthread
- * @async_entry: link entry
- * @cb: callback function, must list_del and destroy this list before
- * returning
- *
- * When inserted into the w1_master async_list, w1_process will execute
- * the callback.  Embed this into the structure with the command details.
+ * struct w1_family_ops - operations for a family type
+ * @add_slave: add_slave
+ * @remove_slave: remove_slave
+ * @groups: sysfs group
  */
-struct w1_async_cmd {
-       struct list_head        async_entry;
-       void (*cb)(struct w1_master *dev, struct w1_async_cmd *async_cmd);
+struct w1_family_ops {
+       int  (*add_slave)(struct w1_slave *sl);
+       void (*remove_slave)(struct w1_slave *sl);
+       const struct attribute_group **groups;
 };
 
-int w1_create_master_attributes(struct w1_master *);
-void w1_destroy_master_attributes(struct w1_master *master);
-void w1_search(struct w1_master *dev, u8 search_type, w1_slave_found_callback cb);
-void w1_search_devices(struct w1_master *dev, u8 search_type, w1_slave_found_callback cb);
-/* call w1_unref_slave to release the reference counts w1_search_slave added */
-struct w1_slave *w1_search_slave(struct w1_reg_num *id);
-/* decrements the reference on sl->master and sl, and cleans up if zero
- * returns the reference count after it has been decremented */
-int w1_unref_slave(struct w1_slave *sl);
-void w1_slave_found(struct w1_master *dev, u64 rn);
-void w1_search_process_cb(struct w1_master *dev, u8 search_type,
-       w1_slave_found_callback cb);
-struct w1_slave *w1_slave_search_device(struct w1_master *dev,
-       struct w1_reg_num *rn);
-struct w1_master *w1_search_master_id(u32 id);
-
-/* Disconnect and reconnect devices in the given family.  Used for finding
- * unclaimed devices after a family has been registered or releasing devices
- * after a family has been unregistered.  Set attach to 1 when a new family
- * has just been registered, to 0 when it has been unregistered.
+/**
+ * struct w1_family - reference counted family structure.
+ * @family_entry:      family linked list
+ * @fid:               8 bit family identifier
+ * @fops:              operations for this family
+ * @refcnt:            reference counter
+ */
+struct w1_family {
+       struct list_head        family_entry;
+       u8                      fid;
+
+       struct w1_family_ops    *fops;
+
+       atomic_t                refcnt;
+};
+
+int w1_register_family(struct w1_family *family);
+void w1_unregister_family(struct w1_family *family);
+
+/**
+ * module_w1_driver() - Helper macro for registering a 1-Wire families
+ * @__w1_family: w1_family struct
+ *
+ * Helper macro for 1-Wire families which do not do anything special in module
+ * init/exit. This eliminates a lot of boilerplate. Each module may only
+ * use this macro once, and calling it replaces module_init() and module_exit()
  */
-void w1_reconnect_slaves(struct w1_family *f, int attach);
-int w1_attach_slave_device(struct w1_master *dev, struct w1_reg_num *rn);
-/* 0 success, otherwise EBUSY */
-int w1_slave_detach(struct w1_slave *sl);
+#define module_w1_family(__w1_family) \
+       module_driver(__w1_family, w1_register_family, \
+                       w1_unregister_family)
 
 u8 w1_triplet(struct w1_master *dev, int bdir);
 void w1_write_8(struct w1_master *, u8);
@@ -321,16 +315,6 @@ static inline struct w1_master* dev_to_w1_master(struct device *dev)
        return container_of(dev, struct w1_master, dev);
 }
 
-extern struct device_driver w1_master_driver;
-extern struct device w1_master_device;
-extern int w1_max_slave_count;
-extern int w1_max_slave_ttl;
-extern struct list_head w1_masters;
-extern struct mutex w1_mlock;
-
-extern int w1_process_callbacks(struct w1_master *dev);
-extern int w1_process(void *);
-
 #endif /* __KERNEL__ */
 
-#endif /* __W1_H */
+#endif /* __LINUX_W1_H */