2005-04-17 00:20:36 +02:00
|
|
|
/* i2c-core.c - a device driver for the iic-bus interface */
|
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
/* Copyright (C) 1995-99 Simon G. Vogl
|
|
|
|
|
|
|
|
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.
|
|
|
|
|
|
|
|
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., 675 Mass Ave, Cambridge, MA 02139, USA. */
|
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
|
2007-10-19 23:21:04 +02:00
|
|
|
/* With some changes from Kyösti Mälkki <kmalkki@cc.hut.fi>.
|
2005-04-17 00:20:36 +02:00
|
|
|
All SMBus-related things are written by Frodo Looijaard <frodol@dds.nl>
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
SMBus 2.0 support by Mark Studebaker <mdsxyz123@yahoo.com> and
|
|
|
|
Jean Delvare <khali@linux-fr.org> */
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/kernel.h>
|
|
|
|
#include <linux/errno.h>
|
|
|
|
#include <linux/slab.h>
|
|
|
|
#include <linux/i2c.h>
|
|
|
|
#include <linux/init.h>
|
|
|
|
#include <linux/idr.h>
|
2005-10-29 20:07:23 +02:00
|
|
|
#include <linux/platform_device.h>
|
2006-01-11 10:50:26 +01:00
|
|
|
#include <linux/mutex.h>
|
2007-02-13 22:09:00 +01:00
|
|
|
#include <linux/completion.h>
|
2008-01-27 18:14:50 +01:00
|
|
|
#include <linux/hardirq.h>
|
|
|
|
#include <linux/irqflags.h>
|
2005-04-17 00:20:36 +02:00
|
|
|
#include <asm/uaccess.h>
|
|
|
|
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
#include "i2c-core.h"
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
static DEFINE_MUTEX(core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
static DEFINE_IDR(i2c_adapter_idr);
|
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
#define is_newstyle_driver(d) ((d)->probe || (d)->remove || (d)->detect)
|
|
|
|
|
|
|
|
static int i2c_detect(struct i2c_adapter *adapter, struct i2c_driver *driver);
|
2007-02-13 22:09:00 +01:00
|
|
|
|
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
|
2008-04-29 23:11:39 +02:00
|
|
|
static const struct i2c_device_id *i2c_match_id(const struct i2c_device_id *id,
|
|
|
|
const struct i2c_client *client)
|
|
|
|
{
|
|
|
|
while (id->name[0]) {
|
|
|
|
if (strcmp(client->name, id->name) == 0)
|
|
|
|
return id;
|
|
|
|
id++;
|
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
static int i2c_device_match(struct device *dev, struct device_driver *drv)
|
|
|
|
{
|
2007-05-01 23:26:30 +02:00
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
struct i2c_driver *driver = to_i2c_driver(drv);
|
|
|
|
|
|
|
|
/* make legacy i2c drivers bypass driver model probing entirely;
|
|
|
|
* such drivers scan each i2c adapter/bus themselves.
|
|
|
|
*/
|
2007-05-01 23:26:30 +02:00
|
|
|
if (!is_newstyle_driver(driver))
|
2007-05-01 23:26:30 +02:00
|
|
|
return 0;
|
|
|
|
|
2008-04-29 23:11:39 +02:00
|
|
|
/* match on an id table if there is one */
|
|
|
|
if (driver->id_table)
|
|
|
|
return i2c_match_id(driver->id_table, client) != NULL;
|
|
|
|
|
2008-05-18 20:49:41 +02:00
|
|
|
return 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
#ifdef CONFIG_HOTPLUG
|
|
|
|
|
|
|
|
/* uevent helps with hotplug: modprobe -q $(MODALIAS) */
|
2007-08-14 15:15:12 +02:00
|
|
|
static int i2c_device_uevent(struct device *dev, struct kobj_uevent_env *env)
|
2007-05-01 23:26:30 +02:00
|
|
|
{
|
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
|
|
|
|
/* by definition, legacy drivers can't hotplug */
|
2008-04-29 23:11:39 +02:00
|
|
|
if (dev->driver)
|
2007-05-01 23:26:30 +02:00
|
|
|
return 0;
|
|
|
|
|
2008-05-18 20:49:41 +02:00
|
|
|
if (add_uevent_var(env, "MODALIAS=%s%s",
|
|
|
|
I2C_MODULE_PREFIX, client->name))
|
|
|
|
return -ENOMEM;
|
2007-05-01 23:26:30 +02:00
|
|
|
dev_dbg(dev, "uevent\n");
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
#else
|
|
|
|
#define i2c_device_uevent NULL
|
|
|
|
#endif /* CONFIG_HOTPLUG */
|
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
static int i2c_device_probe(struct device *dev)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-05-01 23:26:30 +02:00
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
struct i2c_driver *driver = to_i2c_driver(dev->driver);
|
2008-03-12 14:15:00 +01:00
|
|
|
int status;
|
2007-05-01 23:26:30 +02:00
|
|
|
|
2008-07-14 22:38:30 +02:00
|
|
|
if (!driver->probe || !driver->id_table)
|
2007-05-01 23:26:30 +02:00
|
|
|
return -ENODEV;
|
|
|
|
client->driver = driver;
|
|
|
|
dev_dbg(dev, "probe\n");
|
2008-04-29 23:11:39 +02:00
|
|
|
|
2008-07-14 22:38:30 +02:00
|
|
|
status = driver->probe(client, i2c_match_id(driver->id_table, client));
|
2008-03-12 14:15:00 +01:00
|
|
|
if (status)
|
|
|
|
client->driver = NULL;
|
|
|
|
return status;
|
2007-02-13 22:09:00 +01:00
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
static int i2c_device_remove(struct device *dev)
|
|
|
|
{
|
2007-05-01 23:26:30 +02:00
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
struct i2c_driver *driver;
|
|
|
|
int status;
|
|
|
|
|
|
|
|
if (!dev->driver)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
driver = to_i2c_driver(dev->driver);
|
|
|
|
if (driver->remove) {
|
|
|
|
dev_dbg(dev, "remove\n");
|
|
|
|
status = driver->remove(client);
|
|
|
|
} else {
|
|
|
|
dev->driver = NULL;
|
|
|
|
status = 0;
|
|
|
|
}
|
|
|
|
if (status == 0)
|
|
|
|
client->driver = NULL;
|
|
|
|
return status;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
static void i2c_device_shutdown(struct device *dev)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-02-13 22:09:00 +01:00
|
|
|
struct i2c_driver *driver;
|
|
|
|
|
|
|
|
if (!dev->driver)
|
|
|
|
return;
|
|
|
|
driver = to_i2c_driver(dev->driver);
|
|
|
|
if (driver->shutdown)
|
|
|
|
driver->shutdown(to_i2c_client(dev));
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
static int i2c_device_suspend(struct device * dev, pm_message_t mesg)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-02-13 22:09:00 +01:00
|
|
|
struct i2c_driver *driver;
|
|
|
|
|
|
|
|
if (!dev->driver)
|
|
|
|
return 0;
|
|
|
|
driver = to_i2c_driver(dev->driver);
|
|
|
|
if (!driver->suspend)
|
|
|
|
return 0;
|
|
|
|
return driver->suspend(to_i2c_client(dev), mesg);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
static int i2c_device_resume(struct device * dev)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-02-13 22:09:00 +01:00
|
|
|
struct i2c_driver *driver;
|
|
|
|
|
|
|
|
if (!dev->driver)
|
|
|
|
return 0;
|
|
|
|
driver = to_i2c_driver(dev->driver);
|
|
|
|
if (!driver->resume)
|
|
|
|
return 0;
|
|
|
|
return driver->resume(to_i2c_client(dev));
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
static void i2c_client_release(struct device *dev)
|
|
|
|
{
|
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
complete(&client->released);
|
|
|
|
}
|
|
|
|
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
static void i2c_client_dev_release(struct device *dev)
|
|
|
|
{
|
|
|
|
kfree(to_i2c_client(dev));
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
static ssize_t show_client_name(struct device *dev, struct device_attribute *attr, char *buf)
|
|
|
|
{
|
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
|
|
|
return sprintf(buf, "%s\n", client->name);
|
|
|
|
}
|
|
|
|
|
|
|
|
static ssize_t show_modalias(struct device *dev, struct device_attribute *attr, char *buf)
|
|
|
|
{
|
|
|
|
struct i2c_client *client = to_i2c_client(dev);
|
2008-05-18 20:49:41 +02:00
|
|
|
return sprintf(buf, "%s%s\n", I2C_MODULE_PREFIX, client->name);
|
2007-05-01 23:26:30 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static struct device_attribute i2c_dev_attrs[] = {
|
|
|
|
__ATTR(name, S_IRUGO, show_client_name, NULL),
|
|
|
|
/* modalias helps coldplug: modprobe $(cat .../modalias) */
|
|
|
|
__ATTR(modalias, S_IRUGO, show_modalias, NULL),
|
|
|
|
{ },
|
|
|
|
};
|
|
|
|
|
2008-07-14 22:38:35 +02:00
|
|
|
struct bus_type i2c_bus_type = {
|
2007-02-13 22:09:00 +01:00
|
|
|
.name = "i2c",
|
2007-05-01 23:26:30 +02:00
|
|
|
.dev_attrs = i2c_dev_attrs,
|
2007-02-13 22:09:00 +01:00
|
|
|
.match = i2c_device_match,
|
2007-05-01 23:26:30 +02:00
|
|
|
.uevent = i2c_device_uevent,
|
2007-02-13 22:09:00 +01:00
|
|
|
.probe = i2c_device_probe,
|
|
|
|
.remove = i2c_device_remove,
|
|
|
|
.shutdown = i2c_device_shutdown,
|
|
|
|
.suspend = i2c_device_suspend,
|
|
|
|
.resume = i2c_device_resume,
|
2006-01-05 15:37:50 +01:00
|
|
|
};
|
2008-07-14 22:38:35 +02:00
|
|
|
EXPORT_SYMBOL_GPL(i2c_bus_type);
|
2006-01-05 15:37:50 +01:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
|
|
|
|
/**
|
|
|
|
* i2c_verify_client - return parameter as i2c_client, or NULL
|
|
|
|
* @dev: device, probably from some driver model iterator
|
|
|
|
*
|
|
|
|
* When traversing the driver model tree, perhaps using driver model
|
|
|
|
* iterators like @device_for_each_child(), you can't assume very much
|
|
|
|
* about the nodes you find. Use this function to avoid oopses caused
|
|
|
|
* by wrongly treating some non-I2C device as an i2c_client.
|
|
|
|
*/
|
|
|
|
struct i2c_client *i2c_verify_client(struct device *dev)
|
|
|
|
{
|
|
|
|
return (dev->bus == &i2c_bus_type)
|
|
|
|
? to_i2c_client(dev)
|
|
|
|
: NULL;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(i2c_verify_client);
|
|
|
|
|
|
|
|
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
/**
|
|
|
|
* i2c_new_device - instantiate an i2c device for use with a new style driver
|
|
|
|
* @adap: the adapter managing the device
|
|
|
|
* @info: describes one I2C device; bus_num is ignored
|
2007-07-12 14:12:28 +02:00
|
|
|
* Context: can sleep
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
*
|
|
|
|
* Create a device to work with a new style i2c driver, where binding is
|
|
|
|
* handled through driver model probe()/remove() methods. This call is not
|
|
|
|
* appropriate for use by mainboad initialization logic, which usually runs
|
|
|
|
* during an arch_initcall() long before any i2c_adapter could exist.
|
|
|
|
*
|
|
|
|
* This returns the new i2c client, which may be saved for later use with
|
|
|
|
* i2c_unregister_device(); or NULL to indicate an error.
|
|
|
|
*/
|
|
|
|
struct i2c_client *
|
|
|
|
i2c_new_device(struct i2c_adapter *adap, struct i2c_board_info const *info)
|
|
|
|
{
|
|
|
|
struct i2c_client *client;
|
|
|
|
int status;
|
|
|
|
|
|
|
|
client = kzalloc(sizeof *client, GFP_KERNEL);
|
|
|
|
if (!client)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
client->adapter = adap;
|
|
|
|
|
|
|
|
client->dev.platform_data = info->platform_data;
|
2007-10-13 23:56:29 +02:00
|
|
|
device_init_wakeup(&client->dev, info->flags & I2C_CLIENT_WAKE);
|
|
|
|
|
|
|
|
client->flags = info->flags & ~I2C_CLIENT_WAKE;
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
client->addr = info->addr;
|
|
|
|
client->irq = info->irq;
|
|
|
|
|
|
|
|
strlcpy(client->name, info->type, sizeof(client->name));
|
|
|
|
|
|
|
|
/* a new style driver may be bound to this device when we
|
|
|
|
* return from this function, or any later moment (e.g. maybe
|
|
|
|
* hotplugging will load the driver module). and the device
|
|
|
|
* refcount model is the standard driver model one.
|
|
|
|
*/
|
|
|
|
status = i2c_attach_client(client);
|
|
|
|
if (status < 0) {
|
|
|
|
kfree(client);
|
|
|
|
client = NULL;
|
|
|
|
}
|
|
|
|
return client;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(i2c_new_device);
|
|
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
* i2c_unregister_device - reverse effect of i2c_new_device()
|
|
|
|
* @client: value returned from i2c_new_device()
|
2007-07-12 14:12:28 +02:00
|
|
|
* Context: can sleep
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
*/
|
|
|
|
void i2c_unregister_device(struct i2c_client *client)
|
2007-05-01 23:26:30 +02:00
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter = client->adapter;
|
|
|
|
struct i2c_driver *driver = client->driver;
|
|
|
|
|
|
|
|
if (driver && !is_newstyle_driver(driver)) {
|
|
|
|
dev_err(&client->dev, "can't unregister devices "
|
|
|
|
"with legacy drivers\n");
|
|
|
|
WARN_ON(1);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2008-07-14 22:38:36 +02:00
|
|
|
if (adapter->client_unregister) {
|
|
|
|
if (adapter->client_unregister(client)) {
|
|
|
|
dev_warn(&client->dev,
|
|
|
|
"client_unregister [%s] failed\n",
|
|
|
|
client->name);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
mutex_lock(&adapter->clist_lock);
|
|
|
|
list_del(&client->list);
|
|
|
|
mutex_unlock(&adapter->clist_lock);
|
|
|
|
|
|
|
|
device_unregister(&client->dev);
|
|
|
|
}
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL_GPL(i2c_unregister_device);
|
2007-05-01 23:26:30 +02:00
|
|
|
|
|
|
|
|
2008-05-11 20:37:06 +02:00
|
|
|
static const struct i2c_device_id dummy_id[] = {
|
|
|
|
{ "dummy", 0 },
|
|
|
|
{ },
|
|
|
|
};
|
|
|
|
|
2008-04-29 23:11:39 +02:00
|
|
|
static int dummy_probe(struct i2c_client *client,
|
|
|
|
const struct i2c_device_id *id)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int dummy_remove(struct i2c_client *client)
|
2008-01-27 18:14:52 +01:00
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct i2c_driver dummy_driver = {
|
|
|
|
.driver.name = "dummy",
|
2008-04-29 23:11:39 +02:00
|
|
|
.probe = dummy_probe,
|
|
|
|
.remove = dummy_remove,
|
2008-05-11 20:37:06 +02:00
|
|
|
.id_table = dummy_id,
|
2008-01-27 18:14:52 +01:00
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
|
|
|
* i2c_new_dummy - return a new i2c device bound to a dummy driver
|
|
|
|
* @adapter: the adapter managing the device
|
|
|
|
* @address: seven bit address to be used
|
|
|
|
* Context: can sleep
|
|
|
|
*
|
|
|
|
* This returns an I2C client bound to the "dummy" driver, intended for use
|
|
|
|
* with devices that consume multiple addresses. Examples of such chips
|
|
|
|
* include various EEPROMS (like 24c04 and 24c08 models).
|
|
|
|
*
|
|
|
|
* These dummy devices have two main uses. First, most I2C and SMBus calls
|
|
|
|
* except i2c_transfer() need a client handle; the dummy will be that handle.
|
|
|
|
* And second, this prevents the specified address from being bound to a
|
|
|
|
* different driver.
|
|
|
|
*
|
|
|
|
* This returns the new i2c client, which should be saved for later use with
|
|
|
|
* i2c_unregister_device(); or NULL to indicate an error.
|
|
|
|
*/
|
|
|
|
struct i2c_client *
|
2008-05-11 20:37:06 +02:00
|
|
|
i2c_new_dummy(struct i2c_adapter *adapter, u16 address)
|
2008-01-27 18:14:52 +01:00
|
|
|
{
|
|
|
|
struct i2c_board_info info = {
|
2008-05-11 20:37:06 +02:00
|
|
|
I2C_BOARD_INFO("dummy", address),
|
2008-01-27 18:14:52 +01:00
|
|
|
};
|
|
|
|
|
|
|
|
return i2c_new_device(adapter, &info);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(i2c_new_dummy);
|
|
|
|
|
2007-02-13 22:09:00 +01:00
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
|
2007-05-01 23:26:28 +02:00
|
|
|
/* I2C bus adapters -- one roots each I2C or SMBUS segment */
|
|
|
|
|
2007-10-13 23:56:30 +02:00
|
|
|
static void i2c_adapter_dev_release(struct device *dev)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-05-01 23:26:28 +02:00
|
|
|
struct i2c_adapter *adap = to_i2c_adapter(dev);
|
2005-04-17 00:20:36 +02:00
|
|
|
complete(&adap->dev_released);
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:28 +02:00
|
|
|
static ssize_t
|
|
|
|
show_adapter_name(struct device *dev, struct device_attribute *attr, char *buf)
|
|
|
|
{
|
2007-05-01 23:26:28 +02:00
|
|
|
struct i2c_adapter *adap = to_i2c_adapter(dev);
|
2007-05-01 23:26:28 +02:00
|
|
|
return sprintf(buf, "%s\n", adap->name);
|
|
|
|
}
|
2007-01-04 13:07:04 +01:00
|
|
|
|
2007-05-01 23:26:28 +02:00
|
|
|
static struct device_attribute i2c_adapter_attrs[] = {
|
|
|
|
__ATTR(name, S_IRUGO, show_adapter_name, NULL),
|
|
|
|
{ },
|
|
|
|
};
|
2007-01-04 13:07:04 +01:00
|
|
|
|
2007-10-13 23:56:30 +02:00
|
|
|
static struct class i2c_adapter_class = {
|
2007-01-04 13:07:04 +01:00
|
|
|
.owner = THIS_MODULE,
|
|
|
|
.name = "i2c-adapter",
|
2007-05-01 23:26:28 +02:00
|
|
|
.dev_attrs = i2c_adapter_attrs,
|
2005-04-17 00:20:36 +02:00
|
|
|
};
|
|
|
|
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
static void i2c_scan_static_board_info(struct i2c_adapter *adapter)
|
|
|
|
{
|
|
|
|
struct i2c_devinfo *devinfo;
|
|
|
|
|
|
|
|
mutex_lock(&__i2c_board_lock);
|
|
|
|
list_for_each_entry(devinfo, &__i2c_board_list, list) {
|
|
|
|
if (devinfo->busnum == adapter->nr
|
|
|
|
&& !i2c_new_device(adapter,
|
|
|
|
&devinfo->board_info))
|
|
|
|
printk(KERN_ERR "i2c-core: can't create i2c%d-%04x\n",
|
|
|
|
i2c_adapter_id(adapter),
|
|
|
|
devinfo->board_info.addr);
|
|
|
|
}
|
|
|
|
mutex_unlock(&__i2c_board_lock);
|
|
|
|
}
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
static int i2c_do_add_adapter(struct device_driver *d, void *data)
|
|
|
|
{
|
|
|
|
struct i2c_driver *driver = to_i2c_driver(d);
|
|
|
|
struct i2c_adapter *adap = data;
|
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
/* Detect supported devices on that bus, and instantiate them */
|
|
|
|
i2c_detect(adap, driver);
|
|
|
|
|
|
|
|
/* Let legacy drivers scan this bus for matching devices */
|
2008-01-27 18:14:49 +01:00
|
|
|
if (driver->attach_adapter) {
|
|
|
|
/* We ignore the return code; if it fails, too bad */
|
|
|
|
driver->attach_adapter(adap);
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
static int i2c_register_adapter(struct i2c_adapter *adap)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-01-27 18:14:49 +01:00
|
|
|
int res = 0, dummy;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2006-01-18 23:16:04 +01:00
|
|
|
mutex_init(&adap->bus_lock);
|
|
|
|
mutex_init(&adap->clist_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
INIT_LIST_HEAD(&adap->clients);
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2007-05-01 23:26:31 +02:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/* Add the adapter to the driver core.
|
|
|
|
* If the parent pointer is not set up,
|
|
|
|
* we add this adapter to the host bus.
|
|
|
|
*/
|
2007-01-04 13:07:04 +01:00
|
|
|
if (adap->dev.parent == NULL) {
|
2005-04-17 00:20:36 +02:00
|
|
|
adap->dev.parent = &platform_bus;
|
2007-02-13 22:09:04 +01:00
|
|
|
pr_debug("I2C adapter driver [%s] forgot to specify "
|
|
|
|
"physical device\n", adap->name);
|
2007-01-04 13:07:04 +01:00
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
sprintf(adap->dev.bus_id, "i2c-%d", adap->nr);
|
|
|
|
adap->dev.release = &i2c_adapter_dev_release;
|
2007-05-01 23:26:27 +02:00
|
|
|
adap->dev.class = &i2c_adapter_class;
|
2006-08-15 18:26:30 +02:00
|
|
|
res = device_register(&adap->dev);
|
|
|
|
if (res)
|
|
|
|
goto out_list;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2005-07-31 19:02:53 +02:00
|
|
|
dev_dbg(&adap->dev, "adapter [%s] registered\n", adap->name);
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
/* create pre-declared device nodes for new-style drivers */
|
|
|
|
if (adap->nr < __i2c_first_dynamic_bus_num)
|
|
|
|
i2c_scan_static_board_info(adap);
|
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
/* Notify drivers */
|
2008-01-27 18:14:49 +01:00
|
|
|
dummy = bus_for_each_drv(&i2c_bus_type, NULL, adap,
|
|
|
|
i2c_do_add_adapter);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
out_unlock:
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
return res;
|
2006-08-15 18:26:30 +02:00
|
|
|
|
|
|
|
out_list:
|
|
|
|
idr_remove(&i2c_adapter_idr, adap->nr);
|
|
|
|
goto out_unlock;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
/**
|
|
|
|
* i2c_add_adapter - declare i2c adapter, use dynamic bus number
|
|
|
|
* @adapter: the adapter to add
|
2007-07-12 14:12:28 +02:00
|
|
|
* Context: can sleep
|
2007-05-01 23:26:31 +02:00
|
|
|
*
|
|
|
|
* This routine is used to declare an I2C adapter when its bus number
|
|
|
|
* doesn't matter. Examples: for I2C adapters dynamically added by
|
|
|
|
* USB links or PCI plugin cards.
|
|
|
|
*
|
|
|
|
* When this returns zero, a new bus number was allocated and stored
|
|
|
|
* in adap->nr, and the specified adapter became available for clients.
|
|
|
|
* Otherwise, a negative errno value is returned.
|
|
|
|
*/
|
|
|
|
int i2c_add_adapter(struct i2c_adapter *adapter)
|
|
|
|
{
|
|
|
|
int id, res = 0;
|
|
|
|
|
|
|
|
retry:
|
|
|
|
if (idr_pre_get(&i2c_adapter_idr, GFP_KERNEL) == 0)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2007-05-01 23:26:31 +02:00
|
|
|
/* "above" here means "above or equal to", sigh */
|
|
|
|
res = idr_get_new_above(&i2c_adapter_idr, adapter,
|
|
|
|
__i2c_first_dynamic_bus_num, &id);
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2007-05-01 23:26:31 +02:00
|
|
|
|
|
|
|
if (res < 0) {
|
|
|
|
if (res == -EAGAIN)
|
|
|
|
goto retry;
|
|
|
|
return res;
|
|
|
|
}
|
|
|
|
|
|
|
|
adapter->nr = id;
|
|
|
|
return i2c_register_adapter(adapter);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(i2c_add_adapter);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* i2c_add_numbered_adapter - declare i2c adapter, use static bus number
|
|
|
|
* @adap: the adapter to register (with adap->nr initialized)
|
2007-07-12 14:12:28 +02:00
|
|
|
* Context: can sleep
|
2007-05-01 23:26:31 +02:00
|
|
|
*
|
|
|
|
* This routine is used to declare an I2C adapter when its bus number
|
2008-03-23 20:28:20 +01:00
|
|
|
* matters. For example, use it for I2C adapters from system-on-chip CPUs,
|
|
|
|
* or otherwise built in to the system's mainboard, and where i2c_board_info
|
2007-05-01 23:26:31 +02:00
|
|
|
* is used to properly configure I2C devices.
|
|
|
|
*
|
|
|
|
* If no devices have pre-been declared for this bus, then be sure to
|
|
|
|
* register the adapter before any dynamically allocated ones. Otherwise
|
|
|
|
* the required bus ID may not be available.
|
|
|
|
*
|
|
|
|
* When this returns zero, the specified adapter became available for
|
|
|
|
* clients using the bus number provided in adap->nr. Also, the table
|
|
|
|
* of I2C devices pre-declared using i2c_register_board_info() is scanned,
|
|
|
|
* and the appropriate driver model device nodes are created. Otherwise, a
|
|
|
|
* negative errno value is returned.
|
|
|
|
*/
|
|
|
|
int i2c_add_numbered_adapter(struct i2c_adapter *adap)
|
|
|
|
{
|
|
|
|
int id;
|
|
|
|
int status;
|
|
|
|
|
|
|
|
if (adap->nr & ~MAX_ID_MASK)
|
|
|
|
return -EINVAL;
|
|
|
|
|
|
|
|
retry:
|
|
|
|
if (idr_pre_get(&i2c_adapter_idr, GFP_KERNEL) == 0)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2007-05-01 23:26:31 +02:00
|
|
|
/* "above" here means "above or equal to", sigh;
|
|
|
|
* we need the "equal to" result to force the result
|
|
|
|
*/
|
|
|
|
status = idr_get_new_above(&i2c_adapter_idr, adap, adap->nr, &id);
|
|
|
|
if (status == 0 && id != adap->nr) {
|
|
|
|
status = -EBUSY;
|
|
|
|
idr_remove(&i2c_adapter_idr, id);
|
|
|
|
}
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2007-05-01 23:26:31 +02:00
|
|
|
if (status == -EAGAIN)
|
|
|
|
goto retry;
|
|
|
|
|
|
|
|
if (status == 0)
|
|
|
|
status = i2c_register_adapter(adap);
|
|
|
|
return status;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(i2c_add_numbered_adapter);
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
static int i2c_do_del_adapter(struct device_driver *d, void *data)
|
|
|
|
{
|
|
|
|
struct i2c_driver *driver = to_i2c_driver(d);
|
|
|
|
struct i2c_adapter *adapter = data;
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
struct i2c_client *client, *_n;
|
2008-01-27 18:14:49 +01:00
|
|
|
int res;
|
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
/* Remove the devices we created ourselves */
|
|
|
|
list_for_each_entry_safe(client, _n, &driver->clients, detected) {
|
|
|
|
if (client->adapter == adapter) {
|
|
|
|
dev_dbg(&adapter->dev, "Removing %s at 0x%x\n",
|
|
|
|
client->name, client->addr);
|
|
|
|
list_del(&client->detected);
|
|
|
|
i2c_unregister_device(client);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
if (!driver->detach_adapter)
|
|
|
|
return 0;
|
|
|
|
res = driver->detach_adapter(adapter);
|
|
|
|
if (res)
|
|
|
|
dev_err(&adapter->dev, "detach_adapter failed (%d) "
|
|
|
|
"for driver [%s]\n", res, driver->driver.name);
|
|
|
|
return res;
|
|
|
|
}
|
|
|
|
|
2007-07-12 14:12:28 +02:00
|
|
|
/**
|
|
|
|
* i2c_del_adapter - unregister I2C adapter
|
|
|
|
* @adap: the adapter being unregistered
|
|
|
|
* Context: can sleep
|
|
|
|
*
|
|
|
|
* This unregisters an I2C adapter which was previously registered
|
|
|
|
* by @i2c_add_adapter or @i2c_add_numbered_adapter.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
int i2c_del_adapter(struct i2c_adapter *adap)
|
|
|
|
{
|
2008-07-14 22:38:26 +02:00
|
|
|
struct i2c_client *client, *_n;
|
2005-04-17 00:20:36 +02:00
|
|
|
int res = 0;
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* First make sure that this adapter was ever added */
|
2008-01-27 18:14:48 +01:00
|
|
|
if (idr_find(&i2c_adapter_idr, adap->nr) != adap) {
|
2005-07-31 19:02:53 +02:00
|
|
|
pr_debug("i2c-core: attempting to delete unregistered "
|
|
|
|
"adapter [%s]\n", adap->name);
|
2005-04-17 00:20:36 +02:00
|
|
|
res = -EINVAL;
|
|
|
|
goto out_unlock;
|
|
|
|
}
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
/* Tell drivers about this removal */
|
|
|
|
res = bus_for_each_drv(&i2c_bus_type, NULL, adap,
|
|
|
|
i2c_do_del_adapter);
|
|
|
|
if (res)
|
|
|
|
goto out_unlock;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* detach any active clients. This must be done first, because
|
2005-05-19 21:40:38 +02:00
|
|
|
* it can fail; in which case we give up. */
|
2008-07-14 22:38:26 +02:00
|
|
|
list_for_each_entry_safe(client, _n, &adap->clients, list) {
|
2007-05-01 23:26:30 +02:00
|
|
|
struct i2c_driver *driver;
|
|
|
|
|
|
|
|
driver = client->driver;
|
|
|
|
|
|
|
|
/* new style, follow standard driver model */
|
|
|
|
if (!driver || is_newstyle_driver(driver)) {
|
|
|
|
i2c_unregister_device(client);
|
|
|
|
continue;
|
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/* legacy drivers create and remove clients themselves */
|
|
|
|
if ((res = driver->detach_client(client))) {
|
2005-07-31 19:02:53 +02:00
|
|
|
dev_err(&adap->dev, "detach_client failed for client "
|
|
|
|
"[%s] at address 0x%02x\n", client->name,
|
2005-04-17 00:20:36 +02:00
|
|
|
client->addr);
|
|
|
|
goto out_unlock;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* clean up the sysfs representation */
|
|
|
|
init_completion(&adap->dev_released);
|
|
|
|
device_unregister(&adap->dev);
|
|
|
|
|
|
|
|
/* wait for sysfs to drop all references */
|
|
|
|
wait_for_completion(&adap->dev_released);
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
/* free bus id */
|
2005-04-17 00:20:36 +02:00
|
|
|
idr_remove(&i2c_adapter_idr, adap->nr);
|
|
|
|
|
2005-07-31 19:02:53 +02:00
|
|
|
dev_dbg(&adap->dev, "adapter [%s] unregistered\n", adap->name);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-16 19:30:05 +02:00
|
|
|
/* Clear the device structure in case this adapter is ever going to be
|
|
|
|
added again */
|
|
|
|
memset(&adap->dev, 0, sizeof(adap->dev));
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
out_unlock:
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
return res;
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_del_adapter);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
|
2008-07-14 22:38:19 +02:00
|
|
|
static int __attach_adapter(struct device *dev, void *data)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter = to_i2c_adapter(dev);
|
|
|
|
struct i2c_driver *driver = data;
|
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
i2c_detect(adapter, driver);
|
|
|
|
|
|
|
|
/* Legacy drivers scan i2c busses directly */
|
|
|
|
if (driver->attach_adapter)
|
|
|
|
driver->attach_adapter(adapter);
|
2008-07-14 22:38:19 +02:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/*
|
|
|
|
* An i2c_driver is used with one or more i2c_client (device) nodes to access
|
|
|
|
* i2c slave chips, on a bus instance associated with some i2c_adapter. There
|
|
|
|
* are two models for binding the driver to its device: "new style" drivers
|
|
|
|
* follow the standard Linux driver model and just respond to probe() calls
|
|
|
|
* issued if the driver core sees they match(); "legacy" drivers create device
|
|
|
|
* nodes themselves.
|
2005-04-17 00:20:36 +02:00
|
|
|
*/
|
|
|
|
|
2005-12-07 00:33:15 +01:00
|
|
|
int i2c_register_driver(struct module *owner, struct i2c_driver *driver)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2006-02-05 23:28:21 +01:00
|
|
|
int res;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/* new style driver methods can't mix with legacy ones */
|
2007-05-01 23:26:30 +02:00
|
|
|
if (is_newstyle_driver(driver)) {
|
2007-05-01 23:26:30 +02:00
|
|
|
if (driver->attach_adapter || driver->detach_adapter
|
|
|
|
|| driver->detach_client) {
|
|
|
|
printk(KERN_WARNING
|
|
|
|
"i2c-core: driver [%s] is confused\n",
|
|
|
|
driver->driver.name);
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/* add the driver to the list of i2c drivers in the driver core */
|
2005-12-07 00:33:15 +01:00
|
|
|
driver->driver.owner = owner;
|
2005-04-17 00:20:36 +02:00
|
|
|
driver->driver.bus = &i2c_bus_type;
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
/* for new style drivers, when registration returns the driver core
|
|
|
|
* will have called probe() for all matching-but-unbound devices.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
res = driver_register(&driver->driver);
|
|
|
|
if (res)
|
2006-02-05 23:28:21 +01:00
|
|
|
return res;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2006-02-05 23:28:21 +01:00
|
|
|
|
2005-11-26 20:34:05 +01:00
|
|
|
pr_debug("i2c-core: driver [%s] registered\n", driver->driver.name);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
INIT_LIST_HEAD(&driver->clients);
|
|
|
|
/* Walk the adapters that are already present */
|
2008-05-22 23:21:08 +02:00
|
|
|
class_for_each_device(&i2c_adapter_class, NULL, driver,
|
|
|
|
__attach_adapter);
|
2007-05-01 23:26:28 +02:00
|
|
|
|
2008-07-14 22:38:19 +02:00
|
|
|
mutex_unlock(&core_lock);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(i2c_register_driver);
|
|
|
|
|
|
|
|
static int __detach_adapter(struct device *dev, void *data)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter = to_i2c_adapter(dev);
|
|
|
|
struct i2c_driver *driver = data;
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
struct i2c_client *client, *_n;
|
|
|
|
|
|
|
|
list_for_each_entry_safe(client, _n, &driver->clients, detected) {
|
|
|
|
dev_dbg(&adapter->dev, "Removing %s at 0x%x\n",
|
|
|
|
client->name, client->addr);
|
|
|
|
list_del(&client->detected);
|
|
|
|
i2c_unregister_device(client);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (is_newstyle_driver(driver))
|
|
|
|
return 0;
|
2008-07-14 22:38:19 +02:00
|
|
|
|
|
|
|
/* Have a look at each adapter, if clients of this driver are still
|
|
|
|
* attached. If so, detach them to be able to kill the driver
|
|
|
|
* afterwards.
|
|
|
|
*/
|
|
|
|
if (driver->detach_adapter) {
|
|
|
|
if (driver->detach_adapter(adapter))
|
|
|
|
dev_err(&adapter->dev,
|
|
|
|
"detach_adapter failed for driver [%s]\n",
|
|
|
|
driver->driver.name);
|
|
|
|
} else {
|
2008-07-14 22:38:26 +02:00
|
|
|
struct i2c_client *client, *_n;
|
2008-07-14 22:38:19 +02:00
|
|
|
|
2008-07-14 22:38:26 +02:00
|
|
|
list_for_each_entry_safe(client, _n, &adapter->clients, list) {
|
2008-07-14 22:38:19 +02:00
|
|
|
if (client->driver != driver)
|
|
|
|
continue;
|
|
|
|
dev_dbg(&adapter->dev,
|
|
|
|
"detaching client [%s] at 0x%02x\n",
|
|
|
|
client->name, client->addr);
|
|
|
|
if (driver->detach_client(client))
|
|
|
|
dev_err(&adapter->dev, "detach_client "
|
|
|
|
"failed for client [%s] at 0x%02x\n",
|
|
|
|
client->name, client->addr);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2006-02-05 23:28:21 +01:00
|
|
|
return 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/**
|
|
|
|
* i2c_del_driver - unregister I2C driver
|
|
|
|
* @driver: the driver being unregistered
|
2007-07-12 14:12:28 +02:00
|
|
|
* Context: can sleep
|
2007-05-01 23:26:30 +02:00
|
|
|
*/
|
2007-05-01 23:26:32 +02:00
|
|
|
void i2c_del_driver(struct i2c_driver *driver)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-05-22 23:21:08 +02:00
|
|
|
class_for_each_device(&i2c_adapter_class, NULL, driver,
|
|
|
|
__detach_adapter);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
driver_unregister(&driver->driver);
|
2005-11-26 20:34:05 +01:00
|
|
|
pr_debug("i2c-core: driver [%s] unregistered\n", driver->driver.name);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_del_driver);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2007-05-01 23:26:30 +02:00
|
|
|
/* ------------------------------------------------------------------------- */
|
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
static int __i2c_check_addr(struct device *dev, void *addrp)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-01-27 18:14:51 +01:00
|
|
|
struct i2c_client *client = i2c_verify_client(dev);
|
|
|
|
int addr = *(int *)addrp;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
if (client && client->addr == addr)
|
|
|
|
return -EBUSY;
|
2005-04-17 00:20:36 +02:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2007-11-15 19:24:02 +01:00
|
|
|
static int i2c_check_addr(struct i2c_adapter *adapter, int addr)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-01-27 18:14:51 +01:00
|
|
|
return device_for_each_child(&adapter->dev, &addr, __i2c_check_addr);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
int i2c_attach_client(struct i2c_client *client)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter = client->adapter;
|
2006-08-15 18:26:30 +02:00
|
|
|
int res = 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
client->dev.parent = &client->adapter->dev;
|
|
|
|
client->dev.bus = &i2c_bus_type;
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
|
|
|
|
if (client->driver)
|
|
|
|
client->dev.driver = &client->driver->driver;
|
|
|
|
|
2007-05-22 19:49:16 +02:00
|
|
|
if (client->driver && !is_newstyle_driver(client->driver)) {
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
client->dev.release = i2c_client_release;
|
2007-05-22 19:49:16 +02:00
|
|
|
client->dev.uevent_suppress = 1;
|
|
|
|
} else
|
i2c: Add i2c_board_info and i2c_new_device()
This provides partial support for new-style I2C driver binding. It builds
on "struct i2c_board_info" declarations that identify I2C devices on a given
board. This is needed on systems with I2C devices that can't be fully probed
and/or autoconfigured, such as many embedded Linux configurations where the
way a given I2C device is wired may affect how it must be used.
There are two models for declaring such devices:
* LATE -- using a public function i2c_new_device(). This lets modules
declare I2C devices found *AFTER* a given I2C adapter becomes available.
For example, a PCI card could create adapters giving access to utility
chips on that card, and this would be used to associate those chips with
those adapters.
* EARLY -- from arch_initcall() level code, using a non-exported function
i2c_register_board_info(). This copies the declarations *BEFORE* such
an i2c_adapter becomes available, arranging that i2c_new_device() will
be called later when i2c-core registers the relevant i2c_adapter.
For example, arch/.../.../board-*.c files would declare the I2C devices
along with their platform data, and I2C devices would behave much like
PNPACPI devices. (That is, both enumerate from board-specific tables.)
To match the exported i2c_new_device(), the previously-private function
i2c_unregister_device() is now exported.
Pending later patches using these new APIs, this is effectively a NOP.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-05-01 23:26:31 +02:00
|
|
|
client->dev.release = i2c_client_dev_release;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
snprintf(&client->dev.bus_id[0], sizeof(client->dev.bus_id),
|
|
|
|
"%d-%04x", i2c_adapter_id(adapter), client->addr);
|
2006-08-15 18:26:30 +02:00
|
|
|
res = device_register(&client->dev);
|
|
|
|
if (res)
|
2008-01-27 18:14:51 +01:00
|
|
|
goto out_err;
|
|
|
|
|
|
|
|
mutex_lock(&adapter->clist_lock);
|
|
|
|
list_add_tail(&client->list, &adapter->clients);
|
2006-08-15 18:26:30 +02:00
|
|
|
mutex_unlock(&adapter->clist_lock);
|
2006-09-30 17:18:59 +02:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
dev_dbg(&adapter->dev, "client [%s] registered with bus id %s\n",
|
|
|
|
client->name, client->dev.bus_id);
|
|
|
|
|
2006-09-30 17:18:59 +02:00
|
|
|
if (adapter->client_register) {
|
|
|
|
if (adapter->client_register(client)) {
|
|
|
|
dev_dbg(&adapter->dev, "client_register "
|
|
|
|
"failed for client [%s] at 0x%02x\n",
|
|
|
|
client->name, client->addr);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
2006-08-15 18:26:30 +02:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
out_err:
|
2006-08-15 18:26:30 +02:00
|
|
|
dev_err(&adapter->dev, "Failed to attach i2c client %s at 0x%02x "
|
|
|
|
"(%d)\n", client->name, client->addr, res);
|
2006-09-30 17:18:59 +02:00
|
|
|
return res;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_attach_client);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
int i2c_detach_client(struct i2c_client *client)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter = client->adapter;
|
|
|
|
int res = 0;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
if (adapter->client_unregister) {
|
|
|
|
res = adapter->client_unregister(client);
|
|
|
|
if (res) {
|
|
|
|
dev_err(&client->dev,
|
2005-07-29 21:15:29 +02:00
|
|
|
"client_unregister [%s] failed, "
|
|
|
|
"client not detached\n", client->name);
|
2005-04-17 00:20:36 +02:00
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2006-01-18 23:16:04 +01:00
|
|
|
mutex_lock(&adapter->clist_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
list_del(&client->list);
|
2008-01-27 18:14:51 +01:00
|
|
|
mutex_unlock(&adapter->clist_lock);
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
init_completion(&client->released);
|
|
|
|
device_unregister(&client->dev);
|
|
|
|
wait_for_completion(&client->released);
|
|
|
|
|
|
|
|
out:
|
|
|
|
return res;
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_detach_client);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:48 +01:00
|
|
|
/**
|
|
|
|
* i2c_use_client - increments the reference count of the i2c client structure
|
|
|
|
* @client: the client being referenced
|
|
|
|
*
|
|
|
|
* Each live reference to a client should be refcounted. The driver model does
|
|
|
|
* that automatically as part of driver binding, so that most drivers don't
|
|
|
|
* need to do this explicitly: they hold a reference until they're unbound
|
|
|
|
* from the device.
|
|
|
|
*
|
|
|
|
* A pointer to the client with the incremented reference counter is returned.
|
|
|
|
*/
|
|
|
|
struct i2c_client *i2c_use_client(struct i2c_client *client)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-07-14 22:38:24 +02:00
|
|
|
if (client && get_device(&client->dev))
|
|
|
|
return client;
|
|
|
|
return NULL;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_use_client);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:48 +01:00
|
|
|
/**
|
|
|
|
* i2c_release_client - release a use of the i2c client structure
|
|
|
|
* @client: the client being no longer referenced
|
|
|
|
*
|
|
|
|
* Must be called when a user of a client is finished with it.
|
|
|
|
*/
|
|
|
|
void i2c_release_client(struct i2c_client *client)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-07-14 22:38:24 +02:00
|
|
|
if (client)
|
|
|
|
put_device(&client->dev);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_release_client);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
struct i2c_cmd_arg {
|
|
|
|
unsigned cmd;
|
|
|
|
void *arg;
|
|
|
|
};
|
|
|
|
|
|
|
|
static int i2c_cmd(struct device *dev, void *_arg)
|
|
|
|
{
|
|
|
|
struct i2c_client *client = i2c_verify_client(dev);
|
|
|
|
struct i2c_cmd_arg *arg = _arg;
|
|
|
|
|
|
|
|
if (client && client->driver && client->driver->command)
|
|
|
|
client->driver->command(client, arg->cmd, arg->arg);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
void i2c_clients_command(struct i2c_adapter *adap, unsigned int cmd, void *arg)
|
|
|
|
{
|
2008-01-27 18:14:51 +01:00
|
|
|
struct i2c_cmd_arg cmd_arg;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-01-27 18:14:51 +01:00
|
|
|
cmd_arg.cmd = cmd;
|
|
|
|
cmd_arg.arg = arg;
|
|
|
|
device_for_each_child(&adap->dev, &cmd_arg, i2c_cmd);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_clients_command);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
static int __init i2c_init(void)
|
|
|
|
{
|
|
|
|
int retval;
|
|
|
|
|
|
|
|
retval = bus_register(&i2c_bus_type);
|
|
|
|
if (retval)
|
|
|
|
return retval;
|
2008-01-27 18:14:52 +01:00
|
|
|
retval = class_register(&i2c_adapter_class);
|
|
|
|
if (retval)
|
|
|
|
goto bus_err;
|
|
|
|
retval = i2c_add_driver(&dummy_driver);
|
|
|
|
if (retval)
|
|
|
|
goto class_err;
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
class_err:
|
|
|
|
class_unregister(&i2c_adapter_class);
|
|
|
|
bus_err:
|
|
|
|
bus_unregister(&i2c_bus_type);
|
|
|
|
return retval;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void __exit i2c_exit(void)
|
|
|
|
{
|
2008-01-27 18:14:52 +01:00
|
|
|
i2c_del_driver(&dummy_driver);
|
2005-04-17 00:20:36 +02:00
|
|
|
class_unregister(&i2c_adapter_class);
|
|
|
|
bus_unregister(&i2c_bus_type);
|
|
|
|
}
|
|
|
|
|
|
|
|
subsys_initcall(i2c_init);
|
|
|
|
module_exit(i2c_exit);
|
|
|
|
|
|
|
|
/* ----------------------------------------------------
|
|
|
|
* the functional interface to the i2c busses.
|
|
|
|
* ----------------------------------------------------
|
|
|
|
*/
|
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_transfer - execute a single or combined I2C message
|
|
|
|
* @adap: Handle to I2C bus
|
|
|
|
* @msgs: One or more messages to execute before STOP is issued to
|
|
|
|
* terminate the operation; each message begins with a START.
|
|
|
|
* @num: Number of messages to be executed.
|
|
|
|
*
|
|
|
|
* Returns negative errno, else the number of messages executed.
|
|
|
|
*
|
|
|
|
* Note that there is no requirement that each message be sent to
|
|
|
|
* the same slave address, although that is the most common model.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
int i2c_transfer(struct i2c_adapter * adap, struct i2c_msg *msgs, int num)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/* REVISIT the fault reporting model here is weak:
|
|
|
|
*
|
|
|
|
* - When we get an error after receiving N bytes from a slave,
|
|
|
|
* there is no way to report "N".
|
|
|
|
*
|
|
|
|
* - When we get a NAK after transmitting N bytes to a slave,
|
|
|
|
* there is no way to report "N" ... or to let the master
|
|
|
|
* continue executing the rest of this combined message, if
|
|
|
|
* that's the appropriate response.
|
|
|
|
*
|
|
|
|
* - When for example "num" is two and we successfully complete
|
|
|
|
* the first message but get an error part way through the
|
|
|
|
* second, it's unclear whether that should be reported as
|
|
|
|
* one (discarding status on the second message) or errno
|
|
|
|
* (discarding status on the first one).
|
|
|
|
*/
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
if (adap->algo->master_xfer) {
|
|
|
|
#ifdef DEBUG
|
|
|
|
for (ret = 0; ret < num; ret++) {
|
|
|
|
dev_dbg(&adap->dev, "master_xfer[%d] %c, addr=0x%02x, "
|
2007-05-01 23:26:29 +02:00
|
|
|
"len=%d%s\n", ret, (msgs[ret].flags & I2C_M_RD)
|
|
|
|
? 'R' : 'W', msgs[ret].addr, msgs[ret].len,
|
|
|
|
(msgs[ret].flags & I2C_M_RECV_LEN) ? "+" : "");
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
2008-01-27 18:14:50 +01:00
|
|
|
if (in_atomic() || irqs_disabled()) {
|
|
|
|
ret = mutex_trylock(&adap->bus_lock);
|
|
|
|
if (!ret)
|
|
|
|
/* I2C activity is ongoing. */
|
|
|
|
return -EAGAIN;
|
|
|
|
} else {
|
|
|
|
mutex_lock_nested(&adap->bus_lock, adap->level);
|
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
ret = adap->algo->master_xfer(adap,msgs,num);
|
2006-01-18 23:16:04 +01:00
|
|
|
mutex_unlock(&adap->bus_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
return ret;
|
|
|
|
} else {
|
|
|
|
dev_dbg(&adap->dev, "I2C level transfers not supported\n");
|
2008-07-14 22:38:23 +02:00
|
|
|
return -EOPNOTSUPP;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_transfer);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_master_send - issue a single I2C message in master transmit mode
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @buf: Data that will be written to the slave
|
|
|
|
* @count: How many bytes to write
|
|
|
|
*
|
|
|
|
* Returns negative errno, or else the number of bytes written.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
int i2c_master_send(struct i2c_client *client,const char *buf ,int count)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
struct i2c_adapter *adap=client->adapter;
|
|
|
|
struct i2c_msg msg;
|
|
|
|
|
2005-05-07 22:58:46 +02:00
|
|
|
msg.addr = client->addr;
|
|
|
|
msg.flags = client->flags & I2C_M_TEN;
|
|
|
|
msg.len = count;
|
|
|
|
msg.buf = (char *)buf;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2005-05-07 22:58:46 +02:00
|
|
|
ret = i2c_transfer(adap, &msg, 1);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2005-05-07 22:58:46 +02:00
|
|
|
/* If everything went ok (i.e. 1 msg transmitted), return #bytes
|
|
|
|
transmitted, else error code. */
|
|
|
|
return (ret == 1) ? count : ret;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_master_send);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_master_recv - issue a single I2C message in master receive mode
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @buf: Where to store data read from slave
|
|
|
|
* @count: How many bytes to read
|
|
|
|
*
|
|
|
|
* Returns negative errno, or else the number of bytes read.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
int i2c_master_recv(struct i2c_client *client, char *buf ,int count)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adap=client->adapter;
|
|
|
|
struct i2c_msg msg;
|
|
|
|
int ret;
|
2005-05-07 22:58:46 +02:00
|
|
|
|
|
|
|
msg.addr = client->addr;
|
|
|
|
msg.flags = client->flags & I2C_M_TEN;
|
|
|
|
msg.flags |= I2C_M_RD;
|
|
|
|
msg.len = count;
|
|
|
|
msg.buf = buf;
|
|
|
|
|
|
|
|
ret = i2c_transfer(adap, &msg, 1);
|
|
|
|
|
|
|
|
/* If everything went ok (i.e. 1 msg transmitted), return #bytes
|
|
|
|
transmitted, else error code. */
|
|
|
|
return (ret == 1) ? count : ret;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_master_recv);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* ----------------------------------------------------
|
|
|
|
* the i2c address scanning function
|
|
|
|
* Will not work for 10-bit addresses!
|
|
|
|
* ----------------------------------------------------
|
|
|
|
*/
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
static int i2c_probe_address(struct i2c_adapter *adapter, int addr, int kind,
|
|
|
|
int (*found_proc) (struct i2c_adapter *, int, int))
|
|
|
|
{
|
|
|
|
int err;
|
|
|
|
|
|
|
|
/* Make sure the address is valid */
|
|
|
|
if (addr < 0x03 || addr > 0x77) {
|
|
|
|
dev_warn(&adapter->dev, "Invalid probe address 0x%02x\n",
|
|
|
|
addr);
|
|
|
|
return -EINVAL;
|
2005-07-31 21:20:43 +02:00
|
|
|
}
|
|
|
|
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
/* Skip if already in use */
|
|
|
|
if (i2c_check_addr(adapter, addr))
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* Make sure there is something at this address, unless forced */
|
2005-08-09 20:28:10 +02:00
|
|
|
if (kind < 0) {
|
|
|
|
if (i2c_smbus_xfer(adapter, addr, 0, 0, 0,
|
|
|
|
I2C_SMBUS_QUICK, NULL) < 0)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* prevent 24RF08 corruption */
|
|
|
|
if ((addr & ~0x0f) == 0x50)
|
|
|
|
i2c_smbus_xfer(adapter, addr, 0, 0, 0,
|
|
|
|
I2C_SMBUS_QUICK, NULL);
|
|
|
|
}
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
|
|
|
|
/* Finally call the custom detection function */
|
|
|
|
err = found_proc(adapter, addr, kind);
|
|
|
|
/* -ENODEV can be returned if there is a chip at the given address
|
|
|
|
but it isn't supported by this chip driver. We catch it here as
|
|
|
|
this isn't an error. */
|
2006-09-03 22:25:04 +02:00
|
|
|
if (err == -ENODEV)
|
|
|
|
err = 0;
|
|
|
|
|
|
|
|
if (err)
|
|
|
|
dev_warn(&adapter->dev, "Client creation failed at 0x%x (%d)\n",
|
|
|
|
addr, err);
|
|
|
|
return err;
|
2005-07-31 21:20:43 +02:00
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
int i2c_probe(struct i2c_adapter *adapter,
|
2008-01-27 18:14:46 +01:00
|
|
|
const struct i2c_client_address_data *address_data,
|
2005-04-17 00:20:36 +02:00
|
|
|
int (*found_proc) (struct i2c_adapter *, int, int))
|
|
|
|
{
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
int i, err;
|
2005-04-17 00:20:36 +02:00
|
|
|
int adap_id = i2c_adapter_id(adapter);
|
|
|
|
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
/* Force entries are done first, and are not affected by ignore
|
|
|
|
entries */
|
|
|
|
if (address_data->forces) {
|
2008-01-27 18:14:46 +01:00
|
|
|
const unsigned short * const *forces = address_data->forces;
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
int kind;
|
|
|
|
|
|
|
|
for (kind = 0; forces[kind]; kind++) {
|
|
|
|
for (i = 0; forces[kind][i] != I2C_CLIENT_END;
|
|
|
|
i += 2) {
|
|
|
|
if (forces[kind][i] == adap_id
|
|
|
|
|| forces[kind][i] == ANY_I2C_BUS) {
|
|
|
|
dev_dbg(&adapter->dev, "found force "
|
|
|
|
"parameter for adapter %d, "
|
|
|
|
"addr 0x%02x, kind %d\n",
|
|
|
|
adap_id, forces[kind][i + 1],
|
|
|
|
kind);
|
|
|
|
err = i2c_probe_address(adapter,
|
|
|
|
forces[kind][i + 1],
|
|
|
|
kind, found_proc);
|
|
|
|
if (err)
|
|
|
|
return err;
|
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
}
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2005-09-25 16:50:06 +02:00
|
|
|
/* Stop here if we can't use SMBUS_QUICK */
|
|
|
|
if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_QUICK)) {
|
|
|
|
if (address_data->probe[0] == I2C_CLIENT_END
|
|
|
|
&& address_data->normal_i2c[0] == I2C_CLIENT_END)
|
2006-12-10 21:21:31 +01:00
|
|
|
return 0;
|
2005-09-25 16:50:06 +02:00
|
|
|
|
|
|
|
dev_warn(&adapter->dev, "SMBus Quick command not supported, "
|
|
|
|
"can't probe for chips\n");
|
2008-07-14 22:38:23 +02:00
|
|
|
return -EOPNOTSUPP;
|
2005-09-25 16:50:06 +02:00
|
|
|
}
|
|
|
|
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
/* Probe entries are done second, and are not affected by ignore
|
|
|
|
entries either */
|
|
|
|
for (i = 0; address_data->probe[i] != I2C_CLIENT_END; i += 2) {
|
|
|
|
if (address_data->probe[i] == adap_id
|
|
|
|
|| address_data->probe[i] == ANY_I2C_BUS) {
|
|
|
|
dev_dbg(&adapter->dev, "found probe parameter for "
|
|
|
|
"adapter %d, addr 0x%02x\n", adap_id,
|
|
|
|
address_data->probe[i + 1]);
|
|
|
|
err = i2c_probe_address(adapter,
|
|
|
|
address_data->probe[i + 1],
|
|
|
|
-1, found_proc);
|
|
|
|
if (err)
|
|
|
|
return err;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
/* Normal entries are done last, unless shadowed by an ignore entry */
|
|
|
|
for (i = 0; address_data->normal_i2c[i] != I2C_CLIENT_END; i += 1) {
|
|
|
|
int j, ignore;
|
|
|
|
|
|
|
|
ignore = 0;
|
|
|
|
for (j = 0; address_data->ignore[j] != I2C_CLIENT_END;
|
|
|
|
j += 2) {
|
|
|
|
if ((address_data->ignore[j] == adap_id ||
|
|
|
|
address_data->ignore[j] == ANY_I2C_BUS)
|
|
|
|
&& address_data->ignore[j + 1]
|
|
|
|
== address_data->normal_i2c[i]) {
|
|
|
|
dev_dbg(&adapter->dev, "found ignore "
|
|
|
|
"parameter for adapter %d, "
|
|
|
|
"addr 0x%02x\n", adap_id,
|
|
|
|
address_data->ignore[j + 1]);
|
2006-07-01 17:01:59 +02:00
|
|
|
ignore = 1;
|
|
|
|
break;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
}
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
if (ignore)
|
2005-04-17 00:20:36 +02:00
|
|
|
continue;
|
|
|
|
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
dev_dbg(&adapter->dev, "found normal entry for adapter %d, "
|
|
|
|
"addr 0x%02x\n", adap_id,
|
|
|
|
address_data->normal_i2c[i]);
|
|
|
|
err = i2c_probe_address(adapter, address_data->normal_i2c[i],
|
|
|
|
-1, found_proc);
|
|
|
|
if (err)
|
|
|
|
return err;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
[PATCH] I2C: Rewrite i2c_probe
i2c_probe was quite complex and slow, so I rewrote it in a more
efficient and hopefully clearer way.
Note that this slightly changes the way the module parameters are
handled. This shouldn't change anything for the most common cases
though.
For one thing, the function now respects the order of the parameters
for address probing. It used to always do lower addresses first. The
new approach gives the user more control.
For another, ignore addresses don't overrule probe addresses anymore.
This could have been restored the way it was at the cost of a few more
lines of code, but I don't think it's worth it. Both lists are given
as module parameters, so a user would be quite silly to specify the
same addresses in both lists. The normal addresses list is the only
one that isn't controlled by a module parameter, thus is the only one
the user may reasonably want to remove an address from.
Another significant change is the fact that i2c_probe() will no more
stop when a detection function returns -ENODEV. Just because a driver
found a chip it doesn't support isn't a valid reason to stop all
probings for this one driver. This closes the long standing lm_sensors
ticket #1807.
http://www2.lm-sensors.nu/~lm78/readticket.cgi?ticket=1807
I updated the documentation accordingly.
In terms of algorithmic complexity, the new code is way better. If
I is the ignore address count, P the probe address count, N the
normal address count and F the force address count, the old code
was doing 128 * (F + I + P + N) iterations max, while the new code
does F + P + ((I+1) * N) iterations max. For the most common case
where F, I and P are empty, this is down from 128 * N to N.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-08-09 20:17:55 +02:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
return 0;
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_probe);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
i2c: Add detection capability to new-style drivers
Add a mechanism to let new-style i2c drivers optionally autodetect
devices they would support on selected buses and ask i2c-core to
instantiate them. This is a replacement for legacy i2c drivers, much
cleaner.
Where drivers had to implement both a legacy i2c_driver and a
new-style i2c_driver so far, this mechanism makes it possible to get
rid of the legacy i2c_driver and implement both enumerated and
detected device support with just one (new-style) i2c_driver.
Here is a quick conversion guide for these drivers, step by step:
* Delete the legacy driver definition, registration and removal.
Delete the attach_adapter and detach_client methods of the legacy
driver.
* Change the prototype of the legacy detect function from
static int foo_detect(struct i2c_adapter *adapter, int address, int kind);
to
static int foo_detect(struct i2c_client *client, int kind,
struct i2c_board_info *info);
* Set the new-style driver detect callback to this new function, and
set its address_data to &addr_data (addr_data is generally provided
by I2C_CLIENT_INSMOD.)
* Add the appropriate class to the new-style driver. This is
typically the class the legacy attach_adapter method was checking
for. Class checking is now mandatory (done by i2c-core.) See
<linux/i2c.h> for the list of available classes.
* Remove the i2c_client allocation and freeing from the detect
function. A pre-allocated client is now handed to you by i2c-core,
and is freed automatically.
* Make the detect function fill the type field of the i2c_board_info
structure it was passed as a parameter, and return 0, on success. If
the detection fails, return -ENODEV.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2008-07-14 22:38:36 +02:00
|
|
|
/* Separate detection function for new-style drivers */
|
|
|
|
static int i2c_detect_address(struct i2c_client *temp_client, int kind,
|
|
|
|
struct i2c_driver *driver)
|
|
|
|
{
|
|
|
|
struct i2c_board_info info;
|
|
|
|
struct i2c_adapter *adapter = temp_client->adapter;
|
|
|
|
int addr = temp_client->addr;
|
|
|
|
int err;
|
|
|
|
|
|
|
|
/* Make sure the address is valid */
|
|
|
|
if (addr < 0x03 || addr > 0x77) {
|
|
|
|
dev_warn(&adapter->dev, "Invalid probe address 0x%02x\n",
|
|
|
|
addr);
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Skip if already in use */
|
|
|
|
if (i2c_check_addr(adapter, addr))
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* Make sure there is something at this address, unless forced */
|
|
|
|
if (kind < 0) {
|
|
|
|
if (i2c_smbus_xfer(adapter, addr, 0, 0, 0,
|
|
|
|
I2C_SMBUS_QUICK, NULL) < 0)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* prevent 24RF08 corruption */
|
|
|
|
if ((addr & ~0x0f) == 0x50)
|
|
|
|
i2c_smbus_xfer(adapter, addr, 0, 0, 0,
|
|
|
|
I2C_SMBUS_QUICK, NULL);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Finally call the custom detection function */
|
|
|
|
memset(&info, 0, sizeof(struct i2c_board_info));
|
|
|
|
info.addr = addr;
|
|
|
|
err = driver->detect(temp_client, kind, &info);
|
|
|
|
if (err) {
|
|
|
|
/* -ENODEV is returned if the detection fails. We catch it
|
|
|
|
here as this isn't an error. */
|
|
|
|
return err == -ENODEV ? 0 : err;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Consistency check */
|
|
|
|
if (info.type[0] == '\0') {
|
|
|
|
dev_err(&adapter->dev, "%s detection function provided "
|
|
|
|
"no name for 0x%x\n", driver->driver.name,
|
|
|
|
addr);
|
|
|
|
} else {
|
|
|
|
struct i2c_client *client;
|
|
|
|
|
|
|
|
/* Detection succeeded, instantiate the device */
|
|
|
|
dev_dbg(&adapter->dev, "Creating %s at 0x%02x\n",
|
|
|
|
info.type, info.addr);
|
|
|
|
client = i2c_new_device(adapter, &info);
|
|
|
|
if (client)
|
|
|
|
list_add_tail(&client->detected, &driver->clients);
|
|
|
|
else
|
|
|
|
dev_err(&adapter->dev, "Failed creating %s at 0x%02x\n",
|
|
|
|
info.type, info.addr);
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int i2c_detect(struct i2c_adapter *adapter, struct i2c_driver *driver)
|
|
|
|
{
|
|
|
|
const struct i2c_client_address_data *address_data;
|
|
|
|
struct i2c_client *temp_client;
|
|
|
|
int i, err = 0;
|
|
|
|
int adap_id = i2c_adapter_id(adapter);
|
|
|
|
|
|
|
|
address_data = driver->address_data;
|
|
|
|
if (!driver->detect || !address_data)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* Set up a temporary client to help detect callback */
|
|
|
|
temp_client = kzalloc(sizeof(struct i2c_client), GFP_KERNEL);
|
|
|
|
if (!temp_client)
|
|
|
|
return -ENOMEM;
|
|
|
|
temp_client->adapter = adapter;
|
|
|
|
|
|
|
|
/* Force entries are done first, and are not affected by ignore
|
|
|
|
entries */
|
|
|
|
if (address_data->forces) {
|
|
|
|
const unsigned short * const *forces = address_data->forces;
|
|
|
|
int kind;
|
|
|
|
|
|
|
|
for (kind = 0; forces[kind]; kind++) {
|
|
|
|
for (i = 0; forces[kind][i] != I2C_CLIENT_END;
|
|
|
|
i += 2) {
|
|
|
|
if (forces[kind][i] == adap_id
|
|
|
|
|| forces[kind][i] == ANY_I2C_BUS) {
|
|
|
|
dev_dbg(&adapter->dev, "found force "
|
|
|
|
"parameter for adapter %d, "
|
|
|
|
"addr 0x%02x, kind %d\n",
|
|
|
|
adap_id, forces[kind][i + 1],
|
|
|
|
kind);
|
|
|
|
temp_client->addr = forces[kind][i + 1];
|
|
|
|
err = i2c_detect_address(temp_client,
|
|
|
|
kind, driver);
|
|
|
|
if (err)
|
|
|
|
goto exit_free;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Stop here if we can't use SMBUS_QUICK */
|
|
|
|
if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_QUICK)) {
|
|
|
|
if (address_data->probe[0] == I2C_CLIENT_END
|
|
|
|
&& address_data->normal_i2c[0] == I2C_CLIENT_END)
|
|
|
|
goto exit_free;
|
|
|
|
|
|
|
|
dev_warn(&adapter->dev, "SMBus Quick command not supported, "
|
|
|
|
"can't probe for chips\n");
|
|
|
|
err = -EOPNOTSUPP;
|
|
|
|
goto exit_free;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Stop here if the classes do not match */
|
|
|
|
if (!(adapter->class & driver->class))
|
|
|
|
goto exit_free;
|
|
|
|
|
|
|
|
/* Probe entries are done second, and are not affected by ignore
|
|
|
|
entries either */
|
|
|
|
for (i = 0; address_data->probe[i] != I2C_CLIENT_END; i += 2) {
|
|
|
|
if (address_data->probe[i] == adap_id
|
|
|
|
|| address_data->probe[i] == ANY_I2C_BUS) {
|
|
|
|
dev_dbg(&adapter->dev, "found probe parameter for "
|
|
|
|
"adapter %d, addr 0x%02x\n", adap_id,
|
|
|
|
address_data->probe[i + 1]);
|
|
|
|
temp_client->addr = address_data->probe[i + 1];
|
|
|
|
err = i2c_detect_address(temp_client, -1, driver);
|
|
|
|
if (err)
|
|
|
|
goto exit_free;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Normal entries are done last, unless shadowed by an ignore entry */
|
|
|
|
for (i = 0; address_data->normal_i2c[i] != I2C_CLIENT_END; i += 1) {
|
|
|
|
int j, ignore;
|
|
|
|
|
|
|
|
ignore = 0;
|
|
|
|
for (j = 0; address_data->ignore[j] != I2C_CLIENT_END;
|
|
|
|
j += 2) {
|
|
|
|
if ((address_data->ignore[j] == adap_id ||
|
|
|
|
address_data->ignore[j] == ANY_I2C_BUS)
|
|
|
|
&& address_data->ignore[j + 1]
|
|
|
|
== address_data->normal_i2c[i]) {
|
|
|
|
dev_dbg(&adapter->dev, "found ignore "
|
|
|
|
"parameter for adapter %d, "
|
|
|
|
"addr 0x%02x\n", adap_id,
|
|
|
|
address_data->ignore[j + 1]);
|
|
|
|
ignore = 1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (ignore)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
dev_dbg(&adapter->dev, "found normal entry for adapter %d, "
|
|
|
|
"addr 0x%02x\n", adap_id,
|
|
|
|
address_data->normal_i2c[i]);
|
|
|
|
temp_client->addr = address_data->normal_i2c[i];
|
|
|
|
err = i2c_detect_address(temp_client, -1, driver);
|
|
|
|
if (err)
|
|
|
|
goto exit_free;
|
|
|
|
}
|
|
|
|
|
|
|
|
exit_free:
|
|
|
|
kfree(temp_client);
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
struct i2c_client *
|
|
|
|
i2c_new_probed_device(struct i2c_adapter *adap,
|
|
|
|
struct i2c_board_info *info,
|
|
|
|
unsigned short const *addr_list)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
/* Stop here if the bus doesn't support probing */
|
|
|
|
if (!i2c_check_functionality(adap, I2C_FUNC_SMBUS_READ_BYTE)) {
|
|
|
|
dev_err(&adap->dev, "Probing not supported\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; addr_list[i] != I2C_CLIENT_END; i++) {
|
|
|
|
/* Check address validity */
|
|
|
|
if (addr_list[i] < 0x03 || addr_list[i] > 0x77) {
|
|
|
|
dev_warn(&adap->dev, "Invalid 7-bit address "
|
|
|
|
"0x%02x\n", addr_list[i]);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Check address availability */
|
2008-01-27 18:14:51 +01:00
|
|
|
if (i2c_check_addr(adap, addr_list[i])) {
|
2007-05-01 23:26:31 +02:00
|
|
|
dev_dbg(&adap->dev, "Address 0x%02x already in "
|
|
|
|
"use, not probing\n", addr_list[i]);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Test address responsiveness
|
|
|
|
The default probe method is a quick write, but it is known
|
|
|
|
to corrupt the 24RF08 EEPROMs due to a state machine bug,
|
|
|
|
and could also irreversibly write-protect some EEPROMs, so
|
|
|
|
for address ranges 0x30-0x37 and 0x50-0x5f, we use a byte
|
|
|
|
read instead. Also, some bus drivers don't implement
|
|
|
|
quick write, so we fallback to a byte read it that case
|
|
|
|
too. */
|
|
|
|
if ((addr_list[i] & ~0x07) == 0x30
|
|
|
|
|| (addr_list[i] & ~0x0f) == 0x50
|
|
|
|
|| !i2c_check_functionality(adap, I2C_FUNC_SMBUS_QUICK)) {
|
2008-08-10 22:56:15 +02:00
|
|
|
union i2c_smbus_data data;
|
|
|
|
|
2007-05-01 23:26:31 +02:00
|
|
|
if (i2c_smbus_xfer(adap, addr_list[i], 0,
|
|
|
|
I2C_SMBUS_READ, 0,
|
2008-08-10 22:56:15 +02:00
|
|
|
I2C_SMBUS_BYTE, &data) >= 0)
|
2007-05-01 23:26:31 +02:00
|
|
|
break;
|
|
|
|
} else {
|
|
|
|
if (i2c_smbus_xfer(adap, addr_list[i], 0,
|
|
|
|
I2C_SMBUS_WRITE, 0,
|
|
|
|
I2C_SMBUS_QUICK, NULL) >= 0)
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (addr_list[i] == I2C_CLIENT_END) {
|
|
|
|
dev_dbg(&adap->dev, "Probing failed, no device found\n");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
info->addr = addr_list[i];
|
|
|
|
return i2c_new_device(adap, info);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(i2c_new_probed_device);
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
struct i2c_adapter* i2c_get_adapter(int id)
|
|
|
|
{
|
|
|
|
struct i2c_adapter *adapter;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_lock(&core_lock);
|
2005-06-28 06:21:30 +02:00
|
|
|
adapter = (struct i2c_adapter *)idr_find(&i2c_adapter_idr, id);
|
|
|
|
if (adapter && !try_module_get(adapter->owner))
|
|
|
|
adapter = NULL;
|
|
|
|
|
2008-01-27 18:14:49 +01:00
|
|
|
mutex_unlock(&core_lock);
|
2005-06-28 06:21:30 +02:00
|
|
|
return adapter;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_get_adapter);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
void i2c_put_adapter(struct i2c_adapter *adap)
|
|
|
|
{
|
|
|
|
module_put(adap->owner);
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_put_adapter);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* The SMBus parts */
|
|
|
|
|
2006-12-10 21:21:31 +01:00
|
|
|
#define POLY (0x1070U << 3)
|
2005-04-17 00:20:36 +02:00
|
|
|
static u8
|
|
|
|
crc8(u16 data)
|
|
|
|
{
|
|
|
|
int i;
|
2006-12-10 21:21:31 +01:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
for(i = 0; i < 8; i++) {
|
2006-12-10 21:21:31 +01:00
|
|
|
if (data & 0x8000)
|
2005-04-17 00:20:36 +02:00
|
|
|
data = data ^ POLY;
|
|
|
|
data = data << 1;
|
|
|
|
}
|
|
|
|
return (u8)(data >> 8);
|
|
|
|
}
|
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* Incremental CRC8 over count bytes in the array pointed to by p */
|
|
|
|
static u8 i2c_smbus_pec(u8 crc, u8 *p, size_t count)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for(i = 0; i < count; i++)
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
crc = crc8((crc ^ p[i]) << 8);
|
2005-04-17 00:20:36 +02:00
|
|
|
return crc;
|
|
|
|
}
|
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* Assume a 7-bit address, which is reasonable for SMBus */
|
|
|
|
static u8 i2c_smbus_msg_pec(u8 pec, struct i2c_msg *msg)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* The address will be sent first */
|
|
|
|
u8 addr = (msg->addr << 1) | !!(msg->flags & I2C_M_RD);
|
|
|
|
pec = i2c_smbus_pec(pec, &addr, 1);
|
|
|
|
|
|
|
|
/* The data buffer follows */
|
|
|
|
return i2c_smbus_pec(pec, msg->buf, msg->len);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* Used for write only transactions */
|
|
|
|
static inline void i2c_smbus_add_pec(struct i2c_msg *msg)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
msg->buf[msg->len] = i2c_smbus_msg_pec(0, msg);
|
|
|
|
msg->len++;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* Return <0 on CRC error
|
|
|
|
If there was a write before this read (most cases) we need to take the
|
|
|
|
partial CRC from the write part into account.
|
|
|
|
Note that this function does modify the message (we need to decrease the
|
|
|
|
message length to hide the CRC byte from the caller). */
|
|
|
|
static int i2c_smbus_check_pec(u8 cpec, struct i2c_msg *msg)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
u8 rpec = msg->buf[--msg->len];
|
|
|
|
cpec = i2c_smbus_msg_pec(cpec, msg);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
if (rpec != cpec) {
|
|
|
|
pr_debug("i2c-core: Bad PEC 0x%02x vs. 0x%02x\n",
|
|
|
|
rpec, cpec);
|
2008-07-14 22:38:23 +02:00
|
|
|
return -EBADMSG;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2006-12-10 21:21:31 +01:00
|
|
|
return 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_read_byte - SMBus "receive byte" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
*
|
|
|
|
* This executes the SMBus "receive byte" protocol, returning negative errno
|
|
|
|
* else the byte received from the device.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_read_byte(struct i2c_client *client)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
|
|
|
|
|
|
|
status = i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_READ, 0,
|
|
|
|
I2C_SMBUS_BYTE, &data);
|
|
|
|
return (status < 0) ? status : data.byte;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_read_byte);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_write_byte - SMBus "send byte" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @value: Byte to be sent
|
|
|
|
*
|
|
|
|
* This executes the SMBus "send byte" protocol, returning negative errno
|
|
|
|
* else zero on success.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_write_byte(struct i2c_client *client, u8 value)
|
|
|
|
{
|
|
|
|
return i2c_smbus_xfer(client->adapter,client->addr,client->flags,
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
I2C_SMBUS_WRITE, value, I2C_SMBUS_BYTE, NULL);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_write_byte);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_read_byte_data - SMBus "read byte" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @command: Byte interpreted by slave
|
|
|
|
*
|
|
|
|
* This executes the SMBus "read byte" protocol, returning negative errno
|
|
|
|
* else a data byte received from the device.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_read_byte_data(struct i2c_client *client, u8 command)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
|
|
|
|
|
|
|
status = i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_READ, command,
|
|
|
|
I2C_SMBUS_BYTE_DATA, &data);
|
|
|
|
return (status < 0) ? status : data.byte;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_read_byte_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_write_byte_data - SMBus "write byte" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @command: Byte interpreted by slave
|
|
|
|
* @value: Byte being written
|
|
|
|
*
|
|
|
|
* This executes the SMBus "write byte" protocol, returning negative errno
|
|
|
|
* else zero on success.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_write_byte_data(struct i2c_client *client, u8 command, u8 value)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
|
|
|
data.byte = value;
|
|
|
|
return i2c_smbus_xfer(client->adapter,client->addr,client->flags,
|
|
|
|
I2C_SMBUS_WRITE,command,
|
|
|
|
I2C_SMBUS_BYTE_DATA,&data);
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_write_byte_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_read_word_data - SMBus "read word" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @command: Byte interpreted by slave
|
|
|
|
*
|
|
|
|
* This executes the SMBus "read word" protocol, returning negative errno
|
|
|
|
* else a 16-bit unsigned "word" received from the device.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_read_word_data(struct i2c_client *client, u8 command)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
|
|
|
|
|
|
|
status = i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_READ, command,
|
|
|
|
I2C_SMBUS_WORD_DATA, &data);
|
|
|
|
return (status < 0) ? status : data.word;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_read_word_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_write_word_data - SMBus "write word" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @command: Byte interpreted by slave
|
|
|
|
* @value: 16-bit "word" being written
|
|
|
|
*
|
|
|
|
* This executes the SMBus "write word" protocol, returning negative errno
|
|
|
|
* else zero on success.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_write_word_data(struct i2c_client *client, u8 command, u16 value)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
|
|
|
data.word = value;
|
|
|
|
return i2c_smbus_xfer(client->adapter,client->addr,client->flags,
|
|
|
|
I2C_SMBUS_WRITE,command,
|
|
|
|
I2C_SMBUS_WORD_DATA,&data);
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_write_word_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2007-10-13 23:56:31 +02:00
|
|
|
/**
|
2008-07-14 22:38:24 +02:00
|
|
|
* i2c_smbus_read_block_data - SMBus "block read" protocol
|
2007-10-13 23:56:31 +02:00
|
|
|
* @client: Handle to slave device
|
2008-07-14 22:38:24 +02:00
|
|
|
* @command: Byte interpreted by slave
|
2007-10-13 23:56:31 +02:00
|
|
|
* @values: Byte array into which data will be read; big enough to hold
|
|
|
|
* the data returned by the slave. SMBus allows at most 32 bytes.
|
|
|
|
*
|
2008-07-14 22:38:24 +02:00
|
|
|
* This executes the SMBus "block read" protocol, returning negative errno
|
|
|
|
* else the number of data bytes in the slave's response.
|
2007-10-13 23:56:31 +02:00
|
|
|
*
|
|
|
|
* Note that using this function requires that the client's adapter support
|
|
|
|
* the I2C_FUNC_SMBUS_READ_BLOCK_DATA functionality. Not all adapter drivers
|
|
|
|
* support this; its emulation through I2C messaging relies on a specific
|
|
|
|
* mechanism (I2C_M_RECV_LEN) which may not be implemented.
|
|
|
|
*/
|
2007-05-01 23:26:34 +02:00
|
|
|
s32 i2c_smbus_read_block_data(struct i2c_client *client, u8 command,
|
|
|
|
u8 *values)
|
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
2007-05-01 23:26:34 +02:00
|
|
|
|
2008-07-14 22:38:23 +02:00
|
|
|
status = i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_READ, command,
|
|
|
|
I2C_SMBUS_BLOCK_DATA, &data);
|
|
|
|
if (status)
|
|
|
|
return status;
|
2007-05-01 23:26:34 +02:00
|
|
|
|
|
|
|
memcpy(values, &data.block[1], data.block[0]);
|
|
|
|
return data.block[0];
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(i2c_smbus_read_block_data);
|
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_write_block_data - SMBus "block write" protocol
|
|
|
|
* @client: Handle to slave device
|
|
|
|
* @command: Byte interpreted by slave
|
|
|
|
* @length: Size of data block; SMBus allows at most 32 bytes
|
|
|
|
* @values: Byte array which will be written.
|
|
|
|
*
|
|
|
|
* This executes the SMBus "block write" protocol, returning negative errno
|
|
|
|
* else zero on success.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_write_block_data(struct i2c_client *client, u8 command,
|
2006-06-12 21:42:20 +02:00
|
|
|
u8 length, const u8 *values)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2006-01-18 23:14:55 +01:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
if (length > I2C_SMBUS_BLOCK_MAX)
|
|
|
|
length = I2C_SMBUS_BLOCK_MAX;
|
|
|
|
data.block[0] = length;
|
2006-01-18 23:14:55 +01:00
|
|
|
memcpy(&data.block[1], values, length);
|
2005-04-17 00:20:36 +02:00
|
|
|
return i2c_smbus_xfer(client->adapter,client->addr,client->flags,
|
|
|
|
I2C_SMBUS_WRITE,command,
|
|
|
|
I2C_SMBUS_BLOCK_DATA,&data);
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_write_block_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* Returns the number of read bytes */
|
i2c: Fix the i2c_smbus_read_i2c_block_data() prototype
Let the drivers specify how many bytes they want to read with
i2c_smbus_read_i2c_block_data(). So far, the block count was
hard-coded to I2C_SMBUS_BLOCK_MAX (32), which did not make much sense.
Many driver authors complained about this before, and I believe it's
about time to fix it. Right now, authors have to do technically stupid
things, such as individual byte reads or full-fledged I2C messaging,
to work around the problem. We do not want to encourage that.
I even found that some bus drivers (e.g. i2c-amd8111) already
implemented I2C block read the "right" way, that is, they didn't
follow the old, broken standard. The fact that it was never noticed
before just shows how little i2c_smbus_read_i2c_block_data() was used,
which isn't that surprising given how broken its prototype was so far.
There are some obvious compatiblity considerations:
* This changes the i2c_smbus_read_i2c_block_data() prototype. Users
outside the kernel tree will notice at compilation time, and will
have to update their code.
* User-space has access to i2c_smbus_xfer() directly using i2c-dev, so
the changed expectations would affect tools such as i2cdump. In order
to preserve binary compatibility, we give I2C_SMBUS_I2C_BLOCK_DATA
a new numeric value, and define I2C_SMBUS_I2C_BLOCK_BROKEN with the
old numeric value. When i2c-dev receives a transaction with the
old value, it can convert it to the new format on the fly.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-07-12 14:12:29 +02:00
|
|
|
s32 i2c_smbus_read_i2c_block_data(struct i2c_client *client, u8 command,
|
|
|
|
u8 length, u8 *values)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
2006-01-18 23:14:55 +01:00
|
|
|
|
i2c: Fix the i2c_smbus_read_i2c_block_data() prototype
Let the drivers specify how many bytes they want to read with
i2c_smbus_read_i2c_block_data(). So far, the block count was
hard-coded to I2C_SMBUS_BLOCK_MAX (32), which did not make much sense.
Many driver authors complained about this before, and I believe it's
about time to fix it. Right now, authors have to do technically stupid
things, such as individual byte reads or full-fledged I2C messaging,
to work around the problem. We do not want to encourage that.
I even found that some bus drivers (e.g. i2c-amd8111) already
implemented I2C block read the "right" way, that is, they didn't
follow the old, broken standard. The fact that it was never noticed
before just shows how little i2c_smbus_read_i2c_block_data() was used,
which isn't that surprising given how broken its prototype was so far.
There are some obvious compatiblity considerations:
* This changes the i2c_smbus_read_i2c_block_data() prototype. Users
outside the kernel tree will notice at compilation time, and will
have to update their code.
* User-space has access to i2c_smbus_xfer() directly using i2c-dev, so
the changed expectations would affect tools such as i2cdump. In order
to preserve binary compatibility, we give I2C_SMBUS_I2C_BLOCK_DATA
a new numeric value, and define I2C_SMBUS_I2C_BLOCK_BROKEN with the
old numeric value. When i2c-dev receives a transaction with the
old value, it can convert it to the new format on the fly.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-07-12 14:12:29 +02:00
|
|
|
if (length > I2C_SMBUS_BLOCK_MAX)
|
|
|
|
length = I2C_SMBUS_BLOCK_MAX;
|
|
|
|
data.block[0] = length;
|
2008-07-14 22:38:23 +02:00
|
|
|
status = i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_READ, command,
|
|
|
|
I2C_SMBUS_I2C_BLOCK_DATA, &data);
|
|
|
|
if (status < 0)
|
|
|
|
return status;
|
2006-01-18 23:14:55 +01:00
|
|
|
|
|
|
|
memcpy(values, &data.block[1], data.block[0]);
|
|
|
|
return data.block[0];
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_read_i2c_block_data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2006-01-09 05:19:18 +01:00
|
|
|
s32 i2c_smbus_write_i2c_block_data(struct i2c_client *client, u8 command,
|
2006-06-12 21:42:20 +02:00
|
|
|
u8 length, const u8 *values)
|
2006-01-09 05:19:18 +01:00
|
|
|
{
|
|
|
|
union i2c_smbus_data data;
|
|
|
|
|
|
|
|
if (length > I2C_SMBUS_BLOCK_MAX)
|
|
|
|
length = I2C_SMBUS_BLOCK_MAX;
|
|
|
|
data.block[0] = length;
|
|
|
|
memcpy(data.block + 1, values, length);
|
|
|
|
return i2c_smbus_xfer(client->adapter, client->addr, client->flags,
|
|
|
|
I2C_SMBUS_WRITE, command,
|
|
|
|
I2C_SMBUS_I2C_BLOCK_DATA, &data);
|
|
|
|
}
|
2007-05-01 23:26:31 +02:00
|
|
|
EXPORT_SYMBOL(i2c_smbus_write_i2c_block_data);
|
2006-01-09 05:19:18 +01:00
|
|
|
|
2006-12-10 21:21:31 +01:00
|
|
|
/* Simulate a SMBus command using the i2c protocol
|
2005-04-17 00:20:36 +02:00
|
|
|
No checking of parameters is done! */
|
2006-12-10 21:21:31 +01:00
|
|
|
static s32 i2c_smbus_xfer_emulated(struct i2c_adapter * adapter, u16 addr,
|
2005-04-17 00:20:36 +02:00
|
|
|
unsigned short flags,
|
2006-12-10 21:21:31 +01:00
|
|
|
char read_write, u8 command, int size,
|
2005-04-17 00:20:36 +02:00
|
|
|
union i2c_smbus_data * data)
|
|
|
|
{
|
|
|
|
/* So we need to generate a series of msgs. In the case of writing, we
|
|
|
|
need to use only one message; when reading, we need two. We initialize
|
|
|
|
most things with sane defaults, to keep the code below somewhat
|
|
|
|
simpler. */
|
2005-09-25 17:01:11 +02:00
|
|
|
unsigned char msgbuf0[I2C_SMBUS_BLOCK_MAX+3];
|
|
|
|
unsigned char msgbuf1[I2C_SMBUS_BLOCK_MAX+2];
|
2005-04-17 00:20:36 +02:00
|
|
|
int num = read_write == I2C_SMBUS_READ?2:1;
|
2006-12-10 21:21:31 +01:00
|
|
|
struct i2c_msg msg[2] = { { addr, flags, 1, msgbuf0 },
|
2005-04-17 00:20:36 +02:00
|
|
|
{ addr, flags | I2C_M_RD, 0, msgbuf1 }
|
|
|
|
};
|
|
|
|
int i;
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
u8 partial_pec = 0;
|
2008-07-14 22:38:23 +02:00
|
|
|
int status;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
msgbuf0[0] = command;
|
|
|
|
switch(size) {
|
|
|
|
case I2C_SMBUS_QUICK:
|
|
|
|
msg[0].len = 0;
|
|
|
|
/* Special case: The read/write field is used as data */
|
|
|
|
msg[0].flags = flags | (read_write==I2C_SMBUS_READ)?I2C_M_RD:0;
|
|
|
|
num = 1;
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_BYTE:
|
|
|
|
if (read_write == I2C_SMBUS_READ) {
|
|
|
|
/* Special case: only a read! */
|
|
|
|
msg[0].flags = I2C_M_RD | flags;
|
|
|
|
num = 1;
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_BYTE_DATA:
|
|
|
|
if (read_write == I2C_SMBUS_READ)
|
|
|
|
msg[1].len = 1;
|
|
|
|
else {
|
|
|
|
msg[0].len = 2;
|
|
|
|
msgbuf0[1] = data->byte;
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_WORD_DATA:
|
|
|
|
if (read_write == I2C_SMBUS_READ)
|
|
|
|
msg[1].len = 2;
|
|
|
|
else {
|
|
|
|
msg[0].len=3;
|
|
|
|
msgbuf0[1] = data->word & 0xff;
|
2006-09-03 22:24:00 +02:00
|
|
|
msgbuf0[2] = data->word >> 8;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_PROC_CALL:
|
|
|
|
num = 2; /* Special case */
|
|
|
|
read_write = I2C_SMBUS_READ;
|
|
|
|
msg[0].len = 3;
|
|
|
|
msg[1].len = 2;
|
|
|
|
msgbuf0[1] = data->word & 0xff;
|
2006-09-03 22:24:00 +02:00
|
|
|
msgbuf0[2] = data->word >> 8;
|
2005-04-17 00:20:36 +02:00
|
|
|
break;
|
|
|
|
case I2C_SMBUS_BLOCK_DATA:
|
|
|
|
if (read_write == I2C_SMBUS_READ) {
|
2007-05-01 23:26:29 +02:00
|
|
|
msg[1].flags |= I2C_M_RECV_LEN;
|
|
|
|
msg[1].len = 1; /* block length will be added by
|
|
|
|
the underlying bus driver */
|
2005-04-17 00:20:36 +02:00
|
|
|
} else {
|
|
|
|
msg[0].len = data->block[0] + 2;
|
|
|
|
if (msg[0].len > I2C_SMBUS_BLOCK_MAX + 2) {
|
2008-07-14 22:38:23 +02:00
|
|
|
dev_err(&adapter->dev,
|
|
|
|
"Invalid block write size %d\n",
|
|
|
|
data->block[0]);
|
|
|
|
return -EINVAL;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2005-09-25 17:01:11 +02:00
|
|
|
for (i = 1; i < msg[0].len; i++)
|
2005-04-17 00:20:36 +02:00
|
|
|
msgbuf0[i] = data->block[i-1];
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_BLOCK_PROC_CALL:
|
2007-05-01 23:26:29 +02:00
|
|
|
num = 2; /* Another special case */
|
|
|
|
read_write = I2C_SMBUS_READ;
|
|
|
|
if (data->block[0] > I2C_SMBUS_BLOCK_MAX) {
|
2008-07-14 22:38:23 +02:00
|
|
|
dev_err(&adapter->dev,
|
|
|
|
"Invalid block write size %d\n",
|
2007-05-01 23:26:29 +02:00
|
|
|
data->block[0]);
|
2008-07-14 22:38:23 +02:00
|
|
|
return -EINVAL;
|
2007-05-01 23:26:29 +02:00
|
|
|
}
|
|
|
|
msg[0].len = data->block[0] + 2;
|
|
|
|
for (i = 1; i < msg[0].len; i++)
|
|
|
|
msgbuf0[i] = data->block[i-1];
|
|
|
|
msg[1].flags |= I2C_M_RECV_LEN;
|
|
|
|
msg[1].len = 1; /* block length will be added by
|
|
|
|
the underlying bus driver */
|
|
|
|
break;
|
2005-04-17 00:20:36 +02:00
|
|
|
case I2C_SMBUS_I2C_BLOCK_DATA:
|
|
|
|
if (read_write == I2C_SMBUS_READ) {
|
i2c: Fix the i2c_smbus_read_i2c_block_data() prototype
Let the drivers specify how many bytes they want to read with
i2c_smbus_read_i2c_block_data(). So far, the block count was
hard-coded to I2C_SMBUS_BLOCK_MAX (32), which did not make much sense.
Many driver authors complained about this before, and I believe it's
about time to fix it. Right now, authors have to do technically stupid
things, such as individual byte reads or full-fledged I2C messaging,
to work around the problem. We do not want to encourage that.
I even found that some bus drivers (e.g. i2c-amd8111) already
implemented I2C block read the "right" way, that is, they didn't
follow the old, broken standard. The fact that it was never noticed
before just shows how little i2c_smbus_read_i2c_block_data() was used,
which isn't that surprising given how broken its prototype was so far.
There are some obvious compatiblity considerations:
* This changes the i2c_smbus_read_i2c_block_data() prototype. Users
outside the kernel tree will notice at compilation time, and will
have to update their code.
* User-space has access to i2c_smbus_xfer() directly using i2c-dev, so
the changed expectations would affect tools such as i2cdump. In order
to preserve binary compatibility, we give I2C_SMBUS_I2C_BLOCK_DATA
a new numeric value, and define I2C_SMBUS_I2C_BLOCK_BROKEN with the
old numeric value. When i2c-dev receives a transaction with the
old value, it can convert it to the new format on the fly.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-07-12 14:12:29 +02:00
|
|
|
msg[1].len = data->block[0];
|
2005-04-17 00:20:36 +02:00
|
|
|
} else {
|
|
|
|
msg[0].len = data->block[0] + 1;
|
2005-10-08 00:15:59 +02:00
|
|
|
if (msg[0].len > I2C_SMBUS_BLOCK_MAX + 1) {
|
2008-07-14 22:38:23 +02:00
|
|
|
dev_err(&adapter->dev,
|
|
|
|
"Invalid block write size %d\n",
|
|
|
|
data->block[0]);
|
|
|
|
return -EINVAL;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
for (i = 1; i <= data->block[0]; i++)
|
|
|
|
msgbuf0[i] = data->block[i];
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
default:
|
2008-07-14 22:38:23 +02:00
|
|
|
dev_err(&adapter->dev, "Unsupported transaction %d\n", size);
|
|
|
|
return -EOPNOTSUPP;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
i = ((flags & I2C_CLIENT_PEC) && size != I2C_SMBUS_QUICK
|
|
|
|
&& size != I2C_SMBUS_I2C_BLOCK_DATA);
|
|
|
|
if (i) {
|
|
|
|
/* Compute PEC if first message is a write */
|
|
|
|
if (!(msg[0].flags & I2C_M_RD)) {
|
2006-12-10 21:21:31 +01:00
|
|
|
if (num == 1) /* Write only */
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
i2c_smbus_add_pec(&msg[0]);
|
|
|
|
else /* Write followed by read */
|
|
|
|
partial_pec = i2c_smbus_msg_pec(0, &msg[0]);
|
|
|
|
}
|
|
|
|
/* Ask for PEC if last message is a read */
|
|
|
|
if (msg[num-1].flags & I2C_M_RD)
|
2006-12-10 21:21:31 +01:00
|
|
|
msg[num-1].len++;
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
}
|
|
|
|
|
2008-07-14 22:38:23 +02:00
|
|
|
status = i2c_transfer(adapter, msg, num);
|
|
|
|
if (status < 0)
|
|
|
|
return status;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
/* Check PEC if last message is a read */
|
|
|
|
if (i && (msg[num-1].flags & I2C_M_RD)) {
|
2008-07-14 22:38:23 +02:00
|
|
|
status = i2c_smbus_check_pec(partial_pec, &msg[num-1]);
|
|
|
|
if (status < 0)
|
|
|
|
return status;
|
[PATCH] i2c: SMBus PEC support rewrite, 2 of 3
This is my rewrite of the SMBus PEC support. The original
implementation was known to have bugs (credits go to Hideki Iwamoto
for reporting many of them recently), and was incomplete due to a
conceptual limitation.
The rewrite affects only software PEC. Hardware PEC needs very little
code and is mostly untouched.
Technically, both implementations differ in that the original one
was emulating PEC in software by modifying the contents of an
i2c_smbus_data union (changing the transaction to a different type),
while the new one works one level lower, on i2c_msg structures (working
on message contents). Due to the definition of the i2c_smbus_data union,
not all SMBus transactions could be handled (at least not without
changing the definition of this union, which would break user-space
compatibility), and those which could had to be implemented
individually. At the opposite, adding PEC to an i2c_msg structure
can be done on any SMBus transaction with common code.
Advantages of the new implementation:
* It's about twice as small (from ~136 lines before to ~70 now, only
counting i2c-core, including blank and comment lines). The memory
used by i2c-core is down by ~640 bytes (~3.5%).
* Easier to validate, less tricky code. The code being common to all
transactions by design, the risk that a bug can stay uncovered is
lower.
* All SMBus transactions have PEC support in I2C emulation mode
(providing the non-PEC transaction is also implemented). Transactions
which have no emulation code right now will get PEC support for free
when they finally get implemented.
* Allows for code simplifications in header files and bus drivers
(patch follows).
Drawbacks (I guess there had to be at least one):
* PEC emulation for non-PEC capable non-I2C SMBus masters was dropped.
It was based on SMBus tricks and doesn't quite fit in the new design.
I don't think it's really a problem, as the benefit was certainly
not worth the additional complexity, but it's only fair that I at
least mention it.
Lastly, let's note that the new implementation does slightly affect
compatibility (both in kernel and user-space), but doesn't actually
break it. Some defines will be dropped, but the code can always be
changed in a way that will work with both the old and the new
implementations. It shouldn't be a problem as there doesn't seem to be
many users of SMBus PEC to date anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2005-10-26 21:28:55 +02:00
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
if (read_write == I2C_SMBUS_READ)
|
|
|
|
switch(size) {
|
|
|
|
case I2C_SMBUS_BYTE:
|
|
|
|
data->byte = msgbuf0[0];
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_BYTE_DATA:
|
|
|
|
data->byte = msgbuf1[0];
|
|
|
|
break;
|
2006-12-10 21:21:31 +01:00
|
|
|
case I2C_SMBUS_WORD_DATA:
|
2005-04-17 00:20:36 +02:00
|
|
|
case I2C_SMBUS_PROC_CALL:
|
|
|
|
data->word = msgbuf1[0] | (msgbuf1[1] << 8);
|
|
|
|
break;
|
|
|
|
case I2C_SMBUS_I2C_BLOCK_DATA:
|
i2c: Fix the i2c_smbus_read_i2c_block_data() prototype
Let the drivers specify how many bytes they want to read with
i2c_smbus_read_i2c_block_data(). So far, the block count was
hard-coded to I2C_SMBUS_BLOCK_MAX (32), which did not make much sense.
Many driver authors complained about this before, and I believe it's
about time to fix it. Right now, authors have to do technically stupid
things, such as individual byte reads or full-fledged I2C messaging,
to work around the problem. We do not want to encourage that.
I even found that some bus drivers (e.g. i2c-amd8111) already
implemented I2C block read the "right" way, that is, they didn't
follow the old, broken standard. The fact that it was never noticed
before just shows how little i2c_smbus_read_i2c_block_data() was used,
which isn't that surprising given how broken its prototype was so far.
There are some obvious compatiblity considerations:
* This changes the i2c_smbus_read_i2c_block_data() prototype. Users
outside the kernel tree will notice at compilation time, and will
have to update their code.
* User-space has access to i2c_smbus_xfer() directly using i2c-dev, so
the changed expectations would affect tools such as i2cdump. In order
to preserve binary compatibility, we give I2C_SMBUS_I2C_BLOCK_DATA
a new numeric value, and define I2C_SMBUS_I2C_BLOCK_BROKEN with the
old numeric value. When i2c-dev receives a transaction with the
old value, it can convert it to the new format on the fly.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
2007-07-12 14:12:29 +02:00
|
|
|
for (i = 0; i < data->block[0]; i++)
|
2005-04-17 00:20:36 +02:00
|
|
|
data->block[i+1] = msgbuf1[i];
|
|
|
|
break;
|
2007-05-01 23:26:29 +02:00
|
|
|
case I2C_SMBUS_BLOCK_DATA:
|
|
|
|
case I2C_SMBUS_BLOCK_PROC_CALL:
|
|
|
|
for (i = 0; i < msgbuf1[0] + 1; i++)
|
|
|
|
data->block[i] = msgbuf1[i];
|
|
|
|
break;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2008-07-14 22:38:24 +02:00
|
|
|
/**
|
|
|
|
* i2c_smbus_xfer - execute SMBus protocol operations
|
|
|
|
* @adapter: Handle to I2C bus
|
|
|
|
* @addr: Address of SMBus slave on that bus
|
|
|
|
* @flags: I2C_CLIENT_* flags (usually zero or I2C_CLIENT_PEC)
|
|
|
|
* @read_write: I2C_SMBUS_READ or I2C_SMBUS_WRITE
|
|
|
|
* @command: Byte interpreted by slave, for protocols which use such bytes
|
|
|
|
* @protocol: SMBus protocol operation to execute, such as I2C_SMBUS_PROC_CALL
|
|
|
|
* @data: Data to be read or written
|
|
|
|
*
|
|
|
|
* This executes an SMBus protocol operation, and returns a negative
|
|
|
|
* errno code else zero on success.
|
|
|
|
*/
|
2005-04-17 00:20:36 +02:00
|
|
|
s32 i2c_smbus_xfer(struct i2c_adapter * adapter, u16 addr, unsigned short flags,
|
2008-07-14 22:38:24 +02:00
|
|
|
char read_write, u8 command, int protocol,
|
2005-04-17 00:20:36 +02:00
|
|
|
union i2c_smbus_data * data)
|
|
|
|
{
|
|
|
|
s32 res;
|
|
|
|
|
|
|
|
flags &= I2C_M_TEN | I2C_CLIENT_PEC;
|
|
|
|
|
|
|
|
if (adapter->algo->smbus_xfer) {
|
2006-01-18 23:16:04 +01:00
|
|
|
mutex_lock(&adapter->bus_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
res = adapter->algo->smbus_xfer(adapter,addr,flags,read_write,
|
2008-07-14 22:38:24 +02:00
|
|
|
command, protocol, data);
|
2006-01-18 23:16:04 +01:00
|
|
|
mutex_unlock(&adapter->bus_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
} else
|
|
|
|
res = i2c_smbus_xfer_emulated(adapter,addr,flags,read_write,
|
2008-07-14 22:38:24 +02:00
|
|
|
command, protocol, data);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
return res;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(i2c_smbus_xfer);
|
|
|
|
|
|
|
|
MODULE_AUTHOR("Simon G. Vogl <simon@tk.uni-linz.ac.at>");
|
|
|
|
MODULE_DESCRIPTION("I2C-Bus main module");
|
|
|
|
MODULE_LICENSE("GPL");
|