Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
> From: eric miao <[EMAIL PROTECTED]> > Date: Wed, 19 Dec 2007 16:40:04 +0800 > Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated > > use drivers/gpio/pca9539.c instead. > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > Acked-by: Jean Delvare <[EMAIL PROTECTED]> > --- > Documentation/i2c/chips/pca9539 |3 +++ > drivers/i2c/chips/Kconfig |7 +-- > 2 files changed, 8 insertions(+), 2 deletions(-) Yeah, this version looks unlikely to break anyone's systems. > @@ -75,6 +75,9 @@ config SENSORS_PCA9539 > This driver can also be built as a module. If so, the module > will be called pca9539. > > + This driver is deprecated and will be dropped soon. Use > + drivers/gpio/pca9539.c instead. > + > config SENSORS_PCF8591 > tristate "Philips PCF8591" > depends on EXPERIMENTAL > I suggest defining a date, such as "will be removed in the 2.6.26 kernel", and adding it to Documentation/feature-removal-schedule.txt ... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
From: eric miao [EMAIL PROTECTED] Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] Acked-by: Jean Delvare [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) Yeah, this version looks unlikely to break anyone's systems. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL I suggest defining a date, such as will be removed in the 2.6.26 kernel, and adding it to Documentation/feature-removal-schedule.txt ... -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
On Dec 19, 2007 5:01 PM, Jean Delvare <[EMAIL PROTECTED]> wrote: > Hi Eric, > > > On Wed, 19 Dec 2007 16:45:00 +0800, eric miao wrote: > > Updated as follows, the driver name is left unchanged, while > > Kconfig and Documentation are modified so that > > 1. mark it as deprecated > > 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 > > > > From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 > > From: eric miao <[EMAIL PROTECTED]> > > Date: Wed, 19 Dec 2007 16:40:04 +0800 > > Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated > > > > use drivers/gpio/pca9539.c instead. > > > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > > --- > > Documentation/i2c/chips/pca9539 |3 +++ > > drivers/i2c/chips/Kconfig |7 +-- > > 2 files changed, 8 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/i2c/chips/pca9539 > > b/Documentation/i2c/chips/pca9539 > > index c4fce6a..1d81c53 100644 > > --- a/Documentation/i2c/chips/pca9539 > > +++ b/Documentation/i2c/chips/pca9539 > > @@ -1,6 +1,9 @@ > > Kernel driver pca9539 > > = > > > > +NOTE: this driver is deprecated and will be dropped soon, use > > +drivers/gpio/pca9539.c instead. > > + > > Supported chips: > >* Philips PCA9539 > > Prefix: 'pca9539' > > diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig > > index 2e1c24f..54fd302 100644 > > --- a/drivers/i2c/chips/Kconfig > > +++ b/drivers/i2c/chips/Kconfig > > @@ -66,8 +66,8 @@ config SENSORS_PCF8574 > > hardware. If unsure, say N. > > > > config SENSORS_PCA9539 > > - tristate "Philips PCA9539 16-bit I/O port" > > - depends on EXPERIMENTAL > > + tristate "Philips PCA9539 16-bit I/O port (DEPRECATED)" > > + depends on EXPERIMENTAL && !GPIO_PCA9539 > > If I remember correctly how the Kconfig language works, this will allow > for both drivers to be built as modules at the same time. Given that > they have the same name, which one will be loaded by "modprobe > pca9539"? I think that you should instead express the dependency as > "GPIO_PCA9539=n". > Indeed, fixed. > > help > > If you say yes here you get support for the Philips PCA9539 > > 16-bit I/O port. > > @@ -75,6 +75,9 @@ config SENSORS_PCA9539 > > This driver can also be built as a module. If so, the module > > will be called pca9539. > > > > + This driver is deprecated and will be dropped soon. Use > > + drivers/gpio/pca9539.c instead. > > + > > config SENSORS_PCF8591 > > tristate "Philips PCF8591" > > depends on EXPERIMENTAL > > Other than that I'm fine with this approach, note however that it will > have to go through David rather than me, as I can't merge this before > the new pca9539 driver. > > Thanks, > -- > Jean Delvare > OK, I'll then add your Acked-by :-). Updated as follows: >8 - >From 2bd2deff2f417543f0f17ec1aa32d421cc15cf23 Mon Sep 17 00:00:00 2001 From: eric miao <[EMAIL PROTECTED]> Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao <[EMAIL PROTECTED]> Acked-by: Ben Gardner <[EMAIL PROTECTED]> Acked-by: Jean Delvare <[EMAIL PROTECTED]> --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..7a216f8 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate "Philips PCA9539 16-bit I/O port" - depends on EXPERIMENTAL + tristate "Philips PCA9539 16-bit I/O port (DEPRECATED)" + depends on EXPERIMENTAL && GPIO_PCA9539 = "n" help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate "Philips PCF8591" depends on EXPERIMENTAL -- 1.5.2.5.GIT -- Cheers - eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi Eric, On Wed, 19 Dec 2007 16:45:00 +0800, eric miao wrote: > Updated as follows, the driver name is left unchanged, while > Kconfig and Documentation are modified so that > 1. mark it as deprecated > 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 > > From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 > From: eric miao <[EMAIL PROTECTED]> > Date: Wed, 19 Dec 2007 16:40:04 +0800 > Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated > > use drivers/gpio/pca9539.c instead. > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > --- > Documentation/i2c/chips/pca9539 |3 +++ > drivers/i2c/chips/Kconfig |7 +-- > 2 files changed, 8 insertions(+), 2 deletions(-) > > diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 > index c4fce6a..1d81c53 100644 > --- a/Documentation/i2c/chips/pca9539 > +++ b/Documentation/i2c/chips/pca9539 > @@ -1,6 +1,9 @@ > Kernel driver pca9539 > = > > +NOTE: this driver is deprecated and will be dropped soon, use > +drivers/gpio/pca9539.c instead. > + > Supported chips: >* Philips PCA9539 > Prefix: 'pca9539' > diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig > index 2e1c24f..54fd302 100644 > --- a/drivers/i2c/chips/Kconfig > +++ b/drivers/i2c/chips/Kconfig > @@ -66,8 +66,8 @@ config SENSORS_PCF8574 > hardware. If unsure, say N. > > config SENSORS_PCA9539 > - tristate "Philips PCA9539 16-bit I/O port" > - depends on EXPERIMENTAL > + tristate "Philips PCA9539 16-bit I/O port (DEPRECATED)" > + depends on EXPERIMENTAL && !GPIO_PCA9539 If I remember correctly how the Kconfig language works, this will allow for both drivers to be built as modules at the same time. Given that they have the same name, which one will be loaded by "modprobe pca9539"? I think that you should instead express the dependency as "GPIO_PCA9539=n". > help > If you say yes here you get support for the Philips PCA9539 > 16-bit I/O port. > @@ -75,6 +75,9 @@ config SENSORS_PCA9539 > This driver can also be built as a module. If so, the module > will be called pca9539. > > + This driver is deprecated and will be dropped soon. Use > + drivers/gpio/pca9539.c instead. > + > config SENSORS_PCF8591 > tristate "Philips PCF8591" > depends on EXPERIMENTAL Other than that I'm fine with this approach, note however that it will have to go through David rather than me, as I can't merge this before the new pca9539 driver. Thanks, -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Updated as follows, the driver name is left unchanged, while Kconfig and Documentation are modified so that 1. mark it as deprecated 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 >From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 From: eric miao <[EMAIL PROTECTED]> Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao <[EMAIL PROTECTED]> Acked-by: Ben Gardner <[EMAIL PROTECTED]> --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..54fd302 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate "Philips PCA9539 16-bit I/O port" - depends on EXPERIMENTAL + tristate "Philips PCA9539 16-bit I/O port (DEPRECATED)" + depends on EXPERIMENTAL && !GPIO_PCA9539 help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate "Philips PCF8591" depends on EXPERIMENTAL -- 1.5.2.5.GIT Cheers - eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Updated as follows, the driver name is left unchanged, while Kconfig and Documentation are modified so that 1. mark it as deprecated 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..54fd302 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL + tristate Philips PCA9539 16-bit I/O port (DEPRECATED) + depends on EXPERIMENTAL !GPIO_PCA9539 help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL -- 1.5.2.5.GIT Cheers - eric -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi Eric, On Wed, 19 Dec 2007 16:45:00 +0800, eric miao wrote: Updated as follows, the driver name is left unchanged, while Kconfig and Documentation are modified so that 1. mark it as deprecated 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..54fd302 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL + tristate Philips PCA9539 16-bit I/O port (DEPRECATED) + depends on EXPERIMENTAL !GPIO_PCA9539 If I remember correctly how the Kconfig language works, this will allow for both drivers to be built as modules at the same time. Given that they have the same name, which one will be loaded by modprobe pca9539? I think that you should instead express the dependency as GPIO_PCA9539=n. help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL Other than that I'm fine with this approach, note however that it will have to go through David rather than me, as I can't merge this before the new pca9539 driver. Thanks, -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
On Dec 19, 2007 5:01 PM, Jean Delvare [EMAIL PROTECTED] wrote: Hi Eric, On Wed, 19 Dec 2007 16:45:00 +0800, eric miao wrote: Updated as follows, the driver name is left unchanged, while Kconfig and Documentation are modified so that 1. mark it as deprecated 2. exclusive selection of SENSOR_PCA9539 and GPIO_PCA9539 From c58dc1119355dc94d80763aef9d9bc999abda6df Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..54fd302 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL + tristate Philips PCA9539 16-bit I/O port (DEPRECATED) + depends on EXPERIMENTAL !GPIO_PCA9539 If I remember correctly how the Kconfig language works, this will allow for both drivers to be built as modules at the same time. Given that they have the same name, which one will be loaded by modprobe pca9539? I think that you should instead express the dependency as GPIO_PCA9539=n. Indeed, fixed. help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL Other than that I'm fine with this approach, note however that it will have to go through David rather than me, as I can't merge this before the new pca9539 driver. Thanks, -- Jean Delvare OK, I'll then add your Acked-by :-). Updated as follows: 8 - From 2bd2deff2f417543f0f17ec1aa32d421cc15cf23 Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Wed, 19 Dec 2007 16:40:04 +0800 Subject: [PATCH] gpiolib: mark drivers/i2c/chips/pca9539.c as deprecated use drivers/gpio/pca9539.c instead. Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] Acked-by: Jean Delvare [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 |3 +++ drivers/i2c/chips/Kconfig |7 +-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 index c4fce6a..1d81c53 100644 --- a/Documentation/i2c/chips/pca9539 +++ b/Documentation/i2c/chips/pca9539 @@ -1,6 +1,9 @@ Kernel driver pca9539 = +NOTE: this driver is deprecated and will be dropped soon, use +drivers/gpio/pca9539.c instead. + Supported chips: * Philips PCA9539 Prefix: 'pca9539' diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..7a216f8 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -66,8 +66,8 @@ config SENSORS_PCF8574 hardware. If unsure, say N. config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL + tristate Philips PCA9539 16-bit I/O port (DEPRECATED) + depends on EXPERIMENTAL GPIO_PCA9539 = n help If you say yes here you get support for the Philips PCA9539 16-bit I/O port. @@ -75,6 +75,9 @@ config SENSORS_PCA9539 This driver can also be built as a module. If so, the module will be called pca9539. + This driver is deprecated and will be dropped soon. Use + drivers/gpio/pca9539.c instead. + config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL -- 1.5.2.5.GIT -- Cheers - eric -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi Eric, On Tue, 18 Dec 2007 09:54:42 +0800, eric miao wrote: > Well, I guess it would be a smooth path if we rename the > drivers/i2c/chips/pca9539.c > since that's old style I2C driver, which means the driver name is not > so useful external > so the impact is actually minimum. That's fine with me, just send me a patch. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi Eric, On Tue, 18 Dec 2007 09:54:42 +0800, eric miao wrote: Well, I guess it would be a smooth path if we rename the drivers/i2c/chips/pca9539.c since that's old style I2C driver, which means the driver name is not so useful external so the impact is actually minimum. That's fine with me, just send me a patch. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Well, I guess it would be a smooth path if we rename the drivers/i2c/chips/pca9539.c since that's old style I2C driver, which means the driver name is not so useful external so the impact is actually minimum. On Dec 18, 2007 4:29 AM, Jean Delvare <[EMAIL PROTECTED]> wrote: > Hi David, > > > On Mon, 17 Dec 2007 10:09:53 -0800, David Brownell wrote: > > > Date: Mon, 17 Dec 2007 14:33:27 +0800 > > > From: "eric miao" <[EMAIL PROTECTED]> > > > > > > for the following reasons: > > > > > > 1. there is currently no known users of this driver > > > > > > 2. the functionality of this driver is well supported with the recent > > >proposed drivers/gpio/pca9539.c, using GPIO_LIB > > > > > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > > > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > > > --- > > > Documentation/i2c/chips/pca9539 | 47 > > > drivers/i2c/chips/Kconfig | 10 -- > > > drivers/i2c/chips/Makefile |1 - > > > drivers/i2c/chips/pca9539.c | 196 -- > > > > Jean, do you sign off on this? In any case I think this should > > be going through your I2C patches. > > > > I'd be a trifle uneasy just deleting this, because it's possible > > there are *unknown* users ... and also because nobody's yet done > > a userspace interface to the gpiolib infrastructure. (It seems > > to be the usual case of nobody wanting such a thing quite enough > > to write the code.) > > > > I'd be more comfortable marking it as obsolete and flagging it > > for removal a release or two after Eric's new version merges ... > > though maybe that's just paranoia. > > I'm fine with this and I agree that it would be safer, however please > note that both drivers are mutually exclusive because they have the > same name, meaning that deprecating the old driver is not enough, you > also need Kconfig magic to make sure that both drivers aren't built at > the same time. Or alternatively the old driver could be renamed... I > don't really care myself, I'll take whatever patch you or Eric submit. > > -- > Jean Delvare > -- Cheers - eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi David, On Mon, 17 Dec 2007 10:09:53 -0800, David Brownell wrote: > > Date: Mon, 17 Dec 2007 14:33:27 +0800 > > From: "eric miao" <[EMAIL PROTECTED]> > > > > for the following reasons: > > > > 1. there is currently no known users of this driver > > > > 2. the functionality of this driver is well supported with the recent > >proposed drivers/gpio/pca9539.c, using GPIO_LIB > > > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > > --- > > Documentation/i2c/chips/pca9539 | 47 > > drivers/i2c/chips/Kconfig | 10 -- > > drivers/i2c/chips/Makefile |1 - > > drivers/i2c/chips/pca9539.c | 196 -- > > Jean, do you sign off on this? In any case I think this should > be going through your I2C patches. > > I'd be a trifle uneasy just deleting this, because it's possible > there are *unknown* users ... and also because nobody's yet done > a userspace interface to the gpiolib infrastructure. (It seems > to be the usual case of nobody wanting such a thing quite enough > to write the code.) > > I'd be more comfortable marking it as obsolete and flagging it > for removal a release or two after Eric's new version merges ... > though maybe that's just paranoia. I'm fine with this and I agree that it would be safer, however please note that both drivers are mutually exclusive because they have the same name, meaning that deprecating the old driver is not enough, you also need Kconfig magic to make sure that both drivers aren't built at the same time. Or alternatively the old driver could be renamed... I don't really care myself, I'll take whatever patch you or Eric submit. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
> Date: Mon, 17 Dec 2007 14:33:27 +0800 > From: "eric miao" <[EMAIL PROTECTED]> > > for the following reasons: > > 1. there is currently no known users of this driver > > 2. the functionality of this driver is well supported with the recent >proposed drivers/gpio/pca9539.c, using GPIO_LIB > > Signed-off-by: eric miao <[EMAIL PROTECTED]> > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > --- > Documentation/i2c/chips/pca9539 | 47 > drivers/i2c/chips/Kconfig | 10 -- > drivers/i2c/chips/Makefile |1 - > drivers/i2c/chips/pca9539.c | 196 -- Jean, do you sign off on this? In any case I think this should be going through your I2C patches. I'd be a trifle uneasy just deleting this, because it's possible there are *unknown* users ... and also because nobody's yet done a userspace interface to the gpiolib infrastructure. (It seems to be the usual case of nobody wanting such a thing quite enough to write the code.) I'd be more comfortable marking it as obsolete and flagging it for removal a release or two after Eric's new version merges ... though maybe that's just paranoia. - Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Date: Mon, 17 Dec 2007 14:33:27 +0800 From: eric miao [EMAIL PROTECTED] for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 | 47 drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 -- Jean, do you sign off on this? In any case I think this should be going through your I2C patches. I'd be a trifle uneasy just deleting this, because it's possible there are *unknown* users ... and also because nobody's yet done a userspace interface to the gpiolib infrastructure. (It seems to be the usual case of nobody wanting such a thing quite enough to write the code.) I'd be more comfortable marking it as obsolete and flagging it for removal a release or two after Eric's new version merges ... though maybe that's just paranoia. - Dave -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Hi David, On Mon, 17 Dec 2007 10:09:53 -0800, David Brownell wrote: Date: Mon, 17 Dec 2007 14:33:27 +0800 From: eric miao [EMAIL PROTECTED] for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 | 47 drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 -- Jean, do you sign off on this? In any case I think this should be going through your I2C patches. I'd be a trifle uneasy just deleting this, because it's possible there are *unknown* users ... and also because nobody's yet done a userspace interface to the gpiolib infrastructure. (It seems to be the usual case of nobody wanting such a thing quite enough to write the code.) I'd be more comfortable marking it as obsolete and flagging it for removal a release or two after Eric's new version merges ... though maybe that's just paranoia. I'm fine with this and I agree that it would be safer, however please note that both drivers are mutually exclusive because they have the same name, meaning that deprecating the old driver is not enough, you also need Kconfig magic to make sure that both drivers aren't built at the same time. Or alternatively the old driver could be renamed... I don't really care myself, I'll take whatever patch you or Eric submit. -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Well, I guess it would be a smooth path if we rename the drivers/i2c/chips/pca9539.c since that's old style I2C driver, which means the driver name is not so useful external so the impact is actually minimum. On Dec 18, 2007 4:29 AM, Jean Delvare [EMAIL PROTECTED] wrote: Hi David, On Mon, 17 Dec 2007 10:09:53 -0800, David Brownell wrote: Date: Mon, 17 Dec 2007 14:33:27 +0800 From: eric miao [EMAIL PROTECTED] for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 | 47 drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 -- Jean, do you sign off on this? In any case I think this should be going through your I2C patches. I'd be a trifle uneasy just deleting this, because it's possible there are *unknown* users ... and also because nobody's yet done a userspace interface to the gpiolib infrastructure. (It seems to be the usual case of nobody wanting such a thing quite enough to write the code.) I'd be more comfortable marking it as obsolete and flagging it for removal a release or two after Eric's new version merges ... though maybe that's just paranoia. I'm fine with this and I agree that it would be safer, however please note that both drivers are mutually exclusive because they have the same name, meaning that deprecating the old driver is not enough, you also need Kconfig magic to make sure that both drivers aren't built at the same time. Or alternatively the old driver could be renamed... I don't really care myself, I'll take whatever patch you or Eric submit. -- Jean Delvare -- Cheers - eric -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
[ Updated according to Jean's suggestion, thanks ] >From 5b4d907da17d57ec168643ebd847278e8d7267f9 Mon Sep 17 00:00:00 2001 From: eric miao <[EMAIL PROTECTED]> Date: Sat, 15 Dec 2007 12:07:26 +0800 Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c and related files for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao <[EMAIL PROTECTED]> Acked-by: Ben Gardner <[EMAIL PROTECTED]> --- Documentation/i2c/chips/pca9539 | 47 - drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 --- 4 files changed, 0 insertions(+), 254 deletions(-) delete mode 100644 Documentation/i2c/chips/pca9539 delete mode 100644 drivers/i2c/chips/pca9539.c diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 deleted file mode 100644 index c4fce6a..000 --- a/Documentation/i2c/chips/pca9539 +++ /dev/null @@ -1,47 +0,0 @@ -Kernel driver pca9539 -= - -Supported chips: - * Philips PCA9539 -Prefix: 'pca9539' -Addresses scanned: 0x74 - 0x77 -Datasheet: -http://www.semiconductors.philips.com/acrobat/datasheets/PCA9539_2.pdf - -Author: Ben Gardner <[EMAIL PROTECTED]> - - -Description - -The Philips PCA9539 is a 16 bit low power I/O device. -All 16 lines can be individually configured as an input or output. -The input sense can also be inverted. -The 16 lines are split between two bytes. - - -Sysfs entries -- - -Each is a byte that maps to the 8 I/O bits. -A '0' suffix is for bits 0-7, while '1' is for bits 8-15. - -input[01] - read the current value -output[01]- sets the output value -direction[01] - direction of each bit: 1=input, 0=output -invert[01]- toggle the input bit sense - -input reads the actual state of the line and is always available. -The direction defaults to input for all channels. - - -General Remarks - -Note that each output, direction, and invert entry controls 8 lines. -You should use the read, modify, write sequence. -For example. to set output bit 0 of 1. - val=$(cat output0) - val=$(( $val | 1 )) - echo $val > output0 - diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..a676f57 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -65,16 +65,6 @@ config SENSORS_PCF8574 These devices are hard to detect and rarely found on mainstream hardware. If unsure, say N. -config SENSORS_PCA9539 - tristate "Philips PCA9539 16-bit I/O port" - depends on EXPERIMENTAL - help - If you say yes here you get support for the Philips PCA9539 - 16-bit I/O port. - - This driver can also be built as a module. If so, the module - will be called pca9539. - config SENSORS_PCF8591 tristate "Philips PCF8591" depends on EXPERIMENTAL diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile index ca924e1..bc9e9ca 100644 --- a/drivers/i2c/chips/Makefile +++ b/drivers/i2c/chips/Makefile @@ -8,7 +8,6 @@ obj-$(CONFIG_DS1682)+= ds1682.o obj-$(CONFIG_SENSORS_EEPROM) += eeprom.o obj-$(CONFIG_SENSORS_MAX6875) += max6875.o obj-$(CONFIG_SENSORS_M41T00) += m41t00.o -obj-$(CONFIG_SENSORS_PCA9539) += pca9539.o obj-$(CONFIG_SENSORS_PCF8574) += pcf8574.o obj-$(CONFIG_SENSORS_PCF8591) += pcf8591.o obj-$(CONFIG_ISP1301_OMAP) += isp1301_omap.o diff --git a/drivers/i2c/chips/pca9539.c b/drivers/i2c/chips/pca9539.c deleted file mode 100644 index f43c4e7..000 --- a/drivers/i2c/chips/pca9539.c +++ /dev/null @@ -1,196 +0,0 @@ -/* -pca9539.c - 16-bit I/O port with interrupt and reset - -Copyright (C) 2005 Ben Gardner <[EMAIL PROTECTED]> - -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; version 2 of the License. -*/ - -#include -#include -#include -#include -#include - -/* Addresses to scan */ -static unsigned short normal_i2c[] = {0x74, 0x75, 0x76, 0x77, I2C_CLIENT_END}; - -/* Insmod parameters */ -I2C_CLIENT_INSMOD_1(pca9539); - -enum pca9539_cmd -{ - PCA9539_INPUT_0 = 0, - PCA9539_INPUT_1 = 1, - PCA9539_OUTPUT_0= 2, - PCA9539_OUTPUT_1= 3, - PCA9539_INVERT_0= 4, - PCA9539_INVERT_1= 5, - PCA9539_DIRECTION_0 = 6, - PCA9539_DIRECTION_1 = 7, -}; - -static int pca9539_attach_adapter(struct i2c_adapter *adapter); -static int pca9539_detect(struct i2c_adapter *adapter, int address, int kind); -static int pca9539_detach_client(struct i2c_client *client); - -/* This is the driver that will be inserted */ -static struct
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Eric, On Sat, 15 Dec 2007 12:16:50 +0800, eric miao wrote: > From 0bca662f68e7ffe84f333d7d26df25d846713db2 Mon Sep 17 00:00:00 2001 > From: eric miao <[EMAIL PROTECTED]> > Date: Sat, 15 Dec 2007 12:07:26 +0800 > Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c > > for the following reasons: > > 1. there is currently no known users of this driver > > 2. the functionality of this driver is well supported with the recent >proposed drivers/gpio/pca9539.c, using GPIO_LIB > This is fine with me, however... > Signed-off-by: eric miao <[EMAIL PROTECTED]> > Acked-by: Ben Gardner <[EMAIL PROTECTED]> > --- > drivers/i2c/chips/Kconfig | 10 -- > drivers/i2c/chips/Makefile |1 - > drivers/i2c/chips/pca9539.c | 196 > --- ... please also delete Documentation/i2c/chips/pca9539. > 3 files changed, 0 insertions(+), 207 deletions(-) > delete mode 100644 drivers/i2c/chips/pca9539.c -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
[ Updated according to Jean's suggestion, thanks ] From 5b4d907da17d57ec168643ebd847278e8d7267f9 Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Sat, 15 Dec 2007 12:07:26 +0800 Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c and related files for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- Documentation/i2c/chips/pca9539 | 47 - drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 --- 4 files changed, 0 insertions(+), 254 deletions(-) delete mode 100644 Documentation/i2c/chips/pca9539 delete mode 100644 drivers/i2c/chips/pca9539.c diff --git a/Documentation/i2c/chips/pca9539 b/Documentation/i2c/chips/pca9539 deleted file mode 100644 index c4fce6a..000 --- a/Documentation/i2c/chips/pca9539 +++ /dev/null @@ -1,47 +0,0 @@ -Kernel driver pca9539 -= - -Supported chips: - * Philips PCA9539 -Prefix: 'pca9539' -Addresses scanned: 0x74 - 0x77 -Datasheet: -http://www.semiconductors.philips.com/acrobat/datasheets/PCA9539_2.pdf - -Author: Ben Gardner [EMAIL PROTECTED] - - -Description - -The Philips PCA9539 is a 16 bit low power I/O device. -All 16 lines can be individually configured as an input or output. -The input sense can also be inverted. -The 16 lines are split between two bytes. - - -Sysfs entries -- - -Each is a byte that maps to the 8 I/O bits. -A '0' suffix is for bits 0-7, while '1' is for bits 8-15. - -input[01] - read the current value -output[01]- sets the output value -direction[01] - direction of each bit: 1=input, 0=output -invert[01]- toggle the input bit sense - -input reads the actual state of the line and is always available. -The direction defaults to input for all channels. - - -General Remarks - -Note that each output, direction, and invert entry controls 8 lines. -You should use the read, modify, write sequence. -For example. to set output bit 0 of 1. - val=$(cat output0) - val=$(( $val | 1 )) - echo $val output0 - diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..a676f57 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -65,16 +65,6 @@ config SENSORS_PCF8574 These devices are hard to detect and rarely found on mainstream hardware. If unsure, say N. -config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL - help - If you say yes here you get support for the Philips PCA9539 - 16-bit I/O port. - - This driver can also be built as a module. If so, the module - will be called pca9539. - config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile index ca924e1..bc9e9ca 100644 --- a/drivers/i2c/chips/Makefile +++ b/drivers/i2c/chips/Makefile @@ -8,7 +8,6 @@ obj-$(CONFIG_DS1682)+= ds1682.o obj-$(CONFIG_SENSORS_EEPROM) += eeprom.o obj-$(CONFIG_SENSORS_MAX6875) += max6875.o obj-$(CONFIG_SENSORS_M41T00) += m41t00.o -obj-$(CONFIG_SENSORS_PCA9539) += pca9539.o obj-$(CONFIG_SENSORS_PCF8574) += pcf8574.o obj-$(CONFIG_SENSORS_PCF8591) += pcf8591.o obj-$(CONFIG_ISP1301_OMAP) += isp1301_omap.o diff --git a/drivers/i2c/chips/pca9539.c b/drivers/i2c/chips/pca9539.c deleted file mode 100644 index f43c4e7..000 --- a/drivers/i2c/chips/pca9539.c +++ /dev/null @@ -1,196 +0,0 @@ -/* -pca9539.c - 16-bit I/O port with interrupt and reset - -Copyright (C) 2005 Ben Gardner [EMAIL PROTECTED] - -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; version 2 of the License. -*/ - -#include linux/module.h -#include linux/init.h -#include linux/slab.h -#include linux/i2c.h -#include linux/hwmon-sysfs.h - -/* Addresses to scan */ -static unsigned short normal_i2c[] = {0x74, 0x75, 0x76, 0x77, I2C_CLIENT_END}; - -/* Insmod parameters */ -I2C_CLIENT_INSMOD_1(pca9539); - -enum pca9539_cmd -{ - PCA9539_INPUT_0 = 0, - PCA9539_INPUT_1 = 1, - PCA9539_OUTPUT_0= 2, - PCA9539_OUTPUT_1= 3, - PCA9539_INVERT_0= 4, - PCA9539_INVERT_1= 5, - PCA9539_DIRECTION_0 = 6, - PCA9539_DIRECTION_1 = 7, -}; - -static int pca9539_attach_adapter(struct i2c_adapter *adapter); -static int pca9539_detect(struct i2c_adapter *adapter, int address, int kind); -static int pca9539_detach_client(struct i2c_client *client); - -/* This is the
Re: [PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
Eric, On Sat, 15 Dec 2007 12:16:50 +0800, eric miao wrote: From 0bca662f68e7ffe84f333d7d26df25d846713db2 Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Sat, 15 Dec 2007 12:07:26 +0800 Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB This is fine with me, however... Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 --- ... please also delete Documentation/i2c/chips/pca9539. 3 files changed, 0 insertions(+), 207 deletions(-) delete mode 100644 drivers/i2c/chips/pca9539.c -- Jean Delvare -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
>From 0bca662f68e7ffe84f333d7d26df25d846713db2 Mon Sep 17 00:00:00 2001 From: eric miao <[EMAIL PROTECTED]> Date: Sat, 15 Dec 2007 12:07:26 +0800 Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao <[EMAIL PROTECTED]> Acked-by: Ben Gardner <[EMAIL PROTECTED]> --- drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 --- 3 files changed, 0 insertions(+), 207 deletions(-) delete mode 100644 drivers/i2c/chips/pca9539.c diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..a676f57 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -65,16 +65,6 @@ config SENSORS_PCF8574 These devices are hard to detect and rarely found on mainstream hardware. If unsure, say N. -config SENSORS_PCA9539 - tristate "Philips PCA9539 16-bit I/O port" - depends on EXPERIMENTAL - help - If you say yes here you get support for the Philips PCA9539 - 16-bit I/O port. - - This driver can also be built as a module. If so, the module - will be called pca9539. - config SENSORS_PCF8591 tristate "Philips PCF8591" depends on EXPERIMENTAL diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile index ca924e1..bc9e9ca 100644 --- a/drivers/i2c/chips/Makefile +++ b/drivers/i2c/chips/Makefile @@ -8,7 +8,6 @@ obj-$(CONFIG_DS1682)+= ds1682.o obj-$(CONFIG_SENSORS_EEPROM) += eeprom.o obj-$(CONFIG_SENSORS_MAX6875) += max6875.o obj-$(CONFIG_SENSORS_M41T00) += m41t00.o -obj-$(CONFIG_SENSORS_PCA9539) += pca9539.o obj-$(CONFIG_SENSORS_PCF8574) += pcf8574.o obj-$(CONFIG_SENSORS_PCF8591) += pcf8591.o obj-$(CONFIG_ISP1301_OMAP) += isp1301_omap.o diff --git a/drivers/i2c/chips/pca9539.c b/drivers/i2c/chips/pca9539.c deleted file mode 100644 index f43c4e7..000 --- a/drivers/i2c/chips/pca9539.c +++ /dev/null @@ -1,196 +0,0 @@ -/* -pca9539.c - 16-bit I/O port with interrupt and reset - -Copyright (C) 2005 Ben Gardner <[EMAIL PROTECTED]> - -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; version 2 of the License. -*/ - -#include -#include -#include -#include -#include - -/* Addresses to scan */ -static unsigned short normal_i2c[] = {0x74, 0x75, 0x76, 0x77, I2C_CLIENT_END}; - -/* Insmod parameters */ -I2C_CLIENT_INSMOD_1(pca9539); - -enum pca9539_cmd -{ - PCA9539_INPUT_0 = 0, - PCA9539_INPUT_1 = 1, - PCA9539_OUTPUT_0= 2, - PCA9539_OUTPUT_1= 3, - PCA9539_INVERT_0= 4, - PCA9539_INVERT_1= 5, - PCA9539_DIRECTION_0 = 6, - PCA9539_DIRECTION_1 = 7, -}; - -static int pca9539_attach_adapter(struct i2c_adapter *adapter); -static int pca9539_detect(struct i2c_adapter *adapter, int address, int kind); -static int pca9539_detach_client(struct i2c_client *client); - -/* This is the driver that will be inserted */ -static struct i2c_driver pca9539_driver = { - .driver = { - .name = "pca9539", - }, - .attach_adapter = pca9539_attach_adapter, - .detach_client = pca9539_detach_client, -}; - -struct pca9539_data { - struct i2c_client client; -}; - -/* following are the sysfs callback functions */ -static ssize_t pca9539_show(struct device *dev, struct device_attribute *attr, - char *buf) -{ - struct sensor_device_attribute *psa = to_sensor_dev_attr(attr); - struct i2c_client *client = to_i2c_client(dev); - return sprintf(buf, "%d\n", i2c_smbus_read_byte_data(client, -psa->index)); -} - -static ssize_t pca9539_store(struct device *dev, struct device_attribute *attr, -const char *buf, size_t count) -{ - struct sensor_device_attribute *psa = to_sensor_dev_attr(attr); - struct i2c_client *client = to_i2c_client(dev); - unsigned long val = simple_strtoul(buf, NULL, 0); - if (val > 0xff) - return -EINVAL; - i2c_smbus_write_byte_data(client, psa->index, val); - return count; -} - -/* Define the device attributes */ - -#define PCA9539_ENTRY_RO(name, cmd_idx) \ - static SENSOR_DEVICE_ATTR(name, S_IRUGO, pca9539_show, NULL, cmd_idx) - -#define PCA9539_ENTRY_RW(name, cmd_idx) \ - static SENSOR_DEVICE_ATTR(name, S_IRUGO | S_IWUSR, pca9539_show, \ - pca9539_store, cmd_idx) - -PCA9539_ENTRY_RO(input0, PCA9539_INPUT_0); -PCA9539_ENTRY_RO(input1, PCA9539_INPUT_1);
[PATCH 2.6.24-rc5-mm 3/3] gpiolib: obsolete drivers/i2c/chips/pca9539.c
From 0bca662f68e7ffe84f333d7d26df25d846713db2 Mon Sep 17 00:00:00 2001 From: eric miao [EMAIL PROTECTED] Date: Sat, 15 Dec 2007 12:07:26 +0800 Subject: [PATCH] gpiolib: obsolete drivers/i2c/chips/pca9539.c for the following reasons: 1. there is currently no known users of this driver 2. the functionality of this driver is well supported with the recent proposed drivers/gpio/pca9539.c, using GPIO_LIB Signed-off-by: eric miao [EMAIL PROTECTED] Acked-by: Ben Gardner [EMAIL PROTECTED] --- drivers/i2c/chips/Kconfig | 10 -- drivers/i2c/chips/Makefile |1 - drivers/i2c/chips/pca9539.c | 196 --- 3 files changed, 0 insertions(+), 207 deletions(-) delete mode 100644 drivers/i2c/chips/pca9539.c diff --git a/drivers/i2c/chips/Kconfig b/drivers/i2c/chips/Kconfig index 2e1c24f..a676f57 100644 --- a/drivers/i2c/chips/Kconfig +++ b/drivers/i2c/chips/Kconfig @@ -65,16 +65,6 @@ config SENSORS_PCF8574 These devices are hard to detect and rarely found on mainstream hardware. If unsure, say N. -config SENSORS_PCA9539 - tristate Philips PCA9539 16-bit I/O port - depends on EXPERIMENTAL - help - If you say yes here you get support for the Philips PCA9539 - 16-bit I/O port. - - This driver can also be built as a module. If so, the module - will be called pca9539. - config SENSORS_PCF8591 tristate Philips PCF8591 depends on EXPERIMENTAL diff --git a/drivers/i2c/chips/Makefile b/drivers/i2c/chips/Makefile index ca924e1..bc9e9ca 100644 --- a/drivers/i2c/chips/Makefile +++ b/drivers/i2c/chips/Makefile @@ -8,7 +8,6 @@ obj-$(CONFIG_DS1682)+= ds1682.o obj-$(CONFIG_SENSORS_EEPROM) += eeprom.o obj-$(CONFIG_SENSORS_MAX6875) += max6875.o obj-$(CONFIG_SENSORS_M41T00) += m41t00.o -obj-$(CONFIG_SENSORS_PCA9539) += pca9539.o obj-$(CONFIG_SENSORS_PCF8574) += pcf8574.o obj-$(CONFIG_SENSORS_PCF8591) += pcf8591.o obj-$(CONFIG_ISP1301_OMAP) += isp1301_omap.o diff --git a/drivers/i2c/chips/pca9539.c b/drivers/i2c/chips/pca9539.c deleted file mode 100644 index f43c4e7..000 --- a/drivers/i2c/chips/pca9539.c +++ /dev/null @@ -1,196 +0,0 @@ -/* -pca9539.c - 16-bit I/O port with interrupt and reset - -Copyright (C) 2005 Ben Gardner [EMAIL PROTECTED] - -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; version 2 of the License. -*/ - -#include linux/module.h -#include linux/init.h -#include linux/slab.h -#include linux/i2c.h -#include linux/hwmon-sysfs.h - -/* Addresses to scan */ -static unsigned short normal_i2c[] = {0x74, 0x75, 0x76, 0x77, I2C_CLIENT_END}; - -/* Insmod parameters */ -I2C_CLIENT_INSMOD_1(pca9539); - -enum pca9539_cmd -{ - PCA9539_INPUT_0 = 0, - PCA9539_INPUT_1 = 1, - PCA9539_OUTPUT_0= 2, - PCA9539_OUTPUT_1= 3, - PCA9539_INVERT_0= 4, - PCA9539_INVERT_1= 5, - PCA9539_DIRECTION_0 = 6, - PCA9539_DIRECTION_1 = 7, -}; - -static int pca9539_attach_adapter(struct i2c_adapter *adapter); -static int pca9539_detect(struct i2c_adapter *adapter, int address, int kind); -static int pca9539_detach_client(struct i2c_client *client); - -/* This is the driver that will be inserted */ -static struct i2c_driver pca9539_driver = { - .driver = { - .name = pca9539, - }, - .attach_adapter = pca9539_attach_adapter, - .detach_client = pca9539_detach_client, -}; - -struct pca9539_data { - struct i2c_client client; -}; - -/* following are the sysfs callback functions */ -static ssize_t pca9539_show(struct device *dev, struct device_attribute *attr, - char *buf) -{ - struct sensor_device_attribute *psa = to_sensor_dev_attr(attr); - struct i2c_client *client = to_i2c_client(dev); - return sprintf(buf, %d\n, i2c_smbus_read_byte_data(client, -psa-index)); -} - -static ssize_t pca9539_store(struct device *dev, struct device_attribute *attr, -const char *buf, size_t count) -{ - struct sensor_device_attribute *psa = to_sensor_dev_attr(attr); - struct i2c_client *client = to_i2c_client(dev); - unsigned long val = simple_strtoul(buf, NULL, 0); - if (val 0xff) - return -EINVAL; - i2c_smbus_write_byte_data(client, psa-index, val); - return count; -} - -/* Define the device attributes */ - -#define PCA9539_ENTRY_RO(name, cmd_idx) \ - static SENSOR_DEVICE_ATTR(name, S_IRUGO, pca9539_show, NULL, cmd_idx) - -#define PCA9539_ENTRY_RW(name, cmd_idx) \ - static SENSOR_DEVICE_ATTR(name, S_IRUGO | S_IWUSR, pca9539_show, \ - pca9539_store, cmd_idx) - -PCA9539_ENTRY_RO(input0, PCA9539_INPUT_0);