On Thu, Jan 16, 2014 at 7:01 AM, Sascha Hauer wrote:
> Before we had the PWM framework we used to have a barebone PWM api. The
> HAVE_PWM Kconfig symbol used to be selected by the PWM drivers to specify
> the PWM API is present in the kernel. Since the last legacy driver is gone
> the HAVE_PWM
On Thu, Jan 16, 2014 at 7:01 AM, Sascha Hauer s.ha...@pengutronix.de wrote:
Before we had the PWM framework we used to have a barebone PWM api. The
HAVE_PWM Kconfig symbol used to be selected by the PWM drivers to specify
the PWM API is present in the kernel. Since the last legacy driver is
If MODPUBKEY is specified and other than default ./signing_key.x509, use
that key instead of generating one on-the-fly.
Signed-off-by: Eric Miao
Cc: David Howells
Cc: Dan Willemsen
---
kernel/Makefile | 8
1 file changed, 8 insertions(+)
diff --git a/kernel/Makefile b/kernel
Even if MODULE_SIG_FORCE is turned on, it is still useful if module
can export sig_enforce, so user space will know if module signature
is turned on and forced.
Signed-off-by: Eric Miao
Cc: David Howells
Cc: Dan Willemsen
---
kernel/module.c | 8
1 file changed, 8 insertions(+)
diff
Even if MODULE_SIG_FORCE is turned on, it is still useful if module
can export sig_enforce, so user space will know if module signature
is turned on and forced.
Signed-off-by: Eric Miao eric.m...@nvidia.com
Cc: David Howells dhowe...@redhat.com
Cc: Dan Willemsen dwillem...@nvidia.com
---
kernel
If MODPUBKEY is specified and other than default ./signing_key.x509, use
that key instead of generating one on-the-fly.
Signed-off-by: Eric Miao eric.m...@nvidia.com
Cc: David Howells dhowe...@redhat.com
Cc: Dan Willemsen dwillem...@nvidia.com
---
kernel/Makefile | 8
1 file changed, 8
On Sat, Jul 6, 2013 at 1:42 PM, Michael Opdenacker
wrote:
> This flag is a NOOP since 2.6.35 and can be removed.
>
> Signed-off-by: Michael Opdenacker
Indeed. Acked-by: Eric Miao
> ---
> arch/arm/mach-pxa/am200epd.c | 3 +--
> arch/arm/mach-pxa/am300epd.c |
On Sat, Jul 6, 2013 at 1:42 PM, Michael Opdenacker
michael.opdenac...@free-electrons.com wrote:
This flag is a NOOP since 2.6.35 and can be removed.
Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com
Indeed. Acked-by: Eric Miao eric.y.m...@gmail.com
---
arch/arm/mach
NIG_OF is defined, but PWM is not defined in device tree
> configuration file. The device
> can still match the driver is the id_table is matched or name is matched.
> So I covered addtional situation
> 1. PWM is not defined in DT configuration file but CONFIG_OF is defined.
>
the possiblility that event CONFIG_OF is defined, but PWM is
not defined in DT file, and we still can use old way to probe the
device.
Yeah I was thinking maybe we should not keep the legacy working if
CONFIG_OF is defined, but that should be OK:
Acked-by: Eric Miao eric.y.m...@gmail.com
iff --git a/drivers/pwm/pwm-pxa.c b/drivers/pwm/pwm-pxa.c
> index aa4bea7..c8d59a2 100644
> --- a/drivers/pwm/pwm-pxa.c
> +++ b/drivers/pwm/pwm-pxa.c
> @@ -9,6 +9,8 @@
> *
> * 2008-02-13 initial version
> * eric miao
> + * 2013-04-24 add device tree suppo
ao Xie
Looks good to me,
Acked-by: Eric Miao
> ---
> drivers/pwm/pwm-pxa.c | 12 +---
> 1 files changed, 1 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/pwm/pwm-pxa.c b/drivers/pwm/pwm-pxa.c
> index dee6ab55..aa4bea7 100644
> --- a/drivers/pwm/pwm-p
On Mon, May 6, 2013 at 9:29 AM, Chao Xie wrote:
> The PWM driver is not only used by ARCH_PXA but also ARCH_MMP.
>
> Signed-off-by: Chao Xie
Looks good to me.
Acked-by: Eric Miao
> ---
> drivers/pwm/Kconfig |2 +-
> 1 files changed, 1 insertions(+), 1 deletions
On Mon, May 6, 2013 at 9:29 AM, Chao Xie chao@marvell.com wrote:
The PWM driver is not only used by ARCH_PXA but also ARCH_MMP.
Signed-off-by: Chao Xie chao@marvell.com
Looks good to me.
Acked-by: Eric Miao eric.y.m...@gmail.com
---
drivers/pwm/Kconfig |2 +-
1 files changed
@marvell.com
Looks good to me,
Acked-by: Eric Miao eric.y.m...@gmail.com
---
drivers/pwm/pwm-pxa.c | 12 +---
1 files changed, 1 insertions(+), 11 deletions(-)
diff --git a/drivers/pwm/pwm-pxa.c b/drivers/pwm/pwm-pxa.c
index dee6ab55..aa4bea7 100644
--- a/drivers/pwm/pwm-pxa.c
+++ b
(-)
diff --git a/drivers/pwm/pwm-pxa.c b/drivers/pwm/pwm-pxa.c
index aa4bea7..c8d59a2 100644
--- a/drivers/pwm/pwm-pxa.c
+++ b/drivers/pwm/pwm-pxa.c
@@ -9,6 +9,8 @@
*
* 2008-02-13 initial version
* eric miao eric.m...@marvell.com
+ * 2013-04-24 add device tree support
ed-off-by: Andy Shevchenko
>> Reported-by: Mika Westerberg
>> Cc: Eric Miao
>> Cc: Greg Kroah-Hartman
>> ---
>> drivers/base/platform.c |4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/base/platform.c
ed-off-by: Andy Shevchenko
>> Reported-by: Mika Westerberg
>> Cc: Eric Miao
>> Cc: Greg Kroah-Hartman
>> ---
>> drivers/base/platform.c |4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/base/platform.c
-by: Mika Westerberg mika.westerberg@xxx
Cc: Eric Miao eric.miao@xxx
Cc: Greg Kroah-Hartman gregkh@xxx
---
drivers/base/platform.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base/platform.c b/drivers/base/platform.c
index c0b8df3
-by: Mika Westerberg mika.westerberg@xxx
Cc: Eric Miao eric.miao@xxx
Cc: Greg Kroah-Hartman gregkh@xxx
---
drivers/base/platform.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base/platform.c b/drivers/base/platform.c
index c0b8df3
ondary_pwm? */
> { "pxa25x-pwm", 0 },
> - { "pxa27x-pwm", 0 | HAS_SECONDARY_PWM },
> - { "pxa168-pwm", 1 },
> - { "pxa910-pwm", 1 },
> + { "pxa27x-pwm", HAS_SECONDARY_PWM },
> + { "pxa168-pwm
On Mon, Apr 1, 2013 at 3:32 PM, Axel Lin wrote:
> 2013/4/1 Eric Miao :
>> On Sun, Mar 31, 2013 at 11:04 PM, Axel Lin wrote:
>>> clk_enable/clk_disable maintain an enable_count, clk_prepare and
>>> clk_unprepare
>>> also maintain a prepare_count. Th
On Sun, Mar 31, 2013 at 11:04 PM, Axel Lin wrote:
> clk_enable/clk_disable maintain an enable_count, clk_prepare and clk_unprepare
> also maintain a prepare_count. These APIs will do prepare/enable when the
> first
> user calling these APIs, and do disable/unprepare when the corresponding
>
On Mon, Apr 1, 2013 at 12:12 AM, Axel Lin wrote:
> The driver_data field was used to store information about PWM_ID_BASE and
> HAS_SECONDARY_PWM. PWM_ID_BASE is not used now after convert to pwm framework.
> This patch stores the pwm_nr in driver_data field to simplify the code.
>
>
On Mon, Apr 1, 2013 at 12:12 AM, Axel Lin axel@ingics.com wrote:
The driver_data field was used to store information about PWM_ID_BASE and
HAS_SECONDARY_PWM. PWM_ID_BASE is not used now after convert to pwm framework.
This patch stores the pwm_nr in driver_data field to simplify the code.
On Sun, Mar 31, 2013 at 11:04 PM, Axel Lin axel@ingics.com wrote:
clk_enable/clk_disable maintain an enable_count, clk_prepare and clk_unprepare
also maintain a prepare_count. These APIs will do prepare/enable when the
first
user calling these APIs, and do disable/unprepare when the
On Mon, Apr 1, 2013 at 3:32 PM, Axel Lin axel@ingics.com wrote:
2013/4/1 Eric Miao eric.y.m...@gmail.com:
On Sun, Mar 31, 2013 at 11:04 PM, Axel Lin axel@ingics.com wrote:
clk_enable/clk_disable maintain an enable_count, clk_prepare and
clk_unprepare
also maintain a prepare_count
},
- { pxa27x-pwm, 0 | HAS_SECONDARY_PWM },
- { pxa168-pwm, 1 },
- { pxa910-pwm, 1 },
+ { pxa27x-pwm, HAS_SECONDARY_PWM },
+ { pxa168-pwm, 0 },
+ { pxa910-pwm, 0 },
{ },
};
MODULE_DEVICE_TABLE(platform, pwm_id_table);
Looks good to me, Acked-by: Eric Miao
On Sun, Mar 31, 2013 at 2:45 PM, Viresh Kumar wrote:
> On 31 March 2013 10:54, Eric Miao wrote:
>> On Sun, Mar 31, 2013 at 11:52 AM, Viresh Kumar
>> wrote:
>>> Eric/Haojian,
>>>
>>> On 25 March 2013 15:41, Viresh Kumar wrote:
>>>> This p
On Sun, Mar 31, 2013 at 2:45 PM, Viresh Kumar viresh.ku...@linaro.org wrote:
On 31 March 2013 10:54, Eric Miao eric.y.m...@gmail.com wrote:
On Sun, Mar 31, 2013 at 11:52 AM, Viresh Kumar viresh.ku...@linaro.org
wrote:
Eric/Haojian,
On 25 March 2013 15:41, Viresh Kumar viresh.ku
On Sun, Mar 31, 2013 at 11:52 AM, Viresh Kumar wrote:
> Eric/Haojian,
>
> On 25 March 2013 15:41, Viresh Kumar wrote:
>> This patch moves cpufreq driver of ARM based pxa3xx platform to
>> drivers/cpufreq.
>>
>> Cc: Eric Miao
Acked-by: Eric Miao
>
On Sun, Mar 31, 2013 at 11:52 AM, Viresh Kumar viresh.ku...@linaro.org wrote:
Eric/Haojian,
On 25 March 2013 15:41, Viresh Kumar viresh.ku...@linaro.org wrote:
This patch moves cpufreq driver of ARM based pxa3xx platform to
drivers/cpufreq.
Cc: Eric Miao eric.y.m...@gmail.com
Acked
On Mon, Jan 7, 2013 at 6:44 PM, Mika Westerberg
wrote:
> In addition fix following warnings seen when compiling 64-bit:
>
> drivers/spi/spi-pxa2xx.c: In function ‘map_dma_buffers’:
> drivers/spi/spi-pxa2xx.c:384:7: warning: cast from pointer to integer of
> different size
On Mon, Jan 7, 2013 at 6:44 PM, Mika Westerberg
mika.westerb...@linux.intel.com wrote:
In addition fix following warnings seen when compiling 64-bit:
drivers/spi/spi-pxa2xx.c: In function ‘map_dma_buffers’:
drivers/spi/spi-pxa2xx.c:384:7: warning: cast from pointer to integer of
different
t; eventually lead to a complete removal of struct sys_timer.
>
> Cc: Eric Miao
> Cc: Russell King
> Cc: Haojian Zhuang
> Signed-off-by: Stephen Warren
Acked-by: Eric Miao
> ---
> arch/arm/mach-pxa/time.c | 76
> +++--
to be removed, and
eventually lead to a complete removal of struct sys_timer.
Cc: Eric Miao eric.y.m...@gmail.com
Cc: Russell King li...@arm.linux.org.uk
Cc: Haojian Zhuang haojian.zhu...@gmail.com
Signed-off-by: Stephen Warren swar...@nvidia.com
Acked-by: Eric Miao eric.y.m...@gmail.com
On Fri, Oct 5, 2012 at 3:13 AM, Thierry Reding
wrote:
> On Thu, Oct 04, 2012 at 08:48:54PM +0200, Lars-Peter Clausen wrote:
>> On 10/04/2012 08:29 PM, Thierry Reding wrote:
>> > On Thu, Oct 04, 2012 at 05:00:23PM +0200, Lars-Peter Clausen wrote:
>> >> On 10/04/2012 08:06 AM, Thierry Reding wrote:
st remove the function.
>
> Without this patch, building corgi_defconfig results in:
>
> /home/arnd/linux-arm/arch/arm/mach-pxa/sharpsl_pm.c:693:12: warning:
> 'sharpsl_fatal_check' defined but not used [-Wunused-function]
>
> Signed-off-by: Arnd Bergmann
> Cc: Pavel M
Brabec u...@penguin.cz
Cc: Eric Miao eric.y.m...@gmail.com
Cc: Haojian Zhuang haojian.zhu...@gmail.com
Acked-by: Eric Miao eric.y.m...@gmail.com
Let's get it cleaned up firstly. One can always reference the history for a
sample implementation later if this is needed in the future.
---
arch/arm
On Fri, Oct 5, 2012 at 3:13 AM, Thierry Reding
thierry.red...@avionic-design.de wrote:
On Thu, Oct 04, 2012 at 08:48:54PM +0200, Lars-Peter Clausen wrote:
On 10/04/2012 08:29 PM, Thierry Reding wrote:
On Thu, Oct 04, 2012 at 05:00:23PM +0200, Lars-Peter Clausen wrote:
On 10/04/2012 08:06 AM,
On Fri, Sep 21, 2012 at 3:19 PM, Wei Yongjun wrote:
> From: Wei Yongjun
>
> In case of error, the function clk_get() returns ERR_PTR()
> and never returns NULL pointer. The NULL test in the error
> handling should be replaced with IS_ERR().
>
> dpatch engine is used to auto generated this patch.
On Fri, Sep 21, 2012 at 3:19 PM, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function clk_get() returns ERR_PTR()
and never returns NULL pointer. The NULL test in the error
handling should be replaced with IS_ERR().
dpatch
On Tue, Sep 18, 2012 at 1:56 PM, Haojian Zhuang
wrote:
> On Tue, Sep 18, 2012 at 1:43 PM, Eric Miao wrote:
>> On Mon, Sep 17, 2012 at 6:56 PM, Linus Walleij
>> wrote:
>>> On Fri, Sep 14, 2012 at 4:36 AM, Wei Yongjun wrote:
>>>
>>>> From:
On Tue, Sep 18, 2012 at 1:56 PM, Haojian Zhuang
haojian.zhu...@gmail.com wrote:
On Tue, Sep 18, 2012 at 1:43 PM, Eric Miao eric.y.m...@gmail.com wrote:
On Mon, Sep 17, 2012 at 6:56 PM, Linus Walleij linus.wall...@linaro.org
wrote:
On Fri, Sep 14, 2012 at 4:36 AM, Wei Yongjun weiyj
tp://coccinelle.lip6.fr/)
>>
>> Signed-off-by: Wei Yongjun
>
> PXA maintainers: does this look OK?
I seem to have Acked this already in another mail, if that got lost, here
it is:
Acked-by: Eric Miao
>
> Yours,
> Linus Walleij
--
To unsubscribe from this list: send t
this.
(http://coccinelle.lip6.fr/)
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
PXA maintainers: does this look OK?
I seem to have Acked this already in another mail, if that got lost, here
it is:
Acked-by: Eric Miao eric.y.m...@gmail.com
Yours,
Linus Walleij
--
To unsubscribe
On Fri, Sep 14, 2012 at 10:30 AM, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Using for_each_set_bit() to simplify the code.
>
> spatch with a semantic match is used to found this.
> (http://coccinelle.lip6.fr/)
>
> Signed-off-by: Wei Yongjun
Great API, this is g
...@trendmicro.com.cn
Great API, this is good. Acked-by: Eric Miao eric.y.m...@gmail.com
---
arch/arm/mach-mmp/irq.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/arm/mach-mmp/irq.c b/arch/arm/mach-mmp/irq.c
index e60c7d9..3c71246 100644
--- a/arch/arm/mach-mmp/irq.c
On Mon, Aug 6, 2012 at 1:56 AM, Daniel Mack wrote:
> On 05.08.2012 18:56, Haojian Zhuang wrote:
>> On Mon, Aug 6, 2012 at 12:22 AM, Daniel Mack wrote:
>>> On 24.07.2012 20:01, Daniel Mack wrote:
On 23.07.2012 18:51, Dmitry Torokhov wrote:
> On Thu, Jul 19, 2012 at 05:36:12PM +0200,
On Mon, Aug 6, 2012 at 1:56 AM, Daniel Mack zon...@gmail.com wrote:
On 05.08.2012 18:56, Haojian Zhuang wrote:
On Mon, Aug 6, 2012 at 12:22 AM, Daniel Mack zon...@gmail.com wrote:
On 24.07.2012 20:01, Daniel Mack wrote:
On 23.07.2012 18:51, Dmitry Torokhov wrote:
On Thu, Jul 19, 2012 at
On Fri, Feb 22, 2008 at 12:26 AM, Liam Girdwood
<[EMAIL PROTECTED]> wrote:
> On Thu, 2008-02-21 at 08:41 +, Russell King - ARM Linux wrote:
> > On Wed, Feb 20, 2008 at 05:08:46PM +, Liam Girdwood wrote:
> > > This patch series provides a generic framework to allow device drivers
> > >
On Fri, Feb 22, 2008 at 12:26 AM, Liam Girdwood
[EMAIL PROTECTED] wrote:
On Thu, 2008-02-21 at 08:41 +, Russell King - ARM Linux wrote:
On Wed, Feb 20, 2008 at 05:08:46PM +, Liam Girdwood wrote:
This patch series provides a generic framework to allow device drivers
to control
On Jan 31, 2008 3:21 PM, David Brownell <[EMAIL PROTECTED]> wrote:
> On Wednesday 30 January 2008, Kumar Gala wrote:
> > Was wondering if anyone was looking at the cause of this warning in
> > top of linus's tree (8af03e782cae1e0a0f530ddd22301cdd12cf9dc0)
> >
> > drivers/spi/Kconfig:156:warning:
On Jan 31, 2008 3:21 PM, David Brownell [EMAIL PROTECTED] wrote:
On Wednesday 30 January 2008, Kumar Gala wrote:
Was wondering if anyone was looking at the cause of this warning in
top of linus's tree (8af03e782cae1e0a0f530ddd22301cdd12cf9dc0)
drivers/spi/Kconfig:156:warning: 'select'
> > From: eric miao <[EMAIL PROTECTED]>
> >
> > This patch adds the generic IRQ support for the PCA9539 on-chip GPIOs.
>
> This one bothers me a bit on some technical grounds. One problem is
> that these chips are not designed for reliable IRQ management,
From: eric miao [EMAIL PROTECTED]
This patch adds the generic IRQ support for the PCA9539 on-chip GPIOs.
This one bothers me a bit on some technical grounds. One problem is
that these chips are not designed for reliable IRQ management, so no
matter what a driver does it can't deliver
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. ma
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 PRO
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
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
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 current
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
[ 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
[updated according to David's suggestion to handle the error
of I2C transfer]
>From c9b78718488dadc702f40789bd532d1f1765d76e Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:24:36 +0800
Subject: [PATCH] gpiolib: add Generic IRQ support for 16-bit PCA
[ forget about the previous patch, sorry for my carelessness not to
free the chip structure, below is the correct one ]
>From c4be69e8dad28dc75e80b393f9c60f740cca7047 Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiol
[ Yup, it's an issue, patch updated as below:]
>From 8de0246423cbbd0c6bb03a20baf61d360930c350 Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiolib: basic support for 16-bit PCA9539 GPIO expander
1. use 16-bit regist
[ Yup, it's an issue, patch updated as below:]
From 8de0246423cbbd0c6bb03a20baf61d360930c350 Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiolib: basic support for 16-bit PCA9539 GPIO expander
1. use 16-bit register access
[ forget about the previous patch, sorry for my carelessness not to
free the chip structure, below is the correct one ]
From c4be69e8dad28dc75e80b393f9c60f740cca7047 Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiolib: basic
[updated according to David's suggestion to handle the error
of I2C transfer]
From c9b78718488dadc702f40789bd532d1f1765d76e Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:24:36 +0800
Subject: [PATCH] gpiolib: add Generic IRQ support for 16-bit PCA9539
GPIO
[ 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
Jean,
I'd like to postpone the corresponding change to the point that
polling i2c patch is merged.
On Dec 15, 2007 12:16 AM, Jean Delvare <[EMAIL PROTECTED]> wrote:
> Hi Eric,
>
>
> On Mon, 10 Dec 2007 17:37:05 +0800, eric miao wrote:
> > Support for PCA9539 as a GPIO ch
>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
>From b45be77acbf592b9c2085ed03ab5f16d780fa8c7 Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:24:36 +0800
Subject: [PATCH] gpiolib: add Generic IRQ support for 16-bit PCA9539
GPIO expander
This patch adds the generic IRQ support for the PCA953
>From 5ebe07236b99587296cbf603a965d284ceaf Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiolib: basic support for 16-bit PCA9539 GPIO expander
1. use 16-bit register access to simplify the logic, cac
[updated] support for PCA9539 as a GPIO chip is separated into three patches:
0001 - gpiolib: basic support for 16-bit PCA9539 GPIO expander
0002 - gpiolib: add Generic IRQ support for 16-bit PCA9539 GPIO expander
0003 - gpiolib: obsolete drivers/i2c/chips/pca9539.c
The 2nd one uses workqueue
I'd like to create another thread in LKML for the updated version, sorry.
On Dec 15, 2007 11:56 AM, eric miao <[EMAIL PROTECTED]> wrote:
> OK,
>
> Here's the updated version, which
> 1. modify the author info but still preserve Ben's credit in the source head
> 2. Alphab
initial output/direction register settings
Also I'd like to fire another patch to obsolete drivers/i2c/chips/pca9539.c
as everyone agreed.
>From 5ebe07236b99587296cbf603a965d284ceaf Mon Sep 17 00:00:00 2001
From: eric miao <[EMAIL PROTECTED]>
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subjec
initial output/direction register settings
Also I'd like to fire another patch to obsolete drivers/i2c/chips/pca9539.c
as everyone agreed.
From 5ebe07236b99587296cbf603a965d284ceaf Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH
I'd like to create another thread in LKML for the updated version, sorry.
On Dec 15, 2007 11:56 AM, eric miao [EMAIL PROTECTED] wrote:
OK,
Here's the updated version, which
1. modify the author info but still preserve Ben's credit in the source head
2. Alphabetic order in Kconfig/Makefile
3
[updated] support for PCA9539 as a GPIO chip is separated into three patches:
0001 - gpiolib: basic support for 16-bit PCA9539 GPIO expander
0002 - gpiolib: add Generic IRQ support for 16-bit PCA9539 GPIO expander
0003 - gpiolib: obsolete drivers/i2c/chips/pca9539.c
The 2nd one uses workqueue
From 5ebe07236b99587296cbf603a965d284ceaf Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:19:12 +0800
Subject: [PATCH] gpiolib: basic support for 16-bit PCA9539 GPIO expander
1. use 16-bit register access to simplify the logic, cache OUTPUT
From b45be77acbf592b9c2085ed03ab5f16d780fa8c7 Mon Sep 17 00:00:00 2001
From: eric miao [EMAIL PROTECTED]
Date: Mon, 10 Dec 2007 17:24:36 +0800
Subject: [PATCH] gpiolib: add Generic IRQ support for 16-bit PCA9539
GPIO expander
This patch adds the generic IRQ support for the PCA9539 on-chip GPIOs
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
Jean,
I'd like to postpone the corresponding change to the point that
polling i2c patch is merged.
On Dec 15, 2007 12:16 AM, Jean Delvare [EMAIL PROTECTED] wrote:
Hi Eric,
On Mon, 10 Dec 2007 17:37:05 +0800, eric miao wrote:
Support for PCA9539 as a GPIO chip is separated into two patches
On Dec 10, 2007 6:14 PM, David Brownell <[EMAIL PROTECTED]> wrote:
> On Monday 10 December 2007, eric miao wrote:
> > +config GPIO_PCA9539_GENERIC_IRQ
> > +bool " Generic IRQ support for PCA9539"
> > +depends on GPIO_PCA9539=y
>
> Also depends o
-in.
Signed-off-by: eric miao <[EMAIL PROTECTED]>
Acked-by: Ben Gardner <[EMAIL PROTECTED]>
---
drivers/gpio/Kconfig | 10 +++-
drivers/gpio/pca9539.c | 184
2 files changed, 193 insertions(+), 1 deletions(-)
diff --git a/drivers/gp
1. use 16-bit register access to simplify the logic, cache OUTPUT
and DIRECTION registers for fast access
2. platform code is required to setup
a) the numbering of GPIO for PCA9539 (base and number)
c) pass "pca9539_platform_data" within "i2c_board_info"
Signed-off-b
Support for PCA9539 as a GPIO chip is separated into two patches:
0001 - gpiolib: basic support for 16-bit PCA9539 GPIO expander
0002 - gpiolib: add Generic IRQ support for 16-bit PCA9539 GPIO expander
the 2nd one uses workqueue for IRQ handling due to the interrupt mode
nature of the i2c-core,
Support for PCA9539 as a GPIO chip is separated into two patches:
0001 - gpiolib: basic support for 16-bit PCA9539 GPIO expander
0002 - gpiolib: add Generic IRQ support for 16-bit PCA9539 GPIO expander
the 2nd one uses workqueue for IRQ handling due to the interrupt mode
nature of the i2c-core,
1. use 16-bit register access to simplify the logic, cache OUTPUT
and DIRECTION registers for fast access
2. platform code is required to setup
a) the numbering of GPIO for PCA9539 (base and number)
c) pass pca9539_platform_data within i2c_board_info
Signed-off-by: eric miao [EMAIL
-in.
Signed-off-by: eric miao [EMAIL PROTECTED]
Acked-by: Ben Gardner [EMAIL PROTECTED]
---
drivers/gpio/Kconfig | 10 +++-
drivers/gpio/pca9539.c | 184
2 files changed, 193 insertions(+), 1 deletions(-)
diff --git a/drivers/gpio/Kconfig b/drivers
On Dec 10, 2007 6:14 PM, David Brownell [EMAIL PROTECTED] wrote:
On Monday 10 December 2007, eric miao wrote:
+config GPIO_PCA9539_GENERIC_IRQ
+bool Generic IRQ support for PCA9539
+depends on GPIO_PCA9539=y
Also depends on GENERIC_HARDIRQS, right? (You should let
the Kconfig UI handle
with chips with different numbers of GPIOs,
> and to avoid holes in GPIOs number sequences. Those holes can cost a
> lot of unusable irq_desc space for GPIOs that act as IRQs.
>
> Based on a patch from Eric Miao.
>
> # NOT signed-off yet ... purely for comment. It's been sanit
as IRQs.
Based on a patch from Eric Miao.
# NOT signed-off yet ... purely for comment. It's been sanity tested.
---
The question I'm most interested in is whether it's worth paying the
extra data memory. I'm currently leaning towards yes, mostly since
it'll let me be lazy about some
started ? "\n" : "",
- chip->base, chip->base + chip->ngpio - 1,
- chip->label ? : "generic",
- chip->can_sleep ? ", can sleep" : "");
-
OK, here's the all-in-one patch based on David's most recent gpiolib fix,
Changes include:
1. use per gpio structure "gpio_desc", thus eliminating the restriction of
ARCH_GPIOS_PER_CHIP, thus making it possible leaving no holes in GPIOs
numbering
Note: the number of GPIOs on different
OK, here's the all-in-one patch based on David's most recent gpiolib fix,
Changes include:
1. use per gpio structure gpio_desc, thus eliminating the restriction of
ARCH_GPIOS_PER_CHIP, thus making it possible leaving no holes in GPIOs
numbering
Note: the number of GPIOs on different
)
- chip-dbg_show(s, chip);
- else
- gpiolib_dbg_show(s, chip);
+ gpio++;
}
return 0;
}
--
1.5.2.5.GIT
On Nov 28, 2007 3:29 AM, David Brownell [EMAIL PROTECTED] wrote:
On Tuesday 27 November 2007, eric miao wrote
On Nov 21, 2007 11:54 AM, ian <[EMAIL PROTECTED]> wrote:
> On Wed, 2007-11-21 at 10:23 +0800, eric miao wrote:
> > Roughly went through the patch, looks good, here comes the remind, though
> > :-)
> >
> > 1. is it possible to use some name other than &qu
Roughly went through the patch, looks good, here comes the remind, though :-)
1. is it possible to use some name other than "soc_core", maybe
"tmio_core" so that
other multifunction chips sharing a core base will live easier.
2. those C++ style comments "//" are not so pleasant...
- eric
On
1 - 100 of 138 matches
Mail list logo