lkml.org 
[lkml]   [2007]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [2.6 patch] drivers/i2c/i2c-core.c: make code static
Hi Adrian,

On Sun, 29 Jul 2007 16:57:08 +0200, Adrian Bunk wrote:
> After the i2c-isa removal some code can become static.
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
>
> ---
>
> drivers/i2c/i2c-core.c | 7 +++----
> include/linux/i2c.h | 2 --
> 2 files changed, 3 insertions(+), 6 deletions(-)
>
> --- linux-2.6.23-rc1-mm1/include/linux/i2c.h.old 2007-07-26 12:46:44.000000000 +0200
> +++ linux-2.6.23-rc1-mm1/include/linux/i2c.h 2007-07-26 12:46:53.000000000 +0200
> @@ -35,8 +35,6 @@
> #include <linux/sched.h> /* for completion */
> #include <linux/mutex.h>
>
> -extern struct bus_type i2c_bus_type;
> -
> /* --- General options ------------------------------------------------ */
>
> struct i2c_msg;
> --- linux-2.6.23-rc1-mm1/drivers/i2c/i2c-core.c.old 2007-07-26 08:59:44.000000000 +0200
> +++ linux-2.6.23-rc1-mm1/drivers/i2c/i2c-core.c 2007-07-26 09:01:42.000000000 +0200
> @@ -190,7 +190,7 @@
> { },
> };
>
> -struct bus_type i2c_bus_type = {
> +static struct bus_type i2c_bus_type = {
> .name = "i2c",
> .dev_attrs = i2c_dev_attrs,
> .match = i2c_device_match,
> @@ -201,7 +201,6 @@
> .suspend = i2c_device_suspend,
> .resume = i2c_device_resume,
> };
> -EXPORT_SYMBOL_GPL(i2c_bus_type);

This one was left exported on request by David Brownell. See this
discussion thread:
http://lists.lm-sensors.org/pipermail/i2c/2007-March/000924.html

I am personally fine with removing this export, as it can always be
added later if needed. But if others share David's point of view, I am
also fine leaving the export as is.

>
> /**
> * i2c_new_device - instantiate an i2c device for use with a new style driver
> @@ -285,7 +284,7 @@
>
> /* I2C bus adapters -- one roots each I2C or SMBUS segment */
>
> -void i2c_adapter_dev_release(struct device *dev)
> +static void i2c_adapter_dev_release(struct device *dev)
> {
> struct i2c_adapter *adap = to_i2c_adapter(dev);
> complete(&adap->dev_released);
> @@ -303,7 +302,7 @@
> { },
> };
>
> -struct class i2c_adapter_class = {
> +static struct class i2c_adapter_class = {
> .owner = THIS_MODULE,
> .name = "i2c-adapter",
> .dev_attrs = i2c_adapter_attrs,

These ones are unquestionably correct, good catch, thanks.

--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-08-09 11:09    [W:0.244 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site