On Thu, Aug 9, 2012 at 8:28 PM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Wed, Aug 8, 2012 at 10:40 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Add *remove* callback so that necessary cleanup operations
On Wed, Aug 8, 2012 at 7:33 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
On Wed, Aug 8, 2012 at 7:28 PM, Tarun Kanti DebBarma tarun.ka...@ti.com
wrote:
Add *remove* callback so that necessary cleanup operations are
performed when device is unregistered. The device is deleted
from
On Wed, Aug 8, 2012 at 10:40 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Add *remove* callback so that necessary cleanup operations are
performed when device is unregistered. The device is deleted
from the list and associated clock handle is released
Sourav,
On Mon, Jul 30, 2012 at 2:13 PM, Poddar, Sourav sourav.pod...@ti.com wrote:
Hi All,
I tried using gpio as an interrupt line for my driver
(drivers/staging/iio/light/tsl2x7x_core.c) for omap5.
The interrupt line number was directly passed to the driver using
device tree. But what I
Hi Roger,
On Mon, Jul 9, 2012 at 4:46 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
Hi Roger,
On Mon, Jul 9, 2012 at 3:13 PM, Roger Quadros rog...@ti.com wrote:
Hi,
Just bumped across this patch and have a query.
On 03/16/2012 04:05 PM, Tarun Kanti DebBarma wrote:
There is no more
On Wed, Jul 11, 2012 at 11:21 PM, Paul Walmsley p...@pwsan.com wrote:
On Wed, 11 Jul 2012, Tarun Kanti DebBarma wrote:
Since the bank-dbck initialization in a one time operation there
is no need to keep this within gpio_debounce(). Therefore, moving
clk_get(bank-dbck) to omap_gpio_mod_init().
Hi Roger,
On Mon, Jul 9, 2012 at 3:13 PM, Roger Quadros rog...@ti.com wrote:
Hi,
Just bumped across this patch and have a query.
On 03/16/2012 04:05 PM, Tarun Kanti DebBarma wrote:
There is no more need to have saved_wakeup because bank-context.wake_en
already holds that value. So getting
On Mon, Jul 9, 2012 at 5:21 PM, Roger Quadros rog...@ti.com wrote:
Tarun,
On 07/09/2012 02:16 PM, DebBarma, Tarun Kanti wrote:
Hi Roger,
On Mon, Jul 9, 2012 at 3:13 PM, Roger Quadros rog...@ti.com wrote:
Hi,
Just bumped across this patch and have a query.
On 03/16/2012 04:05 PM, Tarun
Hi Tony,
On Thu, Jul 5, 2012 at 2:58 PM, Tony Lindgren t...@atomide.com wrote:
* DebBarma, Tarun Kanti tarun.ka...@ti.com [120704 21:07]:
On Thu, Jun 14, 2012 at 4:28 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
Just to let you know that this cleanup is essential for Benoit's
Hi Tony,
On Thu, Jun 14, 2012 at 4:28 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
Tony,
On Thu, Jun 14, 2012 at 4:24 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Apr 20, 2012 at 6:09 PM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
The devm API usage in probe
On Fri, Jun 29, 2012 at 6:29 AM, Franky Lin fran...@broadcom.com wrote:
On 06/28/2012 04:54 PM, Jon Hunter wrote:
I am wondering if this could be the bug ... on start-up I see that we do
a context restore on bank1 during the probe which is before we have done
the first suspend! In other
On Tue, Jun 26, 2012 at 11:50 PM, Franky Lin fran...@broadcom.com wrote:
On 06/26/2012 12:21 AM, DebBarma, Tarun Kanti wrote:
On Tue, Jun 26, 2012 at 2:22 AM, Franky Lin fran...@broadcom.com wrote:
Hi Kevin, Tarun,
We are using the expansion connector A on Panda board to mount a SDIO
WiFi
On Tue, Jun 26, 2012 at 2:22 AM, Franky Lin fran...@broadcom.com wrote:
Hi Kevin, Tarun,
We are using the expansion connector A on Panda board to mount a SDIO WiFi
dongle on MMC2 with a level triggered interrupt signal connected to GPIO
138. It's been working fine until 3.5 rc1. The board
On Mon, Jun 25, 2012 at 11:00 AM, Rajendra Nayak rna...@ti.com wrote:
On Saturday 23 June 2012 12:47 AM, Paul Walmsley wrote:
Hi
On Fri, 22 Jun 2012, Rajendra Nayak wrote:
In preparation of OMAP moving to Common Clk Framework(CCF) add
clk_prepare()
for omap gpio debounce clock.
On Mon, Jun 25, 2012 at 11:48 AM, NeilBrown ne...@suse.de wrote:
On Thu, 21 Jun 2012 12:04:26 +0530 DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Thu, Jun 21, 2012 at 8:46 AM, NeilBrown ne...@suse.de wrote:
On Thu, 14 Jun 2012 23:24:10 +0530 DebBarma, Tarun Kanti
tarun.ka...@ti.com
On Mon, Jun 25, 2012 at 12:32 PM, Rajendra Nayak rna...@ti.com wrote:
On Monday 25 June 2012 11:41 AM, DebBarma, Tarun Kanti wrote:
On Mon, Jun 25, 2012 at 11:00 AM, Rajendra Nayakrna...@ti.com wrote:
On Saturday 23 June 2012 12:47 AM, Paul Walmsley wrote:
Hi
On Fri, 22 Jun 2012
On Thu, Jun 21, 2012 at 8:46 AM, NeilBrown ne...@suse.de wrote:
On Thu, 14 Jun 2012 23:24:10 +0530 DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Thu, Jun 14, 2012 at 5:45 AM, NeilBrown ne...@suse.de wrote:
On Fri, 11 May 2012 17:30:48 -0700 Kevin Hilman khil...@ti.com wrote:
Hi
On Fri, Apr 20, 2012 at 6:09 PM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
The devm API usage in probe() simplifies error handling operation.
Since iclk is not used in the driver it is removed from wherever
not needed.
Corrected the timer fck name mis-match between clock44xx_data.c and
On Thu, Jun 14, 2012 at 5:45 AM, NeilBrown ne...@suse.de wrote:
On Fri, 11 May 2012 17:30:48 -0700 Kevin Hilman khil...@ti.com wrote:
Hi Grant,
Here's the final round of GPIO cleanups for v3.5. This branch is based
on my for_3.5/fixes/gpio branch you just pulled.
Kevin
Hi.
I'm not
On Wed, May 30, 2012 at 2:51 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Timers in PER domain periodically report old time from TCRR in
posted mode if ick 4*fck. Therefore, set timer to non-posted
whenever ick 4*fck for all timers.
Is there an
On Fri, May 18, 2012 at 3:51 AM, Kevin Hilman khil...@ti.com wrote:
Tarun, Santosh,
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
We do checking for bank-enabled_non_wakeup_gpios in order
to skip redundant operations. Somehow, the check got missed
while doing the cleanup series.
Just to
On Fri, May 18, 2012 at 5:26 AM, Kevin Hilman khil...@ti.com wrote:
Tony Lindgren t...@atomide.com writes:
* Kevin Hilman khil...@ti.com [120517 15:29]:
I just noticed that this patch has caused some strange problems, notably
with the GPIO IRQ used by smsc911x NIC (Overo, Zoom3, 2430SDP,
On Fri, May 18, 2012 at 3:51 AM, Kevin Hilman khil...@ti.com wrote:
Tarun, Santosh,
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
We do checking for bank-enabled_non_wakeup_gpios in order
to skip redundant operations. Somehow, the check got missed
while doing the cleanup series.
Just to
On Thu, May 17, 2012 at 1:44 AM, Jon Hunter jon-hun...@ti.com wrote:
Hi Benoit,
On 05/16/2012 04:28 AM, Cousson, Benoit wrote:
Hi Jon,
On 5/16/2012 1:35 AM, Jon Hunter wrote:
From: Jon Hunterjon-hun...@ti.com
In order to migrate the dmtimer driver to support device-tree I found
that it
Hi,
On Thu, May 10, 2012 at 3:06 AM, Janusz Krzysztofik
jkrzy...@tis.icnet.pl wrote:
On Mon, 7 May 2012 10:52:28 DebBarma, Tarun Kanti wrote:
On Sun, May 6, 2012 at 3:25 AM, Grazvydas Ignotas nota...@gmail.com wrote:
On Mon, Apr 30, 2012 at 10:20 AM, Tarun Kanti DebBarma
tarun.ka...@ti.com
Hi,
On Sun, May 6, 2012 at 3:25 AM, Grazvydas Ignotas nota...@gmail.com wrote:
I've noticed that current mainline enables _all_ possible GPIO
interrupts, this patch fixes that problem.
OK, thanks.
Also Grant doesn't seem to be on CC so might not be able to pick this up.
I have added Cc: in
On Sat, Apr 28, 2012 at 3:48 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Breaking commit: ab985f0f7c2c0ef90b7c832f0c04f470dda0593d
Initialization of irqenable, irqstatus registers is the common
operation done in this function for all OMAP platforms,
On Wed, Apr 25, 2012 at 12:09 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
On Wed, Apr 25, 2012 at 10:04 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Apr 24, 2012 at 9:34 PM, Tony Lindgren t...@atomide.com wrote:
* DebBarma, Tarun Kanti tarun.ka...@ti.com [120424 08:40
On Wed, Apr 25, 2012 at 7:15 PM, Russell King - ARM Linux
li...@arm.linux.org.uk wrote:
On Wed, Apr 25, 2012 at 06:24:14PM +0530, DebBarma, Tarun Kanti wrote:
On Wed, Apr 25, 2012 at 12:09 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
On Wed, Apr 25, 2012 at 10:04 AM, DebBarma, Tarun
Hi Janusz,
On Tue, Apr 24, 2012 at 12:24 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Sat, Apr 21, 2012 at 7:33 PM, Janusz Krzysztofik
jkrzy...@tis.icnet.pl wrote:
On Thursday 02 of February 2012 23:00:37 Tarun Kanti DebBarma wrote:
With register offsets now defined for respective
On Tue, Apr 24, 2012 at 9:34 PM, Tony Lindgren t...@atomide.com wrote:
* DebBarma, Tarun Kanti tarun.ka...@ti.com [120424 08:40]:
Hi Janusz,
On Tue, Apr 24, 2012 at 12:24 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Sat, Apr 21, 2012 at 7:33 PM, Janusz Krzysztofik
jkrzy
On Sat, Apr 21, 2012 at 7:33 PM, Janusz Krzysztofik
jkrzy...@tis.icnet.pl wrote:
On Thursday 02 of February 2012 23:00:37 Tarun Kanti DebBarma wrote:
With register offsets now defined for respective OMAP versions we can get rid
of cpu_class_* checks. This function now has common initialization
On Mon, Apr 23, 2012 at 11:18 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
We do not use iclk anywhere in the dmtimer driver and so removing it.
Hence removing the timer iclk entries from OMAP4 clkdev table as well.
What about the OMAP2/3 clkdev
On Wed, Apr 18, 2012 at 1:45 PM, Paul Walmsley p...@pwsan.com wrote:
Hi Tarun
On Wed, 18 Apr 2012, DebBarma, Tarun Kanti wrote:
That' right, I confirmed that clk_get(NULL, fck) does not work.
Yes, you and Benoît are right, I should have looked more closely at what
was being patched
On Mon, Apr 16, 2012 at 8:33 PM, Hiremath, Vaibhav hvaib...@ti.com wrote:
On Mon, Apr 16, 2012 at 17:55:24, DebBarma, Tarun Kanti wrote:
We do not use iclk anywhere in the dmtimer driver and so removing it.
Hence removing the timer iclk entries from OMAP4 clkdev table as well.
Signed-off
On Mon, Apr 16, 2012 at 10:20 PM, Paul Walmsley p...@pwsan.com wrote:
Hi Tarun,
by the way,
On Mon, 16 Apr 2012, Tarun Kanti DebBarma wrote:
We do not use iclk anywhere in the dmtimer driver and so removing it.
Hence removing the timer iclk entries from OMAP4 clkdev table as well.
On Mon, Apr 16, 2012 at 8:23 PM, Hiremath, Vaibhav hvaib...@ti.com wrote:
On Mon, Apr 16, 2012 at 17:55:22, DebBarma, Tarun Kanti wrote:
The devm API usage in probe() simplifies error handling operation.
Since iclk is not used in the driver it is removed from wherever
not needed. The fclk
On Mon, Apr 16, 2012 at 8:32 PM, Shubhrajyoti shubhrajy...@ti.com wrote:
On Monday 16 April 2012 05:55 PM, Tarun Kanti DebBarma wrote:
Replace the regular kzalloc and ioremap with the devm_ equivalent
to simplify error handling. We don't need kree() anymore in
omap_dm_timer_remove().
Also
On Thu, Apr 12, 2012 at 6:04 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Since hwmod framework now manages sysconfig context save/restore
there is no more need to touch this register in driver. Hence,
remove restore of sysconfig register in
On Mon, Mar 26, 2012 at 7:14 PM, Shubhrajyoti D shubhrajy...@ti.com wrote:
From: Benoit Cousson b-cous...@ti.com
bus_num was used to reference the mcspi controller instance in a fixed array.
Remove this array and store this information directly inside drvdata
structure.
bus_num is now just
On Thu, Mar 29, 2012 at 10:28 AM, Olof Johansson o...@lixom.net wrote:
Hi,
Sorry for the slow reply, I noticed this when dealing with merge
conflicts pulling in this patch and others from Tony.
On Mon, Mar 19, 2012 at 5:06 AM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
With dynamic
On Wed, Mar 21, 2012 at 7:36 PM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Wed, Mar 21, 2012 at 8:10 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 20, 2012 at 11:31 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma
On Tue, Mar 20, 2012 at 2:30 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 3/20/2012 12:39 AM, Tony Lindgren wrote:
* Cousson, Benoitb-cous...@ti.com [120319 16:00]:
Hi Tony,
On 3/19/2012 8:17 PM, Tony Lindgren wrote:
* Tarun Kanti DebBarmatarun.ka...@ti.com [120319 05:09]:
These
On Tue, Mar 20, 2012 at 11:31 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
In _enable_gpio_irqbank() when bank-regs-set_irqenable is valid,
gpio_mask can be directly set by writing to set_irqenable register
without overwriting current value. In order
On Wed, Mar 21, 2012 at 8:10 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 20, 2012 at 11:31 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
In _enable_gpio_irqbank() when bank-regs-set_irqenable is valid,
gpio_mask can be directly set
On Mon, Mar 19, 2012 at 5:47 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
On Mon, Mar 19, 2012 at 5:36 PM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
These two patches incorporate changes to OMAP1 and OMAP2 platforms
s/two/four
I guess that's what you mean since there are 4
On Tue, Mar 20, 2012 at 6:07 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
The cleanup is mostly getting rid of redundant fields in struct gpio_bank{}
as we already have them as part of bank-context now. Also, remove un-used
variable from
On Fri, Mar 16, 2012 at 7:16 PM, Cousson, Benoit b-cous...@ti.com wrote:
Hi Tarun,
On 3/16/2012 1:24 PM, Tarun Kanti DebBarma wrote:
These two patches incorporate changes to OMAP1 and OMAP2 platforms
board files whereby older references to OMAP_GPIO_IRQ macro are
now replaced with
On Thu, Mar 15, 2012 at 3:35 AM, Kevin Hilman khil...@ti.com wrote:
Tarun,
Can you investigate an abort during boot on 3630/Zoom3?
Both Tony and I are seeing the abort below on 3630/Zoom3. I'm using
arm-soc/for-next and Tony is using linux-next, but we see the same abort.
The crash looks
On Tue, Mar 13, 2012 at 3:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
In the existing _set_gpio_dataout_*() implementation, the dataout
register is overwritten every time the function is called. This is
not intended behavior because that would end
On Tue, Mar 13, 2012 at 11:33 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 3:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
In the existing _set_gpio_dataout_*() implementation, the dataout
register is overwritten every
On Tue, Mar 13, 2012 at 12:03 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 11:33 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 3:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes
On Tue, Mar 13, 2012 at 9:57 PM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Tue, Mar 13, 2012 at 12:03 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 11:33 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue
On Tue, Mar 13, 2012 at 12:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
The cleanup is mostly getting rid of redundant fields in struct gpio_bank{}
as we already have them as part of bank-context now. Also, remove un-used
variable from
On Tue, Mar 13, 2012 at 1:23 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 12:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
The cleanup is mostly getting rid of redundant fields in struct gpio_bank{}
as we already have
On Tue, Mar 13, 2012 at 1:38 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 1:23 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Mar 13, 2012 at 12:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
The cleanup
On Tue, Mar 13, 2012 at 9:27 AM, Grant Likely grant.lik...@secretlab.ca wrote:
On Mon, 12 Mar 2012 15:28:42 -0700, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
[...]
Oops... I am sorry.
I was in the process of updating the branch with latest comments
On Tue, Mar 13, 2012 at 3:39 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
In _enable_gpio_irqbank() when bank-regs-set_irqenable is valid,
gpio_mask can be directly set by writing to set_irqenable register
without overwriting current value.
Ouch.
On Wed, Mar 7, 2012 at 4:45 PM, Tarun Kanti DebBarma tarun.ka...@ti.com wrote:
This local variable is just assigned zero and then OR'ed
with isr. It does not appear to serve any purpose and so
removing it.
Missed following comments from Kevin given in v2. Sorry about that.
I have updated the
On Thu, Mar 8, 2012 at 12:49 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
On Thu, Mar 8, 2012 at 4:58 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Wed, Mar 7, 2012 at 5:37 PM, Santosh Shilimkar
santosh.shilim...@ti.com wrote:
On Wednesday 07 March 2012 12:16 PM, Tarun
On Wed, Mar 7, 2012 at 5:33 PM, Santosh Shilimkar
santosh.shilim...@ti.com wrote:
On Wednesday 07 March 2012 12:16 PM, Tarun Kanti DebBarma wrote:
There are two functions, _set_gpio_dataout_reg() and _set_gpio_dataout_mask()
which writes to dataout register and the dataout context must be
On Wed, Mar 7, 2012 at 5:37 PM, Santosh Shilimkar
santosh.shilim...@ti.com wrote:
On Wednesday 07 March 2012 12:16 PM, Tarun Kanti DebBarma wrote:
In omap_gpio_runtime_resume() the context restore should be independent
of bank-enabled_non_wakeup_gpios. This was preventing context restore
of
On Tue, Feb 28, 2012 at 5:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Since we already have bank-context.wake_en to keep track
of gpios which are wakeup enabled, there is no need to have
this field any more.
Signed-off-by: Tarun Kanti DebBarma
On Tue, Feb 28, 2012 at 3:09 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Tue, Feb 28, 2012 at 5:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Since we already have bank-context.wake_en to keep track
of gpios which are wakeup enabled
On Wed, Feb 29, 2012 at 1:59 AM, Cousson, Benoit b-cous...@ti.com wrote:
+ Kevin
Hi Tarun,
On 2/27/2012 8:11 AM, DebBarma, Tarun Kanti wrote:
On Sat, Feb 25, 2012 at 6:34 PM, Russell King - ARM Linux
li...@arm.linux.org.uk wrote:
Can someone explain to me this:
#define GPIO_INDEX
On Wed, Feb 29, 2012 at 12:15 AM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Tue, Feb 28, 2012 at 5:24 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Since we already have bank-context.wake_en to keep track
On Tue, Feb 28, 2012 at 5:20 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
There is no more need to have saved_wakeup. Instead we can use
context.wake_en which holds the current wakeup enable register
context. This also means that the read from wakeup
On Tue, Feb 28, 2012 at 5:32 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
This local variable is just assigned zero and then OR'ed
with isr. It does not appear to serve any purpose and so
removing it.
Signed-off-by: Tarun Kanti DebBarma
On Sat, Feb 25, 2012 at 6:34 PM, Russell King - ARM Linux
li...@arm.linux.org.uk wrote:
Can someone explain to me this:
#define GPIO_INDEX(bank, gpio) (gpio % bank-width)
#define GPIO_BIT(bank, gpio) (1 GPIO_INDEX(bank, gpio))
static int _get_gpio_datain(struct gpio_bank *bank, int gpio)
{
On Fri, Feb 24, 2012 at 3:41 PM, Cousson, Benoit b-cous...@ti.com wrote:
+ Tarun
On 2/24/2012 12:08 AM, Tony Lindgren wrote:
* Cousson, Benoitb-cous...@ti.com [120223 14:14]:
The GPIO driver is still relying on internal OMAP IRQ defines that
are not relevant anymore if OMAP is built with
Hi Benoit,
On Fri, Feb 24, 2012 at 4:07 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Feb 24, 2012 at 3:41 PM, Cousson, Benoit b-cous...@ti.com wrote:
+ Tarun
On 2/24/2012 12:08 AM, Tony Lindgren wrote:
* Cousson, Benoitb-cous...@ti.com [120223 14:14]:
The GPIO driver
On Fri, Feb 24, 2012 at 7:02 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 2/24/2012 2:24 PM, DebBarma, Tarun Kanti wrote:
Hi Benoit,
On Fri, Feb 24, 2012 at 4:07 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Feb 24, 2012 at 3:41 PM, Cousson, Benoitb-cous...@ti.com
wrote
Hi Benoit,
On Fri, Feb 24, 2012 at 7:26 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 2/24/2012 2:53 PM, DebBarma, Tarun Kanti wrote:
On Fri, Feb 24, 2012 at 7:02 PM, Cousson, Benoitb-cous...@ti.com wrote:
On 2/24/2012 2:24 PM, DebBarma, Tarun Kanti wrote:
Hi Benoit,
On Fri, Feb 24
On Wed, Feb 22, 2012 at 12:31 AM, Kevin Hilman khil...@ti.com wrote:
While both level- and edge-triggered GPIOs are capable of generating
interrupts, only edge-triggered GPIOs are capable of generating a
module-level wakeup to the PRCM (c.f. 34xx NDA TRM section 25.5.3.2.)
In order to ensure
Hi Benoit,
On Wed, Feb 15, 2012 at 9:34 PM, Benoit Cousson b-cous...@ti.com wrote:
Replace the regular kzalloc and ioremap with the devm_ equivalent
to simplify error handling.
Add the missing devm_request_mem_region to reserve the region used
by the driver.
Signed-off-by: Benoit Cousson
On Wed, Feb 15, 2012 at 9:34 PM, Benoit Cousson b-cous...@ti.com wrote:
The driver does not need anymore any id to identify the GPIO instance.
Remove every occurence of the bank-id inside the driver.
Remove two trailing spaces.
Add a dev variable for better readability in probe.
Remove
On Thu, Feb 16, 2012 at 12:05 PM, Grant Likely
grant.lik...@secretlab.ca wrote:
On Thu, Feb 16, 2012 at 11:11:52AM +0530, DebBarma, Tarun Kanti wrote:
Hi Benoit,
On Wed, Feb 15, 2012 at 9:34 PM, Benoit Cousson b-cous...@ti.com wrote:
Replace the regular kzalloc and ioremap with the devm_
On Sat, Feb 11, 2012 at 1:25 AM, Kevin Hilman khil...@ti.com wrote:
Tony Lindgren t...@atomide.com writes:
[...]
..but are there also some patches in your series that should also be
applied separately as fixes for the -rc cycle?
IMO, there aren't any things in this series that fall into
Kevin,
On Sat, Feb 4, 2012 at 2:31 AM, Kevin Hilman khil...@ti.com wrote:
Hi Tarun,
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
The following changes since commit 62aa2b537c6f5957afd98e29f96897419ed5ebab:
Linus Torvalds (1):
Linux 3.3-rc2
are available in the git
On Fri, Dec 16, 2011 at 12:58 PM, Shilimkar, Santosh
santosh.shilim...@ti.com wrote:
Tarun,
On Wed, Dec 14, 2011 at 1:50 AM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
This series is continuation of cleanup of OMAP GPIO driver and fixes.
The cleanup include getting rid of cpu_is_* checks
On Mon, Dec 5, 2011 at 7:27 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Mon, Nov 7, 2011 at 7:19 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Nov 4, 2011 at 10:10 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Currently
On Mon, Nov 7, 2011 at 7:19 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Nov 4, 2011 at 10:10 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Currently debounce clock state is not tracked in the system.
??
bank-dbck_enable_mask ?
As I
On Fri, Nov 4, 2011 at 3:14 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
This series is continuation of cleanup of OMAP GPIO driver and fixes.
Using this series on 3630/Zoom3, UART wakeups no longer work from
suspend. That suggests that GPIO wakeups
On Fri, Nov 4, 2011 at 2:57 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Nov 4, 2011 at 3:14 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
This series is continuation of cleanup of OMAP GPIO driver and fixes.
Using this series on 3630
On Mon, Nov 7, 2011 at 5:35 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Nov 4, 2011 at 10:23 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
From: Nishanth Menon n...@ti.com
GPIO IP revisions such as those used in OMAP4 have
On Mon, Nov 21, 2011 at 10:12 PM, Ramirez Luna, Omar
omar.rami...@ti.com wrote:
Hi,
On Tue, Nov 8, 2011 at 12:00 AM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
Since omap_dm_timer_write_reg/__omap_dm_timer_write is now modified
to use timer-func_base OCP_CFG should not use this wrapper
On Thu, Nov 24, 2011 at 1:24 AM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Mon, Nov 7, 2011 at 5:35 PM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Fri, Nov 4, 2011 at 10:23 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma
On Thu, Nov 17, 2011 at 6:23 AM, Omar Ramirez Luna or.r...@gmail.com wrote:
On Wed, Nov 16, 2011 at 12:18 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
...
My use case is as follows:
DSP sends a mailbox message to ARM, this triggers a tasklet in mailbox
for processing the message
On Tue, Nov 15, 2011 at 11:27 PM, Omar Ramirez Luna or.r...@gmail.com wrote:
Hi Tarun,
On Tue, Sep 20, 2011 at 6:30 AM, Tarun Kanti DebBarma
tarun.ka...@ti.com wrote:
Clock is enabled only when timer is started and disabled when the the timer
is stopped. Therefore before accessing registers
On Fri, Nov 4, 2011 at 10:43 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Call runtime pm APIs pm_runtime_get_sync() and pm_runtime_put_sync()
for enabling/disabling clocks appropriately. Remove syscore_ops and
instead use SET_RUNTIME_PM_OPS macro.
On Fri, Nov 4, 2011 at 10:27 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
From: Nishanth Menon n...@ti.com
GPIO debounce registers need to be saved and restored for proper functioning
of driver.
OK.
To save the registers, we cannot cut the clock
On Fri, Nov 4, 2011 at 10:23 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
From: Nishanth Menon n...@ti.com
GPIO IP revisions such as those used in OMAP4 have a set_dataout
while the previous revisions used a single dataout register.
Depending on
On Fri, Nov 4, 2011 at 10:10 PM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
Currently debounce clock state is not tracked in the system.
??
bank-dbck_enable_mask ?
As I understand, this only tells which all GPIOs have debounce timeout enabled.
But,
On Fri, Nov 4, 2011 at 3:14 AM, Kevin Hilman khil...@ti.com wrote:
Tarun Kanti DebBarma tarun.ka...@ti.com writes:
This series is continuation of cleanup of OMAP GPIO driver and fixes.
Using this series on 3630/Zoom3, UART wakeups no longer work from
suspend. That suggests that GPIO wakeups
On Mon, Oct 10, 2011 at 11:10 AM, DebBarma, Tarun Kanti
tarun.ka...@ti.com wrote:
On Wed, Oct 5, 2011 at 3:06 AM, Cousson, Benoit b-cous...@ti.com wrote:
Hi Tarun,
On 10/4/2011 11:20 PM, Cousson, Benoit wrote:
Hi Tony,
Here is the series to fix the warning and remove the redundant
latency
On Wed, Oct 5, 2011 at 3:06 AM, Cousson, Benoit b-cous...@ti.com wrote:
Hi Tarun,
On 10/4/2011 11:20 PM, Cousson, Benoit wrote:
Hi Tony,
Here is the series to fix the warning and remove the redundant
latency structure that be removed since the timer runtime PM
adaptation was just pulled.
On Tue, Sep 27, 2011 at 4:40 AM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
[...]
As pointed out by Kevin, debounce clock was not getting disabled.
In my testing I was somehow grepping CORE power domain instead
of PER power domain and hence missed
On Tue, Sep 27, 2011 at 9:52 PM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
On Tue, Sep 27, 2011 at 4:40 AM, Kevin Hilman khil...@ti.com wrote:
DebBarma, Tarun Kanti tarun.ka...@ti.com writes:
[...]
As pointed out by Kevin, debounce clock
On Mon, Sep 26, 2011 at 10:55 PM, Tony Lindgren t...@atomide.com wrote:
* DebBarma, Tarun Kanti tarun.ka...@ti.com [110923 01:54]:
Hi Tony,
[...]
I've applied these into dmtimer branch with some changes to simplify
things further. I've also merged it into linux-omap master branch
[...]
- Added the debounce clock fix in the end.
Thanks. Glad you found and fixed it.
Rather than add this patch as a fix at the end, I prefer if the problem
is fixed in the original patches that added/created the problem.
Sure. I will put the changes in respective patches.
--
Tarun
[...]
1 - 100 of 314 matches
Mail list logo
